Improve LoggerUtils support and add TestLoggerExtension #3123
+1,183
−772
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR improves
LoggerUtils
and adds supporting JUnit5 annotations in reactor-core tests:LoggerUtils
now has support to redirect rather than copy/capture log messages when thelogger factory is installed early.
In reactor-core tests, a
TestLoggerExtension
is added that sets up aTestLogger
and activates capture/redirection viaLoggerUtils
:TestLogger
(configured depending on test annotations) andsets up
LoggerUtils
with saidTestLogger
ParameterResolver
it injects theTestLogger
into the test casedisableCapture()
The Extension should be applied to individual tests. Convenience annotations are provided that fine tune the
TestLogger
that will be injected:@Capture
should the loggers capture log output, ie. go both to original logger and TestLogger@Redirect
should the loggers redirect log output, ie. only go to TestLogger