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 2016/01/09 00:40:25 UTC

svn commit: r1723817 - /commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/issues/Exec57Test.java

Author: sgoeschl
Date: Fri Jan  8 23:40:25 2016
New Revision: 1723817

URL: http://svn.apache.org/viewvc?rev=1723817&view=rev
Log:
[EXEC-86] Disable "testExecutionOfBackgroundProcess" for Unix as well

Modified:
    commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/issues/Exec57Test.java

Modified: commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/issues/Exec57Test.java
URL: http://svn.apache.org/viewvc/commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/issues/Exec57Test.java?rev=1723817&r1=1723816&r2=1723817&view=diff
==============================================================================
--- commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/issues/Exec57Test.java (original)
+++ commons/proper/exec/trunk/src/test/java/org/apache/commons/exec/issues/Exec57Test.java Fri Jan  8 23:40:25 2016
@@ -23,6 +23,7 @@ import org.apache.commons.exec.CommandLi
 import org.apache.commons.exec.DefaultExecutor;
 import org.apache.commons.exec.OS;
 import org.apache.commons.exec.PumpStreamHandler;
+import org.junit.Ignore;
 import org.junit.Test;
 
 import java.io.IOException;
@@ -42,21 +43,12 @@ public class Exec57Test extends Abstract
      * an ExecuteException is thrown to notify the caller. But this case the threads are still
      * around causing a resource leak.
      *
-     * @TODO [EXEC-57] Broken for Mac OS X
+     * @TODO [EXEC-57] Broken for Mac OS X & Linux
      */
+    @Ignore("Broken for Unix-based systems")
     @Test(timeout = TEST_TIMEOUT)
     public void testExecutionOfBackgroundProcess() throws IOException {
 
-        if (OS.isFamilyMac()) {
-            testIsBrokenForCurrentOperatingSystem();
-            return;
-        }
-
-        if (!OS.isFamilyUnix()) {
-            testNotSupportedForCurrentOperatingSystem();
-            return;
-        }
-
         final CommandLine cmdLine = new CommandLine("sh").addArgument("-c").addArgument("./src/test/scripts/issues/exec-57-nohup.sh", false);
         final DefaultExecutor executor = new DefaultExecutor();
         final PumpStreamHandler pumpStreamHandler = new PumpStreamHandler(System.out, System.err);