Merge lp:~liuyq0307/lava-android-test/modification-sleep-helloworld into lp:~asac/lava-android-test/helloworld-simple

Proposed by Yongqin Liu
Status: Needs review
Proposed branch: lp:~liuyq0307/lava-android-test/modification-sleep-helloworld
Merge into: lp:~asac/lava-android-test/helloworld-simple
Diff against target: 61 lines (+41/-3)
2 files modified
lava_android_test/test_definitions/helloworld.py (+3/-3)
lava_android_test/test_definitions/sleep.py (+38/-0)
To merge this branch: bzr merge lp:~liuyq0307/lava-android-test/modification-sleep-helloworld
Reviewer Review Type Date Requested Status
Andy Doan (community) Approve
Alexander Sack Pending
Review via email: mp+110352@code.launchpad.net

Description of the change

did some small modification
and add the sleep script here

To post a comment you must log in.
168. By Yongqin Liu

modify the pattern error

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

looks good.

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

would be good to have any type of default ... like one minute.

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

i've got an updated branch with this that also includes the ability to have a default option. I'll do the merge.

review: Approve

Unmerged revisions

168. By Yongqin Liu

modify the pattern error

167. By Yongqin Liu

add some small modification

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lava_android_test/test_definitions/helloworld.py'
--- lava_android_test/test_definitions/helloworld.py 2012-06-14 12:45:44 +0000
+++ lava_android_test/test_definitions/helloworld.py 2012-06-14 15:28:17 +0000
@@ -21,11 +21,11 @@
21import lava_android_test.config21import lava_android_test.config
22import lava_android_test.testdef22import lava_android_test.testdef
2323
24test_name = 'hello_world'24test_name = 'helloworld'
2525
26INSTALL_STEPS_ADB_PRE = []26INSTALL_STEPS_ADB_PRE = []
27ADB_SHELL_STEPS = ['echo hello_world: PASS']27ADB_SHELL_STEPS = ['echo helloworld: PASS']
28PATTERN = "^\s*(?P<test_case_id>[^:]*?):\s+(?P<result>(PASS|FAIL)?)\s+-\s+"28PATTERN = "^\s*(?P<test_case_id>[^:]+?):\s+(?P<result>(PASS|FAIL)?)\s*$"
2929
30inst = lava_android_test.testdef.AndroidTestInstaller(30inst = lava_android_test.testdef.AndroidTestInstaller(
31 steps_adb_pre=INSTALL_STEPS_ADB_PRE)31 steps_adb_pre=INSTALL_STEPS_ADB_PRE)
3232
=== added file 'lava_android_test/test_definitions/sleep.py'
--- lava_android_test/test_definitions/sleep.py 1970-01-01 00:00:00 +0000
+++ lava_android_test/test_definitions/sleep.py 2012-06-14 15:28:17 +0000
@@ -0,0 +1,38 @@
1# Copyright (c) 2012 Linaro
2
3# Author: Linaro Validation Team <linaro-dev@lists.linaro.org>
4#
5# This file is part of LAVA Android Test.
6#
7#
8# This program is free software: you can redistribute it and/or modify
9# it under the terms of the GNU General Public License as published by
10# the Free Software Foundation, either version 3 of the License, or
11# (at your option) any later version.
12#
13# This program is distributed in the hope that it will be useful,
14# but WITHOUT ANY WARRANTY; without even the implied warranty of
15# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16# GNU General Public License for more details.
17#
18# You should have received a copy of the GNU General Public License
19# along with this program. If not, see <http://www.gnu.org/licenses/>.
20
21import lava_android_test.config
22import lava_android_test.testdef
23
24test_name = 'sleep'
25
26INSTALL_STEPS_ADB_PRE = []
27ADB_SHELL_STEPS = ['sleep $(OPTIONS); echo sleep_$(OPTIONS): PASS']
28PATTERN = "^\s*(?P<test_case_id>[^:]+?):\s+(?P<result>(PASS|FAIL)?)\s*$"
29
30inst = lava_android_test.testdef.AndroidTestInstaller(
31 steps_adb_pre=INSTALL_STEPS_ADB_PRE)
32run = lava_android_test.testdef.AndroidTestRunner(
33 adbshell_steps=ADB_SHELL_STEPS)
34parser = lava_android_test.testdef.AndroidTestParser(PATTERN)
35testobj = lava_android_test.testdef.AndroidTest(testname=test_name,
36 installer=inst,
37 runner=run,
38 parser=parser)

Subscribers

People subscribed via source and target branches

to all changes: