Ignore generated methods for default arguments (#603)

Co-authored-by: Tim Moore <[email protected]>

Author: 2 people

src/main/scala/tools/jackson/module/scala/introspect/ScalaAnnotationIntrospectorModule.scala

@@ -64,7 +64,7 @@ class ScalaAnnotationIntrospectorInstance(scalaAnnotationIntrospectorModule: Sca
   override def hasIgnoreMarker(mapperConfig: MapperConfig[_], m: AnnotatedMember): Boolean = {
     val name = m.getName
     //special cases to prevent shadow fields associated with lazy vals being serialized
-    name == "0bitmap$1" || name.endsWith("$lzy1") || super.hasIgnoreMarker(mapperConfig, m)
+    name == "0bitmap$1" || name.endsWith("$lzy1") || name.contains("$default$") || super.hasIgnoreMarker(mapperConfig, m)
   }
 
   private def hasCreatorAnnotation(a: Annotated): Boolean = {

src/test/scala/tools/jackson/module/scala/introspect/ScalaAnnotationIntrospectorTest.scala

@@ -31,6 +31,9 @@ object ScalaAnnotationIntrospectorTest {
     @JsonProperty def getValue: Int = value
     @JsonProperty def setValue(value: Int): Unit = { this.value = value }
   }
+  class GeneratedDefaultArgumentClass {
+    def getValue(value: String = "default"): String = value
+  }
 
   case class CaseClassWithDefault(a: String = "defaultParam", b: Option[String] = Some("optionDefault"), c: Option[String])
 
@@ -251,6 +254,12 @@ class ScalaAnnotationIntrospectorTest extends FixtureAnyFlatSpec with Matchers {
     cache.get(new ClassKey(classOf[CaseClassWithDefault])) should not be(null)
   }
 
+  it should "ignore a generated default argument method" in { mapper =>
+    val bean = new GeneratedDefaultArgumentClass
+    val allProps = getProps(mapper, bean)
+    allProps shouldBe empty
+  }
+
   private def getProps(mapper: ObjectMapper, bean: AnyRef) = {
     val classIntrospector = mapper.serializationConfig().classIntrospectorInstance()
     val beanDescription: BeanDescription = classIntrospector.introspectForSerialization(mapper.constructType(bean.getClass))

src/test/scala/tools/jackson/module/scala/ser/CaseClassSerializerTest.scala

@@ -204,4 +204,13 @@ class CaseClassSerializerTest extends SerializerTest {
   it should "serialize ClassWithOnlyUnitField" in {
     serialize(ClassWithOnlyUnitField(())) shouldEqual """{}"""
   }
+
+  it should "not find properties for default arguments" in {
+    case class GeneratedDefaultArgumentClass() {
+      def getValue(value: String = "default"): String = value
+    }
+
+    serialize(GeneratedDefaultArgumentClass()) shouldEqual "{}"
+  }
+
 }