On Tue, 26 Jan 2021 22:55:32 GMT, Brent Christian <bchri...@openjdk.org> wrote:
>> Mahendra Chhipa has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Implemented the review comments. > > test/jdk/java/lang/Class/getEnclosingClass/EnclosingClassTest.java line 126: > >> 124: Path pkg1Dir = Paths.get(SRC_DIR + "/pkg1"); >> 125: FileUtils.deleteFileTreeWithRetry(pkg1Dir); >> 126: } > > I'm not convinced the `@AfterClass` method is necessary. Pre-cleanup is > already done on LL94-95. And occasionally, test-generated artifacts are > helpful in post-mortem analysis. To keep the source code repo clean from temporary generated files, these files are removed after test. Test logs have the information about the generated files for the Postmortem analysis. ------------- PR: https://git.openjdk.java.net/jdk/pull/2170
