Merge pull request #937 from rubocop-hq/predicate-matcher-fix-doc-mistake

bquorning · web-flow · commit fad338729759 · 2020-06-18T09:22:03.000+02:00
Fix mistake in RSpec/PredicateMatcher doc
diff --git a/lib/rubocop/cop/rspec/predicate_matcher.rb b/lib/rubocop/cop/rspec/predicate_matcher.rb
@@ -277,12 +277,12 @@ def replacement_matcher(node)
       #   expect(foo).to be_something
       #
       #   # also good - It checks "true" strictly.
-      #   expect(foo).to be(true)
+      #   expect(foo.something?).to be(true)
       #
       # @example Strict: false, EnforcedStyle: inflected
       #   # bad
       #   expect(foo.something?).to be_truthy
-      #   expect(foo).to be(true)
+      #   expect(foo.something?).to be(true)
       #
       #   # good
       #   expect(foo).to be_something
diff --git a/manual/cops_rspec.md b/manual/cops_rspec.md
@@ -2452,14 +2452,14 @@ expect(foo.something?).to be_truthy
 expect(foo).to be_something
 
 # also good - It checks "true" strictly.
-expect(foo).to be(true)
+expect(foo.something?).to be(true)
 ```
 #### Strict: false, EnforcedStyle: inflected
 
 ```ruby
 # bad
 expect(foo.something?).to be_truthy
-expect(foo).to be(true)
+expect(foo.something?).to be(true)
 
 # good
 expect(foo).to be_something

Original file line number	Diff line number	Diff line change
`@@ -277,12 +277,12 @@ def replacement_matcher(node)`
`277`	`277`	`# expect(foo).to be_something`
`278`	`278`	`#`
`279`	`279`	`# # also good - It checks "true" strictly.`
`280`		`- # expect(foo).to be(true)`
	`280`	`+ # expect(foo.something?).to be(true)`
`281`	`281`	`#`
`282`	`282`	`# @example Strict: false, EnforcedStyle: inflected`
`283`	`283`	`# # bad`
`284`	`284`	`# expect(foo.something?).to be_truthy`
`285`		`- # expect(foo).to be(true)`
	`285`	`+ # expect(foo.something?).to be(true)`
`286`	`286`	`#`
`287`	`287`	`# # good`
`288`	`288`	`# expect(foo).to be_something`