Bug 176030 - Layout test http/tests/websocket/tests/hybi/send-object-tostring-check.html is flaky
Summary: Layout test http/tests/websocket/tests/hybi/send-object-tostring-check.html i...
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-08-28 11:59 PDT by Matt Lewis
Modified: 2021-10-29 10:50 PDT (History)
8 users (show)

See Also:


Attachments
Patch (2.28 KB, patch)
2019-06-18 14:59 PDT, Russell Epstein
no flags Details | Formatted Diff | Diff
Patch (2.28 KB, patch)
2019-06-18 15:05 PDT, Russell Epstein
no flags Details | Formatted Diff | Diff
Update Test Expectations (2.43 KB, patch)
2019-06-18 15:12 PDT, Russell Epstein
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Lewis 2017-08-28 11:59:44 PDT
http/tests/websocket/tests/hybi/send-object-tostring-check.html is a flaky failure on multiple WK2 platforms according to the flakiness dashboard. More specifically it shows the test is flaky on iOS WK2 and El Capitan WK2 Debug. The test failed once on Sierra WK2 Release.

History:
https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=http%2Ftests%2Fwebsocket%2Ftests%2Fhybi%2Fsend-object-tostring-check.html

current build:
https://build.webkit.org/results/Apple%20El%20Capitan%20Debug%20WK2%20(Tests)/r221258%20(2749)/results.html

Diff:
--- /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/http/tests/websocket/tests/hybi/send-object-tostring-check-expected.txt
+++ /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/http/tests/websocket/tests/hybi/send-object-tostring-check-actual.txt
@@ -1,9 +1,11 @@
+CONSOLE MESSAGE: WebSocket network error: The operation couldn’t be completed. Connection reset by peer
+CONSOLE MESSAGE: WebSocket network error: The operation couldn’t be completed. Connection reset by peer
 WebSocket: Object's toString method should be called only once.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 PASS testObj.callCounter is 1
-PASS closeEvent.wasClean is true
+FAIL closeEvent.wasClean should be true. Was false.
 PASS successfullyParsed is true
 
 TEST COMPLETE
Comment 1 Russell Epstein 2019-06-18 14:50:24 PDT
This is still happening as of 246561 with the same diff on Mac Debug WK2 and iOS WK2.

History: 

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=http%2Ftests%2Fwebsocket%2Ftests%2Fhybi%2Fsend-object-tostring-check.html



Diff:

@@ -1,9 +1,11 @@
+CONSOLE MESSAGE: WebSocket network error: The operation couldn’t be completed. Connection reset by peer
+CONSOLE MESSAGE: WebSocket network error: The operation couldn’t be completed. Connection reset by peer
 WebSocket: Object's toString method should be called only once.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 PASS testObj.callCounter is 1
-PASS closeEvent.wasClean is true
+FAIL closeEvent.wasClean should be true. Was false.
 PASS successfullyParsed is true
 
 TEST COMPLETE
Comment 2 Russell Epstein 2019-06-18 14:59:05 PDT
Created attachment 372388 [details]
Patch
Comment 3 Russell Epstein 2019-06-18 15:05:00 PDT
Created attachment 372389 [details]
Patch
Comment 4 Russell Epstein 2019-06-18 15:12:12 PDT
Created attachment 372390 [details]
Update Test Expectations
Comment 5 WebKit Commit Bot 2019-06-18 15:31:34 PDT
Comment on attachment 372390 [details]
Update Test Expectations

Clearing flags on attachment: 372390

Committed r246574: <https://trac.webkit.org/changeset/246574>
Comment 6 WebKit Commit Bot 2019-06-18 15:31:36 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Radar WebKit Bug Importer 2019-06-18 15:32:22 PDT
<rdar://problem/51869153>
Comment 8 Russell Epstein 2019-06-18 15:36:31 PDT
Marked test as flakey in https://trac.webkit.org/changeset/246574/webkit
Comment 9 Truitt Savell 2019-06-18 15:41:28 PDT
Reopening for investigation.
Comment 10 ayumi_kojima 2021-10-29 10:50:21 PDT
The test marked pass/fail on iOS here is now flaky timing out. Updated expectations https://trac.webkit.org/changeset/285040/webkit