Issue 2322004: Add support for sequences of fully automated tests.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 2322004: Add support for sequences of fully automated tests. (Closed)

Created:
10 years, 7 months ago by Tammo Spalink

Modified:
9 years, 7 months ago

Reviewers:
kmixter1, Tom Wai-Hong Tam, Nick Sanders

CC:
chromium-os-reviews_chromium.org, sosa+cc_chromium.org, seano, ericli, petkov+cc_chromium.org

Base URL:
ssh://git@chromiumos-git/autotest.git

Visibility:
Public.

Description

Add support for sequences of fully automated tests. Paired with CL: http://codereview.chromium.org/2367001

Patch Set 1 #

Total comments: 2

Created: 10 years, 7 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+133 lines, -70 lines)			Patch
	M	client/site_tests/factory_Dummy/factory_Dummy.py	View		2 chunks	+5 lines, -4 lines	0 comments	Download
	M	client/site_tests/suite_Factory/control.ui	View		6 chunks	+128 lines, -66 lines	2 comments	Download

Messages

Total messages: 3 (0 generated)

Expand Messages | Collapse Messages

Tammo Spalink

Tested and works for me -- including multiple reboot.

10 years, 7 months ago (2010-05-28 09:38:59 UTC) #1

Nick Sanders

lgtm w/ or w/o suggestion below http://codereview.chromium.org/2322004/diff/1/3 File client/site_tests/suite_Factory/control.ui (right): http://codereview.chromium.org/2322004/diff/1/3#newcode93 client/site_tests/suite_Factory/control.ui:93: formal_name='hardware_SAT'), is there ...

10 years, 6 months ago (2010-06-02 08:06:36 UTC) #2

Tammo Spalink

10 years, 6 months ago (2010-06-07 03:40:20 UTC) #3

http://codereview.chromium.org/2322004/diff/1/3
File client/site_tests/suite_Factory/control.ui (right):

http://codereview.chromium.org/2322004/diff/1/3#newcode93
client/site_tests/suite_Factory/control.ui:93: formal_name='hardware_SAT'),
On 2010/06/02 08:06:36, Nick Sanders wrote:
> is there any way to get the test list specified in only one location? 

You mean like scrape the names out from the step_runin function?  We could do
this -- but then how to annotate with prettier names?  Or do we not care?  This
list here is really just to tell the UI which status from the status file to
associate with which pretty name in the list...

Expand Messages | Collapse Messages