RFR: 8314186: runtime/8176717/TestInheritFD.java failed with "Log file was leaked" [v2]

[Johan Sjölen]

On Wed, 10 Jan 2024 09:44:22 GMT, Leo Korinth <lkorinth at openjdk.org> wrote:

>> No need for the "I am confused" part. This case may be unexpected but if we reach it then it is an error.
>
> I understood it as Johan wanted the exception to be reported as an "Error" instead of a "Fail" to show that the testcase is bad and not (necessarily) the JVM.

Yes, what Leo said. Some way of signalling to the CI/CD system that it's the test that didn't function correctly, not that the test found an issue. Seems like that doesn't exist, so differentiating the two cases with a log line is good enough.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/17325#discussion_r1447649270