RFR: 8253761: Wrong URI syntax printed by jar --describe-module

[Alan Bateman]

On Mon, 5 Oct 2020 14:08:32 GMT, Christian Stein <cstein at openjdk.org> wrote:

>> I think the change exposes issues in a couple of tests. It looks like partialUpdateFooModuleInfo
>> (modularJar/Basic.java) expects the wrong result, as does some of the tests in mmrjar/Basic.java.
>
> I can't find the `modularJar/Basic.java` test run in the logs. In which job/tier/part does it hide?
> 
>> I think the change exposes issues in a couple of tests.
> 
> The one that fails the linked "pre-submit test" run above is already reported as
> https://bugs.openjdk.java.net/browse/JDK-8249095 ("Aux.java" is not a valid file name on Windows).
>> It looks like partialUpdateFooModuleInfo (modularJar/Basic.java) expects the wrong result, as does some of the tests in
>> mmrjar/Basic.java.
> 
> Something for a follow-up issue/PR?

The tests are test/jdk/tools/jar/modularJar/Basic.java and test/jdk/tools/jar/mmrjar/Basic.java. I've verified that
they will fail with the update to the jar tool so we'll need to change this as part of this PR.

The tests are in the core_tools test group, which seems to be tier2. I suspect the pre-submit job is running tier1.

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

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