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

Issue 709813 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

factory: Better test list

Project Member Reported by hungte@chromium.org, Apr 9 2017

Issue description

Before migrating to new JSON based test list, there's some intermediate steps we may do, for example:

- Current test list is usually based on 'TestListArg', which combines both test helper functions and options.
  We should isolate that into several sub components, just like JSON test list was designed - having a TestListArg that contains only constants (or simple properties), then a test list helper to create test items with simple function calls.

- Also, we should eliminate dynamic test list data by execution of commands (either using Device API or simple local execution). In otherwords, we should prevent deciding thermal sensors by dut.thermal.* calls. Also not deciding test list params by sys_utils.InChroot.

As an example, we can start migrating Reef ToT or Eve ToT.
 

Comment 1 by hungte@chromium.org, Apr 12 2017

Owner: yhong@chromium.org
assign to yhong@.
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 25 2017

Project Member

Comment 3 by bugdroid1@chromium.org, Apr 25 2017

Project Member

Comment 4 by bugdroid1@chromium.org, Apr 25 2017

Project Member

Comment 5 by bugdroid1@chromium.org, Apr 25 2017

Comment 6 by hungte@chromium.org, Jul 26 2017

Cc: -youcheng@chromium.org -yhong@chromium.org -phoenixshen@chromium.org -stimim@chromium.org cromeos-factory-eng@google.com

Comment 7 Deleted

Comment 8 Deleted

Comment 9 by hungte@chromium.org, Sep 21 2017

Status: Fixed (was: Untriaged)
ToT is in good shape now.

Comment 10 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 11 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)

Sign in to add a comment