Issue metadata
Sign in to add a comment
|
2 tests in http/tests/cache/post-*.php are flaky when run in random order |
||||||||||||||||||||||
Issue descriptionThe tests: http/tests/cache/post-redirect-get.php http/tests/cache/post-with-cached-subresources.php appear to be flaky when run in random order. Steps to reproduce: This should pass consistently: Tools/Scripts/run-webkit-tests http/tests/cache/ This should fail intermittently: Tools/Scripts/run-webkit-tests http/tests/cache/ --order=random We're working on removing order dependency from Blink layout tests. More context here: https://docs.google.com/document/d/11-628Ny6hY__kfIv__zVD0m3rA0OB9tQLLC_gJI_PNg/edit
,
Nov 11 2016
Assuming this is a bug in the tests and no the cache, I don't think bugs in layout tests are considered network issues?
,
Nov 11 2016
,
Nov 14 2016
,
Nov 18 2016
,
Nov 29 2016
,
Nov 29 2016
,
Nov 30 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9f7c2d8eddada5b0fe0b123312b4e041b67c3879 commit 9f7c2d8eddada5b0fe0b123312b4e041b67c3879 Author: yhirano <yhirano@chromium.org> Date: Wed Nov 30 07:54:16 2016 Fix http/tests/cache random-order flakiness post-image-to-verify.php touches a file to check if it has been loaded, but multiple tests use the file. This change fixes the issue by separating the temporary file. BUG= 663865 Review-Url: https://codereview.chromium.org/2532103003 Cr-Commit-Position: refs/heads/master@{#435144} [modify] https://crrev.com/9f7c2d8eddada5b0fe0b123312b4e041b67c3879/third_party/WebKit/LayoutTests/http/tests/cache/post-redirect-get.php [modify] https://crrev.com/9f7c2d8eddada5b0fe0b123312b4e041b67c3879/third_party/WebKit/LayoutTests/http/tests/cache/post-with-cached-subresources-expected.txt [modify] https://crrev.com/9f7c2d8eddada5b0fe0b123312b4e041b67c3879/third_party/WebKit/LayoutTests/http/tests/cache/post-with-cached-subresources.php [modify] https://crrev.com/9f7c2d8eddada5b0fe0b123312b4e041b67c3879/third_party/WebKit/LayoutTests/http/tests/cache/resources/post-image-to-verify.php
,
Nov 30 2016
Thanks for looking into this! The tests can now be removed from RandomOrderExpectations. I'll take care of that (here's the bug https://crbug.com/669938 ).
,
Dec 1 2016
,
Nov 7 2017
,
Nov 7 2017
Apologies, applied the wrong component in bulk. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by jsb...@chromium.org
, Nov 10 2016