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

Unified Diff: json_reporter.schema.json

Issue 2066113002: Add location information to the JSON reporter. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: cr Created 4 years, 6 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
« no previous file with comments | « doc/json_reporter.md ('k') | lib/src/backend/declarer.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 329686d5edc90bccb19615d1e77f0ae2bf0bd7ee..b34006231ebdce9bf7c672de55723f73c75f1496 100644
--- a/json_reporter.schema.json
+++ b/json_reporter.schema.json
@@ -3,17 +3,50 @@
"description": "An event emitted by the test package's JSON reporter",
"definitions": {
- "Test": {
- "required": ["id", "name", "suiteID", "groupIDs", "metadata"],
+ "GroupEntry": {
+ "required": [
+ "id",
+ "suiteID",
+ "metadata",
+ "line",
+ "column",
+ "url"
+ ],
"properties": {
"id": {"type": "integer", "minimum": 0},
- "name": {"type": "string"},
"suiteID": {"type": "integer", "minimum": 0},
+ "metadata": {"$ref": "#/definitions/Metadata"}
+ },
+ "oneOf": [
+ {
+ "properties": {
+ "line": {"type": "integer", "minimum": 1},
+ "column": {"type": "integer", "minimum": 1},
+ "url": {"type": "string", "format": "uri"}
+ }
+ },
+ {
+ "properties": {
+ "line": {"type": "null"},
+ "column": {"type": "null"},
+ "url": {"type": "null"}
+ }
+ }
+ ]
+ },
+
+ "Test": {
+ "allOf": [{"$ref": "#/definitions/GroupEntry"}],
+ "required": [
+ "name",
+ "groupIDs"
+ ],
+ "properties": {
+ "name": {"type": "string"},
"groupIDs": {
"type": "array",
"items": {"type": "integer", "minimum": 0}
- },
- "metadata": {"$ref": "#/definitions/Metadata"}
+ }
}
},
@@ -29,17 +62,15 @@
},
"Group": {
- "required": ["id", "name", "suiteID"],
+ "allOf": [{"$ref": "#/definitions/GroupEntry"}],
+ "required": ["name"],
"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"}]
},
- "metadata": {"$ref": "#/definitions/Metadata"},
"testCount": {"type": "integer", "minimum": 0}
}
},
« no previous file with comments | « doc/json_reporter.md ('k') | lib/src/backend/declarer.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698