Merge remote-tracking branch 'origen/jruby-9.4'

headius · headius · commit 554704dc3855 · 2025-07-09T14:06:56.000-05:00
diff --git a/core/src/main/java/org/jruby/RubyArray.java b/core/src/main/java/org/jruby/RubyArray.java
@@ -1259,16 +1259,25 @@ public IRubyObject insert(IRubyObject[] args) {
 
     @JRubyMethod(name = "insert", required = 1, rest = true, checkArity = false)
     public IRubyObject insert(ThreadContext context, IRubyObject[] args) {
+        switch (args.length) {
+            case 0:
+                return insert();
+            case 1:
+                return insert(args[0]);
+            case 2:
+                return insert(args[0], args[1]);
+        }
+
         int argc = Arity.checkArgumentCount(context, args, 1, -1);
 
         modifyCheck(context);
 
+        long pos = RubyNumeric.num2long(args[0]);
+
         if (argc == 1) return this;
 
         unpack(context);
 
-        long pos = toLong(context, args[0]);
-
         if (pos == -1) pos = realLength;
         if (pos < 0) pos++;
 
diff --git a/spec/ruby/core/array/insert_spec.rb b/spec/ruby/core/array/insert_spec.rb
@@ -57,6 +57,11 @@
     [].insert(-2).should == []
   end
 
+  # https://bugs.ruby-lang.org/issues/13558
+  it "attempts to coerce a lone position argument" do
+    lambda { [].insert(Object.new) }.should raise_error(TypeError)
+  end
+
   it "tries to convert the passed position argument to an Integer using #to_int" do
     obj = mock('2')
     obj.should_receive(:to_int).and_return(2)
