]> gitweb.fluxo.info Git - puppet-stdlib.git/commit
enhanced the error message of pick function.
authorLeonardo Rodrigues de Mello <l@lmello.eu.org>
Fri, 6 Sep 2013 13:53:47 +0000 (10:53 -0300)
committerLeonardo Rodrigues de Mello <l@lmello.eu.org>
Mon, 16 Sep 2013 13:03:20 +0000 (10:03 -0300)
commit30e994fb6eb91e25a77e60a849f795d60c11af4b
tree3ef170c125734efa71decc06ab54e34efee75e45
parent806430224ad0da860be3761ab83f1e574b64fc60
enhanced the error message of pick function.

When pick function fail return a better error message like
the other stdlib functions, indicating that the error
is on function pick.

This would help people that see the error to identity it is
related to a incorrect use of stdlib function pick, instead of having
to grep all puppet libraries and manifests source for the old message.

I had also changed the spec test.

pick function change spec  as suggested GH-179

Fix the spec test to use expect {}.to instead of lambda {}.should
as explained by Adrienthebo.

"Using expect { }.to is preferred over lambda { }.should.
 In addition it's best practice to do a string match against the
error message to ensure that we're catching the right error,
instead of any error of the right type."

Also fixed a typo on the error message, it was missing one space.

pick function stylish fix as suggested on GH179
lib/puppet/parser/functions/pick.rb
spec/unit/puppet/parser/functions/pick_spec.rb