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

Unified Diff: json_reporter.schema.json

Issue 1637193003: Add suite information to the JSON reporter. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: fix changelog Created 4 years, 11 months 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
« json_reporter.md ('K') | « json_reporter.md ('k') | lib/src/runner.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 946e8e687ad3c82205a2228947622981bb8d1338..00b0a34a18bed3df11cc796b4973bfc18bec395f 100644
--- a/json_reporter.schema.json
+++ b/json_reporter.schema.json
@@ -4,10 +4,11 @@
"definitions": {
"Test": {
- "required": ["id", "name", "groupIDs", "metadata"],
+ "required": ["id", "name", "suiteID", "groupIDs", "metadata"],
"properties": {
"id": {"type": "integer", "minimum": 0},
"name": {"type": "string"},
+ "suiteID": {"type": "integer", "minimum": 0},
"groupIDs": {
"type": "array",
"items": {"type": "integer", "minimum": 0}
@@ -16,15 +17,27 @@
}
},
+ "Suite": {
+ "required": ["id", "platform", "path"],
+ "properties": {
+ "id": {"type": "integer", "minimum": 0},
+ "platform": {
+ "oneOf": [{"type": "string"}, {"type": "null"}]
+ },
+ "path": {"type": "string"}
+ }
+ },
+
"Group": {
- "required": ["id", "name"],
+ "required": ["id", "name", "suiteID"],
"properties": {
"id": {"type": "integer", "minimum": 0},
+ "suiteID": {"type": "integer", "minimum": 0},
"parentID": {
"oneOf": [{"type": "integer", "minimum": 0}, {"type": "null"}]
},
"name": {
- "oneOf": [{"type": "string"}, {"type": "null"}],
+ "oneOf": [{"type": "string"}, {"type": "null"}]
},
"metadata": {"$ref": "#/definitions/Metadata"}
}
@@ -53,7 +66,7 @@
"required": ["protocolVersion", "runnerVersion"],
"properties": {
"type": {"enum": ["start"]},
- "protocolVersion": {"type": "string", "pattern": "^0\.1\."},
+ "protocolVersion": {"type": "string", "pattern": "^0\\.1\\."},
"runnerVersion": {
"oneOf": [{"type": "string"}, {"type": "null"}]
}
@@ -70,6 +83,15 @@
},
{
+ "title": "SuiteEvent",
+ "required": ["suite"],
+ "properties": {
+ "type": {"enum": ["suite"]},
+ "group": {"$ref": "#/definitions/Suite"}
+ }
+ },
+
+ {
"title": "GroupEvent",
"required": ["group"],
"properties": {
@@ -128,7 +150,8 @@
"type": {
"not": {
"enum": [
- "start", "testStart", "group", "print", "error", "testDone", "done"
+ "start", "testStart", "suite", "group", "print", "error",
+ "testDone", "done"
]
}
}
« json_reporter.md ('K') | « json_reporter.md ('k') | lib/src/runner.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698