it's cover? (with a ?)

rspec · Jan 12, 2012 · 95bf234 · 95bf234
1 parent 19c64a5
commit 95bf234
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/lib/rspec/matchers.rb b/lib/rspec/matchers.rb
@@ -353,7 +353,7 @@ def change(receiver=nil, message=nil, &block)
     # ### Warning:: Ruby >= 1.9 only
     def cover(*values)
       BuiltIn::Cover.new(*values)
-    end if (1..2).respond_to?(:cover)
+    end if (1..2).respond_to?(:cover?)
 
     # Passes if <tt>actual == expected</tt>.
     #

diff --git a/lib/rspec/matchers/built_in.rb b/lib/rspec/matchers/built_in.rb
@@ -12,7 +12,7 @@ module BuiltIn
       autoload :BeAKindOf,      'rspec/matchers/built_in/be_kind_of'
       autoload :BeWithin,       'rspec/matchers/built_in/be_within'
       autoload :Change,         'rspec/matchers/built_in/change'
-      autoload :Cover,          'rspec/matchers/built_in/cover' if defined?("[].cover?")
+      autoload :Cover,          'rspec/matchers/built_in/cover' if (1..2).respond_to?(:cover?)
       autoload :Eq,             'rspec/matchers/built_in/eq'
       autoload :Eql,            'rspec/matchers/built_in/eql'
       autoload :Equal,          'rspec/matchers/built_in/equal'

diff --git a/spec/rspec/matchers/cover_spec.rb b/spec/rspec/matchers/cover_spec.rb
@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-if (1..2).respond_to?(:cover)
+if (1..2).respond_to?(:cover?)
   describe "should cover(expected)" do
     context "for a range target" do
       it "passes if target covers expected" do