Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[RESTEASY-2567] Added new test coverage for JSON merge patch #4375

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

honza-kasik
Copy link
Contributor

@honza-kasik honza-kasik commented Nov 6, 2024

Mainly negative test cases, see test documentation for details.

See https://issues.redhat.com/browse/RESTEASY-2567

@honza-kasik honza-kasik changed the title Added new test coverage for JSON merge patch [RESTEASY-2567] Added new test coverage for JSON merge patch Nov 6, 2024
@jamezp
Copy link
Contributor

jamezp commented Nov 6, 2024

@honza-kasik It looks like the format check failed.

Error:  Failed to execute goal net.revelc.code:impsort-maven-plugin:1.11.0:check (check-import-sort) on project resteasy-integration-tests: Imports are not sorted in /home/runner/work/resteasy/resteasy/testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/resource/patch/StudentJsonMergePatchTest.java -> [Help 1]

This can be resolved by simply running mvn clean process-test-classes.

Also, can you squash the commits into a single commit?

@honza-kasik honza-kasik force-pushed the json-merge-patch-test-extension branch from afe3e77 to 5ee0cc2 Compare November 7, 2024 13:24
@jamezp jamezp merged commit cbcbb50 into resteasy:6.2 Nov 8, 2024
12 of 13 checks passed
@jamezp
Copy link
Contributor

jamezp commented Nov 8, 2024

Upstream #4383

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