Closed Bug 1897305 Opened 1 month ago Closed 1 month ago

[wpt-sync] Sync PR 46331 - [wptrunner] Optimize server test order

Categories

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

task

Tracking

(firefox128 fixed)

RESOLVED FIXED
128 Branch
Tracking Status
firefox128 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Jonathan Lee <jonathanjlee@google.com> wrote:

[wptrunner] Optimize server test order

TestEnvironment.ensure_started() synchronously polls that all servers are started. The non-webtransport-h3 servers are often tested first and immediately fail with "Connection refused", which puts them in the pending list. This spins the polling loop at least once and delays startup by at least 0.5s.

Run the webtransport-h3 server test first, which blocks. This provides additional time for the non-webtransport-h3 servers to become ready, which means the TestEnvironment often only needs to poll once.

This speeds up ensure_started() from ~2.2s -> ~1.7s and improves wptrunner's interactive UX (https://crbug.com/339299999). No functional change.

Status: NEW → RESOLVED
Closed: 1 month ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Test result changes from PR not available.
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/47a6b3318b49
[wpt PR 46331] - [wptrunner] Optimize server test order, a=testonly
Status: REOPENED → RESOLVED
Closed: 1 month ago1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 128 Branch
You need to log in before you can comment on or make changes to this bug.