On Tue, 26 Mar 2024 22:33:45 GMT, Alisen Chung <ach...@openjdk.org> wrote:
>> Removing applet usage from manual JFileChooser tests
>
> Alisen Chung has updated the pull request incrementally with one additional
> commit since the last revision:
>
> moved create ui code to helper, change test pass check, add mouse release,
> remove getContentPane
test/jdk/javax/swing/JFileChooser/4150029/bug4150029.java line 86:
> 84: System.out.println("Created sub-directory: " + subDir);
> 85:
> 86: fileChooser = new JFileChooser(subDir);
Not required here as you are initializing in `createAndShowUI` method.
test/jdk/javax/swing/JFileChooser/4150029/bug4150029.java line 137:
> 135: clickBackSpace();
> 136: if (prevDir == crntDir) {
> 137: passed_2 = true;
This condition is not satisfied and hence the test failed in win11 machine. In
linux machine, once the current directory is at `root` then pressing the
backspace key doesn't have any effect.
But looks like it is different in windows and mac. Checked in windows native
explorer, once it reached to `home` folder, it toggles between `home` and
`Desktop` on backspace key press. The condition checked here will never satisfy
and `passed_2` variable remains false.
So, either you can check the condition differently or may remove this check for
windows and mac.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18415#discussion_r1540454288
PR Review Comment: https://git.openjdk.org/jdk/pull/18415#discussion_r1540474274
