> Currently these tests ignore vm flags, In most cases I've updated them to use 
> `ProcessTools.createTestJavaProcessBuilder` this usually required some 
> cleanup also.
> 
> `test/jdk/java/net/ServerSocket/AcceptCauseFileDescriptorLeak.java` and 
> `test/jdk/java/net/URLConnection/6212146/TestDriver.java` have been set to 
> use `@require vm.flagless` because they both use `sh` commands.
> 
> I've ran these changes against tiers 1-3 and everything seems stable

Darragh Clarke has updated the pull request incrementally with two additional 
commits since the last revision:

 - Update test/jdk/java/net/ServerSocket/AcceptInheritHandle.java
   
   Co-authored-by: Andrey Turbanov <turban...@gmail.com>
 - fixed copyright year

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17787/files
  - new: https://git.openjdk.org/jdk/pull/17787/files/a9fcc6c2..75619936

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17787&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17787&range=00-01

  Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/17787.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17787/head:pull/17787

PR: https://git.openjdk.org/jdk/pull/17787

Reply via email to