Return SpawnResults using a List instead of a Set.

Currently we don't care about the list order of SpawnResults. However, we may care about the list order later. Also, if the equals() method for SpawnResults is ever changed then it may be problematic to return SpawnResults in a Set.

Aside: ActionResults use SpawnResults to calculate cumulative execution times for Actions, and may provide other metrics in future.

RELNOTES: None.
PiperOrigin-RevId: 176579460

1 files changed, 1 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/worker/WorkerTestStrategy.java b/src/main/java/com/google/devtools/build/lib/worker/WorkerTestStrategy.java
index e06b6c4f1b..274c953484 100644
--- a/src/main/java/com/google/devtools/build/lib/worker/WorkerTestStrategy.java
+++ b/src/main/java/com/google/devtools/build/lib/worker/WorkerTestStrategy.java
@@ -213,7 +213,7 @@ public class WorkerTestStrategy extends StandaloneTestStrategy {
         builder.setTestCase(details);
       }
 
-      return StandaloneTestResult.create(ImmutableSet.of(), builder.build());
+      return StandaloneTestResult.create(ImmutableList.of(), builder.build());
     } catch (IOException | InterruptedException e) {
       if (worker != null) {
         workerPool.invalidateObject(key, worker);