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