Remove InterruptibleTest#testInterruptibleCommand
This test has been flaky for a long time and does not test what it is supposed to test.
Fixes #248.
--
MOS_MIGRATED_REVID=118668487
diff --git a/src/test/java/com/google/devtools/build/lib/shell/InterruptibleTest.java b/src/test/java/com/google/devtools/build/lib/shell/InterruptibleTest.java
index 68b20ba..6ef9915 100644
--- a/src/test/java/com/google/devtools/build/lib/shell/InterruptibleTest.java
+++ b/src/test/java/com/google/devtools/build/lib/shell/InterruptibleTest.java
@@ -13,10 +13,8 @@
// limitations under the License.
package com.google.devtools.build.lib.shell;
-import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
import org.junit.After;
import org.junit.Before;
@@ -90,32 +88,4 @@
assertTrue("Main thread was not interrupted during command execution!",
mainThread.isInterrupted());
}
-
- /**
- * Test that interrupting a thread in an "interruptible" Command.execute
- * causes preserves the thread's interruptible status, terminates the
- * subprocess, and returns promptly.
- */
- @Test
- public void testInterruptibleCommand() throws Exception {
- try {
- command.execute(Command.NO_INPUT,
- Command.NO_OBSERVER,
- System.out,
- System.err,
- true); // => interruptible
- fail("Subprocess not aborted!");
- } catch (AbnormalTerminationException e) {
- assertEquals("Process terminated by signal 15", // SIGINT
- e.getMessage());
- }
-
- // We don't assert that the interrupter thread has exited; due to prompt
- // termination it might still be running.
-
- // The interrupter thread should have set the main thread's interrupt flag.
- assertTrue("Main thread was not interrupted during command execution!",
- mainThread.isInterrupted());
-
- }
}
