Merge lp:~asac/lava-android-test/big-little-suite-selection-through-OPTIONS into lp:lava-android-test

Proposed by Alexander Sack
Status: Merged
Merged at revision: 192
Proposed branch: lp:~asac/lava-android-test/big-little-suite-selection-through-OPTIONS
Merge into: lp:lava-android-test
Diff against target: 22 lines (+5/-2)
1 file modified
lava_android_test/test_definitions/big_LITTLE.py (+5/-2)
To merge this branch: bzr merge lp:~asac/lava-android-test/big-little-suite-selection-through-OPTIONS
Reviewer Review Type Date Requested Status
Andy Doan (community) Approve
Review via email: mp+118967@code.launchpad.net

Description of the change

not tested, but copied from the sleep.py example.

want to be able to just run one test suite (e.g. -b for basic)

To post a comment you must log in.
Revision history for this message
Alexander Sack (asac) wrote :

fails ...

passes -O -b to the lava-android-test command which is not parsed as -O '-b' ... also i think the binding to the underlying sh ... command needs rework.

Revision history for this message
Alexander Sack (asac) wrote :

ok confirmed that this patch actually works ... was my fault.

Revision history for this message
Andy Doan (doanac) wrote :

looks good to me

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lava_android_test/test_definitions/big_LITTLE.py'
2--- lava_android_test/test_definitions/big_LITTLE.py 2012-07-10 09:16:53 +0000
3+++ lava_android_test/test_definitions/big_LITTLE.py 2012-08-09 14:19:23 +0000
4@@ -31,8 +31,10 @@
5
6 test_name = 'big_LITTLE'
7
8+DEFAULT_OPTIONS='-a'
9+
10 INSTALL_STEPS_ADB_PRE = []
11-ADB_SHELL_STEPS = ['run_stress_switcher_tests.sh -a']
12+ADB_SHELL_STEPS = ['run_stress_switcher_tests.sh $(OPTIONS)']
13 PATTERN = "(?P<test_case_id>.*-*)\s+:\s+(?P<result>(PASS|FAIL))"
14
15 inst = lava_android_test.testdef.AndroidTestInstaller(
16@@ -43,4 +45,5 @@
17 testobj = lava_android_test.testdef.AndroidTest(testname=test_name,
18 installer=inst,
19 runner=run,
20- parser=parser)
21+ parser=parser,
22+ default_options=DEFAULT_OPTIONS)

Subscribers

People subscribed via source and target branches