Looks good to me.
Xuelei
On 7/29/2019 7:03 PM, Weijun Wang wrote:
Turns out a previous code change accidentally ignore the input of a
process, and this test calls this method.
Please review the patch below:
*diff --git a/test/jdk/ProblemList.txt b/test/jdk/ProblemList.txt*
*--- a/test/jdk/ProblemList.txt*
*+++ b/test/jdk/ProblemList.txt*
@@ -661,8 +661,6 @@
sun/security/tools/jarsigner/warnings/BadKeyUsageTest.java
8026393 generic-all
sun/security/tools/jarsigner/PreserveRawManifestEntryAndDigest.java
8228431 solaris-all
-sun/security/tools/jarsigner/Warning.java 8228745
generic-all
-
javax/net/ssl/ServerName/SSLEngineExplorerMatchedSNI.java
8212096 generic-all
javax/net/ssl/DTLS/PacketLossRetransmission.java
8169086 macosx-x64
javax/net/ssl/DTLS/RespondToRetransmit.java
8169086 macosx-x64
*diff --git a/test/lib/jdk/test/lib/process/ProcessTools.java
b/test/lib/jdk/test/lib/process/ProcessTools.java*
*--- a/test/lib/jdk/test/lib/process/ProcessTools.java*
*+++ b/test/lib/jdk/test/lib/process/ProcessTools.java*
@@ -360,7 +360,7 @@
* @return The {@linkplain OutputAnalyzer} instance wrapping the
process.
*/
public static OutputAnalyzer executeProcess(ProcessBuilder pb,
String input) throws Exception {
- return executeProcess(pb, null, null);
+ return executeProcess(pb, input, null);
}
/**
Noreg-self.
Thanks,
Max
