Update release notes wrt #51

Author: cowtowncoder

datetime/src/main/java/com/fasterxml/jackson/datatype/jsr310/deser/key/YearKeyDeserializer.java

@@ -1,21 +1,16 @@
 package com.fasterxml.jackson.datatype.jsr310.deser.key;
 
-import static java.time.temporal.ChronoField.YEAR;
-
 import java.io.IOException;
 import java.time.DateTimeException;
 import java.time.Year;
-import java.time.format.DateTimeFormatter;
-import java.time.format.DateTimeFormatterBuilder;
-import java.time.format.SignStyle;
 
 import com.fasterxml.jackson.databind.DeserializationContext;
 
 public class YearKeyDeserializer extends Jsr310KeyDeserializer {
 
     public static final YearKeyDeserializer INSTANCE = new YearKeyDeserializer();
 
-    private YearKeyDeserializer() {
+    protected YearKeyDeserializer() {
         // singleton
     }
 

datetime/src/test/java/com/fasterxml/jackson/datatype/jsr310/key/YearAsKeyTest.java

@@ -21,15 +21,20 @@ public class YearAsKeyTest extends ModuleTestBase
     private final ObjectReader READER = MAPPER.readerFor(TYPE_REF);
 
     @Test
-    public void testSerialization() throws Exception {
+    public void testKeySerialization() throws Exception {
         assertEquals("Value is incorrect", mapAsString("3141", "test"),
                 MAPPER.writeValueAsString(asMap(Year.of(3141), "test")));
     }
 
     @Test
-    public void testDeserialization() throws Exception {
+    public void testKeyDeserialization() throws Exception {
         assertEquals("Value is incorrect", asMap(Year.of(3141), "test"),
                 READER.readValue(mapAsString("3141", "test")));
+        // Test both padded, unpadded
+        assertEquals("Value is incorrect", asMap(Year.of(476), "test"),
+                READER.readValue(mapAsString("0476", "test")));
+        assertEquals("Value is incorrect", asMap(Year.of(476), "test"),
+                READER.readValue(mapAsString("476", "test")));
     }
 
     @Test(expected = InvalidFormatException.class)
@@ -45,11 +50,15 @@ public void deserializeYearKey_notAYear() throws Exception {
     @Test
     public void serializeAndDeserializeYearKeyUnpadded() throws Exception {
         // fix for issue #51 verify we can deserialize an unpadded year e.g. "1"
-        ObjectMapper unpaddedMapper = newMapper();
         Map<Year, Float> testMap = Collections.singletonMap(Year.of(1), 1F);
-        String serialized = unpaddedMapper.writeValueAsString(testMap);
+        String serialized = MAPPER.writeValueAsString(testMap);
         TypeReference<Map<Year, Float>> yearFloatTypeReference = new TypeReference<Map<Year, Float>>() {};
-        Map<Year, Float> deserialized = unpaddedMapper.readValue(serialized, yearFloatTypeReference);
+        Map<Year, Float> deserialized = MAPPER.readValue(serialized, yearFloatTypeReference);
         assertEquals(testMap, deserialized);
+
+        // actually, check padded as well just to make sure
+        Map<Year, Float> deserialized2 = MAPPER.readValue(aposToQuotes("{'0001':1.0}"),
+                yearFloatTypeReference);
+        assertEquals(testMap, deserialized2);
     }
 }

release-notes/CREDITS-2.x

@@ -62,3 +62,6 @@ Michael O'Keeffe (kupci@github)
   * Contributed fix for #69: `ZonedDateTime` for times before the epoch do not
     serialize correctly
    (2.10.0)
+  * Contributed fix for #51: `YearKeyDeserializer` doesn't work with non-padded
+    year values
+   (2.10.0)

release-notes/VERSION-2.x

@@ -8,6 +8,11 @@ Modules:
 === Releases ===
 ------------------------------------------------------------------------
 
+2.10.0 (not yet released)
+
+#51: `YearKeyDeserializer` doesn't work with non-padded year values
+ (reported by sladkoff@github; fix contributed by Mike [kupci@github])
+
 2.10.0.pr2 (31-Aug-2019)
 
 #69: `ZonedDateTime` for times before the epoch do not serialize correctly