Skip to content

Add failing regression test for #6032 - #6150

Open
Dongnyoung wants to merge 2 commits into
FasterXML:3.xfrom
Dongnyoung:fix-6032-serialization-order
Open

Add failing regression test for #6032#6150
Dongnyoung wants to merge 2 commits into
FasterXML:3.xfrom
Dongnyoung:fix-6032-serialization-order

Conversation

@Dongnyoung

Copy link
Copy Markdown
Contributor

Adds a failing test for #6032.

The new test covers serialization ordering when both:

  • MapperFeature.SORT_CREATOR_PROPERTIES_FIRST
  • MapperFeature.SORT_PROPERTIES_ALPHABETICALLY

are enabled.

It verifies the existing behavior for a constructor annotated with
@ConstructorProperties, where the constructor property is sorted first.

It also adds a @JacksonTestFailureExpected case showing that an unannotated
single-argument constructor is currently treated as a creator property for
serialization ordering, causing bb to be written before aa.

No production code changes are included.

Tested with:

.\mvnw.cmd "-Dtest=tools.jackson.databind.tofix.ConstructorPropertiesSerializationOrder6032Test" "-Dsurefire.useModulePath=false" test

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown

🧪 Code Coverage Report

Metric Coverage Change
Instructions coverage 81.90% 📈 +0.000%
Branches branches 75.49% 📉 -0.010%

Coverage data generated from JaCoCo test results

@gitar-bot

gitar-bot Bot commented Aug 18, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Adds a failing regression test covering serialization ordering when sorting creator properties and properties alphabetically are both enabled. No issues found.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source

@github-actions

Copy link
Copy Markdown

🧪 Code Coverage Report

Metric Coverage Change
Instructions coverage 81.92% 📈 +0.000%
Branches branches 75.56% 📈 +0.010%

Coverage data generated from JaCoCo test results

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants