Merge lp:~liuyq0307/linaro-android-build-tools/monkeyrunner into lp:linaro-android-build-tools

Proposed by Yongqin Liu
Status: Superseded
Proposed branch: lp:~liuyq0307/linaro-android-build-tools/monkeyrunner
Merge into: lp:linaro-android-build-tools
Diff against target: 33 lines (+21/-0)
1 file modified
build-scripts/post-build-lava.py (+21/-0)
To merge this branch: bzr merge lp:~liuyq0307/linaro-android-build-tools/monkeyrunner
Reviewer Review Type Date Requested Status
Paul Sokolovsky Needs Fixing
Review via email: mp+107200@code.launchpad.net

This proposal has been superseded by a proposal from 2012-05-25.

Description of the change

add support for integrating monkeyrunner test from android-build page.

To post a comment you must log in.
Revision history for this message
Yongqin Liu (liuyq0307) wrote :
Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

2 issues:

"##because these action cause android reboot easily" comment is not very clear - to what exactly it relates? Can you either elaborate it (something like "Warning - monkeyrunner actions can cause unexpected Android reboots") or perhaps just remove, because that's probably related more to specific monkeyrunner actions, than to post-build-lava.py's implementation of m-r support.

Also, at lines 18, 19 there's inconsistent indentation/braces placement, it would be nice to have it consistent and match what is used elsewhere in the script.

Thanks.

review: Needs Fixing
433. By Yongqin Liu

update according to the review comment

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'build-scripts/post-build-lava.py'
2--- build-scripts/post-build-lava.py 2012-05-17 17:59:51 +0000
3+++ build-scripts/post-build-lava.py 2012-05-25 05:36:17 +0000
4@@ -118,10 +118,31 @@
5 return test_actions
6
7
8+def gen_monkeyrunner_actions():
9+ test_actions = []
10+ prefix = 'MONKEY_RUNNER_URL_'
11+ test_list = []
12+ for var in os.environ.keys():
13+ if var.startswith(prefix):
14+ test_list.append(var)
15+ test_list.sort()
16+ for var in test_list:
17+ url = os.environ.get(var)
18+ action = {
19+ "command": "lava_android_test_run_monkeyrunner",
20+ "parameters":
21+ {
22+ "url": url
23+ }
24+ }
25+ test_actions.append(action)
26+
27+
28 def gen_test_actions():
29 test_actions = []
30 test_actions.extend(gen_custom_actions())
31 test_actions.extend(gen_test_plan_actions())
32+ test_actions.extend(gen_monkeyrunner_actions())
33 return test_actions
34
35

Subscribers

People subscribed via source and target branches