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