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