Fix flaky test

Author: jhalterman

core/src/test/java/dev/failsafe/issues/Issue231Test.java

@@ -8,19 +8,23 @@
 
 import java.time.Duration;
 import java.util.concurrent.ExecutionException;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Executors;
 import java.util.concurrent.atomic.AtomicBoolean;
 
 import static org.testng.Assert.assertTrue;
 
-@Test(enabled = false)
+@Test
 public class Issue231Test {
   /**
-   * Timeout, even with interruption, should wait for the execution to complete.
+   * Timeout, even with interruption, should wait for the execution to complete before completing the future.
    */
   public void shouldWaitForExecutionCompletion() {
+    // Use a separate executorService for this test in case the common pool is full
+    ExecutorService executorService = Executors.newFixedThreadPool(2);
     Timeout<Object> timeout = Timeout.builder(Duration.ofMillis(100)).withInterrupt().build();
     AtomicBoolean executionCompleted = new AtomicBoolean();
-    Asserts.assertThrows(() -> Failsafe.with(timeout).runAsync(() -> {
+    Asserts.assertThrows(() -> Failsafe.with(timeout).with(executorService).runAsync(() -> {
       try {
         Thread.sleep(1000);
       } catch (InterruptedException ignore) {

pom.xml

@@ -168,7 +168,7 @@
       <plugin>
         <groupId>org.jacoco</groupId>
         <artifactId>jacoco-maven-plugin</artifactId>
-        <version>0.8.6</version>
+        <version>0.8.7</version>
         <executions>
           <execution>
             <id>prepare-agent</id>