When deserializing, `JsonProperty.Access.READ_ONLY` not working in all cases

hi I can’t well english. please your understand.

until 2.9.0 successful. but fail from 2.9.1. 2.9.3 will still fail Has changed spec for access option in JsonProperty? or bug? I attach test code

Test Class

import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.databind.ObjectMapper;
import lombok.AllArgsConstructor;
import lombok.Data;
import lombok.NoArgsConstructor;
import org.junit.Before;
import org.junit.Test;

import java.io.IOException;

import static org.hamcrest.CoreMatchers.is;
import static org.hamcrest.CoreMatchers.nullValue;
import static org.hamcrest.MatcherAssert.assertThat;

public class DeserializeTest {
    private ObjectMapper objectMapper;

    @Before
    public void setUp(){
        this.objectMapper = new ObjectMapper();
    }

    @Test
    public void test() throws IOException {
        String json = "{\"pubtrans\":\"\",\"name\":\"changyong\"}";

        TestClass result = this.objectMapper.readValue(json, TestClass.class);

        // jackson 2.9.0 is success, 2.9.1~2.9.3 is fail
        assertThat(result.getPubtrans(), is(nullValue()));
    }

    @Data
    @AllArgsConstructor
    @NoArgsConstructor
    public static class TestClass{
        @JsonProperty(access = JsonProperty.Access.READ_ONLY)
        private Pubtrans pubtrans;
        private String name;
    }

    enum Pubtrans{
        SUBWAY, BUS
    }
}

[MonkeyDAntoine]

Hello,

I tested the DeserializeTest given by @LichKing-lee and it’s worked with version 2.9.6.

But if I set a value to the JsonProperty : @JsonProperty(value = "pub_trans",access = JsonProperty.Access.READ_ONLY)

and of course change the json in the test : String json = "{\"pub_trans\":\"A\",\"name\":\"changyong\"}";

The unit test fails with the exception com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "pub_trans"

It’s seems that I cannot combine access and value. I still found a workaround by adding @JsonIgnoreProperties(value="pub_trans", allowGetters=true) at class-level of the TestClass

[cowtowncoder]

Looks like READ_ONLY is properly detected, added for Creator-property too. Not yet sure why it is still found – probably since creator property lookup is bit different in property-based approach, so maybe just need to add another pass over those. And then decide how to change behavior (remove entry in secondary lookup, change handler to no-operation).