Merge branch '2.x' into 3.x

Author: cowtowncoder

cbor/src/test/java/module-info.java

@@ -24,4 +24,5 @@
     opens tools.jackson.dataformat.cbor.seq;
     opens tools.jackson.dataformat.cbor.testutil;
     opens tools.jackson.dataformat.cbor.testutil.failure;
+    opens tools.jackson.dataformat.cbor.tofix;
 }

cbor/src/test/java/tools/jackson/dataformat/cbor/tofix/StringRef599Test.java

@@ -0,0 +1,45 @@
+package tools.jackson.dataformat.cbor.tofix;
+
+import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
+
+import tools.jackson.databind.ObjectMapper;
+
+import tools.jackson.dataformat.cbor.CBORGenerator;
+import tools.jackson.dataformat.cbor.CBORTestBase;
+import tools.jackson.dataformat.cbor.CBORWriteFeature;
+import tools.jackson.dataformat.cbor.testutil.failure.JacksonTestFailureExpected;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+import org.junit.jupiter.api.Test;
+
+public class StringRef599Test extends CBORTestBase
+{
+    static class AB599 {
+        public String a;
+        public String b;
+    }
+
+    @JsonIgnoreProperties(ignoreUnknown = true)
+    static class B599 {
+        public String b;
+    }
+    
+    private final ObjectMapper MAPPER = cborMapper(cborFactoryBuilder()
+            .enable(CBORWriteFeature.STRINGREF)
+            .build());
+
+    // [dataformats-binary#599]
+    @JacksonTestFailureExpected
+    @Test
+    public void testStringRef() throws Exception
+    {
+        AB599 ab = new AB599();
+        ab.a = "foo";
+        // important: has to be same String value to use StringRef
+        ab.b = ab.a;
+        byte[] cbor = MAPPER.writeValueAsBytes(ab);
+        B599 b = MAPPER.readValue(cbor, B599.class);
+        assertEquals(ab.b, b.b);
+    }    
+}