org.chromium.android_webview.test.AwQuotaManagerBridgeTest#testDeleteOriginWithAppCache flaky timeouts |
|||
Issue descriptionhttps://build.chromium.org/p/tryserver.chromium.android/builders/linux_android_rel_ng/builds/45324: C 777.579s Main [FAIL] org.chromium.android_webview.test.AwQuotaManagerBridgeTest#testDeleteOriginWithAppCache: C 777.579s Main junit.framework.AssertionFailedError: Criteria not met in allotted time.
,
Apr 1 2016
Running this test in a loop exposed other problems. The test server is failing to shutdown :/
,
Apr 1 2016
I guess you mean ctstestserver. Do you have hush's fix?
,
Apr 1 2016
this is the change: https://android-review.googlesource.com/#/c/209838/
,
Apr 1 2016
No. This is org.chromium.net.test.util.TestWebServer, but it originally was forked from the cts server. afaict this is not ssl though, so doesn't look related. I'll file another bug for that.
,
Apr 1 2016
Forked that to crbug.com/600021
,
May 6 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/24c4d329de36e9fba9a79f491709428da78ffd34 commit 24c4d329de36e9fba9a79f491709428da78ffd34 Author: boliu <boliu@chromium.org> Date: Fri May 06 23:32:59 2016 aw: Disable quota manager tests Too flaky. See crbug.com/609980 . BUG= 609980 , 599192 Review-Url: https://codereview.chromium.org/1960833002 Cr-Commit-Position: refs/heads/master@{#392202} [modify] https://crrev.com/24c4d329de36e9fba9a79f491709428da78ffd34/android_webview/javatests/src/org/chromium/android_webview/test/AwQuotaManagerBridgeTest.java
,
May 9 2016
Hi Bo, These tests can be re-enabled now after you've fixed the shutdown deadlock
,
May 9 2016
These tests has nothing to do with the test server shutdown issue. I ran one of these tests overnight to test the test server shutdown fix. I think the flakes here are all from tests leaving storage data around after it's done, so repeating a single test isn't helping. Not sure. |
|||
►
Sign in to add a comment |
|||
Comment 1 by boliu@chromium.org
, Apr 1 2016