(MODULES-3543) Fix define_with_params to handle undef properly

As described in PUP-6422,

	ensure_resources('File[/tmp/a]', { owner => undef })

would not actually create the file. This fixes it, and adds tests to prove
it.

1 files changed, 7 insertions, 0 deletions

diff --git a/spec/functions/ensure_resource_spec.rb b/spec/functions/ensure_resource_spec.rb
index c4f2cbd..57b5123 100755
--- a/spec/functions/ensure_resource_spec.rb
+++ b/spec/functions/ensure_resource_spec.rb
@@ -28,6 +28,13 @@ describe 'ensure_resource' do
       it { expect(lambda { catalogue }).to contain_user('username2').without_ensure }
     end
 
+    describe 'after running ensure_resource("user", "username1", { "gid" => undef })' do
+      before { subject.call(['User', 'username1', { "gid" => :undef }]) }
+
+      # this lambda is required due to strangeness within rspec-puppet's expectation handling
+      it { expect(lambda { catalogue }).to contain_user('username1').with_ensure('present') }
+    end
+
     describe 'after running ensure_resource("user", ["username1", "username2"], {})' do
       before { subject.call(['User', ['username1', 'username2'], {}]) }