RFR: 8281705: SourceLauncherTest.testSystemProperty isn't being run [v2]
Jaikiran Pai
jpai at openjdk.java.net
Wed Feb 23 01:42:26 UTC 2022
> 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
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/7456/files
- new: https://git.openjdk.java.net/jdk/pull/7456/files/cc170b6f..970b2a0e
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7456&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7456&range=00-01
Stats: 14358 lines in 429 files changed: 9621 ins; 2862 del; 1875 mod
Patch: https://git.openjdk.java.net/jdk/pull/7456.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/7456/head:pull/7456
PR: https://git.openjdk.java.net/jdk/pull/7456
More information about the compiler-dev
mailing list