New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Throw IdlHarnessError outside try #10280

Merged
merged 1 commit into from Apr 3, 2018

Conversation

Projects
None yet
4 participants
@lukebjerring
Contributor

lukebjerring commented Apr 3, 2018

Extracted from the reverted #10240.

Improves the error message for unexpected outcomes in idlharness.js tests.


This change is Reviewable

@lukebjerring lukebjerring requested a review from foolip Apr 3, 2018

@wpt-pr-bot wpt-pr-bot added the infra label Apr 3, 2018

@wpt-pr-bot wpt-pr-bot requested review from ayg, gsnedders and jgraham Apr 3, 2018

@w3c-bots

This comment has been minimized.

w3c-bots commented Apr 3, 2018

Build PASSED

Started: 2018-04-03 14:29:10
Finished: 2018-04-03 14:36:57

View more information about this build on:

@foolip

foolip approved these changes Apr 3, 2018

@foolip foolip merged commit 81e900a into web-platform-tests:master Apr 3, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment