add serialization order tests (JavaBean style)

[pjfanning]

See #3925

There is #3928 for records. This PR focuses on JavaBean style classes.

This highlights 2 problems:

• When JavaBean only has a JsonProperty on the constructor param, this is ignored
• Similar JavaBean that duplicates the JsonProperty annotation - putting it on the field and the constructor param - this upsets the order of the fields in the output.

[pjfanning]

@yihtserns this test result does not seem to match what you say in #3925 . When the JsonProperty annotation only appears on the constructor param, it appears to be ignored. It does not affect the order and it does not even affect the name of the field in the JSON.

[JooHyukKim]

@yihtserns this test result does not seem to match what you say in #3925 .

If what @pjfanning is true, can we revise what was said in the mentioned comment? just in case of confusing anybody

[yihtserns]

You're missing @JsonProperty annotation on the field - I said:

public record NestedRecordOne(
    String id,
    String email,
    @JsonProperty(value = "yikes!") NestedRecordTwo nestedRecordTwo) {
}

...is not equivalent to this:

public class NestedClassOne {
    private String id;
    private String name;
    @JsonProperty(value = "all is fine!")  // 👈 not just here
    private NestedClassTwo nestedClassTwo;

    public NestedClassOne(String id,
                          String name,
                          NestedClassTwo nestedClassTwo) {

        this.id = id;
        this.name = name;
        this.nestedClassTwo = nestedClassTwo;
    }

    // getters
}

...but is actually equivalent to this:

public class NestedClassOne {
    private String id;
    private String name;
    @JsonProperty(value = "all is fine!")  // 👈 here
    private NestedClassTwo nestedClassTwo;

    public NestedClassOne(String id,
                          String name,
                          // 👇 and also here
                          @JsonProperty(value = "all is fine!") NestedClassTwo nestedClassTwo) {

        this.id = id;
        this.name = name;
        this.nestedClassTwo = nestedClassTwo;
    }

    ...
    @JsonProperty(value = "all is fine!")  // 👈 and also here, but this part is not important
    public NestedClassTwo getNestedClassTwo() {
        return this.nestedClassTwo;
    }
}

[pjfanning]

@yihtserns why would you want to put the same JsonProperty annotation in 2 places?

[yihtserns]

why would you want to put the same JsonProperty annotation in 2 places?

I wouldn't - I'm just describing the resulting Record's compiled .class, when a Record component is annotated with @JsonProperty.

[pjfanning]

Thanks @yihtserns - I updated the PR to have 2 separate classes to highlight the 2 separate issues we are seeing

[cowtowncoder]

@pjfanning I think I could proceed with this, but it needs merge/rebase from 2.16 I think.

[pjfanning]

@cowtowncoder the tests in this PR just reproduce a problem. I have not attempted a fix.

[cowtowncoder]

@pjfanning Gotcha. It could be added under ..../failing/ only including failing part, if that would help someone else try solve it. But it can be kept as draft PR too if that makes more sense.