This CL provides a bunch of new parser tests, and updates the

This CL provides a bunch of new parser tests, and updates the
TestExpectations file to skip the ones that fail or crash. It also
updates the TestExpectations parser to support URLs pointing to the
mojo repo's issue tracker.

R=abarth@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/5b16e5567ecb3296660577f989d6afb12183793b

[abarth-chromium, 2014-10-29 01:00:53]

abarth@chromium.org changed reviewers:
+ abarth@chromium.org

[abarth-chromium, 2014-10-29 01:00:54]

lgtm

https://codereview.chromium.org/685593006/diff/1/sky/tests/TestExpectations
File sky/tests/TestExpectations (right):

https://codereview.chromium.org/685593006/diff/1/sky/tests/TestExpectations#n...
sky/tests/TestExpectations:15: #https://github.com/domokit/mojo/issues/15
parser/utf16.sky [ Skip ]
Why is this one commented out?

https://codereview.chromium.org/685593006/diff/1/sky/tests/parser/README.md
File sky/tests/parser/README.md (right):

https://codereview.chromium.org/685593006/diff/1/sky/tests/parser/README.md#n...
sky/tests/parser/README.md:5: We shoulda also test the following (each line is
its own test):
s/shoulda/should/

[Hixie, 2014-10-29 01:56:50]

On 2014/10/29 01:00:54, abarth wrote:
> 
>
https://codereview.chromium.org/685593006/diff/1/sky/tests/TestExpectations#n...
> sky/tests/TestExpectations:15: #https://github.com/domokit/mojo/issues/15
> parser/utf16.sky [ Skip ]
> Why is this one commented out?

Because it crashes currently, so it's listed as [ Crash ] for now lower down,
but once the crash is fixed it'll time out because it doesn't have a script in
it so it'll need commenting out.


>
https://codereview.chromium.org/685593006/diff/1/sky/tests/parser/README.md#n...
> sky/tests/parser/README.md:5: We shoulda also test the following (each line is
> its own test):
> s/shoulda/should/

Fixed.

[Hixie, 2014-10-29 01:58:22]

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