Add (failing) test for #2932

Author: cowtowncoder

src/main/java/com/fasterxml/jackson/databind/introspect/JacksonAnnotationIntrospector.java

@@ -576,9 +576,9 @@ public TypeResolverBuilder<?> findTypeResolver(MapperConfig<?> config,
     public TypeResolverBuilder<?> findPropertyTypeResolver(MapperConfig<?> config,
             AnnotatedMember am, JavaType baseType)
     {
-        /* As per definition of @JsonTypeInfo, should only apply to contents of container
-         * (collection, map) types, not container types themselves:
-         */
+        // As per definition of @JsonTypeInfo, should only apply to contents of container
+        // (collection, map) types, not container types themselves:
+
         // 17-Apr-2016, tatu: For 2.7.4 make sure ReferenceType also included
         if (baseType.isContainerType() || baseType.isReferenceType()) {
             return null;
@@ -591,9 +591,8 @@ public TypeResolverBuilder<?> findPropertyTypeResolver(MapperConfig<?> config,
     public TypeResolverBuilder<?> findPropertyContentTypeResolver(MapperConfig<?> config,
             AnnotatedMember am, JavaType containerType)
     {
-        /* First: let's ensure property is a container type: caller should have
-         * verified but just to be sure
-         */
+        // First: let's ensure property is a container type: caller should have
+        // verified but just to be sure
         if (containerType.getContentType() == null) {
             throw new IllegalArgumentException("Must call method with a container or reference type (got "+containerType+")");
         }

src/test/java/com/fasterxml/jackson/databind/deser/creators/ImplicitParamsForCreatorTest.java

@@ -37,11 +37,14 @@ public XY(int x, int y) {
     /**********************************************************
      */
 
+    private final ObjectMapper MAPPER = jsonMapperBuilder()
+            // important! Chain to also use standard Jackson annotations
+            .annotationIntrospector(new MyParamIntrospector())
+            .build();
+
     public void testNonSingleArgCreator() throws Exception
     {
-        ObjectMapper mapper = new ObjectMapper();
-        mapper.setAnnotationIntrospector(new MyParamIntrospector());
-        XY value = mapper.readValue(aposToQuotes("{'paramName0':1,'paramName1':2}"), XY.class);
+        XY value = MAPPER.readValue(aposToQuotes("{'paramName0':1,'paramName1':2}"), XY.class);
         assertNotNull(value);
         assertEquals(1, value.x);
         assertEquals(2, value.y);

src/test/java/com/fasterxml/jackson/failing/ImplicitParamsForCreator2932Test.java

@@ -0,0 +1,57 @@
+package com.fasterxml.jackson.failing;
+
+import com.fasterxml.jackson.databind.*;
+import com.fasterxml.jackson.databind.annotation.JsonDeserialize;
+import com.fasterxml.jackson.databind.introspect.AnnotatedMember;
+import com.fasterxml.jackson.databind.introspect.AnnotatedParameter;
+import com.fasterxml.jackson.databind.introspect.JacksonAnnotationIntrospector;
+
+public class ImplicitParamsForCreator2932Test extends BaseMapTest
+{
+    @SuppressWarnings("serial")
+    static class MyParamIntrospector extends JacksonAnnotationIntrospector
+    {
+        @Override
+        public String findImplicitPropertyName(AnnotatedMember param) {
+            if (param instanceof AnnotatedParameter) {
+                AnnotatedParameter ap = (AnnotatedParameter) param;
+                return "paramName"+ap.getIndex();
+            }
+            return super.findImplicitPropertyName(param);
+        }
+    }
+
+    // [databind#2932]
+    static class Bean2932
+    {
+        int _a, _b;
+
+//        @JsonCreator
+        public Bean2932(/*@com.fasterxml.jackson.annotation.JsonProperty("paramName0")*/
+                @JsonDeserialize() int a, int b) {
+            _a = a;
+            _b = b;
+        }
+    }
+
+    /*
+    /**********************************************************
+    /* Test methods
+    /**********************************************************
+     */
+
+    private final ObjectMapper MAPPER = jsonMapperBuilder()
+            // important! Chain to also use standard Jackson annotations
+            .annotationIntrospector(new MyParamIntrospector())
+            .build();
+
+    // [databind#2932]
+    public void testJsonCreatorWithOtherAnnotations() throws Exception
+    {
+        Bean2932 bean = MAPPER.readValue(a2q("{'paramName0':1,'paramName1':2}"),
+                Bean2932.class);
+        assertNotNull(bean);
+        assertEquals(1, bean._a);
+        assertEquals(2, bean._b);
+    }
+}