It's fine and necessary fix but it should be done in the integration
repository, not in client. Otherwise, the bug would stay in master for
2 next builds and prevent jtreg starting at all.

+1

-yan

On 01/12/2017 10:04 AM, Prasanta Sadhukhan wrote:
Hi All,

Please review the name change for this regression test which was causing
jtreg to fail as it was not finding the test.

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

Filename was javax/swing/JTable/8133919/DrawGridLInesTest.java [whith
capital I]

but test has jtreg tag @run main DrawGridLinesTest

Fix is to rename the test to correct name.

webrev: http://cr.openjdk.java.net/~psadhukhan/8172558/webrev.00/

Regards
Prasanta

Reply via email to