Skip to content

Conversation

@cppwfs
Copy link
Contributor

@cppwfs cppwfs commented Jan 29, 2026

Replace three-parameter TestUtils.getPropertyValue() calls with the type-safe generic method signature across all Spring Integration test modules. The deprecated method that required an explicit Class<T> parameter has been replaced with the generic method that infers the type parameter.

@cppwfs cppwfs added this to the 7.1.0-M2 milestone Jan 29, 2026
@cppwfs cppwfs requested a review from artembilan January 29, 2026 11:59
Replace three-parameter `TestUtils.getPropertyValue()` calls with the
type-safe generic method signature across all Spring Integration test
modules. The deprecated method that required an explicit `Class<T>`
parameter has been replaced with the generic method that infers the
type parameter.
@cppwfs cppwfs force-pushed the SI-getPropertyValue branch from 78ce528 to f2f5b99 Compare January 29, 2026 11:59
@cppwfs cppwfs changed the title Replace deprecated TestUtils API calls in test classes dd TestUtils.propertyValue() based on generics Jan 29, 2026
@cppwfs cppwfs changed the title dd TestUtils.propertyValue() based on generics Add TestUtils.propertyValue() based on generics Jan 29, 2026
Copy link
Member

@artembilan artembilan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT.
Pulling locally for final review and some clean up.
Thank you very much!

After merging I'll create a https://github.com/spring-projects/spring-integration/wiki page about migration from 7.0 to 7.1. And then I'll ask you to add a note about this change. The point is that new <T> @Nullable T getPropertyValue(Object root, String propertyPath) signature is really a breaking change as we see with our tests. Not a bit deal, but still better to mention and guide properly.

Thanks again!

@artembilan
Copy link
Member

Merged as after some minor clean and addition to the docs: 54a50ba

@artembilan artembilan closed this Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants