diff options
author | Jeff McCune <jeff@puppetlabs.com> | 2012-01-11 17:48:57 -0800 |
---|---|---|
committer | Jeff McCune <jeff@puppetlabs.com> | 2012-01-11 17:48:57 -0800 |
commit | 5273a891b7a4c8065f491ecb9435b64e5bc89245 (patch) | |
tree | da5c20ae17a029456295c9d18861cf8ac86f2a5b /spec/unit/puppet/parser/functions/values_spec.rb | |
parent | c4b836e322deab90f4cf27e8efe4f1a1c91cf235 (diff) | |
parent | dc674419ae2e59e28157358b1a7329401ece12ee (diff) | |
download | puppet-stdlib-5273a891b7a4c8065f491ecb9435b64e5bc89245.tar.gz puppet-stdlib-5273a891b7a4c8065f491ecb9435b64e5bc89245.tar.bz2 |
Merge branch '2.2.x'
* 2.2.x:
(#11901) Fix sort order error in tests for 'keys' and 'values'
Diffstat (limited to 'spec/unit/puppet/parser/functions/values_spec.rb')
-rw-r--r-- | spec/unit/puppet/parser/functions/values_spec.rb | 21 |
1 files changed, 13 insertions, 8 deletions
diff --git a/spec/unit/puppet/parser/functions/values_spec.rb b/spec/unit/puppet/parser/functions/values_spec.rb index f6eb5b6..a5fbbd8 100644 --- a/spec/unit/puppet/parser/functions/values_spec.rb +++ b/spec/unit/puppet/parser/functions/values_spec.rb @@ -6,25 +6,30 @@ describe "the values function" do Puppet::Parser::Functions.autoloader.loadall end - before :each do - @scope = Puppet::Parser::Scope.new - end + let(:scope) { Puppet::Parser::Scope.new } it "should exist" do Puppet::Parser::Functions.function("values").should == "function_values" end it "should raise a ParseError if there is less than 1 arguments" do - lambda { @scope.function_values([]) }.should( raise_error(Puppet::ParseError)) + lambda { scope.function_values([]) }.should( raise_error(Puppet::ParseError)) end it "should return values from a hash" do - result = @scope.function_values([{'a'=>'1','b'=>'2','c'=>'3'}]) - result.should(eq(['1','2','3'])) + result = scope.function_values([{'a'=>'1','b'=>'2','c'=>'3'}]) + # =~ is the RSpec::Matchers::MatchArray matcher. + # A.K.A. "array with same elements" (multiset) matching + result.should =~ %w{ 1 2 3 } end - it "should return values from a hash" do - lambda { @scope.function_values([['a','b','c']]) }.should( raise_error(Puppet::ParseError)) + it "should return a multiset" do + result = scope.function_values([{'a'=>'1','b'=>'3','c'=>'3'}]) + result.should =~ %w{ 1 3 3 } + result.should_not =~ %w{ 1 3 } end + it "should raise a ParseError unless a Hash is provided" do + lambda { scope.function_values([['a','b','c']]) }.should( raise_error(Puppet::ParseError)) + end end |