[Bug #19335] `Integer#remainder` should respect `#coerce` (#7120)

Also `Numeric#remainder` should.

1 files changed, 18 insertions, 5 deletions

diff --git a/test/ruby/test_integer_comb.rb b/test/ruby/test_integer_comb.rb
index 1ad13dd31b..150f45cfd7 100644
--- a/test/ruby/test_integer_comb.rb
+++ b/test/ruby/test_integer_comb.rb
@@ -408,19 +408,32 @@ class TestIntegerComb < Test::Unit::TestCase
   end
 
   def test_remainder
+    coerce = EnvUtil.labeled_class("CoerceNum") do
+      def initialize(num)
+        @num = num
+      end
+      def coerce(other)
+        [other, @num]
+      end
+      def inspect
+        "#{self.class.name}(#@num)"
+      end
+      alias to_s inspect
+    end
+
     VS.each {|a|
-      VS.each {|b|
-        if b == 0
+      (VS + VS.map {|b| [coerce.new(b), b]}).each {|b, i = b|
+        if i == 0
           assert_raise(ZeroDivisionError) { a.divmod(b) }
         else
-          r = a.remainder(b)
+          r = assert_nothing_raised(ArgumentError, "#{a}.remainder(#{b})") {a.remainder(b)}
           assert_kind_of(Integer, r)
           if a < 0
-            assert_operator(-b.abs, :<, r, "#{a}.remainder(#{b})")
+            assert_operator(-i.abs, :<, r, "#{a}.remainder(#{b})")
             assert_operator(0, :>=, r, "#{a}.remainder(#{b})")
           elsif 0 < a
             assert_operator(0, :<=, r, "#{a}.remainder(#{b})")
-            assert_operator(b.abs, :>, r, "#{a}.remainder(#{b})")
+            assert_operator(i.abs, :>, r, "#{a}.remainder(#{b})")
           else
             assert_equal(0, r, "#{a}.remainder(#{b})")
           end