<Swing Dev> [12] RFR JDK-8210052:Enable testing for all the available look and feels in SwingSet3 demo tests

[Vikrant Agarwal]

Hi Shura,

Yes, there were no changes needed to the Test Helpers class itself.
Sure, I will take care not to introduce unrelated formatting changes next time.

Best Regards,
Vikrant

-----Original Message-----
From: Alexandre (Shura) Iline 
Sent: Tuesday, September 04, 2018 8:23 PM
To: Vikrant Agarwal <vikrant.v.agarwal at oracle.com>
Cc: swing-dev at openjdk.java.net; Muneer Kolarkunnu <abdul.kolarkunnu at oracle.com>; Srinivas Mandalika <srinivas.mandalika at oracle.com>
Subject: Re: <Swing Dev> [12] RFR JDK-8210052:Enable testing for all the available look and feels in SwingSet3 demo tests

Vikrant,

There were no changes needed to the TestHelpers class itself, correct?

You should try to avoid formatting changes as it makes it harder to review.

Otherwise looks good.

Thank you.

Shura

> On Sep 4, 2018, at 2:03 AM, Vikrant Agarwal <vikrant.v.agarwal at oracle.com> wrote:
> 
> Hi All,
>  
> Please review the fix to add support for testing for all the available look and feels for the following SwingSet3 Demo Tests:
> ButtonDemoScreenshotTest.java 
> ButtonDemoTest.java 
> ComboBoxDemoTest.java 
> ListDemoTest.java 
> OptionPaneDemoTest.java 
> ProgressBarDemoTest.java 
> ScrollPaneDemoTest.java 
> SpinnerDemoTest.java 
> TabbedPaneDemoTest.java 
> TableDemoTest.java 
> ToggleButtonDemoTest.java 
> TreeDemoTest.java
>  
> Task: https://bugs.openjdk.java.net/browse/JDK-8210052
> Webrev Link: http://cr.openjdk.java.net/~vagarwal/8210052/webrev.1/
> Summary: We use “availableLookAndFeels” dataProvider from TestHelpers class  to run the test iteratively using all the available look and feels.
>  
> Best Regards,
> Vikrant