--- jsr166/src/test/tck/ExecutorCompletionService9Test.java 2016/05/23 22:58:06 1.4 +++ jsr166/src/test/tck/ExecutorCompletionService9Test.java 2016/10/31 15:52:27 1.6 @@ -7,6 +7,7 @@ import java.util.ArrayList; import java.util.Collection; +import java.util.Comparator; import java.util.List; import java.util.Set; import java.util.HashSet; @@ -15,7 +16,6 @@ import java.util.concurrent.CompletionSe import java.util.concurrent.ExecutionException; import java.util.concurrent.Executor; import java.util.concurrent.ExecutorCompletionService; -import java.util.concurrent.ExecutorService; import java.util.concurrent.Future; import junit.framework.Test; @@ -69,10 +69,10 @@ public class ExecutorCompletionService9T use(result); } - HashSet results; + ArrayList results; void use(Integer x) { - if (results == null) results = new HashSet(); + if (results == null) results = new ArrayList(); results.add(x); } @@ -81,6 +81,7 @@ public class ExecutorCompletionService9T */ public void testSolveAll() throws InterruptedException, ExecutionException { + results = null; Set> solvers = Set.of( () -> null, () -> 1, @@ -88,7 +89,8 @@ public class ExecutorCompletionService9T () -> 3, () -> null); solveAll(cachedThreadPool, solvers); - assertEquals(Set.of(1, 2, 3), results); + results.sort(Comparator.naturalOrder()); + assertEquals(List.of(1, 2, 3), results); } /** @@ -96,6 +98,7 @@ public class ExecutorCompletionService9T */ public void testSolveAny() throws InterruptedException { + results = null; Set> solvers = Set.of( () -> { throw new ArithmeticException(); }, () -> null, @@ -103,7 +106,7 @@ public class ExecutorCompletionService9T () -> 2); solveAny(cachedThreadPool, solvers); assertEquals(1, results.size()); - Integer elt = results.iterator().next(); + Integer elt = results.get(0); assertTrue(elt.equals(1) || elt.equals(2)); }