Support @Skip as a suite-level annotation.

Support @Skip as a suite-level annotation.

Closes #14

R=kevmoo@google.com

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

[kevmoo, 2015-04-17 23:56:29]

LGTM, modulo nits

https://codereview.chromium.org/1096693005/diff/1/CHANGELOG.md
File CHANGELOG.md (right):

https://codereview.chromium.org/1096693005/diff/1/CHANGELOG.md#newcode12
CHANGELOG.md:12: [custom html]:
https://github.com/dart-lang/test/blob/master/README.md#skipping-tests
You mean [skip] here?

https://codereview.chromium.org/1096693005/diff/1/lib/src/runner/loader.dart
File lib/src/runner/loader.dart (right):

https://codereview.chromium.org/1096693005/diff/1/lib/src/runner/loader.dart#...
lib/src/runner/loader.dart:138: return new Future.value();
you can just return here – forEach handles null fine

https://codereview.chromium.org/1096693005/diff/1/lib/src/runner/parse_metada...
File lib/src/runner/parse_metadata.dart (right):

https://codereview.chromium.org/1096693005/diff/1/lib/src/runner/parse_metada...
lib/src/runner/parse_metadata.dart:205: _parseSkip(Annotation annotation, String
constructorName, String path) {
Is this a String? Define (and document) return value

[nweiz, 2015-04-18 00:07:34]

Code review changes

[nweiz, 2015-04-18 00:07:39]

https://codereview.chromium.org/1096693005/diff/1/CHANGELOG.md
File CHANGELOG.md (right):

https://codereview.chromium.org/1096693005/diff/1/CHANGELOG.md#newcode12
CHANGELOG.md:12: [custom html]:
https://github.com/dart-lang/test/blob/master/README.md#skipping-tests
On 2015/04/17 23:56:29, kevmoo wrote:
> You mean [skip] here?

Done.

https://codereview.chromium.org/1096693005/diff/1/lib/src/runner/loader.dart
File lib/src/runner/loader.dart (right):

https://codereview.chromium.org/1096693005/diff/1/lib/src/runner/loader.dart#...
lib/src/runner/loader.dart:138: return new Future.value();
On 2015/04/17 23:56:29, kevmoo wrote:
> you can just return here – forEach handles null fine

Done. Future.wait makes me paranoid, because it doesn't.

https://codereview.chromium.org/1096693005/diff/1/lib/src/runner/parse_metada...
File lib/src/runner/parse_metadata.dart (right):

https://codereview.chromium.org/1096693005/diff/1/lib/src/runner/parse_metada...
lib/src/runner/parse_metadata.dart:205: _parseSkip(Annotation annotation, String
constructorName, String path) {
On 2015/04/17 23:56:29, kevmoo wrote:
> Is this a String? Define (and document) return value

Done.

[nweiz, 2015-04-18 00:10:31]

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