You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by sg...@apache.org on 2010/08/14 21:32:47 UTC
svn commit: r985573 -
/commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java
Author: sgoeschl
Date: Sat Aug 14 19:32:47 2010
New Revision: 985573
URL: http://svn.apache.org/viewvc?rev=985573&view=rev
Log:
Improved test for EXEC-34
Modified:
commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java
Modified: commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java
URL: http://svn.apache.org/viewvc/commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java?rev=985573&r1=985572&r2=985573&view=diff
==============================================================================
--- commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java (original)
+++ commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java Sat Aug 14 19:32:47 2010
@@ -581,8 +581,10 @@ public class DefaultExecutorTest extends
// if you comment out the next line the test will fail
Thread.sleep(2000);
// terminate it
+ assertTrue(watchdog.isWatching());
watchdog.destroyProcess();
assertTrue("Watchdog should have killed the process",watchdog.killedProcess());
+ assertFalse("Watchdog is no longer watching the process",watchdog.isWatching());
}
/**