RFR: 8287121: Fix typo in jlink's description resource key lookup

Jaikiran Pai jpai at openjdk.java.net
Tue May 24 05:01:46 UTC 2022


On Sun, 22 May 2022 05:58:25 GMT, Christian Stein <cstein at openjdk.org> wrote:

> Commit https://github.com/openjdk/jdk/commit/655500a4f5e3abcff176599604deceefb6ca6640 for issue [JDK-8286654](https://bugs.openjdk.java.net/browse/JDK-8286654) added an optional description accessor on the `ToolProvider` interface. It included a typo in` jlink`'s description resource key lookup: `"jlink.desciption"`
> 
> This follow-up commit fixes the typo by adding the missing `r` character: `"jlink.description"`.
> 
> This commit also also adds an automated check that ensures all current and future tool provider implementations within the JDK don't throw an exception when invoking their name and description accessor methods. Specific tool names and descriptions are not expected by this general test.

test/jdk/java/util/spi/ToolProviderDescriptionTest.java line 40:

> 38:     public static void main(String... args) throws Exception {
> 39:         List<String> descriptions = listToolDescriptions();
> 40:         if (descriptions.isEmpty()) {

Hello @sormuras,
Won't this condition always be "false", because from what I see in this test code, the `descriptions` list will never be empty since in the `describeTool` method of this test we always return a `String` instance to be added to the `List`, even if there's no `description` for the `ToolProvider` instance.

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

PR: https://git.openjdk.java.net/jdk/pull/8825


More information about the core-libs-dev mailing list