[14] RFR 8228745: sun/security/tools/jarsigner/Warning.java test timeout

Xuelei Fan xuelei.fan at oracle.com
Tue Jul 30 03:12:47 UTC 2019


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
> 


More information about the security-dev mailing list