Java records don't serialize with `MapperFeature.REQUIRE_SETTERS_FOR_GETTERS` since 2.16

[mathijs81]

Search before asking

• I searched in the issues and found nothing similar.

Describe the bug

When a mapper is configured with REQUIRE_SETTERS_FOR_GETTERS, nothing gets serialized for java records, even though the properties are just properties, not just getters.

Version Information

2.16.0

Reproduction

Here's a test that reproduces the issue. This test passes on 2.15.3, but starts failing from 2.16.0 onwards.
(The failure is that {} is returned instead of {"prop":123}.)

import static org.junit.jupiter.api.Assertions.assertEquals;

import org.junit.jupiter.api.Test;

import com.fasterxml.jackson.databind.MapperFeature;
import com.fasterxml.jackson.databind.json.JsonMapper;

class SerializeTest {
    record TestRecord(Object prop) {}

    @Test
    void testSerialize() throws Exception {
        var mapper = JsonMapper.builder().build();
        var mapperWithSetters = JsonMapper.builder().configure(MapperFeature.REQUIRE_SETTERS_FOR_GETTERS, true).build();

        assertEquals("{\"prop\":123}", mapper.writeValueAsString(new TestRecord(123)));
        // This one fails for 2.16.0, works for 2.15.3:
        assertEquals("{\"prop\":123}", mapperWithSetters.writeValueAsString(new TestRecord(123)));
    }

}

Expected behavior

REQUIRE_SETTERS_FOR_GETTERS shouldn't matter for java records

Additional context

I'm running JDK 17 if it matters.

[cowtowncoder]

Just to make sure I understand: this also fails in 2.17(.2)?

Other than that, yeah... seems like Records should ignore this setting.

[mathijs81]

Yes, also fails in 2.17.2 (and also with latest LTS JDK 21)

[cowtowncoder]

Looks like this indeed failed on 2.17.x, but was fixed by something (POJO property introspection rewrite?) in 2.18 branch. Will add passing test, mark as fixed.