Add a JSON reporter.

Add a JSON reporter.

Closes #11

R=kevmoo@google.com

Committed: https://github.com/dart-lang/test/commit/3d6797dc64c33fad9c73b958c8f1ad32e243da2d

[nweiz, 2015-11-21 02:54:47]

nweiz@google.com changed reviewers:
+ kevmoo@google.com

[nweiz, 2015-11-21 02:54:48]

The tests aren't quite finished, but otherwise this is pretty much done.

[kevmoo, 2015-11-21 03:37:18]

Code looks solid.

Add notes to readme about the new feature

I'd love to see working tests, but otherwise LGTM

https://codereview.chromium.org/1461293005/diff/1/json_reporter.schema.json
File json_reporter.schema.json (right):

https://codereview.chromium.org/1461293005/diff/1/json_reporter.schema.json#n...
json_reporter.schema.json:11: "metadata": {"$ref": "#/definitions/Metadata"}
I question wether this is required – especially if all of the values are
empty/defaults

Thoughts?

https://codereview.chromium.org/1461293005/diff/1/test/runner/json_reporter_t...
File test/runner/json_reporter_test.dart (right):

https://codereview.chromium.org/1461293005/diff/1/test/runner/json_reporter_t...
test/runner/json_reporter_test.dart:5: @TestOn("vm")
several type errors, failures etc...you're on this, right?

[nweiz, 2015-11-23 22:24:08]

> Add notes to readme about the new feature

I assume you mean the changelog? Done.

https://codereview.chromium.org/1461293005/diff/1/json_reporter.schema.json
File json_reporter.schema.json (right):

https://codereview.chromium.org/1461293005/diff/1/json_reporter.schema.json#n...
json_reporter.schema.json:11: "metadata": {"$ref": "#/definitions/Metadata"}
On 2015/11/21 03:37:17, kevmoo wrote:
> I question wether this is required – especially if all of the values are
> empty/defaults
> 
> Thoughts?

We need to report when a test is skipped somehow, and this is the most
extensible way to do so. We could omit it if it's just defaults, but it doesn't
take up much room and emitting a consistent structure makes the code easier to
consume.

https://codereview.chromium.org/1461293005/diff/1/test/runner/json_reporter_t...
File test/runner/json_reporter_test.dart (right):

https://codereview.chromium.org/1461293005/diff/1/test/runner/json_reporter_t...
test/runner/json_reporter_test.dart:5: @TestOn("vm")
On 2015/11/21 03:37:17, kevmoo wrote:
> several type errors, failures etc...you're on this, right?

Yes.

[nweiz, 2015-11-23 22:24:20]

Committed patchset #1 (id:1) manually as
3d6797dc64c33fad9c73b958c8f1ad32e243da2d (presubmit successful).