Merge pull request #411 from rspec/allow-mutex-mocks

Allow mocking Mutex.new

Author: JonRowe

lib/rspec/support/reentrant_mutex.rb

@@ -43,7 +43,15 @@ def exit
 
     if defined? ::Mutex
       # On 1.9 and up, this is in core, so we just use the real one
-      Mutex = ::Mutex
+      class Mutex < ::Mutex
+        # If you mock Mutex.new you break our usage of Mutex, so
+        # instead we capture the original method to return Mutexs.
+        NEW_MUTEX_METHOD = Mutex.method(:new)
+
+        def self.new
+          NEW_MUTEX_METHOD.call
+        end
+      end
     else # For 1.8.7
       # :nocov:
       RSpec::Support.require_rspec_support "mutex"

spec/rspec/support/mutex_spec.rb

@@ -0,0 +1,8 @@
+require 'rspec/support/mutex'
+
+RSpec.describe RSpec::Support::Mutex do
+  it "allows ::Mutex to be mocked", :if => defined?(::Mutex) do
+    expect(Mutex).to receive(:new)
+    ::Mutex.new
+  end
+end