[jdk11u-dev] Integrated: 8210112: remove jdk.testlibrary.ProcessTools

[Goetz Lindenmaier]

On Fri, 25 Mar 2022 07:12:36 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

> I want to backport this ot simplify follow-up test backports.
> 
> I had to adapt the change in several means.  This is a list 
> of the changes to resolve failed hunks:
> 
> Already fixed:
> patching file test/jdk/com/sun/jdi/ExclusiveBind.java
> 
> Resolved, 8211171 move JarUtils to top-level testlibrary already backported.
> patching file test/jdk/java/lang/ClassLoader/getResource/automaticmodules/Driver.java
> patching file test/jdk/java/lang/ClassLoader/securityManager/ClassLoaderTest.java
> patching file test/jdk/java/lang/instrument/executableJAR/ExecJarWithAgent.java
> patching file test/jdk/java/rmi/module/ModuleTest.java
> 
> Context
> patching file test/jdk/java/lang/System/OsVersionTest.java
> patching file test/jdk/java/security/Policy/SignedJar/SignedJarTest.java
> patching file test/jdk/java/util/Arrays/TimSortStackSize2.java
> 
> Copyright
> patching file test/jdk/javax/management/remote/mandatory/connection/DefaultAgentFilterTest.java
> 
> Resolved, 8211171 move JarUtils to top-level testlibrary already backported.
> patching file test/jdk/jdk/modules/scenarios/automaticmodules/RunWithAutomaticModules.java
> patching file test/jdk/sun/security/tools/jarsigner/TsacertOptionTest.java
> 
> Copyright
> patching file test/jdk/sun/security/tools/jarsigner/Warning.java
> patching file test/jdk/sun/security/tools/jarsigner/warnings/BadKeyUsageTest.java
> 
> Resolved, 8211171 move JarUtils to top-level testlibrary already backported.
> patching file test/jdk/sun/security/tools/jarsigner/warnings/NoTimestampTest.java
> 
> File not in 11:  --- probably goes away
> test/jdk/sun/tools/jcmd/JcmdBase.java b/test/jdk/sun/tools/jcmd/JcmdBase.java
> patching file test/jdk/sun/tools/jcmd/TestJcmdDefaults.java
> patching file test/jdk/sun/tools/jcmd/TestJcmdSanity.java
> 
> Copyright
> patching file test/jdk/sun/tools/jmap/BasicJMapTest.java
> 
> Resolved
> patching file test/jdk/sun/tools/jstatd/JstatdTest.java
> 
> Resolved, 8211171 move JarUtils to top-level testlibrary already backported.
> patching file test/jdk/tools/jlink/basic/BasicTest.java
> patching file test/jdk/tools/launcher/modules/addexports/manifest/AddExportsAndOpensInManifest.java
> patching file test/jdk/tools/launcher/modules/addreads/AddReadsTest.java
> patching file test/jdk/tools/launcher/modules/illegalaccess/IllegalAccessTest.java
> patching file test/jdk/tools/launcher/modules/patch/basic/PatchTest.java
> patching file test/jdk/tools/launcher/modules/patch/basic/PatchTestWarningError.java
> patching file test/jdk/tools/launcher/modules/validate/ValidateModulesTest.java
> 
> On top of this, I had to adapt a few tests to work after removal of 
> the old testlibrary file.
> 
> Last, I had to move the `change` of  "8217375: jarsigner breaks old signature with long lines in manifest"
> to OutputAnalyzer.java from the old file to the new one.
> 
> Now all test are passing.

This pull request has now been integrated.

Changeset: 9aa728e6
Author:    Goetz Lindenmaier <goetz at openjdk.org>
URL:       https://git.openjdk.java.net/jdk11u-dev/commit/9aa728e61efe366c00ce78ae2d9bbcc56e3d974b
Stats:     2571 lines in 184 files changed: 335 ins; 1687 del; 549 mod

8210112: remove jdk.testlibrary.ProcessTools

Reviewed-by: mdoerr
Backport-of: 21a0458422c462f5ae82a98ce08c6460db26c67e

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

PR: https://git.openjdk.java.net/jdk11u-dev/pull/945