Add unit test for FasterXML#999

Author: cowtowncoder

src/main/java/com/fasterxml/jackson/databind/ser/BeanPropertyWriter.java

@@ -36,7 +36,7 @@
 @JacksonStdImpl // since 2.6. NOTE: sub-classes typically are not
 public class BeanPropertyWriter
     extends PropertyWriter // which extends `ConcreteBeanPropertyBase`
-    implements java.io.Serializable // since 2.6.2
+    implements java.io.Serializable // since 2.6
 {
     // As of 2.7
     private static final long serialVersionUID = 1L;
@@ -582,7 +582,7 @@ public JavaType getFullPropertyType() {
     */
 
     /**
-     * @deprecated Since 2.7, to be removed from 2.8
+     * @deprecated Since 2.7, to be removed from 2.8, use {@link #getType()} instead.
      */
     @Deprecated
     public Class<?> getPropertyType() {
@@ -600,7 +600,7 @@ public Class<?> getPropertyType() {
      *
      * @return The property type, or null if not found.
      *
-     * @deprecated Since 2.7, to be removed from 2.8
+     * @deprecated Since 2.7, to be removed from 2.8, use {@link #getType()} instead.
      */
     @Deprecated
     public Type getGenericPropertyType() {

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

@@ -0,0 +1,51 @@
+package com.fasterxml.jackson.failing;
+
+import com.fasterxml.jackson.annotation.*;
+import com.fasterxml.jackson.core.type.TypeReference;
+import com.fasterxml.jackson.databind.*;
+
+public class ExternalTypeId999Test extends BaseMapTest
+{
+    public static interface Payload { }
+
+    @JsonTypeName("foo")
+    public static class FooPayload implements Payload { }
+
+    @JsonTypeName("bar")
+    public static class BarPayload implements Payload { }
+
+    public static class Message<P extends Payload>
+    {
+        private final String type;
+        @JsonTypeInfo(visible = true, use = JsonTypeInfo.Id.NAME,
+                include = JsonTypeInfo.As.EXTERNAL_PROPERTY, property = "type")
+        @JsonSubTypes({
+                @JsonSubTypes.Type(FooPayload.class),
+                @JsonSubTypes.Type(BarPayload.class) })
+        private final P payload;
+
+        @JsonCreator
+        public Message(@JsonProperty("type") String type,
+                @JsonProperty("payload") P payload)
+        {
+            if (type == null) {
+                throw new RuntimeException("'type' is null");
+            }
+            if (payload == null) {
+                throw new RuntimeException("'payload' is null");
+            }
+            this.type = type;
+            this.payload = payload;
+        }
+    }
+
+
+    public void testExternalTypeId() throws Exception
+    {
+        ObjectMapper objectMapper = new ObjectMapper();
+        Message<?> msg = objectMapper.readValue(
+                "{ \"type\": \"foo\", \"payload\": {} }",
+                new TypeReference<Message<FooPayload>>() { });
+        assertNotNull(msg);
+    }
+}