diff options
-rw-r--r-- | lib/puppet/provider/whole_line/ruby.rb (renamed from lib/puppet/provider/append_line/ruby.rb) | 2 | ||||
-rw-r--r-- | lib/puppet/type/whole_line.rb (renamed from lib/puppet/type/append_line.rb) | 6 | ||||
-rw-r--r-- | spec/unit/puppet/provider/whole_line/ruby_spec.rb (renamed from spec/unit/puppet/provider/append_line/ruby_spec.rb) | 4 | ||||
-rw-r--r-- | spec/unit/puppet/type/append_line_spec.rb | 24 | ||||
-rw-r--r-- | spec/unit/puppet/type/whole_line_spec.rb | 24 | ||||
-rw-r--r-- | tests/whole_line.pp (renamed from tests/append_line.pp) | 4 |
6 files changed, 32 insertions, 32 deletions
diff --git a/lib/puppet/provider/append_line/ruby.rb b/lib/puppet/provider/whole_line/ruby.rb index 5e78659..156f255 100644 --- a/lib/puppet/provider/append_line/ruby.rb +++ b/lib/puppet/provider/whole_line/ruby.rb @@ -1,4 +1,4 @@ -Puppet::Type.type(:append_line).provide(:ruby) do +Puppet::Type.type(:whole_line).provide(:ruby) do def exists? File.readlines(resource[:path]).find do |line| diff --git a/lib/puppet/type/append_line.rb b/lib/puppet/type/whole_line.rb index b3f926c..f231602 100644 --- a/lib/puppet/type/append_line.rb +++ b/lib/puppet/type/whole_line.rb @@ -1,11 +1,11 @@ -Puppet::Type.newtype(:append_line) do +Puppet::Type.newtype(:whole_line) do desc <<-EOT - Type that can append a line to a file if it does not already contain it. + Type that can append whole a line to a file if it does not already contain it. Example: - append_line { 'sudo_rule': + whole_line { 'sudo_rule': path => '/etc/sudoers', line => '%admin ALL=(ALL) ALL', } diff --git a/spec/unit/puppet/provider/append_line/ruby_spec.rb b/spec/unit/puppet/provider/whole_line/ruby_spec.rb index ea28c31..9faff2c 100644 --- a/spec/unit/puppet/provider/append_line/ruby_spec.rb +++ b/spec/unit/puppet/provider/whole_line/ruby_spec.rb @@ -1,12 +1,12 @@ require 'puppet' require 'tempfile' -provider_class = Puppet::Type.type(:append_line).provider(:ruby) +provider_class = Puppet::Type.type(:whole_line).provider(:ruby) describe provider_class do before :each do tmp = Tempfile.new('tmp') @tmpfile = tmp.path tmp.close! - @resource = Puppet::Type::Append_line.new( + @resource = Puppet::Type::Whole_line.new( {:name => 'foo', :path => @tmpfile, :line => 'foo'} ) @provider = provider_class.new(@resource) diff --git a/spec/unit/puppet/type/append_line_spec.rb b/spec/unit/puppet/type/append_line_spec.rb deleted file mode 100644 index d0564c3..0000000 --- a/spec/unit/puppet/type/append_line_spec.rb +++ /dev/null @@ -1,24 +0,0 @@ -require 'puppet' -require 'tempfile' -describe Puppet::Type.type(:append_line) do - before :each do - @append_line = Puppet::Type.type(:append_line).new(:name => 'foo', :line => 'line', :path => '/tmp/path') - end - it 'should accept a line and path' do - @append_line[:line] = 'my_line' - @append_line[:line].should == 'my_line' - end - it 'should accept posix filenames' do - @append_line[:path] = '/tmp/path' - @append_line[:path].should == '/tmp/path' - end - it 'should not accept unqualified path' do - expect { @append_line[:path] = 'file' }.should raise_error(Puppet::Error, /File paths must be fully qualified/) - end - it 'should require that a line is specified' do - expect { Puppet::Type.type(:append_line).new(:name => 'foo', :path => '/tmp/file') }.should raise_error(Puppet::Error, /Both line and path are required attributes/) - end - it 'should require that a file is specified' do - expect { Puppet::Type.type(:append_line).new(:name => 'foo', :line => 'path') }.should raise_error(Puppet::Error, /Both line and path are required attributes/) - end -end diff --git a/spec/unit/puppet/type/whole_line_spec.rb b/spec/unit/puppet/type/whole_line_spec.rb new file mode 100644 index 0000000..f273baa --- /dev/null +++ b/spec/unit/puppet/type/whole_line_spec.rb @@ -0,0 +1,24 @@ +require 'puppet' +require 'tempfile' +describe Puppet::Type.type(:whole_line) do + before :each do + @whole_line = Puppet::Type.type(:whole_line).new(:name => 'foo', :line => 'line', :path => '/tmp/path') + end + it 'should accept a line and path' do + @whole_line[:line] = 'my_line' + @whole_line[:line].should == 'my_line' + end + it 'should accept posix filenames' do + @whole_line[:path] = '/tmp/path' + @whole_line[:path].should == '/tmp/path' + end + it 'should not accept unqualified path' do + expect { @whole_line[:path] = 'file' }.should raise_error(Puppet::Error, /File paths must be fully qualified/) + end + it 'should require that a line is specified' do + expect { Puppet::Type.type(:whole_line).new(:name => 'foo', :path => '/tmp/file') }.should raise_error(Puppet::Error, /Both line and path are required attributes/) + end + it 'should require that a file is specified' do + expect { Puppet::Type.type(:whole_line).new(:name => 'foo', :line => 'path') }.should raise_error(Puppet::Error, /Both line and path are required attributes/) + end +end diff --git a/tests/append_line.pp b/tests/whole_line.pp index f50a833..1302989 100644 --- a/tests/append_line.pp +++ b/tests/whole_line.pp @@ -1,7 +1,7 @@ file { '/tmp/dansfile': ensure => present }-> -append_line { 'dans_line': +whole_line { 'dans_line': line => 'dan is awesome', - #path => '/tmp/dansfile', + path => '/tmp/dansfile', } |