diff options
author | Dan Bode <dan@puppetlabs.com> | 2011-07-27 15:28:44 -0700 |
---|---|---|
committer | Dan Bode <dan@puppetlabs.com> | 2011-07-27 15:28:44 -0700 |
commit | 4540949d96608c6ff7eed07d53294337446fbead (patch) | |
tree | d56d4f7d2723f2b83fed621fbc097aa5a552ed33 /spec/unit/puppet/type/whole_line_spec.rb | |
parent | 9560b6336cc983b65aaff909f4fb2e983dfd7012 (diff) | |
download | puppet-stdlib-4540949d96608c6ff7eed07d53294337446fbead.tar.gz puppet-stdlib-4540949d96608c6ff7eed07d53294337446fbead.tar.bz2 |
(#8665) Change type from append_line to whole_line
Changed the type name from append_line to
whole_line.
After feedback that having a type with a verb in
the name was confusing.
Diffstat (limited to 'spec/unit/puppet/type/whole_line_spec.rb')
-rw-r--r-- | spec/unit/puppet/type/whole_line_spec.rb | 24 |
1 files changed, 24 insertions, 0 deletions
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 |