RFR: 8281705: SourceLauncherTest.testSystemProperty isn't being run [v2]

Jaikiran Pai jpai at openjdk.java.net
Tue Mar 1 13:46:07 UTC 2022


On Wed, 23 Feb 2022 01:42:26 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Can I please get a review of this test-only change which fixes the issue noted in https://bugs.openjdk.java.net/browse/JDK-8281705? 
>> 
>> The change in this commit adds the `@Test` annotation on the test method to make sure it gets run. Before this change, the test run logs showed that `28 tests` were being run and now after this change, the logs report `29 tests` and I can see `testSystemProperty` in the test run logs.
>
> Jaikiran Pai has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
> 
>  - Merge latest from master branch
>  - 8281705: SourceLauncherTest.testSystemProperty isn't being run

Any reviews on this one please?

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

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


More information about the compiler-dev mailing list