Add now-passing test for #3133 (#4231)

Author: JooHyukKim

src/test/java/com/fasterxml/jackson/databind/BaseMapTest.java

@@ -244,7 +244,7 @@ public static ObjectMapper newJsonMapper() {
     }
 
     // @since 2.10
-    protected static JsonMapper.Builder jsonMapperBuilder() {
+    public static JsonMapper.Builder jsonMapperBuilder() {
         return JsonMapper.builder();
     }
 

src/test/java/com/fasterxml/jackson/databind/deser/jdk/BigDecimalForFloatDisabled3133Test.java

@@ -0,0 +1,63 @@
+package com.fasterxml.jackson.databind.deser.jdk;
+
+import java.util.HashMap;
+import java.util.Map;
+
+import com.fasterxml.jackson.annotation.JsonSubTypes;
+import com.fasterxml.jackson.annotation.JsonTypeInfo;
+import com.fasterxml.jackson.databind.DeserializationFeature;
+import com.fasterxml.jackson.databind.ObjectMapper;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
+
+import static com.fasterxml.jackson.databind.BaseMapTest.jsonMapperBuilder;
+import static com.fasterxml.jackson.databind.BaseTest.a2q;
+
+/**
+ * Unit test proving that below issue is fixed.
+ * <p>
+ * [databind#3133] Map deserialization results in different numeric classes based on json
+ * ordering (BigDecimal / Double) when used in combination with @JsonSubTypes
+ */
+public class BigDecimalForFloatDisabled3133Test
+{
+    @JsonTypeInfo(
+            use = JsonTypeInfo.Id.NAME,
+            include = JsonTypeInfo.As.PROPERTY,
+            property = "type")
+
+    @JsonSubTypes({
+            @JsonSubTypes.Type(value = TestMapContainer.class, name = "MAP"),
+    })
+    interface TestJsonTypeInfoInterface { }
+
+    static class TestMapContainer implements TestJsonTypeInfoInterface {
+
+        private Map<String, ? extends Object> map = new HashMap<>();
+
+        public Map<String, ? extends Object> getMap() {
+            return map;
+        }
+
+        public void setMap(Map<String, ? extends Object> map) {
+            this.map = map;
+        }
+    }
+
+    private final ObjectMapper mapper = jsonMapperBuilder()
+            .disable(DeserializationFeature.USE_BIG_DECIMAL_FOR_FLOATS)
+            .build();
+
+    @Test
+    public void testDeserializeWithDifferentOrdering() throws Exception {
+        // case 1 : type first
+        String ordering1 = a2q("{'type': 'MAP','map': { 'doubleValue': 0.1 }}");
+        TestMapContainer model1 = mapper.readValue(ordering1, TestMapContainer.class);
+        Assertions.assertTrue(model1.getMap().get("doubleValue") instanceof Double);
+
+        // case 2 : value first
+        String ordering2 = a2q("{'map': { 'doubleValue': 0.1 }, 'type': 'MAP'}");
+        TestMapContainer model2 = mapper.readValue(ordering2, TestMapContainer.class);
+        Assertions.assertTrue(model2.getMap().get("doubleValue") instanceof Double);
+    }
+}