From 3f86e3a731b9e1be943a55fa12bd5e32716a20ef Mon Sep 17 00:00:00 2001
From: David Schmitt <david@black.co.at>
Date: Thu, 30 Jun 2016 11:06:56 +0100
Subject: (MODULES-3543) Fixup defined_with_params to work on all puppet
 versions

---
 spec/functions/defined_with_params_spec.rb |  4 ++--
 spec/functions/ensure_resource_spec.rb     | 30 ++++++++++++++++++++++++++++--
 2 files changed, 30 insertions(+), 4 deletions(-)

(limited to 'spec/functions')

diff --git a/spec/functions/defined_with_params_spec.rb b/spec/functions/defined_with_params_spec.rb
index e00c423..e2f3abe 100755
--- a/spec/functions/defined_with_params_spec.rb
+++ b/spec/functions/defined_with_params_spec.rb
@@ -26,10 +26,10 @@ describe 'defined_with_params' do
 
   describe 'when passing undef values' do
     let :pre_condition do
-      'file { "/tmp/a": }'
+      'file { "/tmp/a": ensure => present }'
     end
 
     it { is_expected.to run.with_params('File[/tmp/a]', {}).and_return(true) }
-    it { is_expected.to run.with_params('File[/tmp/a]', { 'owner' => :undef }).and_return(true) }
+    it { is_expected.to run.with_params('File[/tmp/a]', { 'ensure' => 'present', 'owner' => :undef }).and_return(true) }
   end
 end
diff --git a/spec/functions/ensure_resource_spec.rb b/spec/functions/ensure_resource_spec.rb
index 57b5123..9a17f5b 100755
--- a/spec/functions/ensure_resource_spec.rb
+++ b/spec/functions/ensure_resource_spec.rb
@@ -10,6 +10,32 @@ describe 'ensure_resource' do
     is_expected.to run.with_params(1,2,3).and_raise_error(Puppet::ParseError)
   }
 
+  context 'given an empty catalog' do
+    describe 'after running ensure_resource("user", "username1", {})' do
+      before { subject.call(['User', 'username1', {}]) }
+
+      # this lambda is required due to strangeness within rspec-puppet's expectation handling
+      it { expect(lambda { catalogue }).to contain_user('username1').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').without_ensure }
+      it { expect(lambda { catalogue }).to contain_user('username1').without_gid }
+    end
+
+    describe 'after running ensure_resource("user", "username1", { ensure => present, gid => undef })' do
+      before { subject.call(['User', 'username1', { 'ensure' => 'present', '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') }
+      it { expect(lambda { catalogue }).to contain_user('username1').without_gid }
+    end
+
+  end
+
   context 'given a catalog with "user { username1: ensure => present }"' do
     let(:pre_condition) { 'user { username1: ensure => present }' }
 
@@ -28,8 +54,8 @@ 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 }]) }
+    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') }
-- 
cgit v1.2.3