commit 37c7992ac317c40a63c347f74dca2018b138d003
author William Hesse <whesse@google.com> Tue Mar 19 11:39:59 2019 +0000
committer William Hesse <whesse@google.com> Tue Mar 19 11:39:59 2019 +0000
tree 68fea4ddac93389c4a1019b9337d7d9b55f7965a
parent af5fc2d4d226ceee5f4bd624f567090d53282071

[testing] Make tests with multiple VMOptions have multiple names.

Change-Id: I16abe0cd68e5af5ee68eaff8b1702aec2a4bf5a6
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/97227
Reviewed-by: Jonas Termansen <sortie@google.com>

tools/testing/dart/test_suite.dart

diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index 2bf1708..9c4a92c 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -804,7 +804,11 @@
       var isCrashExpected = expectations.contains(Expectation.crash);
       var commands = makeCommands(info, vmOptionsVariant, allVmOptions,
           commonArguments, isCrashExpected);
-      enqueueNewTestCase(testName, commands, expectations, info);
+      var variantTestName = testName;
+      if (vmOptionsList.length > 1) {
+        variantTestName = "$testName/$vmOptionsVariant";
+      }
+      enqueueNewTestCase(variantTestName, commands, expectations, info);
     }
   }