RFR: 8168388: GetMousePositionTest fails with the message "Mouse position should not be null" [v3]

Alexander Zuev kizune at openjdk.java.net
Thu Oct 21 06:00:10 UTC 2021


On Thu, 21 Oct 2021 04:24:11 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Alexander Zuev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Call dispose() from the test final block. Also added original exception to the unexpected exception handler for easier triage in the future.
>
> test/jdk/java/awt/Mouse/GetMousePositionTest/GetMousePositionWithOverlay.java line 52:
> 
>> 50:             doTest();
>> 51:         } catch (Exception e) {
>> 52:             if (e instanceof RuntimeException) {
> 
> Do we need this catch block? I guess main() throws Exception so we can just have try-finally block, no?

The catch block was already there and i decided not to remove it and moved the rest of the code instead of the try block.

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

PR: https://git.openjdk.java.net/jdk/pull/6028



More information about the client-libs-dev mailing list