Revert "rlz: experiment checking if more time helps failing guest mode test or not."
This reverts commit 18f2fd4bb569bac8e2bd9ef85cf6073d41d65321.
Reason for revert: 5 minutes did not work
Original change's description:
> rlz: experiment checking if more time helps failing guest mode test or not.
>
> This will be reverted after we get a few runs.
>
> BUG=None
> TEST=None
>
> Change-Id: I6035154ea95bf06100247fe375ced00838c1bd72
> Reviewed-on: https://chromium-review.googlesource.com/1345455
> Commit-Ready: David Haddock <dhaddock@chromium.org>
> Tested-by: David Haddock <dhaddock@chromium.org>
> Reviewed-by: Stephen Lin <stephenlin@google.com>
Bug: None
Change-Id: I9c65ab43dd7e1e4e09d2e68500211193a3273ff2
Reviewed-on: https://chromium-review.googlesource.com/1370631
Commit-Ready: David Haddock <dhaddock@chromium.org>
Tested-by: David Haddock <dhaddock@chromium.org>
Reviewed-by: David Haddock <dhaddock@chromium.org>
diff --git a/client/site_tests/desktopui_CheckRlzPingSent/desktopui_CheckRlzPingSent.py b/client/site_tests/desktopui_CheckRlzPingSent/desktopui_CheckRlzPingSent.py
index db861d4..1c030d7 100644
--- a/client/site_tests/desktopui_CheckRlzPingSent/desktopui_CheckRlzPingSent.py
+++ b/client/site_tests/desktopui_CheckRlzPingSent/desktopui_CheckRlzPingSent.py
@@ -43,11 +43,6 @@
timeout_minutes = 2
timeout = time.time() + 60 * timeout_minutes
- # TODO(dhaddock): Remove after experiment.
- extra_minutes = 5
- extra_timeout = time.time() + 60 * extra_minutes
- test_failed = False
-
# Setup a keyboard emulator to open new tabs and type a search.
with input_playback.InputPlayback() as player:
player.emulate(input_type='keyboard')
@@ -64,17 +59,10 @@
break
else:
if time.time() > timeout:
- test_failed = True
- if time.time() > extra_timeout:
raise error.TestFail('RLZ ping did not send in %d '
- 'minutes.' % extra_minutes)
+ 'minutes.' % timeout_minutes)
time.sleep(10)
- if test_failed:
- raise error.TestFail('Test did not send ping within %d mins but '
- 'did within %d minutes.' % (timeout_minutes,
- extra_timeout))
-
def run_once(self, logged_in=True):
"""