On Tue, 29 Sep 2026 16:34:50 GMT, Martin Fox <[email protected]> wrote:
>> Andy Goryachev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Revert "feature"
>>
>> This reverts commit d1aeb8bceccbfef7723d8cb6cb85896898f8e92d.
>
> tests/manualTests/src/main/java/com/oracle/test/manual/util/ManualTestWindow.java
> line 243:
>
>> 241: String s = TestRunnerApp.getExitCodeString(code);
>> 242: if (code == TestRunner.FAILED) {
>> 243: new AssertionError("Failed Manual Test: " +
>> title).printStackTrace();
>
> This just prints the stack trace from pushing the "Fail" button to the call
> to exit(). It has nothing to do with the test that was just run.
the error and the stack trace are required for automation - you are right that
it adds little detail since it's always the same.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2156#discussion_r4135965041