Bug 180982 - [macOS Debug WK2] Layout Test imported/w3c/web-platform-tests/service-workers/service-worker/registration-updateviacache.https.html is flaky
Summary: [macOS Debug WK2] Layout Test imported/w3c/web-platform-tests/service-workers...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-12-19 10:58 PST by Ryan Haddad
Modified: 2022-06-21 06:20 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2017-12-19 10:58:24 PST
The following layout test is flaky on macOS Debug WK2

imported/w3c/web-platform-tests/service-workers/service-worker/registration-updateviacache.https.html

Flakiness Dashboard:

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=imported%2Fw3c%2Fweb-platform-tests%2Fservice-workers%2Fservice-worker%2Fregistration-updateviacache.https.html

--- /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/service-workers/service-worker/registration-updateviacache.https-expected.txt
+++ /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/service-workers/service-worker/registration-updateviacache.https-actual.txt
@@ -1,22 +1,24 @@
+
+Harness Error (TIMEOUT), message = null
 
 PASS register-with-updateViaCache-undefined 
 PASS register-with-updateViaCache-imports 
 PASS register-with-updateViaCache-all 
 PASS register-with-updateViaCache-none 
 PASS register-with-updateViaCache-undefined-then-undefined 
-PASS register-with-updateViaCache-undefined-then-imports 
-PASS register-with-updateViaCache-undefined-then-all 
-PASS register-with-updateViaCache-undefined-then-none 
-PASS register-with-updateViaCache-imports-then-undefined 
-PASS register-with-updateViaCache-imports-then-imports 
-PASS register-with-updateViaCache-imports-then-all 
-PASS register-with-updateViaCache-imports-then-none 
-PASS register-with-updateViaCache-all-then-undefined 
-PASS register-with-updateViaCache-all-then-imports 
-PASS register-with-updateViaCache-all-then-all 
-PASS register-with-updateViaCache-all-then-none 
-PASS register-with-updateViaCache-none-then-undefined 
-PASS register-with-updateViaCache-none-then-imports 
-PASS register-with-updateViaCache-none-then-all 
-PASS register-with-updateViaCache-none-then-none 
+TIMEOUT register-with-updateViaCache-undefined-then-imports Test timed out
+NOTRUN register-with-updateViaCache-undefined-then-all 
+NOTRUN register-with-updateViaCache-undefined-then-none 
+NOTRUN register-with-updateViaCache-imports-then-undefined 
+NOTRUN register-with-updateViaCache-imports-then-imports 
+NOTRUN register-with-updateViaCache-imports-then-all 
+NOTRUN register-with-updateViaCache-imports-then-none 
+NOTRUN register-with-updateViaCache-all-then-undefined 
+NOTRUN register-with-updateViaCache-all-then-imports 
+NOTRUN register-with-updateViaCache-all-then-all 
+NOTRUN register-with-updateViaCache-all-then-none 
+NOTRUN register-with-updateViaCache-none-then-undefined 
+NOTRUN register-with-updateViaCache-none-then-imports 
+NOTRUN register-with-updateViaCache-none-then-all 
+NOTRUN register-with-updateViaCache-none-then-none
Comment 1 youenn fablet 2017-12-20 08:02:23 PST
It might be that we should change the expectation to:
[ Debug ] imported/w3c/web-platform-tests/service-workers/service-worker/registration-updateviacache.https.html [ Slow ]
Comment 2 Radar WebKit Bug Importer 2017-12-20 08:41:42 PST
<rdar://problem/36156648>
Comment 3 Matt Lewis 2017-12-20 09:17:44 PST
Marked as slow in:
https://trac.webkit.org/changeset/226183/webkit
Comment 4 Radar WebKit Bug Importer 2017-12-20 09:18:04 PST
<rdar://problem/36157216>
Comment 5 youenn fablet 2017-12-20 09:27:07 PST
(In reply to Matt Lewis from comment #3)
> Marked as slow in:
> https://trac.webkit.org/changeset/226183/webkit

Thanks!
Comment 6 Ryan Haddad 2018-03-05 20:32:25 PST
The slow modifier was removed in https://trac.webkit.org/changeset/229192/webkit, so this test is showing up as a failure again.
Comment 7 Ryan Haddad 2018-03-05 20:39:38 PST
Marked the test as slow again in https://trac.webkit.org/changeset/229306/webkit
Comment 8 youenn fablet 2022-06-21 06:20:41 PDT
Adding slow fixes the issue.