Re: RFR: 8319938: com/sun/java/swing/plaf/gtk/TestFileChooserSingleDirectorySelection.java fails with "getSelectedFiles returned empty array for LAF: javax.swing.plaf.metal.MetalLookAndFeel" [v4]

2023-12-05 Thread Abhishek Kumar
On Tue, 5 Dec 2023 18:09:39 GMT, Alexey Ivanov wrote: >> Abhishek Kumar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Review comment fix > > Is it possible to shorten the subject of the issue? It is too long and wordy, > yet it doesn'

Re: RFR: 8319938: com/sun/java/swing/plaf/gtk/TestFileChooserSingleDirectorySelection.java fails with "getSelectedFiles returned empty array for LAF: javax.swing.plaf.metal.MetalLookAndFeel" [v4]

2023-12-05 Thread Alexey Ivanov
On Tue, 5 Dec 2023 12:17:55 GMT, Abhishek Kumar wrote: >> The test fails for JFileChooser selection mode set to `DIRECTORIES_ONLY`. >> For `DIRECTORIES_ONLY `mode, there may not be any directories in home >> directory and due to that test failed. Added the code to create temporary >> directori

Re: RFR: 8319938: com/sun/java/swing/plaf/gtk/TestFileChooserSingleDirectorySelection.java fails with "getSelectedFiles returned empty array for LAF: javax.swing.plaf.metal.MetalLookAndFeel" [v4]

2023-12-05 Thread Alexey Ivanov
On Tue, 5 Dec 2023 12:17:55 GMT, Abhishek Kumar wrote: >> The test fails for JFileChooser selection mode set to `DIRECTORIES_ONLY`. >> For `DIRECTORIES_ONLY `mode, there may not be any directories in home >> directory and due to that test failed. Added the code to create temporary >> directori

Re: RFR: 8319938: com/sun/java/swing/plaf/gtk/TestFileChooserSingleDirectorySelection.java fails with "getSelectedFiles returned empty array for LAF: javax.swing.plaf.metal.MetalLookAndFeel" [v4]

2023-12-05 Thread Abhishek Kumar
> The test fails for JFileChooser selection mode set to `DIRECTORIES_ONLY`. For > `DIRECTORIES_ONLY `mode, there may not be any directories in home directory > and due to that test failed. Added the code to create temporary directories > and files for the test. > Tested the current change on the