Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(307)

Issue 10836187: [chromeos] pyauto tests should auto login by default on chromeos (Closed)

Created:
8 years, 4 months ago by Nirnimesh
Modified:
8 years, 4 months ago
Reviewers:
craigdh, dennis_jeffrey
CC:
chromium-reviews, oshima+watch_chromium.org, dyu1, hashimoto+watch_chromium.org, aboxhall+watch_chromium.org, yoshiki+watch_chromium.org, yuzo+watch_chromium.org, davidbarr+watch_chromium.org, anantha, dmazzoni+watch_chromium.org, dtseng+watch_chromium.org, dennis_jeffrey, ctguil+watch_chromium.org, stevenjb+watch_chromium.org, zork+watch_chromium.org
Visibility:
Public.

Description

[chromeos] pyauto tests should auto login by default on chromeos Tests should auto-login on chromeos by default, since most use cases begin there. Provide a way to override it. This helps with non-login pyauto functional tests on chromeos such that if (for whatever reason) the device gets logged out in the middle of running pyauto_functional.py, individual tests can still manage to carry on. BUG=None TEST=None R=craigdh@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=151311

Patch Set 1 #

Total comments: 1

Patch Set 2 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+59 lines, -27 lines) Patch
M chrome/test/functional/chromeos_accessibility.py View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/test/functional/chromeos_device_policy.py View 5 chunks +14 lines, -14 lines 0 comments Download
M chrome/test/functional/chromeos_login.py View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/test/functional/netflix.py View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/test/pyautolib/chromeos/chromeos_utils.py View 1 1 chunk +4 lines, -9 lines 0 comments Download
M chrome/test/pyautolib/policy_base.py View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/test/pyautolib/pyauto.py View 2 chunks +29 lines, -4 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Nirnimesh
8 years, 4 months ago (2012-08-10 00:45:29 UTC) #1
Nirnimesh
The only interesting change is in pyauto.py The changes in functional/ & chromoes_util.py are to ...
8 years, 4 months ago (2012-08-10 00:49:24 UTC) #2
Nirnimesh
> The change in policy_base.py is make it not override Login() or else it gets ...
8 years, 4 months ago (2012-08-10 00:53:21 UTC) #3
craigdh
LGTM.
8 years, 4 months ago (2012-08-10 16:48:31 UTC) #4
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
8 years, 4 months ago (2012-08-10 18:34:54 UTC) #5
dennis_jeffrey
8 years, 4 months ago (2012-08-13 16:37:35 UTC) #6
LGTM with an optional nit

https://chromiumcodereview.appspot.com/10836187/diff/1/chrome/test/pyautolib/...
File chrome/test/pyautolib/chromeos/chromeos_utils.py (right):

https://chromiumcodereview.appspot.com/10836187/diff/1/chrome/test/pyautolib/...
chrome/test/pyautolib/chromeos/chromeos_utils.py:31: # Should auto-login.
Nothing to do here.
should we add a TODO to remove this function (and this file)?

Powered by Google App Engine
This is Rietveld 408576698