Add a dedicated Result for skipped tests.

test/runner/json_reporter_test.dart

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 @TestOn("vm")
 
 import 'dart:convert';
 
 import 'package:path/path.dart' as p;
 import 'package:scheduled_test/descriptor.dart' as d;
@@ skipping 325 matching lines @@
         test('skip 2', () {}, skip: true);
         test('skip 3', () {}, skip: true);
       """, [
         _start,
         _allSuites(),
         _suite(0),
         _testStart(1, "loading test.dart", groupIDs: []),
         _testDone(1, hidden: true),
         _group(2, testCount: 3),
         _testStart(3, "skip 1", skip: true, line: 6, column: 9),
-        _testDone(3),
+        _testDone(3, skipped: true),
         _testStart(4, "skip 2", skip: true, line: 7, column: 9),
-        _testDone(4),
+        _testDone(4, skipped: true),
         _testStart(5, "skip 3", skip: true, line: 8, column: 9),
-        _testDone(5),
+        _testDone(5, skipped: true),
         _done()
       ]);
     });
 
     test("reports skipped groups", () {
       _expectReport("""
         group('skip', () {
           test('success 1', () {});
           test('success 2', () {});
           test('success 3', () {});
         }, skip: true);
       """, [
         _start,
         _allSuites(),
         _suite(0),
         _testStart(1, "loading test.dart", groupIDs: []),
         _testDone(1, hidden: true),
         _group(2, testCount: 0),
         _group(3, name: "skip", parentID: 2, skip: true, testCount: 0,
             line: 6, column: 9),
         _testStart(4, "skip", groupIDs: [2, 3], skip: true,
             line: 6, column: 9),
-        _testDone(4),
+        _testDone(4, skipped: true),
         _done()
       ]);
     });
 
     test("reports the skip reason if available", () {
       _expectReport("""
         test('skip 1', () {}, skip: 'some reason');
         test('skip 2', () {}, skip: 'or another');
       """, [
         _start,
          _allSuites(),
         _suite(0),
         _testStart(1, "loading test.dart", groupIDs: []),
         _testDone(1, hidden: true),
         _group(2, testCount: 2),
         _testStart(3, "skip 1", skip: "some reason", line: 6, column: 9),
-        _testDone(3),
+        _testDone(3, skipped: true),
         _testStart(4, "skip 2", skip: "or another", line: 7, column: 9),
-        _testDone(4),
+        _testDone(4, skipped: true),
         _done()
       ]);
     });
   });
 
   group("reports line and column numbers for", () {
     test("the first call to setUpAll()", () {
       _expectReport("""
         setUpAll(() {});
         setUpAll(() {});
@@ skipping 221 matching lines @@
   };
 }
 
 /// Returns the event emitted by the JSON reporter indicating that a test
 /// finished.
 ///
 /// The [result] parameter indicates the result of the test. It defaults to
 /// `"success"`.
 ///
 /// The [hidden] parameter indicates whether the test should not be displayed
-/// after finishing.
-Map _testDone(int id, {String result, bool hidden: false}) {
+/// after finishing. The [skipped] parameter indicates whether the test was
+/// skipped.
+Map _testDone(int id, {String result, bool hidden: false,
+    bool skipped: false}) {
   result ??= "success";
-  return {"type": "testDone", "testID": id, "result": result, "hidden": hidden};
+  return {
+    "type": "testDone",
+    "testID": id,
+    "result": result,
+    "hidden": hidden,
+    "skipped": skipped
+  };
 }
 
 /// Returns the event emitted by the JSON reporter indicating that the entire
 /// run finished.
 Map _done({bool success: true}) => {"type": "done", "success": success};
 
 /// Returns the serialized metadata corresponding to [skip].
 Map _metadata({skip}) {
   if (skip == true) {
     return {"skip": true, "skipReason": null};
   } else if (skip is String) {
     return {"skip": true, "skipReason": skip};
   } else {
     return {"skip": false, "skipReason": null};
   }
 }