Skip to content

Commit

Permalink
Highlight bug: eql? is not aliased to the overridden ==.
Browse files Browse the repository at this point in the history
~~~
IPAddress.parse('::1') == IPAddress.parse('::1')   #=> true
IPAddress.parse('::1').eql? IPAddress.parse('::1') #=> false
~~~
  • Loading branch information
smortex committed Mar 19, 2016
1 parent 064502c commit c983123
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
2 changes: 2 additions & 0 deletions test/ipaddress/ipv4_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -361,8 +361,10 @@ def test_method_compare
assert_equal false, ip3 < ip1
# ip1 should be equal to itself
assert_equal true, ip1 == ip1
assert_equal true, ip1.eql?(ip1)
# ip1 should be equal to ip4
assert_equal true, ip1 == ip4
assert_equal true, ip1.eql?(ip4)
# test sorting
arr = ["10.1.1.1/8","10.1.1.1/16","172.16.1.1/14"]
assert_equal arr, [ip1,ip2,ip3].sort.map{|s| s.to_string}
Expand Down
5 changes: 5 additions & 0 deletions test/ipaddress/ipv6_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,11 @@ def test_method_compare
assert_equal false, ip3 < ip1
# ip1 should be equal to itself
assert_equal true, ip1 == ip1
assert_equal true, ip1.eql?(ip1)
# ip1 should be equal to a copy of itself
other = ip1.dup
assert_equal true, ip1 == other
assert_equal true, ip1.eql?(other)
# ip4 should be greater than ip1
assert_equal true, ip1 < ip4
assert_equal false, ip1 > ip4
Expand Down

0 comments on commit c983123

Please sign in to comment.