Closed Bug 1581405 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 19066 - Avoid spurious timeouts in websocket test.

Categories

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

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox71 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla71
Tracking Flags:
Tracking Status
firefox71 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Josh Matthews <josh@joshmatthews.net> wrote:

Avoid spurious timeouts in websocket test.

If a test assertion fails, the rest of the code in that test step will not run. By changing the ordering, we ensure that the test does not timeout because the websocket is never closed the test condition fails.

Ran 2 tests and 4 subtests
OK     : 2
PASS   : 2
FAIL   : 2
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8e2b4e76df02
[wpt PR 19066] - Avoid spurious timeouts in websocket test., a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/e02a3d990ba6
[wpt PR 19066] - Update wpt metadata, a=testonly
You need to log in before you can comment on or make changes to this bug.