-
- Notifications
You must be signed in to change notification settings - Fork 1.6k
Issue: #3166: Introduce --redirect-stdout and --redirect-stderr CLI options to redirect STDOUT and STDERR #4311
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
Issue: #3166: Introduce --redirect-stdout and --redirect-stderr CLI options to redirect STDOUT and STDERR #4311
Conversation
ab51b9a to 216bfdb Compare There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a test for the separate/same file redirect cases to ConsoleLauncherIntegrationTests.
junit-platform-console/src/main/java/org/junit/platform/console/tasks/ConsoleTestExecutor.java Outdated Show resolved Hide resolved
documentation/src/docs/asciidoc/release-notes/release-notes-5.12.0-RC2.adoc Outdated Show resolved Hide resolved
6b9a026 to 91e353e Compare platform-tests/src/test/java/org/junit/platform/console/options/StdStreamTest.java Outdated Show resolved Hide resolved
8fada9f to 85d8577 Compare platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/ArchUnitTests.java Show resolved Hide resolved
documentation/src/docs/asciidoc/release-notes/release-notes-5.13.0-M1.adoc Outdated Show resolved Hide resolved
documentation/src/docs/asciidoc/user-guide/advanced-topics/junit-platform-reporting.adoc Outdated Show resolved Hide resolved
documentation/src/docs/asciidoc/user-guide/advanced-topics/junit-platform-reporting.adoc Outdated Show resolved Hide resolved
...-console/src/main/java/org/junit/platform/console/options/TestConsoleOutputOptionsMixin.java Outdated Show resolved Hide resolved
...-console/src/main/java/org/junit/platform/console/options/TestConsoleOutputOptionsMixin.java Outdated Show resolved Hide resolved
junit-platform-console/src/main/java/org/junit/platform/console/tasks/ConsoleTestExecutor.java Outdated Show resolved Hide resolved
junit-platform-console/src/main/java/org/junit/platform/console/tasks/ConsoleTestExecutor.java Outdated Show resolved Hide resolved
platform-tests/src/test/java/org/junit/platform/console/ConsoleLauncherIntegrationTests.java Outdated Show resolved Hide resolved
platform-tests/src/test/java/org/junit/platform/console/ConsoleLauncherIntegrationTests.java Outdated Show resolved Hide resolved
platform-tests/src/test/java/org/junit/platform/console/options/StdStreamTest.java Outdated Show resolved Hide resolved
d013e56 to 48361ac Compare 13caf16 to 4a81bc9 Compare There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like it's almost complete. 👍
Sorry it took me so long to get back to you!
documentation/src/docs/asciidoc/user-guide/advanced-topics/junit-platform-reporting.adoc Show resolved Hide resolved
...tform-console/src/main/java/org/junit/platform/console/options/TestConsoleOutputOptions.java Outdated Show resolved Hide resolved
junit-platform-console/src/main/java/org/junit/platform/console/tasks/StdStreamHandler.java Outdated Show resolved Hide resolved
junit-platform-console/src/main/java/org/junit/platform/console/tasks/StdStreamHandler.java Show resolved Hide resolved
platform-tests/src/test/java/org/junit/platform/console/ConsoleLauncherIntegrationTests.java Outdated Show resolved Hide resolved
platform-tests/src/test/java/org/junit/platform/console/ConsoleLauncherIntegrationTests.java Outdated Show resolved Hide resolved
0e2bc71 to 9bddb6d Compare …derr CLI options to redirect STDOUT and STDERR
9bddb6d to 59c6e36 Compare
Hi @marcphilipp |
| @Oyster-zx Thank you for your contribution! 👍 |
I second that! 👍 Please note that the documentation was in the wrong section, but I have already addressed that in 3d92bd6. |
| @marcphilipp thank you for the review 😄 |
| @Oyster-zx thank you for your contribution! I've been waiting for this feature for a while now. |
| Legend! I see this is now in a milestone release, so i am looking forward to trying this out tomorrow! |
| Really appreciate your work on this — I’ve been looking forward to this feature for a long time. |
Overview
Add new CLI options --redirect-stdout and --redirect-stderr to redirect stdout and stderr respectively to a file,
if both are set to the same file, stdout and stderr will be merged.
Issue: #3166
Credits to @mobounya for the initial PR.
I hereby agree to the terms of the JUnit Contributor License Agreement.
Definition of Done
@APIannotations