diff options
author | Patrick Carlisle <patrick@puppetlabs.com> | 2012-08-09 14:18:30 -0700 |
---|---|---|
committer | Patrick Carlisle <patrick@puppetlabs.com> | 2012-08-09 14:49:51 -0700 |
commit | 09322ad94398d69081aae44cc770830ac2144b75 (patch) | |
tree | 29b63f3910e4d567d21dc1c5e9f208b0d78155be /spec/unit/puppet/parser/functions/merge_spec.rb | |
parent | a1bab586ba6fcf2b3035898d1002d0a6ef89d7c8 (diff) | |
download | puppet-stdlib-09322ad94398d69081aae44cc770830ac2144b75.tar.gz puppet-stdlib-09322ad94398d69081aae44cc770830ac2144b75.tar.bz2 |
Use rvalue functions correctly
A bug fix in Puppet exposed that several tests were using rvalue functions
incorrectly (this was not properly checked by puppet before). This fixes those
tests.
Diffstat (limited to 'spec/unit/puppet/parser/functions/merge_spec.rb')
-rw-r--r-- | spec/unit/puppet/parser/functions/merge_spec.rb | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/spec/unit/puppet/parser/functions/merge_spec.rb b/spec/unit/puppet/parser/functions/merge_spec.rb index 9e8a619..192da4c 100644 --- a/spec/unit/puppet/parser/functions/merge_spec.rb +++ b/spec/unit/puppet/parser/functions/merge_spec.rb @@ -7,12 +7,16 @@ describe Puppet::Parser::Functions.function(:merge) do describe 'when calling merge from puppet' do it "should not compile when no arguments are passed" do - Puppet[:code] = 'merge()' - expect { scope.compiler.compile }.should raise_error(Puppet::ParseError, /wrong number of arguments/) + Puppet[:code] = '$x = merge()' + expect { + scope.compiler.compile + }.to raise_error(Puppet::ParseError, /wrong number of arguments/) end it "should not compile when 1 argument is passed" do - Puppet[:code] = "$my_hash={'one' => 1}\nmerge($my_hash)" - expect { scope.compiler.compile }.should raise_error(Puppet::ParseError, /wrong number of arguments/) + Puppet[:code] = "$my_hash={'one' => 1}\n$x = merge($my_hash)" + expect { + scope.compiler.compile + }.to raise_error(Puppet::ParseError, /wrong number of arguments/) end end describe 'when calling merge on the scope instance' do |