1551437 - [wpt-sync] Sync PR 16782 - [testharness.js] Tolerate late tests

Status: RESOLVED FIXED

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

Description

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

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

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

jugglinmike <[e-mail]> wrote:

[testharness.js] Tolerate late tests

This reinstates the change originally merged as [1] and subsequently
reverted via [2]. This version of the patch updates the code to pass the
project's linting scripts (by changing the automated test to rely on
step_timeout instead of setTimeout).

[1] 7a02521f636f1c686a73487c3ee9af88ea470a0c
[2] eba8d664eb05a6ca4f7e50fc0e0fa83458ee86cb

---

@foolip CI failed to catch the linting error in the original submission because neither TravisCI nor Taskcluster ran the lint script. The task was being migrated from TravisCI to Taskcluster at the time. We're not sure if the omission was due to a bug in the Taskcluster configuration or a result of differing merge strategies between the two systems, but neither explanation calls for a fix to the infrastructure.

@jgraham wrote in IRC,

jgraham: I wonder how well tested that patch was

jgraham: Not related to the lint breakage, but related to whether it breaks tests

In acknowledgement of that concern, we should also carefully review the results published to wpt.fyi for this pull request.

Comment 1

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

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

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://hg.mozilla.org/integration/mozilla-inbound/rev/57c1f394870b
[wpt PR 16782] - [testharness.js] Tolerate late tests, a=testonly

Comment 4

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/57c1f394870b