summaryrefslogtreecommitdiff
path: root/spec/unit/puppet
diff options
context:
space:
mode:
authorJeff McCune <jeff@puppetlabs.com>2012-05-10 22:00:19 -0700
committerJeff McCune <jeff@puppetlabs.com>2012-05-10 22:04:58 -0700
commit20aacc5a29f767f09ecf5964068392cf3054bddc (patch)
treeed4033182a2cce8838a9ba6ae5bb56eaaee43da2 /spec/unit/puppet
parent1ebfa5daea65a7ccb5e17c1d1021567c91ed580b (diff)
downloadpuppet-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/puppet')
-rw-r--r--spec/unit/puppet/type/file_line_spec.rb14
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/)