New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 622029 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Jun 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

FAFT: 'NoneType' object has no attribute 'dump_whole'"

Project Member Reported by waihong@chromium.org, Jun 21 2016

Issue description

Some FAFT tests failed on Chromeboxes which don't have Chrome EC.
  https://wmatrix.googleplex.com/testrun/unfiltered?test_ids=313157267

Should ignore the EC extracting for firmware updater.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 30 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/133b00c8869cfd20e26a43d37f677eea28fd5214

commit 133b00c8869cfd20e26a43d37f677eea28fd5214
Author: Tom Wai-Hong Tam <waihong@chromium.org>
Date: Tue Jun 21 20:06:19 2016

FAFT: Ignore extracting EC binary on devices without Chrome EC

BUG= chromium:622029 
TEST=Ran the firmware_UpdateFirmwareVersion test on a Chromebox

Change-Id: I233eb038b2a3f96eb33cda91d6c9b2890cf37fbc
Reviewed-on: https://chromium-review.googlesource.com/354484
Commit-Ready: Wai-Hong Tam <waihong@google.com>
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: danny chan <dchan@chromium.org>

[modify] https://crrev.com/133b00c8869cfd20e26a43d37f677eea28fd5214/server/cros/faft/firmware_test.py

Status: Fixed (was: Started)
Labels: VerifyIn-54

Comment 4 by dchan@chromium.org, Oct 7 2016

Labels: VerifyIn-55

Comment 5 by dchan@google.com, Nov 19 2016

Labels: VerifyIn-56

Comment 6 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57

Comment 7 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 8 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 9 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 11 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment