On Fri, 12 Mar 2021 14:31:17 GMT, Florian Kirmaier <[email protected]>
wrote:
>> tests/system/src/test/java/test/com/sun/javafx/application/InitializeJavaFXBase.java
>> line 80:
>>
>>> 78: System.out.println("Finished launch!");
>>> 79: Assert.fail("Error: No Exception was thrown - expected
>>> IllegalStateException");
>>> 80: } catch (IllegalStateException e) {
>>
>> Maybe add a comment that this exception is expected?
>
> I've added a comment and a print-statement that this is expected - on both
> catches for EllegalStateException
The comment looks fine. See my global comment about the print statement.
-------------
PR: https://git.openjdk.java.net/jfx/pull/421
