Closed Bug 1672550 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 26155 - [testharness.js] Fix bug in get_test_name

Categories

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

task

Tracking

(firefox84 fixed)

RESOLVED FIXED
84 Branch
Tracking Status
firefox84 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

b'Stephen McGruer <smcgruer@chromium.org>' wrote:

[testharness.js] Fix bug in get_test_name

The duplicate detection code previously mutated 'trimmed' before
checking the duplicate-detection cache, so repeated subtest names would
all just get '1' appended, rather than 1, 2, 3, ...

PR 26155 applied with additional changes from upstream: 4127ebe2f88a17abaa95400a3ac39173ccd70a70
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9757f8bc5929 [wpt PR 26155] - [testharness.js] Fix bug in get_test_name, a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
You need to log in before you can comment on or make changes to this bug.