diff options
author | Ashley Penney <ashley.penney@puppetlabs.com> | 2014-05-08 13:58:53 -0400 |
---|---|---|
committer | Ashley Penney <ashley.penney@puppetlabs.com> | 2014-05-08 13:58:53 -0400 |
commit | e2297a1ea6b711ca930a21adc56aefe7c602826f (patch) | |
tree | a731fa4c18deac09565cb696b73b29b27c520d98 /spec/functions/merge_spec.rb | |
parent | 17a912ea0afb3fb018170477856c25a95009f2cc (diff) | |
parent | 96e43e69d8496926ad4951534e75b204bb279f22 (diff) | |
download | puppet-stdlib-e2297a1ea6b711ca930a21adc56aefe7c602826f.tar.gz puppet-stdlib-e2297a1ea6b711ca930a21adc56aefe7c602826f.tar.bz2 |
Merge pull request #250 from hunner/move_unit
Move unit tests to spec/functions
Diffstat (limited to 'spec/functions/merge_spec.rb')
-rwxr-xr-x | spec/functions/merge_spec.rb | 52 |
1 files changed, 52 insertions, 0 deletions
diff --git a/spec/functions/merge_spec.rb b/spec/functions/merge_spec.rb new file mode 100755 index 0000000..15a5d94 --- /dev/null +++ b/spec/functions/merge_spec.rb @@ -0,0 +1,52 @@ +#! /usr/bin/env ruby -S rspec + +require 'spec_helper' + +describe Puppet::Parser::Functions.function(:merge) do + let(:scope) { PuppetlabsSpec::PuppetInternals.scope } + + describe 'when calling merge from puppet' do + it "should not compile when no arguments are passed" do + pending("Fails on 2.6.x, see bug #15912") if Puppet.version =~ /^2\.6\./ + 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 + pending("Fails on 2.6.x, see bug #15912") if Puppet.version =~ /^2\.6\./ + 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 + it 'should require all parameters are hashes' do + expect { new_hash = scope.function_merge([{}, '2'])}.to raise_error(Puppet::ParseError, /unexpected argument type String/) + expect { new_hash = scope.function_merge([{}, 2])}.to raise_error(Puppet::ParseError, /unexpected argument type Fixnum/) + end + + it 'should accept empty strings as puppet undef' do + expect { new_hash = scope.function_merge([{}, ''])}.not_to raise_error + end + + it 'should be able to merge two hashes' do + new_hash = scope.function_merge([{'one' => '1', 'two' => '1'}, {'two' => '2', 'three' => '2'}]) + new_hash['one'].should == '1' + new_hash['two'].should == '2' + new_hash['three'].should == '2' + end + + it 'should merge multiple hashes' do + hash = scope.function_merge([{'one' => 1}, {'one' => '2'}, {'one' => '3'}]) + hash['one'].should == '3' + end + + it 'should accept empty hashes' do + scope.function_merge([{},{},{}]).should == {} + end + end +end |