Add a Data Saver integration test for the MissingViaHeader4xx behavior |
|||||
Issue descriptionCurrently, there is no Data Saver integration test for the MissingViaHeader4xx bypass behavior. We should add one.
,
Mar 14 2017
,
Mar 28 2017
This test is flaky, failing about half the time with
======================================================================
FAIL: testMissingViaHeader4xxBypass (bypass.Bypass)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/usr/local/google/home/robertogden/chromium/src/tools/chrome_proxy/webdriver/bypass.py", line 135, in testMissingViaHeader4xxBypass
self.assertEqual(1, histogram['count'])
AssertionError: 1 != 2
Please fix when you get the chance
,
Apr 12 2017
,
Apr 12 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c37ea25b3d66e0081c0fc86c3a24e4e38c48177b commit c37ea25b3d66e0081c0fc86c3a24e4e38c48177b Author: ryansturm <ryansturm@chromium.org> Date: Wed Apr 12 23:44:16 2017 Fixing flaky integration test BUG= 700612 Review-Url: https://codereview.chromium.org/2816823002 Cr-Commit-Position: refs/heads/master@{#464207} [modify] https://crrev.com/c37ea25b3d66e0081c0fc86c3a24e4e38c48177b/tools/chrome_proxy/webdriver/bypass.py
,
Apr 13 2017
Working consistently now, thanks! |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by bugdroid1@chromium.org
, Mar 14 2017