opuppet/puppetlabs-stdlib
4
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Memoize file_line spec instance variables

Browse source 
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.
This commit is contained in:
Jeff McCune 2012-05-10 22:00:19 -07:00
parent 1ebfa5daea
commit 20aacc5a29
1 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
spec/unit/puppet/type/file_line_spec.rb
View file

@ -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/)