test deserialization with Java Record (#645)

* add check for java records

* Update build.sbt

* add test

* Update build.sbt

* Update MRecord.java

* Update build.sbt

Author: pjfanning

build.sbt

@@ -51,6 +51,8 @@ scalaMajorVersion := {
   }
 }
 
+val addJava17Tests: Boolean = compareVersions(System.getProperty("java.version"), "17.0.0") >= 0
+
 mimaPreviousArtifacts := {
   if (scalaReleaseVersion.value > 2)
     Set.empty
@@ -99,6 +101,17 @@ Test / unmanagedSourceDirectories ++= {
   }
 }
 
+Test / unmanagedSourceDirectories ++= {
+  if (addJava17Tests && scalaReleaseVersion.value == 2 && scalaMajorVersion.value >= 13) {
+    Seq(
+      (LocalRootProject / baseDirectory).value / "src" / "test" / "java-17",
+      (LocalRootProject / baseDirectory).value / "src" / "test" / "scala-jdk-17",
+    )
+  } else {
+    Seq.empty
+  }
+}
+
 libraryDependencies ++= Seq(
   "com.fasterxml.jackson.core" % "jackson-core" % jacksonVersion,
   "com.fasterxml.jackson.core" % "jackson-annotations" % jacksonVersion,
@@ -227,3 +240,23 @@ mimaBinaryIssueFilters ++= Seq(
   ProblemFilters.exclude[DirectMissingMethodProblem]("com.fasterxml.jackson.module.scala.introspect.ScalaAnnotationIntrospector.findSerializationInclusionForContent"),
   ProblemFilters.exclude[DirectMissingMethodProblem]("com.fasterxml.jackson.module.scala.introspect.ScalaAnnotationIntrospector.findSerializationInclusion")
 )
+
+def compareVersions(version1: String, version2: String): Int = {
+  var comparisonResult = 0
+  val version1Splits = version1.split("\\.")
+  val version2Splits = version2.split("\\.")
+  val maxLengthOfVersionSplits = Math.max(version1Splits.length, version2Splits.length)
+  var i = 0
+  while (comparisonResult == 0 && i < maxLengthOfVersionSplits) {
+    val v1 = if (i < version1Splits.length) version1Splits(i).toInt
+    else 0
+    val v2 = if (i < version2Splits.length) version2Splits(i).toInt
+    else 0
+    val compare = v1.compareTo(v2)
+    if (compare != 0) {
+      comparisonResult = compare
+    }
+    i += 1
+  }
+  comparisonResult
+}

src/test/java-17/com/fasterxml/jackson/module/scala/deser/MRecord.java

@@ -0,0 +1,23 @@
+package com.fasterxml.jackson.module.scala.deser;
+
+import com.fasterxml.jackson.annotation.JsonCreator;
+import com.fasterxml.jackson.annotation.JsonValue;
+
+public record MRecord(String name, String domain) {
+    @JsonCreator(mode = JsonCreator.Mode.DELEGATING)
+    public static MRecord fromString(String uri) {
+        String[] parts = uri.split("@");
+        if (parts.length > 2 || parts.length == 0) {
+            throw new RuntimeException(String.format("Invalid SIP URI: %s", uri));
+        }
+
+        String[] usernameParts = parts[0].split("\\.");
+            return new MRecord(usernameParts[usernameParts.length - 1], parts.length == 2 ? parts[1] : null);
+        }
+
+    @Override
+    @JsonValue
+    public String toString() {
+        return String.format("%s@%s", name, domain);
+    }
+}

src/test/java-17/com/fasterxml/jackson/module/scala/deser/MRecordWrapper.java

@@ -0,0 +1,3 @@
+package com.fasterxml.jackson.module.scala.deser;
+
+public record MRecordWrapper(MRecord value) {}

src/test/scala-jdk-17/com/fasterxml/jackson/module/scala/deser/RecordTest.scala

@@ -0,0 +1,14 @@
+package com.fasterxml.jackson.module.scala.deser
+
+import com.fasterxml.jackson.module.scala.DefaultScalaModule
+
+class RecordTest extends DeserializerTest {
+  lazy val module: DefaultScalaModule.type = DefaultScalaModule
+
+  "An ObjectMapper with DefaultScalaModule" should "not affect Java Record deserialization" in {
+    val json = "{\"value\": \"a@b\"}"
+    val mapper = newMapper
+    val testVal = mapper.readValue(json, classOf[MRecordWrapper])
+    testVal.value shouldEqual new MRecord("a", "b")
+  }
+}