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

Issue 6677018: autotest/factory_PreFinalCheck: add verified boot key matching test (Closed)

Created:
9 years, 9 months ago by Hung-Te
Modified:
9 years, 7 months ago
CC:
chromium-os-reviews_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli
Visibility:
Public.

Description

autotest/factory_PreFinalCheck: add verified boot key matching test Using dev_debug_vboot to verify all the keys are correct. NOTE: verifying needs to read from system flashrom, and that takes 10-20 seconds on Alex for each trial (pressing space). Before this CL, PreFinalCheck takes 1-2 seconds for each run; by applying this CL, each run needs 10+ seconds. BUG=chrome-os-partner:2634 TEST=on alex with OS in partition B signed with ssd key -> pass OS in B signed with recovery key -> fail Change-Id: I8bac210ff74fa5f2f1c6979e50b349e4da82a203 Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=54ad315

Patch Set 1 #

Patch Set 2 : simple check of flooded key-press events #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -0 lines) Patch
M client/site_tests/factory_PreFinalCheck/factory_PreFinalCheck.py View 1 4 chunks +18 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Hung-Te
9 years, 9 months ago (2011-03-11 11:58:08 UTC) #1
Nick Sanders
lgtm
9 years, 9 months ago (2011-03-11 22:09:12 UTC) #2
Hung-Te
Added a simple check to discard cached key-press events by adding one more second to ...
9 years, 9 months ago (2011-03-14 07:28:36 UTC) #3
Nick Sanders
lgtm. can you file a bug to not cache keypresses between tests, probably we shouldn't ...
9 years, 9 months ago (2011-03-15 01:50:35 UTC) #4
Hung-Te
9 years, 9 months ago (2011-03-15 02:10:44 UTC) #5
On 2011/03/15 01:50:35, Nick Sanders wrote:
> lgtm. can you file a bug to not cache keypresses between tests, probably we
> shouldn't have to track this in each test.
  I've updated http://code.google.com/p/chrome-os-partner/issues/detail?id=1622
to address that - it should be solved at the same time.

Powered by Google App Engine
This is Rietveld 408576698