ReactorContextTestExecutionListener should use named hooks
See original GitHub issueNOTE: This is marked for first time contributors only. If you want the issue, please comment to claim it. If you need any help with this issue, just ask on the ticket and we will be glad to help.
Summary
Currently, ReactorContextTestExecutionListener
resets all previously set onLastOperator
hooks:
https://github.com/spring-projects/spring-security/blob/dca3645850e082d845e17a6572af84aa44b97f12/test/src/main/java/org/springframework/security/test/context/support/ReactorContextTestExecutionListener.java#L66-L68
This might affect users’ code, and it is recommended to use named hooks instead.
Actual Behavior
ReactorContextTestExecutionListener
resets all pre-configured onLastOperator
hooks after test method.
Expected Behavior
ReactorContextTestExecutionListener resets only the hooks it sets.
Version
5.1.1.RELEASE
Issue Analytics
- State:
- Created 5 years ago
- Comments:10 (5 by maintainers)
@rwinch I created a PR. #6122. Kindly review. Your feedback is important to me. Thanks for all the help.
@daejii Thanks for volunteering! The issue is all yours. If you have any questions or need any help, please don’t hesitate to ask! Thanks again!