Skip to content
Browse files

Merge pull request #4229 from lest/range-include

refactor Range#include? to handle ranges with floats
  • Loading branch information...
2 parents e43b2b3 + 952e9d9 commit edfe7d80135465858bda841b0bc5991f14f35295 @fxn fxn committed Dec 30, 2011
View
3 activesupport/lib/active_support/core_ext/range/include_range.rb
@@ -9,7 +9,8 @@ class Range
# (5..9).include?(11) # => false
def include_with_range?(value)
if value.is_a?(::Range)
- min <= value.min && max >= value.max
+ operator = exclude_end? && !value.exclude_end? ? :< : :<=
+ include_without_range?(value.first) && value.last.send(operator, last)
else
include_without_range?(value)
end
View
4 activesupport/test/core_ext/range_ext_test.rb
@@ -53,6 +53,10 @@ def test_should_not_include_overlapping_last
assert !(2..8).include?(5..9)
end
+ def test_should_include_identical_exclusive_with_floats
+ assert (1.0...10.0).include?(1.0...10.0)
+ end
+
def test_blockless_step
assert_equal [1,3,5,7,9], (1..10).step(2)
end

0 comments on commit edfe7d8

Please sign in to comment.
Something went wrong with that request. Please try again.