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

[Vikrant Agarwal]

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 <vikrant.v.agarwal at oracle.com>
Cc: swing-dev at openjdk.java.net; Aleksandre Iline <alexandre.iline at oracle.com>; Muneer Kolarkunnu <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/20190305/70131c67/attachment-0001.html>