Update puppetlabs/stdlib module

[mirror/dsa-puppet.git] / 3rdparty / modules / stdlib / spec / functions / private_spec.rb

diff --git a/3rdparty/modules/stdlib/spec/functions/private_spec.rb b/3rdparty/modules/stdlib/spec/functions/private_spec.rb
index a13be64..cfd78c4 100644 (file)
--- a/3rdparty/modules/stdlib/spec/functions/private_spec.rb
+++ b/3rdparty/modules/stdlib/spec/functions/private_spec.rb
@@ -1,56 +1,52 @@
 require 'spec_helper'
 
 describe 'private' do
-  it 'should issue a warning' do
-    scope.expects(:warning).with("private() DEPRECATED: This function will cease to function on Puppet 4; please use assert_private() before upgrading to puppet 4 for backwards-compatibility, or migrate to the new parser's typing system.")
+  it 'issues a warning' do
+    expect(scope).to receive(:warning).with("private() DEPRECATED: This function will cease to function on Puppet 4; please use assert_private() before upgrading to puppet 4 for backwards-compatibility, or migrate to the new parser's typing system.") # rubocop:disable Metrics/LineLength : unable to cut line to required length
     begin
-      subject.call []
-    rescue
+      subject.execute
+    rescue # rubocop:disable Lint/HandleExceptions
       # ignore this
     end
   end
 
-  context "when called from inside module" do
-    it "should not fail" do
-      scope.expects(:lookupvar).with('module_name').returns('foo')
-      scope.expects(:lookupvar).with('caller_module_name').returns('foo')
+  context 'when called from inside module' do
+    it 'does not fail' do
+      expect(scope).to receive(:lookupvar).with('module_name').and_return('foo')
+      expect(scope).to receive(:lookupvar).with('caller_module_name').and_return('foo')
       expect {
-        subject.call []
+        subject.execute
       }.not_to raise_error
     end
   end
 
-  context "with an explicit failure message" do
-    it "prints the failure message on error" do
-      scope.expects(:lookupvar).with('module_name').returns('foo')
-      scope.expects(:lookupvar).with('caller_module_name').returns('bar')
+  context 'with an explicit failure message' do
+    it 'prints the failure message on error' do
+      expect(scope).to receive(:lookupvar).with('module_name').and_return('foo')
+      expect(scope).to receive(:lookupvar).with('caller_module_name').and_return('bar')
       expect {
-        subject.call ['failure message!']
-      }.to raise_error Puppet::ParseError, /failure message!/
+        subject.execute('failure message!')
+      }.to raise_error(Puppet::ParseError, %r{failure message!})
     end
   end
 
-  context "when called from private class" do
-    it "should fail with a class error message" do
-      scope.expects(:lookupvar).with('module_name').returns('foo')
-      scope.expects(:lookupvar).with('caller_module_name').returns('bar')
-      scope.source.expects(:name).returns('foo::baz')
-      scope.source.expects(:type).returns('hostclass')
-      expect {
-        subject.call []
-      }.to raise_error Puppet::ParseError, /Class foo::baz is private/
+  context 'when called from private class' do
+    it 'fails with a class error message' do
+      expect(scope).to receive(:lookupvar).with('module_name').and_return('foo')
+      expect(scope).to receive(:lookupvar).with('caller_module_name').and_return('bar')
+      expect(scope.source).to receive(:name).and_return('foo::baz')
+      expect(scope.source).to receive(:type).and_return('hostclass')
+      expect { subject.execute }.to raise_error(Puppet::ParseError, %r{Class foo::baz is private})
     end
   end
 
-  context "when called from private definition" do
-    it "should fail with a class error message" do
-      scope.expects(:lookupvar).with('module_name').returns('foo')
-      scope.expects(:lookupvar).with('caller_module_name').returns('bar')
-      scope.source.expects(:name).returns('foo::baz')
-      scope.source.expects(:type).returns('definition')
-      expect {
-        subject.call []
-      }.to raise_error Puppet::ParseError, /Definition foo::baz is private/
+  context 'when called from private definition' do
+    it 'fails with a class error message' do
+      expect(scope).to receive(:lookupvar).with('module_name').and_return('foo')
+      expect(scope).to receive(:lookupvar).with('caller_module_name').and_return('bar')
+      expect(scope.source).to receive(:name).and_return('foo::baz')
+      expect(scope.source).to receive(:type).and_return('definition')
+      expect { subject.execute }.to raise_error(Puppet::ParseError, %r{Definition foo::baz is private})
     end
   end
 end