Closed Bug 1919953 Opened 10 months ago Closed 10 months ago

[wpt-sync] Sync PR 48217 - [wpt] Detect miscategorized crashtests

Categories

(Core :: Layout, task, P4)

task

Tracking

()

RESOLVED FIXED
132 Branch
Tracking Status
firefox132 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Jonathan Lee <jonathanjlee@google.com> wrote:

[wpt] Detect miscategorized crashtests

https://web-platform-tests.org/writing-tests/crashtest.html says:

Crashtests are identified by the string -crash in the filename
immediately before the extension, or by being in a directory called
crashtests.

In particular:

  • css/css-foo/bar-crash-001.html is not a crash test

This CL fixes the names for files intended to run as crash tests 0.
Currently, these tests don't have any coverage 1 because wpt manifest doesn't detect and write them to MANIFEST.json.

Bug: None
Change-Id: I4c2100d9d43b91f4b6a6697a01e0e1754add621f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5868914
Reviewed-by: Morten Stenshorne \<mstensho@chromium.org>
Commit-Queue: Jonathan Lee \<jonathanjlee@google.com>
Cr-Commit-Position: refs/heads/main@{#1356525}

PR 48217 applied with additional changes from upstream: a8c39a102fc211f67dfc4dec645f7cd37538b998
Component: web-platform-tests → Layout
Product: Testing → Core

CI Results

Ran 11 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 16 tests

Status Summary

Firefox

PASS : 16

Chrome

PASS : 15
CRASH: 1

Safari

PASS : 16

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4128ec9ac829 [wpt PR 48217] - [wpt] Detect miscategorized crashtests, a=testonly
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch
You need to log in before you can comment on or make changes to this bug.