Skip to content

[2.10.x] Add test for field ordering (backport #1042) by @levinson#1182

Merged
mkurz merged 1 commit into2.10.xfrom
mergify/bp/2.10.x/pr-1042
Jun 30, 2025
Merged

[2.10.x] Add test for field ordering (backport #1042) by @levinson#1182
mkurz merged 1 commit into2.10.xfrom
mergify/bp/2.10.x/pr-1042

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Jun 30, 2025

Pull Request Checklist

  • Have you read through the contributor guidelines?
  • Have you squashed your commits?
  • Have you added copyright headers to new files? N/A
  • Have you updated the documentation? N/A
  • Have you added tests for any changed functionality? N/A

Purpose

This PR demonstrates #1038. The new test fails due to serialized field order when compiled with Scala 3:

[info] - should preserve field ordering *** FAILED *** (33 milliseconds)
[info]   "{"[a":42,"m1":"foo","b":"baz","c":"bar","z":1.2]}" did not equal "{"[m1":"foo","c":"bar","z":1.2,"a":42,"b":"baz"]}" (JsonSharedSpec.scala:454)

This is an automatic backport of pull request #1042 done by [Mergify](https://mergify.com).

(cherry picked from commit c8946b3)
@mergify mergify bot mentioned this pull request Jun 30, 2025
5 tasks
@mkurz mkurz merged commit aadff80 into 2.10.x Jun 30, 2025
17 checks passed
@mergify mergify bot deleted the mergify/bp/2.10.x/pr-1042 branch June 30, 2025 15:06
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.

1 participant