]> gitweb.fluxo.info Git - puppet-stdlib.git/commit
(#11901) Fix sort order error in tests for 'keys' and 'values'
authorJeff McCune <jeff@puppetlabs.com>
Thu, 12 Jan 2012 01:39:15 +0000 (17:39 -0800)
committerJeff McCune <jeff@puppetlabs.com>
Thu, 12 Jan 2012 01:39:15 +0000 (17:39 -0800)
commitb7a59a80954fdfd9923a0cd560374591e24f00b1
tree04b37f8f26ddca2b8791e9994268a56f8195c92e
parentb082094599f4cd26c9cb0c67ae44880b0dc02407
(#11901) Fix sort order error in tests for 'keys' and 'values'

Between Ruby 1.8.7 p352 and p357 the way arrays were returned when using
keys and values in Ruby changed, and due to assumption about the
ordering our tests are now failing.

This patch fixes the issue by using the =~ operator matcher in rspec.
This matcher is implemented as RSpec::Matchers::MatchArray and performs
multiset equality matching of arrays.  Order doesn't matter, but
duplicate values do.

This patch also switches @scope instance variables to memoized let
methods for clarity in the code.

Original Author: Ken Barber
Reviewed-by: Nick Lewis
This commit closes GH-29
spec/unit/puppet/parser/functions/keys_spec.rb
spec/unit/puppet/parser/functions/values_spec.rb