[jdk11u-dev] RFR: 8244010: Simplify usages of ProcessTools.createJavaProcessBuilder in our tests [v3]

[Matthias Baesken]

On Mon, 10 Oct 2022 09:15:50 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

>> I backport this for parity with 11.0.17-oracle.
>> 
>> 
>> test/hotspot/jtreg/compiler/graalunit/common/GraalUnitTestLauncher.java
>> Simple resolve due to context
>> 
>> test/hotspot/jtreg/compiler/linkage/TestLinkageErrorInGenerateOopMap.java
>> Flags differ. Resolved.
>> 
>> test/hotspot/jtreg/compiler/loopstripmining/CheckLoopStripMining.java
>> Change already in 11.  Skipped.
>> 
>> test/hotspot/jtreg/gc/arguments/TestMaxRAMFlags.java
>> Test not in 11, skipped.
>> 
>> test/hotspot/jtreg/gc/arguments/TestSoftMaxHeapSizeFlag.java
>> Test was added in 13. skipped.
>> 
>> test/hotspot/jtreg/gc/g1/TestMarkStackSizes.java
>> Test was added in 15 by 8238855: Move G1ConcurrentMark flag sanity checks to g1Arguments
>> 
>> test/hotspot/jtreg/gc/metaspace/TestSizeTransitions.java
>> Test was added in 14 by 8227179: Test for new gc+metaspace=info output format
>> 
>> nvdimm tests not i 11
>> 
>> test/hotspot/jtreg/gc/z/TestAllocateHeapAt.java
>> Test was added in 15 by 8239129: ZGC: Allow -XX:AllocateHeapAt to use any filesystem
>> 
>> test/hotspot/jtreg/gc/z/TestHighUsage.java
>> Test was added in 13 by 8224185: ZGC: Introduce "High Usage" rule
>> 
>> test/hotspot/jtreg/gc/z/TestSmallHeap.java
>> Test was added in 14 by 8234338: ZGC: Improve small heap usage
>> 
>> test/hotspot/jtreg/runtime/Shutdown/ShutdownTest.java
>> Test was added in 13 by 8074355: make MutexLocker smarter about non-JavaThreads
>> 
>> test/hotspot/jtreg/runtime/cds/MaxMetaspaceSize.java
>> In 11, the test is at test/hotspot/jtreg/runtime/SharedArchiveFile/MaxMetaspaceSize.java
>> It was moved in 14 by 8202339: [TESTBUG] Consolidate the tests in runtime/SharedArchiveFile and runtime/appcds
>> Applied patch by hand.
>> 
>> test/hotspot/jtreg/runtime/cds/appcds/TestCommon.javaj
>> In 11, the test is in test/hotspot/jtreg/runtime/appcds/TestCommon.java
>> It was moved in 14 by 8202339: [TESTBUG] Consolidate the tests in runtime/SharedArchiveFile and runtime/appcds
>> Applied patch by hand. In addition, I removed makeCommandLineForAppCDS()
>> 
>> test/hotspot/jtreg/runtime/records/RedefineRecord.java
>> Skipped. Feature and test not in 11.
>> 
>> test/hotspot/jtreg/serviceability/logging/TestLogRotation.java
>> Arguments used differ.  Resolved.  Probably clean after pushing 8243568.
>> 
>> test/hotspot/jtreg/serviceability/sa/TestInstanceKlassSize.java
>> test/hotspot/jtreg/serviceability/sa/TestInstanceKlassSizeForInterface.java
>> Resolved due to context.
>
> Goetz Lindenmaier has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits:
> 
>  - Merge
>  - Backport 35af52dd4f5e05d5c9cd65db92f23b136404d157
>  - Backport d84e4f1fd68badf858dacef62314d1c1cbc2b97a

thanks for backporting , looks okay.

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

Marked as reviewed by mbaesken (Reviewer).

PR: https://git.openjdk.org/jdk11u-dev/pull/1397