[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