Merge pull request #848 from schwern/docs/message-chain-cop

Darhazer · web-flow · commit 28131fe92841 · 2019-12-13T16:10:02.000+02:00
Fix RSpec/MessageChain "better" docs.
diff --git a/lib/rubocop/cop/rspec/message_chain.rb b/lib/rubocop/cop/rspec/message_chain.rb
@@ -11,7 +11,7 @@ module RSpec
       #
       #   # better
       #   thing = Thing.new(baz: 42)
-      #   allow(foo).to receive(bar: thing)
+      #   allow(foo).to receive(:bar).and_return(thing)
       #
       class MessageChain < Cop
         MSG = 'Avoid stubbing using `%<method>s`.'
diff --git a/manual/cops_rspec.md b/manual/cops_rspec.md
@@ -1846,7 +1846,7 @@ allow(foo).to receive_message_chain(:bar, :baz).and_return(42)
 
 # better
 thing = Thing.new(baz: 42)
-allow(foo).to receive(bar: thing)
+allow(foo).to receive(:bar).and_return(thing)
 ```
 
 ### References

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ module RSpec`
`11`	`11`	`#`
`12`	`12`	`# # better`
`13`	`13`	`# thing = Thing.new(baz: 42)`
`14`		`- # allow(foo).to receive(bar: thing)`
	`14`	`+ # allow(foo).to receive(:bar).and_return(thing)`
`15`	`15`	`#`
`16`	`16`	`class MessageChain < Cop`
`17`	`17`	MSG = 'Avoid stubbing using `%<method>s`.'