Bug 180982 – [macOS Debug WK2] Layout Test imported/w3c/web-platform-tests/service-workers/service-worker/registration-updateviacache.https.html is flaky

• Component: New Bugs
• Status: RESOLVED FIXED

[Ryan Haddad]

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

[youenn fablet]

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 ]

[Radar WebKit Bug Importer]

<rdar://problem/36156648>

[Matt Lewis]

Marked as slow in:
https://trac.webkit.org/changeset/226183/webkit

[Radar WebKit Bug Importer]

<rdar://problem/36157216>

[youenn fablet]

(In reply to Matt Lewis from comment #3)
> Marked as slow in:
> https://trac.webkit.org/changeset/226183/webkit

Thanks!

[Ryan Haddad]

The slow modifier was removed in https://trac.webkit.org/changeset/229192/webkit, so this test is showing up as a failure again.

[Ryan Haddad]

Marked the test as slow again in https://trac.webkit.org/changeset/229306/webkit

[youenn fablet]

Adding slow fixes the issue.