Cleanup of isolate.status (first step)

tools/testing/dart/test_suite.dart

Index: tools/testing/dart/test_suite.dart
diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index 4f62e29916823b1eb57dac680cdcc58a31bbb52b..5f5e4efcaa360373ec23960e9c920440d88bcd38 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -355,7 +355,10 @@ class CCTestSuite extends TestSuite {
         SummaryReport.add(expectations);
       }
 
-      if (expectations.contains(SKIP)) return;
+      if (expectations.contains(SKIP) ||
+          expectations.contains(SKIP_BY_DESIGN)) {
+        return;
+      }
 
       var args = TestUtils.standardOptions(configuration);
       args.add(testName);
@@ -683,7 +686,10 @@ class StandardTestSuite extends TestSuite {
         SummaryReport.add(expectations);
       }
     }
-    if (expectations.contains(SKIP)) return;
+    if (expectations.contains(SKIP) ||
+        expectations.contains(SKIP_BY_DESIGN)) {
+      return;
+    }
 
     if (configuration['compiler'] != 'none' && info.hasCompileError) {
       // If a compile-time error is expected, and we're testing a
@@ -1907,6 +1913,7 @@ class TestUtils {
 class SummaryReport {
   static int total = 0;
   static int skipped = 0;
+  static int skippedByDesign = 0;
   static int noCrash = 0;
   static int pass = 0;
   static int failOk = 0;
@@ -1919,6 +1926,9 @@ class SummaryReport {
     ++total;
     if (expectations.contains(SKIP)) {
       ++skipped;
+    } else if (expectations.contains(SKIP_BY_DESIGN)) {
+      ++skipped;
+      ++skippedByDesign;
     } else {
       if (expectations.contains(PASS) && expectations.contains(FAIL) &&
           !expectations.contains(CRASH) && !expectations.contains(OK)) {
@@ -1950,7 +1960,7 @@ class SummaryReport {
   static void printReport() {
     if (total == 0) return;
     String report = """Total: $total tests
- * $skipped tests will be skipped
+ * $skipped tests will be skipped ($skippedByDesign skipped by design)
  * $noCrash tests are expected to be flaky but not crash
  * $pass tests are expected to pass
  * $failOk tests are expected to fail that we won't fix