> When an `OutputAnalyzer` instance uses a `LazyOutputBuffer` impl to capture a > process' standard output, it unconditionally writes progress logs to stdout. > This can easily flood the output in tests that spawn a great number of > processes, each with an OutputAnalyzer attached, as part of their operations. > > This change introduce an optional "quiet mode" which allows suppressing the > diagnostic messages emitted by the analyzer. > > (Notes on implementation) > * Introducing a "verbose mode" instead would likely have been more idiomatic, > but would have implied a lot of existing tests opting into this new mode to > retain their current printing behavior. > * It is implemented as an extra parameter in new constructor overloads for > `OutputAnalyzer`, which really is the only way since `LazyOutputBuffer` > starts logging in its constructor.
Frederic Thevenet has updated the pull request incrementally with one additional commit since the last revision: Switch opt-in quiet mode for opt out verbose mode. Fix indent. ------------- Changes: - all: https://git.openjdk.org/jdk/pull/30238/files - new: https://git.openjdk.org/jdk/pull/30238/files/f8110f55..0838790a Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=30238&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=30238&range=00-01 Stats: 15 lines in 2 files changed: 1 ins; 1 del; 13 mod Patch: https://git.openjdk.org/jdk/pull/30238.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/30238/head:pull/30238 PR: https://git.openjdk.org/jdk/pull/30238
