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

Goetz Lindenmaier goetz at openjdk.java.net
Fri Mar 25 07:21:22 UTC 2022


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.

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

Commit messages:
 - Backport 21a0458422c462f5ae82a98ce08c6460db26c67e

Changes: https://git.openjdk.java.net/jdk11u-dev/pull/945/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk11u-dev&pr=945&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8210112
  Stats: 2571 lines in 184 files changed: 335 ins; 1687 del; 549 mod
  Patch: https://git.openjdk.java.net/jdk11u-dev/pull/945.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk11u-dev pull/945/head:pull/945

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


More information about the jdk-updates-dev mailing list