2053621 - [wpt-sync] Sync PR 61149 - Don't set test fail status in web-extensions-helper.js.

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 61149 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/61149
Details from upstream follow.

justinlulejian <[e-mail]> wrote:

Don't set test fail status in web-extensions-helper.js.

web-extension-helper.js was overriding the test status to just test.FAIL, leaving out the test message. This causes a test failure to look like: FAIL message: Undefined.UNDEFINED since the message and stack are undefined.

test.set_status(status, message, stack); is already being handled by testharness.js in the step() method (see line 2878) so we can just rely on that.

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=d28bdec74b191675e5895dfde3a0c03540966d9a

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/b15e95c6c52f
https://hg.mozilla.org/integration/autoland/rev/82d02a30a4ef
[wpt PR 61149] - Don't set test fail status in web-extensions-helper.js., a=testonly

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Test result changes from PR not available.

Comment 5

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/82d02a30a4ef