On Wed, 10 Mar 2021 23:10:06 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> Florian Kirmaier has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   JDK-8263322
>>   updated the javadoc to mention the new case.
>
> 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

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

PR: https://git.openjdk.java.net/jfx/pull/421

Reply via email to