Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(495)

Unified Diff: json_reporter.schema.json

Issue 1469863005: Add JSON protocol support for groups. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: Code review changes Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « json_reporter.md ('k') | lib/src/backend/invoker.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: json_reporter.schema.json
diff --git a/json_reporter.schema.json b/json_reporter.schema.json
index b37e5882383d34b318b7cbe9cb47e34b8235ab31..cf932448927facf80d3f67570221df6f87c7db8f 100644
--- a/json_reporter.schema.json
+++ b/json_reporter.schema.json
@@ -12,6 +12,20 @@
}
},
+ "Group": {
+ "required": ["id", "name"],
+ "properties": {
+ "id": {"type": "integer", "minimum": 0},
+ "parentID": {
+ "oneOf": [{"type": "integer", "minimum": 0}, {"type": "null"}]
+ },
+ "name": {
+ "oneOf": [{"type": "string"}, {"type": "null"}],
+ },
+ "metadata": {"$ref": "#/definitions/Metadata"}
+ }
+ },
+
"Metadata": {
"required": ["skip", "skipReason"],
"properties": {
@@ -52,6 +66,15 @@
},
{
+ "title": "GroupEvent",
+ "required": ["group"],
+ "properties": {
+ "type": {"enum": ["group"]},
+ "group": {"$ref": "#/definitions/Group"}
+ }
+ },
+
+ {
"title": "PrintEvent",
"required": ["message"],
"properties": {
@@ -97,7 +120,9 @@
"properties": {
"type": {
"not": {
- "enum": ["start", "testStart", "print", "error", "testDone", "done"]
+ "enum": [
+ "start", "testStart", "group", "print", "error", "testDone", "done"
+ ]
}
}
}
« no previous file with comments | « json_reporter.md ('k') | lib/src/backend/invoker.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698