<i18n dev> RFR: JDK-8316696: Remove the testing base classes: IntlTest and CollatorTest [v3]

Justin Lu jlu at openjdk.org
Thu Sep 28 18:13:12 UTC 2023


> Please review this PR which removes the i18n related testing base classes `IntlTest` and `CollatorTest` and converts all the tests that use them,
> 
> IntlTest and CollatorTest are testing classes which are extended by tests in `text/`, `util/Locale`, `util/TimeZone`, and `util/Calendar`. The abstract testing classes are quite dated and have caused issues such as: variation between OS, hiding stack trace, and causing tests to spuriously pass.
> 
> This change mainly automates a low level conversion of all the tests (75) using the frameworks; all tests were converted to use JUnit instead. (With the exception of `DateFormatRoundTripTest` due to the nature of the test).
> 
> The main changes can be viewed in the following commits
> 
> scripted changes - [c0ece01 ](https://github.com/openjdk/jdk/commit/c0ece01e91479a020d5c6dce937dc827472b763b)
> - Converts the IntlTest methods logln, log, err, and errln
> - Adds the JUnit Test annotation to tests that should be ran
> - Adjusts the Jtreg tags accordingly
> - Remove the main method
> - Insert initAll() methods for tests that previously adjusted the JVM default Locale/TimeZone in the main method
> 
> manual changes - [9a54910](https://github.com/openjdk/jdk/commit/9a5491065a94a4dc7a05194f3b8330efba8077b7)
> - Some tests that had extensive logic in the main method or did not follow the general IntlTest format had to be manually adjusted
> - Also clarified some tests that had optional argument setup in the main method (now removed)
> 
> removal of IntlTest and CollatorTest - [8ee9f9c](https://github.com/openjdk/jdk/commit/8ee9f9c79f79210ee6244186970d87a1cac05556)
> - Removes both the test classes. 
> - Replaces CollatorTest with CollatorTestUtils
> 
> Tiers 1-3 clean

Justin Lu has updated the pull request incrementally with one additional commit since the last revision:

  cleanup util classes in text/testlib

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/15954/files
  - new: https://git.openjdk.org/jdk/pull/15954/files/f90266f4..3c676794

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15954&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15954&range=01-02

  Stats: 40 lines in 3 files changed: 20 ins; 2 del; 18 mod
  Patch: https://git.openjdk.org/jdk/pull/15954.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15954/head:pull/15954

PR: https://git.openjdk.org/jdk/pull/15954


More information about the i18n-dev mailing list