Better explanation for MessageSpies

kuboon · web-flow · commit dc4cd28454ef · 2022-06-03T11:51:47.000+09:00
diff --git a/lib/rubocop/cop/rspec/message_spies.rb b/lib/rubocop/cop/rspec/message_spies.rb
@@ -12,17 +12,22 @@ module RSpec
       #
       #   # bad
       #   expect(foo).to receive(:bar)
+      #   subject
       #
       #   # good
+      #   allow(foo).to receive(:bar) # or use instance_spy
+      #   subject
       #   expect(foo).to have_received(:bar)
       #
       # @example `EnforcedStyle: receive`
       #
       #   # bad
+      #   subject
       #   expect(foo).to have_received(:bar)
       #
       #   # good
       #   expect(foo).to receive(:bar)
+      #   subject
       #
       class MessageSpies < Base
         include ConfigurableEnforcedStyle

Original file line number	Diff line number	Diff line change
`@@ -12,17 +12,22 @@ module RSpec`
`12`	`12`	`#`
`13`	`13`	`# # bad`
`14`	`14`	`# expect(foo).to receive(:bar)`
	`15`	`+ # subject`
`15`	`16`	`#`
`16`	`17`	`# # good`
	`18`	`+ # allow(foo).to receive(:bar) # or use instance_spy`
	`19`	`+ # subject`
`17`	`20`	`# expect(foo).to have_received(:bar)`
`18`	`21`	`#`
`19`	`22`	# @example `EnforcedStyle: receive`
`20`	`23`	`#`
`21`	`24`	`# # bad`
	`25`	`+ # subject`
`22`	`26`	`# expect(foo).to have_received(:bar)`
`23`	`27`	`#`
`24`	`28`	`# # good`
`25`	`29`	`# expect(foo).to receive(:bar)`
	`30`	`+ # subject`
`26`	`31`	`#`
`27`	`32`	`class MessageSpies < Base`
`28`	`33`	`include ConfigurableEnforcedStyle`