Clean up formatting and language on allow(O.any_instance) code.

rspec · Oct 16, 2013 · 6556ed2 · 6556ed2
1 parent cc5312c
commit 6556ed2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/lib/rspec/mocks/matchers/receive.rb b/lib/rspec/mocks/matchers/receive.rb
@@ -67,7 +67,7 @@ def warn_if_any_instance(expression, subject)
           if AnyInstance::Recorder === subject
             RSpec.warning(
               "`#{expression}(#{subject.klass}.any_instance).to` " <<
-              "is probably not what you meant, it does not #{expression} on " <<
+              "is probably not what you meant, it does not operate on " <<
               "any instance of `#{subject.klass}`. " <<
               "Use `#{expression}_any_instance_of(#{subject.klass}).to` instead."
             )

diff --git a/spec/rspec/mocks/matchers/receive_spec.rb b/spec/rspec/mocks/matchers/receive_spec.rb
@@ -11,15 +11,14 @@ def verify_all
 
       describe "expectations/allowances on any instance recorders" do
         include_context "with syntax", [:expect, :should]
-        include DeprecationHelpers
 
         it "warns about allow(Klass.any_instance).to receive..." do
           expect(RSpec).to receive(:warning).with(/allow.*any_instance.*is probably not what you meant.*allow_any_instance_of.*instead/)
           allow(Object.any_instance).to receive(:foo)
         end
 
         it "includes the correct call site in the allow warning" do
-          expect_warning_with_call_site(__FILE__, __LINE__+1)
+          expect_warning_with_call_site(__FILE__, __LINE__ + 1)
           allow(Object.any_instance).to receive(:foo)
         end
 
@@ -30,7 +29,7 @@ def verify_all
         end
 
         it "includes the correct call site in the expect warning" do
-          expect_warning_with_call_site(__FILE__, __LINE__+1)
+          expect_warning_with_call_site(__FILE__, __LINE__ + 1)
           expect(Object.any_instance).to receive(:foo)
           Object.any_instance.foo
         end