On Mon, 17 Aug 2026 09:20:47 GMT, Alan Bateman <[email protected]> wrote:
>> Naoto Sato has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 726 commits: >> >> - Merge remote-tracking branch 'jdk-sandbox/json' into >> JDK-8381976-Implementation-for-Simple-JSON-API >> - Address comment regarding asDouble wording in class spec >> - Reflects Alan's comments >> - Merge branch 'master' into JDK-8381976-Implementation-for-Simple-JSON-API >> - Merge remote-tracking branch 'jdk-sandbox/json' into >> JDK-8381976-Implementation-for-Simple-JSON-API >> - Removed unused imports >> - Removed unnecessary qualifiers >> - Merge remote-tracking branch 'jdk-sandbox/json' into >> JDK-8381976-Implementation-for-Simple-JSON-API >> - Merge branch 'toDisplayString' into json >> - wording >> - ... and 716 more: https://git.openjdk.org/jdk/compare/16bf2730...76272098 > > test/jdk/jdk/incubator/json/TestOtherImpl.java line 29: > >> 27: * @test >> 28: * @bug 8381976 >> 29: * @summary Tests for other implementations that extend JsonValue >> sub-interfaces > > For now at least, > JsonString/JsonNumber/JsonObject/JsonArray/JsonBoolean/JsonNull are > non-sealed, but the test for this "extension point" only tests two > custom-JsonString methods. Are there other generate tests that could be added > here? Can create tests that verify those sub-interfaces are non-sealed. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32282#discussion_r3797823222
