RFR: 8359061: Update and ProblemList manual test java/awt/Cursor/CursorDragTest/ListDragCursor.java [v9]

Damon Nguyen dnguyen at openjdk.org
Wed Jun 18 22:55:12 UTC 2025


On Wed, 18 Jun 2025 19:20:32 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>The timer updates each second, which sends additional messages — the dragged cursor changes as soon as the displayed time updates.
> 
> Pause the timer, and the drag cursor doesn't change until you release the left mouse button.
> 
> Harshitha was close to discovering it.

That makes a lot of sense. I saw your updates on that issue as well. I guess we can use this workaround in this niche case if necessary. Thanks @aivanov-jdk and @honkar-jdk for your help in looking into this!

> test/jdk/java/awt/Cursor/CursorDragTest/ListDragCursor.java line 46:
> 
>> 44:     static Frame instructionsFrame;
>> 45:     private static final CountDownLatch countDownLatch = new CountDownLatch(1);
>> 46:     static String INSTRUCTIONS = """
> 
> I'd place the fields in this order:
> 
> 
> INSTRUCTIONS
> 
> testFrame
> instructionsFrame
> 
> countDownLatch
> 
> 
> The blank lines are explicit for the code, too. This way the fields come in groups: the instructions describe what the tester will do, so it's reasonable to put them at the top, and this aligns to the most common way in manual tests; then go the two frames of the test; the latch which controls the test follows.
> 
> Personally, I'd also declare all of them `private`.

Implemented and noted!

> test/jdk/java/awt/Cursor/CursorDragTest/ListDragCursor.java line 114:
> 
>> 112:         Panel btnPanel = new Panel();
>> 113:         Button passBtn = new Button("PASS");
>> 114:         Button failBtn = new Button("FAIL");
> 
> Suggestion:
> 
>         Button passBtn = new Button("Pass");
>         Button failBtn = new Button("Fail");
> 
> Title case is good enough.

I thought I updated this already. Thanks.

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

PR Comment: https://git.openjdk.org/jdk/pull/25705#issuecomment-2985950371
PR Review Comment: https://git.openjdk.org/jdk/pull/25705#discussion_r2155664742
PR Review Comment: https://git.openjdk.org/jdk/pull/25705#discussion_r2155657589


More information about the client-libs-dev mailing list