Testharness.js doesnt report errors if reason property not available.
Categories
(Testing :: web-platform-tests, defect)
Tracking
(firefox72 fixed)
Tracking | Status | |
---|---|---|
firefox72 | --- | fixed |
People
(Reporter: automatedtester, Assigned: automatedtester)
References
Details
Attachments
(1 file)
when wpt hit a XPCOM error when running the following error
0:15.54 pid:44228 JavaScript error: http://web-platform.test:8000/resources/testharness.js, line 3698: TypeError: e.reason is undefined
1:10 PM 0:15.54 pid:44228 JavaScript error: , line 0: uncaught exception: undefined
Assignee | ||
Comment 1•5 years ago
|
||
Pushed by dburns@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cb345f708bb2 Have Testharness.js handle errors that do not have a reason property. r=jgraham
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/19897 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Upstream PR was closed without merging
Backout by aciure@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/89f812b6366f Backed out changeset cb345f708bb2 for causing default-reader.any.sharedworker.html wpt to permafail CLOSED TREE
Comment 7•5 years ago
|
||
Looks like this fixed a test so a metadata update is required.
Comment 8•5 years ago
|
||
(In reply to Pulsebot from comment #6)
Backout by aciure@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/89f812b6366f
Backed out changeset cb345f708bb2 for causing
default-reader.any.sharedworker.html wpt to permafail CLOSED TREE
Backed out for causing bug 1559028 to permafail
Failure log https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=272994356&repo=autoland&lineNumber=5141
Backout: https://hg.mozilla.org/integration/autoland/rev/89f812b6366f
Pushed by rmaries@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f9e3c13826e7 Have Testharness.js handle errors that do not have a reason property. r=jgraham
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Comment 11•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Upstream PR merged by moz-wptsync-bot
Assignee | ||
Updated•5 years ago
|
Description
•