Bug 297063
| Summary: | [ macOS wk2 arm64 ] http/tests/xmlhttprequest/chunked-progress-event-expectedLength.html is a flaky text failure | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Bryan Medina <bmedina2> |
| Component: | Tools / Tests | Assignee: | Chris Dumez <cdumez> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | webkit-bot-watchers-bugzilla, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Bryan Medina
http/tests/xmlhttprequest/chunked-progress-event-expectedLength.html
is a flaky text failure on macOS wk2 arm64.
HISTORY:
https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fxmlhttprequest%2Fchunked-progress-event-expectedLength.html
TEXT DIFF:
--- /Volumes/Data/worker/Apple-Sequoia-Debug-AppleSilicon-WK2-Tests/build/layout-test-results/http/tests/xmlhttprequest/chunked-progress-event-expectedLength-expected.txt
+++ /Volumes/Data/worker/Apple-Sequoia-Debug-AppleSilicon-WK2-Tests/build/layout-test-results/http/tests/xmlhttprequest/chunked-progress-event-expectedLength-actual.txt
@@ -4,4 +4,4 @@
PASS should appear below:
-PASS
+
TEXT DIFF URL:
https://build.webkit.org/results/Apple-Sequoia-Debug-AppleSilicon-WK2-Tests/298328%40main%20(2042)/http/tests/xmlhttprequest/chunked-progress-event-expectedLength-diff.txt
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/157765880>
Bryan Medina
I was able to reproduce the text failure locally on my device running macOS Sequoia arm64 Debug ToT using the following command:
run-webkit-tests http/tests/xmlhttprequest/chunked-progress-event-expectedLength.html --iterations=500 -f
EWS
Test gardening commit 298344@main (9af55296b765): <https://commits.webkit.org/298344@main>
Reviewed commits have been landed. Closing PR #49066 and removing active labels.
Alexey Proskuryakov
This is just a bad test, where parsing the HTML races with the test itself. It would be ideal to convert it to a js-test, but for a quick fix, moving test() into unload handler should suffice.
Chris Dumez
Pull request: https://github.com/WebKit/WebKit/pull/53711
EWS
Committed 302878@main (f420b718bfc9): <https://commits.webkit.org/302878@main>
Reviewed commits have been landed. Closing PR #53711 and removing active labels.