<Swing Dev> [13] RFR JDK-8219552:bump jtreg requiredVersion to b14 in test/jdk/sanity/client/

Mon Mar 11 05:31:11 UTC 2019

Gentle Reminder.

Best Regards,

Vikrant

From: Vikrant Agarwal 
Sent: Wednesday, March 06, 2019 9:45 AM
To: Igor Ignatyev <igor.ignatyev at oracle.com>
Cc: Aleksandre Iline <alexandre.iline at oracle.com>; swing-dev at openjdk.java.net
Subject: Re: <Swing Dev> [13] RFR JDK-8219552:bump jtreg requiredVersion to b14 in test/jdk/sanity/client/

Hi Igor,

We are moving exclusiveAccess.dirs to test/jdk/TEST.ROOT to minimize our dependency on ‘test/jdk/sanity/client/TEST.properties’, so that we can remove it in the future if screenshots keyword is no longer required.

Best Regards,

Vikrant

From: Igor Ignatyev 
Sent: Tuesday, March 05, 2019 10:34 PM
To: Vikrant Agarwal <HYPERLINK "mailto:vikrant.v.agarwal at oracle.com"vikrant.v.agarwal at oracle.com>
Cc: HYPERLINK "mailto:swing-dev at openjdk.java.net"swing-dev at openjdk.java.net; Aleksandre Iline <HYPERLINK "mailto:alexandre.iline at oracle.com"alexandre.iline at oracle.com>; Muneer Kolarkunnu <HYPERLINK "mailto:abdul.kolarkunnu at oracle.com"abdul.kolarkunnu at oracle.com>; Sergey Bylokhov <HYPERLINK "mailto:sergey.bylokhov at oracle.com"sergey.bylokhov at oracle.com>
Subject: Re: <Swing Dev> [13] RFR JDK-8219552:bump jtreg requiredVersion to b14 in test/jdk/sanity/client/

Hi Vikrant,

although I still don't understand why we are moving exclusiveAccess.dirs to test/jdk/TEST.ROOT, I'm fine w/ the patch.

Cheers,

-- Igor

On Mar 5, 2019, at 2:18 AM, Vikrant Agarwal <HYPERLINK "mailto:vikrant.v.agarwal at oracle.com"vikrant.v.agarwal at oracle.com> wrote:

Hi Igor,

Here is the updated webrev: http://cr.openjdk.java.net/~vagarwal/8219552/webrev.3/

Summary of changes made:

Removed requiredVersion from TEST.properties and README
Removed 'test/jdk/sanity/client/TEST.ROOT.template' as it is not being used.
It is not necessary for all keywords to be  under TEST.ROOT file. Right now test ButtonDemoScreenshotTest.java runs without any errors and if we remove 'screenshots' keyword from TEST.properties file it throws “Error. Parse Exception: Invalid key: screenshots”exception. So keeping the TEST.properties file.
Added "sanity/client" under exclusiveAccess.dirs to test/jdk/TEST.ROOT and removed exclusiveAccess.dirs  from ‘test/jdk/sanity/client/TEST.properties’
Removed "intermittent" keyword from ButtonDemoScreenshotTest, because intermittent issue is resolved by JDK-8157338
Added "screenshots" keyword to EditorPaneDemoTest as this test also has screenshot tests.
Added GPL legal notice in TEST.properties file

Best Regards,

Vikrant

From: Igor Ignatyev 
Sent: Friday, February 22, 2019 11:35 PM
To: Vikrant Agarwal <HYPERLINK "mailto:vikrant.v.agarwal at oracle.com"vikrant.v.agarwal at oracle.com>
Cc: HYPERLINK "mailto:swing-dev at openjdk.java.net"swing-dev at openjdk.java.net; Aleksandre Iline <HYPERLINK "mailto:alexandre.iline at oracle.com"alexandre.iline at oracle.com>; Muneer Kolarkunnu <HYPERLINK "mailto:abdul.kolarkunnu at oracle.com"abdul.kolarkunnu at oracle.com>
Subject: Re: <Swing Dev> [13] RFR JDK-8219552:bump jtreg requiredVersion to b14 in test/jdk/sanity/client/

Hi Vikrant.

could you please explain how 'test/jdk/sanity/client/TEST.ROOT.template' is used? if it's not used at all, I'd prefer to delete this file for good. 

I also have kinda same question to 'TEST.properties' file. AFAIR jtreg doesn't extending 'keys' in 'TEST.properties' files, which means all keywords must be listed in 'TEST.ROOT' file, and using of any other unregistered keywords leads to 'Error. Parse Exception: Invalid key: ...'.  'screenshots' isn't registered in test/jdk/TEST.ROOT, having it in 'test/jdk/sanity/client/TEST.properties' has no effect, so 'test/jdk/sanity/client/SwingSet/src/ButtonDemoScreenshotTest.java' (the only test w/ screenshots) can't be executed and will always fail w/ error status. so I'm wondering if you doing something w/ test/jdk/sanity/client/ before running these tests.

Thanks,

-- Igor

On Feb 21, 2019, at 9:56 PM, Vikrant Agarwal <HYPERLINK "mailto:vikrant.v.agarwal at oracle.com"vikrant.v.agarwal at oracle.com> wrote:

Hi All,

Please review this small fix for HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-8219552"JDK-8219552:

Bug: https://bugs.openjdk.java.net/browse/JDK-8219552

Webrev: http://cr.openjdk.java.net/~vagarwal/8219552/webrev.0/

Summary: Removed jtreg requiredVersion field from files in test/jdk/sanity/client/ as it is redundant because jtreg requiredVersion is specified in test/jdk/TEST.ROOT which is being updated by HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-8219417"JDK-8219417

Best Regards,

Vikrant

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/swing-dev/attachments/20190310/f56b8dbe/attachment-0001.html>