JDK 10 RFR of test-only bug 8183320, et. al.: Better cleanup of temp files created by [N]IO tests

Alan Bateman Alan.Bateman at oracle.com
Tue Jul 18 11:09:59 UTC 2017


On 17/07/2017 20:49, Brian Burkhalter wrote:
> Please review at your convenience this patch for [1, 2, 3, 4]:
>
> http://cr.openjdk.java.net/~bpb/8183320/webrev.00/
>
> Note that without the existence check at line 60, the SetDefaultProvider test fails with a FileAlreadyExistsException.
>
>
In SetDefaultProvider then it's important that the tests get a unique 
temp directory, otherwise will get interference between the 
testExplodedWithXXXX tests. This should do it:

   Path here = Paths.get("");
   Patch patchdir = Files.createTempDirectory(here, "patch");

-Alan


More information about the nio-dev mailing list