Hi Sergei,
I could see you modified tests only in
/test/java/time/*test*/java/time/format/ directory.
Won't the tests from test/java/time/*tck*/java/time/format/ directory
fail with same issue?
Thanks and Regards,
Nadeesh
On 12/26/2016 8:27 PM, Sergei Kovalev wrote:
Hello Team,
Please review below fix for tests.
Bug ID: https://bugs.openjdk.java.net/browse/JDK-8171958
Web review: http://cr.openjdk.java.net/~skovalev/8171958/webrev.00/
Issue: some tests fails in case of module limitation by
'--limit-module java.base' command line option.
Root cause: The tests uses locale data that stored in separate
resource file "jdk.localedata".
Solution: Add declaration of required module. In same cases a test
file contains many test items, part of which could be executed with
java.base module only. In this cases we can separate the items and
extract that items which depend on locale data and run them
individually. Therefore this proposal contains additional test files
where added "WithLocale" suffix which demonstrate dependency on
resources. Alternative solution could be add a required module
declaration "jdk.localedata" to all files. However this will lead to
unnecessary test coverage reduction.
--
Thanks and Regards,
Nadeesh TV
