Merge pull request #1546 from ydah/fix/1518

Fix a false positive for `RSpec/StubbedMock` when stubbed message expectation with a block and block parameter

CHANGELOG.md

@@ -7,6 +7,7 @@
 - Add new `RSpec/Rails/MinitestAssertions` cop. ([@ydah])
 - Fix a false positive for `RSpec/PendingWithoutReason` when not inside example. ([@ydah])
 - Fix a false negative for `RSpec/PredicateMatcher` when using `include` and `respond_to`. ([@ydah])
+- Fix a false positive for `RSpec/StubbedMock` when stubbed message expectation with a block and block parameter. ([@ydah])
 
 ## 2.16.0 (2022-12-13)
 

lib/rubocop/cop/rspec/stubbed_mock.rb

@@ -91,7 +91,7 @@ class StubbedMock < Base
         #   @param node [RuboCop::AST::Node]
         #   @yield [RuboCop::AST::Node] matcher
         def_node_matcher :matcher_with_return_block, <<~PATTERN
-          (block #message_expectation? args _)  # receive(:foo) { 'bar' }
+          (block #message_expectation? (args) _)  # receive(:foo) { 'bar' }
         PATTERN
 
         # @!method matcher_with_hash(node)

spec/rubocop/cop/rspec/stubbed_mock_spec.rb

@@ -15,6 +15,12 @@
     RUBY
   end
 
+  it 'ignores stubbed message expectation with a block and block parameter' do
+    expect_no_offenses(<<-RUBY)
+      expect(foo).to receive(:bar) { |x| bar }
+    RUBY
+  end
+
   it 'flags stubbed message expectation with argument matching' do
     expect_offense(<<-RUBY)
       expect(foo).to receive(:bar).with(42).and_return('hello world')