Skip to content
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

[Gecko Bug 1433923] Use detailed failure message for wdspec test failures (pytest). #10215

Merged
merged 2 commits into from Mar 28, 2018

Conversation

Projects
None yet
5 participants
@moz-wptsync-bot
Copy link
Collaborator

moz-wptsync-bot commented Mar 28, 2018

When adding failures to the test report, the hook for pytest_runtest_logreport
missed to extract the failure message from the stack, and as such passed in no
message for the failure.
bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1433923
gecko-commit: 849020c741de1e3bca2e22f5902e7d3b40f64ebd
gecko-integration-branch: central
gecko-reviewers: jgraham


This change is Reviewable

Use detailed failure message for wdspec test failures (pytest).
When adding failures to the test report, the hook for pytest_runtest_logreport
missed to extract the failure message from the stack, and as such passed in no
message for the failure.
bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1433923
gecko-commit: 849020c741de1e3bca2e22f5902e7d3b40f64ebd
gecko-integration-branch: central
gecko-reviewers: jgraham
@wpt-pr-bot
Copy link
Collaborator

wpt-pr-bot left a comment

Already reviewed downstream.

@w3c-bots

This comment has been minimized.

Copy link

w3c-bots commented Mar 28, 2018

Build PASSED

Started: 2018-03-28 19:08:40
Finished: 2018-03-28 19:28:24

View more information about this build on:

@moz-wptsync-bot moz-wptsync-bot merged commit 94ab26e into master Mar 28, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1433923 branch Mar 28, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.