[10] RFR of JDK-8173411: Some testng tests check nothing in java time

Amy Lu amy.lu at oracle.com
Wed Oct 11 03:30:17 UTC 2017


Please review the patch to change TestNG.dirs and lib.dirs (def and file 
structure) for test/jdk/java/time.

test/jdk/java/time contains three sets of tests:
nontestng/
tck/
test/

Tests from directory "tck" and "test" are testng tests with properties:
TestNG.dirs = ..
lib.dirs = ../../../lib/testlibrary
lib.build = jdk.testlibrary.RandomFactory

But not all files under "TestNG.dirs" are real tests, some of them are 
"libraries" thus should not be put under "TestNG.dirs" (thus be run as 
testng test). Moreover, due to this def, when one runs tests from "test" 
directory, extra files (nontestng/* and tck/*) will also be compiled 
(which are unnecessary compiling).

In this patch:
Non-test files ("libraries") are moved to "lib" directory;
Real tests previously under "test" dir are moved to test/jdk/, and tests 
under "tck" dir are moved to test/tck/;
test/jdk/ and test/tck/ each has TEST.properties with def:
     TestNG.dirs = .
     lib.dirs = /java/time/lib /lib/testlibrary
MockIOExceptionAppendable.java is not used anywhere, removed.
MockSimplePeriod.java previously exist in both "tck" and "test", now it 
is under "lib".

With this change, non-test file then won't be run as testng test and no 
unnecessary compiling.

bug: https://bugs.openjdk.java.net/browse/JDK-8173411
webrev: http://cr.openjdk.java.net/~amlu/8173411/webrev.00/

Thanks,
Amy



More information about the core-libs-dev mailing list