[jdk17u-dev] RFR: 8316696: Remove the testing base classes: IntlTest and CollatorTest [v3]
Goetz Lindenmaier
goetz at openjdk.org
Thu Jan 16 19:03:11 UTC 2025
> I backport this for parity with 17.0.15-oracle.
>
> Several enhancements touched the files edited in this change
> causing the need to resolve. Among others:
> 8283698: Refactor Locale constructors used in src/test
> 8287868: Localized names update in COMPAT locale provider
> Several tests needed resolving of Copyright. I omit those in the
> following list.
>
> test/jdk/java/text/BreakIterator/BreakIteratorTest.java
> Two of the modified tests not in 17.
>
> test/jdk/java/text/Collator/APITest.java
> test/jdk/java/text/Format/DateFormat/DateFormatTest.java
> I had to edit the quite different test output, but changes trivial.
>
> test/jdk/java/text/Format/NumberFormat/Bug4838107.java
> Later change "8317372: Refactor some NumberFormat tests to use JUnit" was already backported
> including some lines of this change.
>
> test/jdk/java/text/Format/NumberFormat/IntlTestDecimalFormatSymbols.java
> 19 added enhancement "8282625: Formatter caches Locale/DecimalFormatSymbols poorly"
> I had to edit the quite different test output, but changes trivial.
>
> test/jdk/java/util/Locale/LegacyCodesClassInvariant.java
> This includes a bit of test rework I had to resolve, but the changed
> code is identical to 21. The resolve is needed becuase of context
> differences.
>
> test/jdk/java/util/Locale/LocaleEnhanceTest.java
> test/jdk/java/util/Locale/LocaleTest.java
> These needed mostly trivial edits.
>
>
> test/jdk/java/util/TimeZone/TimeZoneTest.java
> Resolved as "8285844: TimeZone.getTimeZone(ZoneOffset) does not work for all
> ZoneOffsets and returns GMT unexpected" not in 17.
Goetz Lindenmaier has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
Resolved parts
-------------
Changes:
- all: https://git.openjdk.org/jdk17u-dev/pull/3214/files
- new: https://git.openjdk.org/jdk17u-dev/pull/3214/files/913d4553..5dba2d43
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=3214&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=3214&range=01-02
Stats: 2 lines in 2 files changed: 1 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk17u-dev/pull/3214.diff
Fetch: git fetch https://git.openjdk.org/jdk17u-dev.git pull/3214/head:pull/3214
PR: https://git.openjdk.org/jdk17u-dev/pull/3214
More information about the jdk-updates-dev
mailing list