malva: Fix ProcessTest#testExitValue

penberg · penberg · commit 4308632ad6b4 · 2010-11-12T18:45:47.000+02:00
The "more LICENSE" command returns successfully on my home laptop. Use "sleep
10" instead to make sure we're checking for exit value before the process under
test has exited.

Signed-off-by: Pekka Enberg &lt;penberg@kernel.org&gt;
diff --git a/src/malva/java/lang/ProcessTest.java b/src/malva/java/lang/ProcessTest.java
@@ -32,10 +32,10 @@ public static void testExitValue() {
     }
 
     try {
-      ProcessBuilder processBuilder = new ProcessBuilder("more", "LICENSE");
+      ProcessBuilder processBuilder = new ProcessBuilder("sleep", "10");
       final Process process = processBuilder.start();
 
-      assertThrows(new Block(){
+      assertThrows(new Block() {
         @Override public void run() {
           process.exitValue();
         }
@@ -119,4 +119,4 @@ public static void main(String[] args) {
     testGetOutputStream();
     testWaitFor();
   }
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -32,10 +32,10 @@ public static void testExitValue() {`
`32`	`32`	`}`
`33`	`33`
`34`	`34`	`try {`
`35`		`- ProcessBuilder processBuilder = new ProcessBuilder("more", "LICENSE");`
	`35`	`+ ProcessBuilder processBuilder = new ProcessBuilder("sleep", "10");`
`36`	`36`	`final Process process = processBuilder.start();`
`37`	`37`
`38`		`- assertThrows(new Block(){`
	`38`	`+ assertThrows(new Block() {`
`39`	`39`	`@Override public void run() {`
`40`	`40`	`process.exitValue();`
`41`	`41`	`}`
`@@ -119,4 +119,4 @@ public static void main(String[] args) {`
`119`	`119`	`testGetOutputStream();`
`120`	`120`	`testWaitFor();`
`121`	`121`	`}`
`122`		`-}`
	`122`	`+}`