Skip to content

Conversation

@rlaope
Copy link
Contributor

@rlaope rlaope commented Sep 9, 2025

fix #35445

This PR adds the @nullable annotation to the parameter of
AbstractJsonPathAssertions.isEqualTo(Object expectedValue).

Also adds a test to verify that passing null works as expected.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Sep 9, 2025
@rlaope rlaope force-pushed the patch-1 branch 2 times, most recently from 37fdacc to 3a0acd8 Compare September 9, 2025 07:04
@sdeleuze sdeleuze self-assigned this Sep 9, 2025
@sdeleuze sdeleuze added this to the 6.2.11 milestone Sep 9, 2025
@sdeleuze sdeleuze added in: test Issues in the test module type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Sep 9, 2025
@sdeleuze
Copy link
Contributor

sdeleuze commented Sep 9, 2025

This PR is hardly usable as not against the branch where we are asked to fix the issue originally, multiple draft commits not squashed, etc. We appreciate contributions but maybe for your next potential try, please take the time to polish it before sending it, and for straightforward changes where we mention in the issue that we will take care, not needed to send a PR that will take more time for us to integrate than directly do the fix. Better to choose non assigned issues that we don't have the time to process. As a consequence, I will close this PR unmerged and reopen the issue.

@sdeleuze sdeleuze closed this Sep 9, 2025
@sdeleuze sdeleuze removed this from the 6.2.11 milestone Sep 9, 2025
@sdeleuze sdeleuze added status: declined A suggestion or change that we don't feel we should currently apply and removed type: enhancement A general enhancement labels Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: test Issues in the test module status: declined A suggestion or change that we don't feel we should currently apply

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing @Nullable on JsonPathAssertions.isEqualTo

3 participants