From 873ff536a637f43ddd69058e3e353f8f807b881d Mon Sep 17 00:00:00 2001
From: Takashi Kajinami <tkajinam@redhat.com>
Date: Mon, 30 Jan 2023 13:38:40 +0900
Subject: [PATCH] Replace mocha by rspec-mocks

puppetlabs_spec_helper recommends rspec-mocks instead of mocha[1] and
it uses rspec-mocks by default instead of mocha since v 5.0.0[2]

This is the prep work to adapt to that migration.

[1] https://github.com/puppetlabs/puppetlabs_spec_helper/#mock_with
[2] https://github.com/puppetlabs/puppetlabs_spec_helper/commit/493f0cbc1c96a3fa67591f3936430a70af74847f

Closes-Bug: #2004135
Change-Id: Ifc3ac7c3c755b1ffb14bfd149891b48a2a8b3142
---
 spec/spec_helper.rb               |  2 ++
 spec/unit/provider/ironic_spec.rb | 22 ++++++++++++----------
 2 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
index 33f27e2d..a99919e7 100644
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -13,6 +13,8 @@ RSpec.configure do |c|
 
   c.module_path = File.join(fixture_path, 'modules')
   c.manifest_dir = File.join(fixture_path, 'manifests')
+
+  c.mock_with :rspec
 end
 
 at_exit { RSpec::Puppet::Coverage.report! }
diff --git a/spec/unit/provider/ironic_spec.rb b/spec/unit/provider/ironic_spec.rb
index 41e57010..c27bbc90 100644
--- a/spec/unit/provider/ironic_spec.rb
+++ b/spec/unit/provider/ironic_spec.rb
@@ -32,7 +32,7 @@ describe Puppet::Provider::Ironic do
 
     it 'should fail if config is empty' do
       conf = {}
-      klass.expects(:ironic_conf).returns(conf)
+      expect(klass).to receive(:ironic_conf).and_return(conf)
       expect do
         klass.ironic_credentials
       end.to raise_error(Puppet::Error, credential_error)
@@ -40,7 +40,7 @@ describe Puppet::Provider::Ironic do
 
     it 'should fail if config does not have keystone_authtoken section.' do
       conf = {'foo' => 'bar'}
-      klass.expects(:ironic_conf).returns(conf)
+      expect(klass).to receive(:ironic_conf).and_return(conf)
       expect do
         klass.ironic_credentials
       end.to raise_error(Puppet::Error, credential_error)
@@ -48,7 +48,7 @@ describe Puppet::Provider::Ironic do
 
     it 'should fail if config does not contain all auth params' do
       conf = {'keystone_authtoken' => {'invalid_value' => 'foo'}}
-      klass.expects(:ironic_conf).returns(conf)
+      expect(klass).to receive(:ironic_conf).and_return(conf)
       expect do
        klass.ironic_credentials
       end.to raise_error(Puppet::Error, credential_error)
@@ -67,18 +67,20 @@ describe Puppet::Provider::Ironic do
         :OS_PROJECT_DOMAIN_NAME => credential_hash['project_domain_name'],
         :OS_USER_DOMAIN_NAME    => credential_hash['user_domain_name'],
       }
-      klass.expects(:get_ironic_credentials).with().returns(credential_hash)
-      klass.expects(:withenv).with(authenv)
+      expect(klass).to receive(:get_ironic_credentials).with(no_args).and_return(credential_hash)
+      expect(klass).to receive(:withenv).with(authenv)
       klass.auth_ironic('test_retries')
     end
 
     ['[Errno 111] Connection refused',
      '(HTTP 400)'].reverse.each do |valid_message|
       it "should retry when ironic cli returns with error #{valid_message}" do
-        klass.expects(:get_ironic_credentials).with().returns({})
-        klass.expects(:sleep).with(10).returns(nil)
-        klass.expects(:ironic).twice.with(['test_retries']).raises(
-          Exception, valid_message).then.returns('')
+        expect(klass).to receive(:get_ironic_credentials).with(no_args).and_return({})
+        expect(klass).to receive(:sleep).with(10).and_return(nil)
+        expect(klass).to receive(:ironic).with(['test_retries']).and_invoke(
+          lambda { |x| raise valid_message },
+          lambda { |x| return '' }
+        )
         klass.auth_ironic('test_retries')
       end
     end
@@ -93,7 +95,7 @@ describe Puppet::Provider::Ironic do
         net1
         net2
       EOT
-      klass.expects(:auth_ironic).returns(output)
+      expect(klass).to receive(:auth_ironic).and_return(output)
       result = klass.list_ironic_resources('foo')
       expect(result).to eql(['net1', 'net2'])
     end