Remove problematic unit test. Tests that depend on Thread.sleep() are generally unreliable.

Author: jrobinso

src/test/java/org/broad/igv/tools/FeatureSearcherTest.java

@@ -154,31 +154,4 @@ public FeatureSource<? extends Feature> getTestBedSource() throws IOException, T
         return TribbleFeatureSource.getFeatureSource(new ResourceLocator(path), genome);
     }
 
-    /**
-     * Test that stopping actually stops the search.
-     * We start searching a window which will never complete
-     * @throws Exception
-     */
-    @Test
-    public void testCancel() throws Exception{
-        FeatureSearcher searcher = new FeatureSearcher(getTestBedSource(), genome, "chr3", 0);
-        searcher.setSearchIncrement(100);
-
-        // Be careful monkeying around here.
-        // The test will exit and shut down the threadExecutor if there is nothing blocking on this thread
-        // This can make it seem like the search aborted prematurely
-        LongRunningTask.getThreadExecutor().execute(searcher);
-        Thread.sleep(500);
-        assertFalse(searcher.isDone());
-        searcher.cancel();
-
-        //Stopping is not instantaneous
-        int counter = 0;
-        while(!searcher.isDone()){
-            Thread.sleep(100);
-            counter++;
-        }
-        assertNull(searcher.getResult());
-        assertTrue(counter < 5);
-    }
 }