RFR: 8251267: CDS tests should use CDSTestUtils.getOutputDir instead of System.getProperty("user.dir")

Yumin Qi minqi at openjdk.java.net
Tue Dec 8 18:46:08 UTC 2020


On Tue, 8 Dec 2020 17:17:09 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

> Replacing
>     `String s = System.getProperty("user.dir");` 
> with
>     `String s = CDSTestUtils.getOutputDir(); `
> 
> Replacing
>     `File dir = new File(System.getProperty("user.dir"));`
> with
>     `File dir = CDSTestUtils.getOutputDirAsFile();`
> 
> Passed tier1, hs-tiers 2/3/4 tests.

Looks good!

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

Marked as reviewed by minqi (Reviewer).

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


More information about the hotspot-runtime-dev mailing list