Fix #569: incorrect overflow reporting

Author: cowtowncoder

ion/src/test/java/com/fasterxml/jackson/dataformat/ion/IonNumberOverflowTest.java

@@ -3,6 +3,8 @@
 import org.hamcrest.Matchers;
 import org.junit.Test;
 
+import com.fasterxml.jackson.core.JsonParser;
+import com.fasterxml.jackson.core.JsonToken;
 import com.fasterxml.jackson.core.exc.InputCoercionException;
 
 import static org.hamcrest.MatcherAssert.assertThat;
@@ -72,4 +74,15 @@ private void _testLongCoercionFail(BigInteger input) throws Exception
         }
     }
 
+    // [dataformats-binary#569]: incorrect overflow fail for long values (from BigInteger)
+    @Test
+    public void testLongAsBigIntegerSize() throws Exception {
+        // Note: Values: Long.MAX_VALUE through Long.MAX_VALUE -7 are considered LONG by Ion.  
+        BigInteger bigIntLongValue = new BigInteger(Long.MAX_VALUE + "").subtract(new BigInteger("10"));
+        IonParser bigIntLongParser = (IonParser) new IonFactory().createParser(bigIntLongValue.toString());
+        assertEquals(JsonToken.VALUE_NUMBER_INT, bigIntLongParser.nextToken());
+        assertEquals(JsonParser.NumberType.BIG_INTEGER, bigIntLongParser.getNumberType());
+        assertEquals(JsonParser.NumberTypeFP.UNKNOWN, bigIntLongParser.getNumberTypeFP());
+        assertEquals(bigIntLongValue.longValue(), bigIntLongParser.getLongValue());
+    }    
 }

release-notes/VERSION-2.x

@@ -16,6 +16,9 @@ Active maintainers:
 
 2.18.4 (not yet released)
 
+#569: (ion) `IonParser` fails to parse some `long` values saying
+  they are out of range when they are not
+ (reported, fix suggested by @seadbrane)
 - (ion) Upgrade `ion-java` to 1.11.10 (from 1.11.9)
 
 2.18.3 (28-Feb-2025)