RFR CODETOOLS-7902339: Handle different look and feels in JFileChooserOperator
Alexandre (Shura) Iline
alexandre.iline at oracle.com
Thu Oct 25 18:18:08 UTC 2018
The JFileChooserOperator.java changes look great.
1. You can use @BeforeMethod even with data provider, it seems. There a seem to be a way to pass parameters to @BeforeMethod. Have you tried that?
2. I am worried about hardcoding “Desktop” even in the tests. That would mean that Jemmy tests only works on English windows. Should we think of localization resource bundles for this case?
3. Please bump minor version
> On Oct 23, 2018, at 11:18 PM, Muneer Kolarkunnu <abdul.kolarkunnu at oracle.com> wrote:
> Hi All,
> Please review fix for the enhancement: https://bugs.openjdk.java.net/browse/CODETOOLS-7902339
> Webrev : http://cr.openjdk.java.net/~akolarkunnu/7902339/webrev.00/
> There are multiple issues in different L&Fs.
> In GTK and Motif L&F:
> There are two JLists, one is to list folders and second one one is to list files, but current JFileChooserOperator implementation didn't consider this, it expects only one JList that is to list files.
> Windows & Windows Classic L&F:
> There is no 'Go Home' button, but there is a toggle button to go desktop. In Windows platform 'Go Home' button usually(In metal and nimbus L&F) navigates to Desktop only.
More information about the jemmy-dev