[11u] Test failing since downport of 8234011: (zipfs) Memory leak in ZipFileSystem.releaseDeflater() was downported

[Lindenmaier, Goetz]

Hi Volker

That sounds good, thanks for investigating!

Best regards,
  Goetz

-----Original Message-----
From: Volker Simonis <volker.simonis at gmail.com> 
Sent: Freitag, 4. September 2020 10:10
To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
Cc: jdk-updates-dev at openjdk.java.net
Subject: Re: [11u] Test failing since downport of 8234011: (zipfs) Memory leak in ZipFileSystem.releaseDeflater() was downported

Hi Goetz,

yes, this is a test problem caused by the way we construct the URI
from a Path. This part of the test had to be changed for the downport
because the original patch used a version of
FileSystems.newFileSystem(..) which directly takes a Path argument and
is not available in jdk11.

I think the fix would be similar to
https://bugs.openjdk.java.net/browse/JDK-8186227. I'll take a look.

Thanks for finding this,
Volker

On Wed, Sep 2, 2020 at 9:23 AM Lindenmaier, Goetz
<goetz.lindenmaier at sap.com> wrote:
>
> Hi,
>
> We see test jdk/nio/zipfs/ReleaseDeflater.java failing on Windows 64-bit since
> "8234011: (zipfs) Memory leak in ZipFileSystem.releaseDeflater()" was downported.
>
> java.lang.IllegalArgumentException: Illegal character in opaque part at index 11: jar:file:C:\\jvmtests\\output_ ....  \\jdk\\nio\\zipfs\\ReleaseDeflater\\ReleaseDeflaterTest.zip
>         at java.base/java.net.URI.create(URI.java:883)
>         at ReleaseDeflater.main(ReleaseDeflater.java:51)
>         at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>         at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
>
> https://bugs.openjdk.java.net/browse/JDK-8234011
> https://hg.openjdk.java.net/jdk-updates/jdk11u-dev/rev/cb299db4a569
>
> I would assume this is a problem with the test, looking at the change.
> Does someone have an idea how to fix this?
>
> Best regards,
>   Goetz.