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

Issue 10834041: [cros] Retail mode test should not attempt to skip OOBE (Closed)

Created:
8 years, 5 months ago by Nirnimesh
Modified:
8 years, 4 months ago
Reviewers:
rkc
CC:
chromium-reviews, dennis_jeffrey, anantha, dyu1, bartfab (slow)
Visibility:
Public.

Description

[cros] Retail mode test should not attempt to skip OOBE There's no OOBE to skip in retail mode. This test broke when I switched on OOBE in test images. R=rkc@chromium.org BUG= TEST= NOTRY=true Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148923

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M chrome/test/functional/chromeos_retail_mode.py View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Nirnimesh
8 years, 5 months ago (2012-07-26 21:27:09 UTC) #1
Nirnimesh
The test doesn't pass yet, but at least it gets further, and fails with: 'JSONInterfaceError: ...
8 years, 5 months ago (2012-07-26 21:32:45 UTC) #2
rkc
LGTM - there is no Oobe for a device in retail mode. On 2012/07/26 21:32:45, ...
8 years, 4 months ago (2012-07-27 20:32:34 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/nirnimesh@chromium.org/10834041/1
8 years, 4 months ago (2012-07-30 00:49:29 UTC) #4
commit-bot: I haz the power
8 years, 4 months ago (2012-07-30 02:56:53 UTC) #5
Change committed as 148923

Powered by Google App Engine
This is Rietveld 408576698