Fix #5319

release-notes/VERSION

@@ -5,6 +5,11 @@ Versions: 3.x (for earlier see VERSION-2.x)
 === Releases ===
 ------------------------------------------------------------------------
 
+Not yet released
+
+#5319: `TreeTraversingParser` does not respect `DeserializationFeature.ACCEPT_FLOAT_AS_INT`
+ (reported by @Artur)
+
 3.0.0-rc10 (19-Sep-2025)
 
 #5272: Replace `ObjectMapper.getRegisteredModuleIds()` (2.x) with

src/main/java/tools/jackson/databind/node/BigIntegerNode.java

@@ -306,10 +306,20 @@ public Optional<BigDecimal> decimalValueOpt() {
      */
 
     @Override
-    protected int _asIntValueUnchecked() {
+    public short _asShortValueUnchecked() {
+        return _value.shortValue();
+    }
+
+    @Override
+    public int _asIntValueUnchecked() {
         return _value.intValue();
     }
 
+    @Override
+    public long _asLongValueUnchecked() {
+        return _value.longValue();
+    }
+
     @Override
     protected float _asFloatValueUnchecked() {
         return _value.floatValue();

src/main/java/tools/jackson/databind/node/DecimalNode.java

@@ -206,17 +206,17 @@ public OptionalDouble asDoubleOpt() {
      */
 
     @Override
-    protected short _asShortValueUnchecked() {
+    public short _asShortValueUnchecked() {
         return _value.shortValue();
     }
 
     @Override
-    protected int _asIntValueUnchecked() {
+    public int _asIntValueUnchecked() {
         return _value.intValue();
     }
 
     @Override
-    protected long _asLongValueUnchecked() {
+    public long _asLongValueUnchecked() {
         return _value.longValue();
     }
 

src/main/java/tools/jackson/databind/node/DoubleNode.java

@@ -160,17 +160,17 @@ public OptionalDouble asDoubleOpt() {
      */
 
     @Override
-    protected short _asShortValueUnchecked() {
+    public short _asShortValueUnchecked() {
         return (short) _value;
     }
 
     @Override
-    protected int _asIntValueUnchecked() {
+    public int _asIntValueUnchecked() {
         return (int) _value;
     }
 
     @Override
-    protected long _asLongValueUnchecked() {
+    public long _asLongValueUnchecked() {
         return (long) _value;
     }
 

src/main/java/tools/jackson/databind/node/FloatNode.java

@@ -130,17 +130,17 @@ public OptionalDouble asDoubleOpt() {
      */
 
     @Override
-    protected short _asShortValueUnchecked() {
+    public short _asShortValueUnchecked() {
         return (short) _value;
     }
 
     @Override
-    protected int _asIntValueUnchecked() {
+    public int _asIntValueUnchecked() {
         return (int) _value;
     }
 
     @Override
-    protected long _asLongValueUnchecked() {
+    public long _asLongValueUnchecked() {
         return (long) _value;
     }
 

src/main/java/tools/jackson/databind/node/IntNode.java

@@ -195,7 +195,17 @@ public Optional<BigInteger> bigIntegerValueOpt() {
      */
 
     @Override
-    protected int _asIntValueUnchecked() {
+    public short _asShortValueUnchecked() {
+        return (short) _value;
+    }
+
+    @Override
+    public long _asLongValueUnchecked() {
+        return _value;
+    }
+
+    @Override
+    public int _asIntValueUnchecked() {
         return _value;
     }
 

src/main/java/tools/jackson/databind/node/LongNode.java

@@ -185,9 +185,19 @@ public Optional<BigInteger> bigIntegerValueOpt() {
      */
 
     @Override
-    protected int _asIntValueUnchecked() {
+    public short _asShortValueUnchecked() {
+        return (short) _value;
+    }
+
+    @Override
+    public int _asIntValueUnchecked() {
         return (int) _value;
     }
+
+    @Override
+    public long _asLongValueUnchecked() {
+        return _value;
+    }
 
     @Override
     protected float _asFloatValueUnchecked() {

src/main/java/tools/jackson/databind/node/NumericFPNode.java

@@ -329,39 +329,12 @@ public Optional<BigDecimal> asDecimalOpt() {
         return Optional.of(_asDecimalValueUnchecked());
     }
 
-    /*
-    /**********************************************************************
-    /* NumericNode extensions
-    /**********************************************************************
-     */
-
     /*
     /**********************************************************************
     /* Abstract methods for sub-classes to implement
     /**********************************************************************
      */
 
-    /**
-     * Method for sub-classes to implement; returns the underlying
-     * value as a {@code short} without any checks (wrt NaN or value range),
-     * so caller must ensure validity prior to calling
-     */
-    protected abstract short _asShortValueUnchecked();
-
-    /**
-     * Method for sub-classes to implement; returns the underlying
-     * value as a {@code int} without any checks (wrt NaN or value range),
-     * so caller must ensure validity prior to calling
-     */
-    protected abstract int _asIntValueUnchecked();
-
-    /**
-     * Method for sub-classes to implement; returns the underlying
-     * value as a {@code long} without any checks (wrt NaN or value range),
-     * so caller must ensure validity prior to calling
-     */
-    protected abstract long _asLongValueUnchecked();
-
     /**
      * Method for sub-classes to implement; returns the underlying
      * value as a {@link BigInteger} without any checks (wrt NaN), so caller

src/main/java/tools/jackson/databind/node/NumericIntNode.java

@@ -180,12 +180,10 @@ public boolean hasFractionalPart() {
 
     /*
     /**********************************************************************
-    /* Abstract methods for sub-classes
+    /* API extensions for sub-classes
     /**********************************************************************
      */
 
-    protected abstract int _asIntValueUnchecked();
-
     protected abstract float _asFloatValueUnchecked();
 
     protected abstract double _asDoubleValueUnchecked();

src/main/java/tools/jackson/databind/node/NumericNode.java

@@ -105,7 +105,7 @@ protected final String _valueDesc() {
 
     /*
     /**********************************************************************
-    /* Extended API
+    /* Extended API, public methods
     /**********************************************************************
      */
 
@@ -141,4 +141,31 @@ protected final String _valueDesc() {
      * "not-a-number" (NaN) value.
      */
     public abstract boolean isNaN();
+
+    /*
+    /**********************************************************************
+    /* Extended API, unsafe access methods
+    /**********************************************************************
+     */
+
+    /**
+     * Method for sub-classes to implement; returns the underlying
+     * value as a {@code short} without any checks (wrt NaN or value range),
+     * so caller must ensure validity prior to calling
+     */
+    public abstract short _asShortValueUnchecked();
+
+    /**
+     * Method for sub-classes to implement; returns the underlying
+     * value as a {@code int} without any checks (wrt NaN or value range),
+     * so caller must ensure validity prior to calling
+     */
+    public abstract int _asIntValueUnchecked();
+
+    /**
+     * Method for sub-classes to implement; returns the underlying
+     * value as a {@code long} without any checks (wrt NaN or value range),
+     * so caller must ensure validity prior to calling
+     */
+    public abstract long _asLongValueUnchecked();
 }

src/main/java/tools/jackson/databind/node/ShortNode.java

@@ -169,7 +169,17 @@ public Optional<BigInteger> bigIntegerValueOpt() {
      */
 
     @Override
-    protected int _asIntValueUnchecked() {
+    public short _asShortValueUnchecked() {
+        return _value;
+    }
+
+    @Override
+    public int _asIntValueUnchecked() {
+        return _value;
+    }
+
+    @Override
+    public long _asLongValueUnchecked() {
         return _value;
     }
 

src/main/java/tools/jackson/databind/node/TreeTraversingParser.java

@@ -317,8 +317,8 @@ public int getIntValue() throws InputCoercionException {
     public int getValueAsInt()
     {
         final NumericNode node = (NumericNode) currentNumericNode(NR_INT);
-        if (node.canConvertToInt()) {
-            return node.intValue();
+        if (node.inIntRange()) {
+            return node._asIntValueUnchecked();
         }
         // !!! TODO: better defaulting/coercion?
         return getIntValue();
@@ -328,13 +328,13 @@ public int getValueAsInt()
     public int getValueAsInt(int defaultValue)
     {
         final NumericNode node = (NumericNode) currentNumericNode(NR_INT);
-        if (node.canConvertToInt()) {
-            return node.intValue();
+        if (node.inIntRange()) {
+            return node._asIntValueUnchecked();
         }
         // !!! TODO: better defaulting/coercion?
         return defaultValue;
     }
-    
+
     @Override
     public long getLongValue() throws InputCoercionException {
         final NumericNode node = (NumericNode) currentNumericNode(NR_LONG);
@@ -354,6 +354,28 @@ public long getLongValue() throws InputCoercionException {
         return node.longValue();
     }
 
+    @Override
+    public long getValueAsLong()
+    {
+        final NumericNode node = (NumericNode) currentNumericNode(NR_INT);
+        if (node.inLongRange()) {
+            return node._asLongValueUnchecked();
+        }
+        // !!! TODO: better defaulting/coercion?
+        return getLongValue();
+    }
+
+    @Override
+    public long getValueAsLong(long defaultValue)
+    {
+        final NumericNode node = (NumericNode) currentNumericNode(NR_INT);
+        if (node.inLongRange()) {
+            return node._asLongValueUnchecked();
+        }
+        // !!! TODO: better defaulting/coercion?
+        return defaultValue;
+    }
+
     @Override
     public Number getNumberValue() throws InputCoercionException {
         return currentNumericNode(-1).numberValue();

src/test/java/tools/jackson/databind/convert/CoerceFloatToIntTest.java

@@ -15,7 +15,6 @@
 import tools.jackson.databind.exc.InvalidFormatException;
 import tools.jackson.databind.exc.MismatchedInputException;
 import tools.jackson.databind.node.JsonNodeFactory;
-import tools.jackson.databind.testutil.failure.JacksonTestFailureExpected;
 import tools.jackson.databind.type.LogicalType;
 
 import static org.junit.jupiter.api.Assertions.*;
@@ -84,7 +83,6 @@ public void testLegacyDoubleToIntCoercion() throws Exception
     }
 
     // [databind#5319]
-    @JacksonTestFailureExpected
     @Test
     public void testLegacyDoubleToIntCoercionJsonNodeToInteger() throws Exception
     {
@@ -98,7 +96,6 @@ public void testLegacyDoubleToIntCoercionJsonNodeToInteger() throws Exception
     }
 
     // [databind#5319]
-    @JacksonTestFailureExpected
     @Test
     public void testLegacyDoubleToIntCoercionJsonNodeToLong() throws Exception
     {
@@ -112,7 +109,6 @@ public void testLegacyDoubleToIntCoercionJsonNodeToLong() throws Exception
     }
 
     // [databind#5319]
-    //@JacksonTestFailureExpected
     @Test
     public void testLegacyDoubleToIntCoercionJsonNodeToBigInteger() throws Exception
     {