Browse files

added Range#count? for Ruby 1.8

Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
  • Loading branch information...
1 parent 289cc15 commit b0da0dd8345b68a4e54c52e398cc65bbc0c4fbb5 @dcrec1 dcrec1 committed with spastorino Feb 19, 2011
View
1 activesupport/lib/active_support/core_ext/range.rb
@@ -2,3 +2,4 @@
require 'active_support/core_ext/range/conversions'
require 'active_support/core_ext/range/include_range'
require 'active_support/core_ext/range/overlaps'
+require 'active_support/core_ext/range/cover'
View
3 activesupport/lib/active_support/core_ext/range/cover.rb
@@ -0,0 +1,3 @@
+class Range
+ alias_method(:cover?, :include?) unless instance_methods.include?(:cover?)
+end
View
12 activesupport/test/core_ext/range_ext_test.rb
@@ -62,4 +62,16 @@ def test_original_step
(1..10).step(2) {|i| array << i }
assert_equal [1,3,5,7,9], array
end
+
+ if RUBY_VERSION < '1.9'
+ def test_cover
+ assert((1..3).cover?(2))
+ assert !(1..3).cover?(4)
+ end
+ else
+ def test_cover_is_not_override
+ range = (1..3)
+ assert range.method(:include?) != range.method(:cover?)
+ end
+ end
end

3 comments on commit b0da0dd

@fxn
Ruby on Rails member

This patch has no documentation.

@fxn
Ruby on Rails member

I have added doc coverage here e2d54fb

Please remember that for your next contribution, provide code, tests, and docs. That's a complete patch.

@tomstuart

Except for the commit message, which is wrong.

Please sign in to comment.