diff options
author | Jeff McCune <jeff@puppetlabs.com> | 2012-05-10 22:00:19 -0700 |
---|---|---|
committer | Jeff McCune <jeff@puppetlabs.com> | 2012-05-10 22:04:58 -0700 |
commit | 20aacc5a29f767f09ecf5964068392cf3054bddc (patch) | |
tree | ed4033182a2cce8838a9ba6ae5bb56eaaee43da2 /spec/unit | |
parent | 1ebfa5daea65a7ccb5e17c1d1021567c91ed580b (diff) | |
download | puppet-stdlib-20aacc5a29f767f09ecf5964068392cf3054bddc.tar.gz puppet-stdlib-20aacc5a29f767f09ecf5964068392cf3054bddc.tar.bz2 |
Memoize file_line spec instance variables
This just changes the instance variables to a memoized let block and
gets ride of the before :each block.
The patch has no change in behavior.
Diffstat (limited to 'spec/unit')
-rw-r--r-- | spec/unit/puppet/type/file_line_spec.rb | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/spec/unit/puppet/type/file_line_spec.rb b/spec/unit/puppet/type/file_line_spec.rb index 7e07c06..a3e13ea 100644 --- a/spec/unit/puppet/type/file_line_spec.rb +++ b/spec/unit/puppet/type/file_line_spec.rb @@ -1,19 +1,19 @@ require 'puppet' require 'tempfile' describe Puppet::Type.type(:file_line) do - before :each do - @file_line = Puppet::Type.type(:file_line).new(:name => 'foo', :line => 'line', :path => '/tmp/path') + let :file_line do + Puppet::Type.type(:file_line).new(:name => 'foo', :line => 'line', :path => '/tmp/path') end it 'should accept a line and path' do - @file_line[:line] = 'my_line' - @file_line[:line].should == 'my_line' + file_line[:line] = 'my_line' + file_line[:line].should == 'my_line' end it 'should accept posix filenames' do - @file_line[:path] = '/tmp/path' - @file_line[:path].should == '/tmp/path' + file_line[:path] = '/tmp/path' + file_line[:path].should == '/tmp/path' end it 'should not accept unqualified path' do - expect { @file_line[:path] = 'file' }.should raise_error(Puppet::Error, /File paths must be fully qualified/) + expect { file_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(:file_line).new(:name => 'foo', :path => '/tmp/file') }.should raise_error(Puppet::Error, /Both line and path are required attributes/) |