[jdk21u-dev] Integrated: 8310683: Refactor StandardCharset/standard.java to use JUnit
Amos Shi
ashi at openjdk.org
Fri Aug 16 17:18:59 UTC 2024
On Thu, 8 Aug 2024 20:20:42 GMT, Amos Shi <ashi at openjdk.org> wrote:
> Backport of [JDK-8310683](https://bugs.openjdk.org/browse/JDK-8310683)
> - This PR contains three commits
> - `commit 1` is the `clean` backport of `Standard.java` file from [JDK-8310047](https://bugs.openjdk.org/browse/JDK-8310047), which is the dependency of current PR [JDK-8310683](https://bugs.openjdk.org/browse/JDK-8310683)
> - `commit 2` is the `clean` backport of [JDK-8310683](https://bugs.openjdk.org/browse/JDK-8310683) original commit
> - `commit 3` is fixing the compile error of the test case, to remove the charsets added by [JDK-8310047](https://bugs.openjdk.org/browse/JDK-8310047) in Java 22
>
>
> /Users/I048686/SAPDevelop/github.com/dev-8310683-21/test/jdk/java/nio/charset/StandardCharsets/Standard.java:140: error: cannot find symbol
> Arguments.of(StandardCharsets.UTF_32BE, "UTF-32BE"),
> ^
> symbol: variable UTF_32BE
> location: class StandardCharsets
> /Users/I048686/SAPDevelop/github.com/dev-8310683-21/test/jdk/java/nio/charset/StandardCharsets/Standard.java:141: error: cannot find symbol
> Arguments.of(StandardCharsets.UTF_32LE, "UTF-32LE"),
> ^
> symbol: variable UTF_32LE
> location: class StandardCharsets
> /Users/I048686/SAPDevelop/github.com/dev-8310683-21/test/jdk/java/nio/charset/StandardCharsets/Standard.java:142: error: cannot find symbol
> Arguments.of(StandardCharsets.UTF_32, "UTF-32")
> ^
> symbol: variable UTF_32
> location: class StandardCharsets
>
>
> Testing
> - Local: Test passed on `MacOS 14.6.1` on Apple M1 Max
> - `Standard.java`: Test results: passed: 1
> - Pipeline: All checks have passed
> - Testing Machine: SAP nightlies Passed on `2024-08-15`
> - Automated jtreg test: `jtreg_jdk_tier2`, Started at `2024-08-15 04:34:01+01:00`
> - java/nio/charset/StandardCharsets/Standard.java: SUCCESSFUL GitHub 📊 - [04:55:15.771 -> 1,774 msec]
This pull request has now been integrated.
Changeset: 4e84d4dc
Author: Amos Shi <ashi at openjdk.org>
URL: https://git.openjdk.org/jdk21u-dev/commit/4e84d4dc514192e4cdf8e2c7c474847832987ab9
Stats: 128 lines in 1 file changed: 49 ins; 28 del; 51 mod
8310683: Refactor StandardCharset/standard.java to use JUnit
Reviewed-by: mdoerr
Backport-of: d82ade354570a930d8282684a8ffd368c613defc
-------------
PR: https://git.openjdk.org/jdk21u-dev/pull/903
More information about the jdk-updates-dev
mailing list