Merge pull request #179 from lmello/function_pick_error_msg

enhanced the error message of pick function.

Author: adrienthebo

lib/puppet/parser/functions/pick.rb

@@ -21,7 +21,7 @@ module Puppet::Parser::Functions
    args.delete(:undefined)
    args.delete("")
    if args[0].to_s.empty? then
-     fail "Must provide non empty value."
+     fail Puppet::ParseError, "pick(): must receive at last one non empty value"
    else
      return args[0]
    end

spec/unit/puppet/parser/functions/pick_spec.rb

@@ -29,6 +29,6 @@
   end
 
   it 'should error if no values are passed' do
-    expect { scope.function_pick([]) }.to raise_error(Puppet::Error, /Must provide non empty value./)
+    expect { scope.function_pick([]) }.to( raise_error(Puppet::ParseError, "pick(): must receive at last one non empty value"))
   end
 end