Fix a couple bugs in the JSON schema.

Fix a couple bugs in the JSON schema.

This adds missing Test.groupIDs and PrintEvent.testID fields, and adds a
description to clarify the purpose of FutureEvent. It also now verifies
that the protocol version is compatible.

R=kevmoo@google.com

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

[nweiz, 2015-11-30 22:16:36]

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

[nweiz, 2015-11-30 22:19:02]

Description was changed from

==========
Fix a couple bugs in the JSON schema.

This adds missing Test.groupIDs and PrintEvent.testID fields, and adds a
description to clarify the purpose of FutureEvent.
==========

to

==========
Fix a couple bugs in the JSON schema.

This adds missing Test.groupIDs and PrintEvent.testID fields, and adds a
description to clarify the purpose of FutureEvent. It also now verifies
that the protocol version is compatible.
==========

[kevmoo, 2015-11-30 22:25:55]

LGTM...but...

Is there a way to validate this? Are you using a tool?

[nweiz, 2015-11-30 22:35:45]

On 2015/11/30 22:25:55, kevmoo wrote:
> Is there a way to validate this? Are you using a tool?

I plan to use the schema to validate the output once the json_schema package is
usable (pending https://github.com/patefacio/json_schema/pull/5), but that won't
catch stuff like this. The schema is intentionally written to be
forwards-compatible, which means it doesn't forbid unknown fields. Maybe we
could do some dynamic modification of it to produce one that does? But that's
something to talk about when we have the ability to actually perform validation
in the tests.

[nweiz, 2015-11-30 22:36:11]

Description was changed from

==========
Fix a couple bugs in the JSON schema.

This adds missing Test.groupIDs and PrintEvent.testID fields, and adds a
description to clarify the purpose of FutureEvent. It also now verifies
that the protocol version is compatible.
==========

to

==========
Fix a couple bugs in the JSON schema.

This adds missing Test.groupIDs and PrintEvent.testID fields, and adds a
description to clarify the purpose of FutureEvent. It also now verifies
that the protocol version is compatible.

R=kevmoo@google.com

Committed:
https://github.com/dart-lang/test/commit/fedd3925735b67f85c188d85e1cc0cf2e108...
==========

[nweiz, 2015-11-30 22:36:13]

Committed patchset #2 (id:20001) manually as
fedd3925735b67f85c188d85e1cc0cf2e1084f48 (presubmit successful).