Merge lp:~doanac/ubuntu-test-cases/simplify-touch into lp:ubuntu-test-cases/smoke-touch-apps
- simplify-touch
- Merge into smoke-touch-apps
Status: | Merged |
---|---|
Merged at revision: | 80 |
Proposed branch: | lp:~doanac/ubuntu-test-cases/simplify-touch |
Merge into: | lp:ubuntu-test-cases/smoke-touch-apps |
Diff against target: |
694 lines (+54/-348) 46 files modified
calendar-app-autopilot/setup.sh (+0/-14) calendar-app-autopilot/ts_control (+1/-1) camera-app-autopilot/setup.sh (+0/-14) camera-app-autopilot/ts_control (+1/-1) friends-app-autopilot/setup.sh (+0/-14) friends-app-autopilot/ts_control (+1/-1) gallery-app-autopilot/setup.sh (+0/-14) gallery-app-autopilot/ts_control (+1/-1) jenkins/jenkins.sh (+12/-15) jenkins/provision.sh (+4/-3) mediaplayer-app-autopilot/setup.sh (+0/-14) mediaplayer-app-autopilot/ts_control (+1/-1) memevent/setup.sh (+0/-14) memevent/ts_control (+1/-2) music-app-autopilot/setup.sh (+0/-14) music-app-autopilot/ts_control (+1/-1) notes-app-autopilot/setup.sh (+0/-14) notes-app-autopilot/ts_control (+1/-1) phone-app-autopilot/setup.sh (+0/-14) phone-app-autopilot/ts_control (+1/-1) phone-app-connected-autopilot/setup.sh (+1/-10) share-app-autopilot/setup.sh (+0/-14) share-app-autopilot/ts_control (+1/-1) ubuntu-calculator-app-autopilot/setup.sh (+0/-14) ubuntu-calculator-app-autopilot/ts_control (+1/-1) ubuntu-clock-app-autopilot/setup.sh (+0/-14) ubuntu-clock-app-autopilot/ts_control (+1/-1) ubuntu-clock-autopilot/master.run (+0/-15) ubuntu-clock-autopilot/setup.sh (+0/-14) ubuntu-clock-autopilot/ts_control (+0/-1) ubuntu-clock-autopilot/tslist.auto (+0/-4) ubuntu-docviewer-app-autopilot/setup.sh (+0/-14) ubuntu-docviewer-app-autopilot/ts_control (+1/-1) ubuntu-filemanager-app-autopilot/setup.sh (+0/-14) ubuntu-filemanager-app-autopilot/ts_control (+1/-1) ubuntu-rssreader-app-autopilot/setup.sh (+0/-14) ubuntu-rssreader-app-autopilot/ts_control (+1/-1) ubuntu-terminal-app-autopilot/setup.sh (+0/-14) ubuntu-terminal-app-autopilot/ts_control (+1/-1) ubuntu-weather-app-autopilot/setup.sh (+0/-14) ubuntu-weather-app-autopilot/ts_control (+1/-1) unity8-autopilot/setup.sh (+1/-9) utils/target/prepare-autopilot-test.sh (+16/-0) utils/target/unlock_screen.sh (+2/-6) webbrowser-app-autopilot/setup.sh (+0/-14) webbrowser-app-autopilot/ts_control (+1/-1) |
To merge this branch: | bzr merge lp:~doanac/ubuntu-test-cases/simplify-touch |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paul Larson | Approve | ||
Javier Collado (community) | Approve | ||
Review via email: mp+181695@code.launchpad.net |
Commit message
Description of the change
This is part 1 of a 3 part MP that will get us to where our smoke-touch-apps tests can be run on either the target or from the host. Its mostly just setting up the ground work for the next 2 MPs
Paul Larson (pwlars) wrote : | # |
+1 this all looks good.
Some random comments (nothing that has to be fixed now):
* Yeah, I had noticed that one time about the fact that we use -s on some adb commands and not other. The -s is actually not needed as long as we have ANDROID_SERIAL exported to the environment. If we ever quit doing that, then we need to use the -s though. We should probably be consistent here, but that's a cleanup thing for another time.
* We can actually remove the workaround (which was previously commented out but left in place by me because we didn't know if we would get asked to turn it back on again) for the ueventd bug. That fix went in *today*! \o/
Andy Doan (doanac) wrote : | # |
On 08/23/2013 05:58 AM, Javier Collado wrote:
> Review: Approve
> - Shell scripts arguments
> While the scripts are documented and it's clear what environment variables have
> to be set to use them, I'd prefer to have arguments, parse them and print the
> usage string in case of failure.
That's a good point. Given all the changes dependent on this change, I
think we should do this in as a follow-on. It would make things nicer
though.
> - Spaces and tabs mixed
> In jenkins.sh it seems that there are some lines that are indented using tabs
> and other ones with spaces (and not always the same number of spaces). Being
> python my main programming language, I'd prefer to use just 4 spaces per
> indentation level.
I converted to tabs. I hate spaces. Guido is a genius for Python but
evil for spaces :)
> - adb commands without serial number
> Some adb commands don't have a serial number (one of them an adb push in this
> merge proposal), neither had the old version of the script, so I guess it
> works, but I feel like I'm mising something with regard to what device uses adb
> by default.
As Paul said, we should probably just remove the -s $ANDROID_SERIAL as a
follow-on.
Paul Larson (pwlars) wrote : | # |
> I converted to tabs. I hate spaces. Guido is a genius for Python but
> evil for spaces :)
>
Nooooo! Not tabs, 4 spaces is good and proper. Also, I can give you my vim
rc files that do it automatically and even alert you to pyflakes issues.
I'm being a little pedantic perhaps, but in all seriousness, we should
agree on some coding standards or we'll continue to hit stuff like this
Preview Diff
1 | === removed file 'calendar-app-autopilot/setup.sh' | |||
2 | --- calendar-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
3 | +++ calendar-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
4 | @@ -1,14 +0,0 @@ | |||
5 | 1 | #!/bin/sh | ||
6 | 2 | |||
7 | 3 | set -e | ||
8 | 4 | |||
9 | 5 | PKGS="autopilot-touch calendar-app-autopilot" | ||
10 | 6 | |||
11 | 7 | MISSING=0 | ||
12 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
13 | 9 | if [ $MISSING -eq 1 ] ; then | ||
14 | 10 | apt-get install -yq --force-yes $PKGS | ||
15 | 11 | else | ||
16 | 12 | echo "setup not needed" | ||
17 | 13 | fi | ||
18 | 14 | /usr/local/bin/unlock_screen.sh | ||
19 | 15 | 0 | ||
20 | === modified file 'calendar-app-autopilot/ts_control' | |||
21 | --- calendar-app-autopilot/ts_control 2013-08-01 22:15:03 +0000 | |||
22 | +++ calendar-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
23 | @@ -1,1 +1,1 @@ | |||
25 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=calendar-app-autopilot prepare-autopilot-test.sh |
26 | 2 | 2 | ||
27 | === removed file 'camera-app-autopilot/setup.sh' | |||
28 | --- camera-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
29 | +++ camera-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
30 | @@ -1,14 +0,0 @@ | |||
31 | 1 | #!/bin/sh | ||
32 | 2 | |||
33 | 3 | set -e | ||
34 | 4 | |||
35 | 5 | PKGS="autopilot-touch camera-app-autopilot" | ||
36 | 6 | |||
37 | 7 | MISSING=0 | ||
38 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
39 | 9 | if [ $MISSING -eq 1 ] ; then | ||
40 | 10 | apt-get install -yq --force-yes $PKGS | ||
41 | 11 | else | ||
42 | 12 | echo "setup not needed" | ||
43 | 13 | fi | ||
44 | 14 | /usr/local/bin/unlock_screen.sh | ||
45 | 15 | 0 | ||
46 | === modified file 'camera-app-autopilot/ts_control' | |||
47 | --- camera-app-autopilot/ts_control 2013-07-10 16:48:31 +0000 | |||
48 | +++ camera-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
49 | @@ -1,1 +1,1 @@ | |||
51 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=camera-app-autopilot prepare-autopilot-test.sh |
52 | 2 | 2 | ||
53 | === removed file 'friends-app-autopilot/setup.sh' | |||
54 | --- friends-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
55 | +++ friends-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
56 | @@ -1,14 +0,0 @@ | |||
57 | 1 | #!/bin/sh | ||
58 | 2 | |||
59 | 3 | set -e | ||
60 | 4 | |||
61 | 5 | PKGS="autopilot-touch friends-app-autopilot" | ||
62 | 6 | |||
63 | 7 | MISSING=0 | ||
64 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
65 | 9 | if [ $MISSING -eq 1 ] ; then | ||
66 | 10 | apt-get install -yq --force-yes $PKGS | ||
67 | 11 | else | ||
68 | 12 | echo "setup not needed" | ||
69 | 13 | fi | ||
70 | 14 | /usr/local/bin/unlock_screen.sh | ||
71 | 15 | 0 | ||
72 | === modified file 'friends-app-autopilot/ts_control' | |||
73 | --- friends-app-autopilot/ts_control 2013-07-11 18:39:09 +0000 | |||
74 | +++ friends-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
75 | @@ -1,1 +1,1 @@ | |||
77 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=friends-app-autopilot prepare-autopilot-test.sh |
78 | 2 | 2 | ||
79 | === removed file 'gallery-app-autopilot/setup.sh' | |||
80 | --- gallery-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
81 | +++ gallery-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
82 | @@ -1,14 +0,0 @@ | |||
83 | 1 | #!/bin/sh | ||
84 | 2 | |||
85 | 3 | set -e | ||
86 | 4 | |||
87 | 5 | PKGS="autopilot-touch gallery-app-autopilot" | ||
88 | 6 | |||
89 | 7 | MISSING=0 | ||
90 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
91 | 9 | if [ $MISSING -eq 1 ] ; then | ||
92 | 10 | apt-get install -yq --force-yes $PKGS | ||
93 | 11 | else | ||
94 | 12 | echo "setup not needed" | ||
95 | 13 | fi | ||
96 | 14 | /usr/local/bin/unlock_screen.sh | ||
97 | 15 | 0 | ||
98 | === modified file 'gallery-app-autopilot/ts_control' | |||
99 | --- gallery-app-autopilot/ts_control 2013-07-11 19:41:56 +0000 | |||
100 | +++ gallery-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
101 | @@ -1,1 +1,1 @@ | |||
103 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=gallery-app-autopilot prepare-autopilot-test.sh |
104 | 2 | 2 | ||
105 | === modified file 'jenkins/jenkins.sh' | |||
106 | --- jenkins/jenkins.sh 2013-08-16 19:34:15 +0000 | |||
107 | +++ jenkins/jenkins.sh 2013-08-23 23:40:38 +0000 | |||
108 | @@ -33,10 +33,10 @@ | |||
109 | 33 | BUILDID=`cat ${RESDIR}/media-info | awk '{ print $(NF)}' | sed -e 's/(//' -e 's/)//'` | 33 | BUILDID=`cat ${RESDIR}/media-info | awk '{ print $(NF)}' | sed -e 's/(//' -e 's/)//'` |
110 | 34 | echo "= TOUCH BUILD DATE:$BUILDID" | 34 | echo "= TOUCH BUILD DATE:$BUILDID" |
111 | 35 | 35 | ||
113 | 36 | adb shell "top -n1 -b" > ${RESDIR}/top.log | 36 | adb shell "top -n1 -b" > ${RESDIR}/top.log |
114 | 37 | 37 | ||
115 | 38 | set -x | 38 | set -x |
117 | 39 | adb shell 'rm -f /var/crash/*' | 39 | adb shell 'rm -f /var/crash/*' |
118 | 40 | if [ -z $QUICK ] ; then | 40 | if [ -z $QUICK ] ; then |
119 | 41 | # get the phone in sane place | 41 | # get the phone in sane place |
120 | 42 | adb -s ${ANDROID_SERIAL} reboot | 42 | adb -s ${ANDROID_SERIAL} reboot |
121 | @@ -51,31 +51,28 @@ | |||
122 | 51 | echo "SKIPPING phone reboot..." | 51 | echo "SKIPPING phone reboot..." |
123 | 52 | fi | 52 | fi |
124 | 53 | 53 | ||
125 | 54 | # push the screen unlocker | ||
126 | 55 | adb push ${BASEDIR}/unlock_screen/unlock_screen.sh /usr/local/bin | ||
127 | 56 | adb push ${BASEDIR}/unlock_screen/unlock_screen.py /usr/local/bin | ||
128 | 57 | # push the runlist over to the test: | 54 | # push the runlist over to the test: |
129 | 58 | adb push ${BASEDIR} ${TESTSUITE_TARGET_BASE} &> /dev/null | 55 | adb push ${BASEDIR} ${TESTSUITE_TARGET_BASE} &> /dev/null |
130 | 59 | ${UTAH_PHABLET_CMD} \ | 56 | ${UTAH_PHABLET_CMD} \ |
131 | 60 | -s ${ANDROID_SERIAL} \ | 57 | -s ${ANDROID_SERIAL} \ |
132 | 61 | --results-dir ${RESDIR} \ | 58 | --results-dir ${RESDIR} \ |
133 | 62 | --skip-install --skip-network --skip-utah \ | 59 | --skip-install --skip-network --skip-utah \ |
135 | 63 | --pull /var/crash \ | 60 | --pull /var/crash \ |
136 | 64 | -l ${TESTSUITE_TARGET}/master.run | 61 | -l ${TESTSUITE_TARGET}/master.run |
138 | 65 | adb shell 'rm -f /var/crash/*' | 62 | adb shell 'rm -f /var/crash/*' |
139 | 66 | 63 | ||
140 | 67 | if ! `grep "^errors: [!0]" < $UTAHFILE >/dev/null` ; then | 64 | if ! `grep "^errors: [!0]" < $UTAHFILE >/dev/null` ; then |
143 | 68 | echo "errors found" | 65 | echo "errors found" |
144 | 69 | EXITCODE=1 | 66 | EXITCODE=1 |
145 | 70 | fi | 67 | fi |
146 | 71 | if ! `grep "^failures: [!0]" < $UTAHFILE >/dev/null` ; then | 68 | if ! `grep "^failures: [!0]" < $UTAHFILE >/dev/null` ; then |
149 | 72 | echo "failures found" | 69 | echo "failures found" |
150 | 73 | EXITCODE=2 | 70 | EXITCODE=2 |
151 | 74 | fi | 71 | fi |
156 | 75 | echo "Results Summary" | 72 | echo "Results Summary" |
157 | 76 | echo "---------------" | 73 | echo "---------------" |
158 | 77 | egrep '^(errors|failures|passes|fetch_errors):' $UTAHFILE | 74 | egrep '^(errors|failures|passes|fetch_errors):' $UTAHFILE |
159 | 78 | exit $EXITCODE | 75 | exit $EXITCODE |
160 | 79 | } | 76 | } |
161 | 80 | 77 | ||
162 | 81 | trap cleanup TERM INT EXIT | 78 | trap cleanup TERM INT EXIT |
163 | 82 | 79 | ||
164 | === modified file 'jenkins/provision.sh' | |||
165 | --- jenkins/provision.sh 2013-08-16 16:35:06 +0000 | |||
166 | +++ jenkins/provision.sh 2013-08-23 23:40:38 +0000 | |||
167 | @@ -4,6 +4,7 @@ | |||
168 | 4 | ## Intersting environment variables that must be set: | 4 | ## Intersting environment variables that must be set: |
169 | 5 | ## ANDROID_SERIAL - specify another android device | 5 | ## ANDROID_SERIAL - specify another android device |
170 | 6 | ## NETWORK_FILE - an alternative network file if you aren't in the lab | 6 | ## NETWORK_FILE - an alternative network file if you aren't in the lab |
171 | 7 | ## TOUCH_IMAGE=--ubuntu-bootstrap (provision with read-only system image) | ||
172 | 7 | 8 | ||
173 | 8 | set -e | 9 | set -e |
174 | 9 | set -x | 10 | set -x |
175 | @@ -16,10 +17,10 @@ | |||
176 | 16 | ANDROID_SERIAL="${ANDROID_SERIAL-015d1884b20c1c0f}" #doanac's nexus7 at home | 17 | ANDROID_SERIAL="${ANDROID_SERIAL-015d1884b20c1c0f}" #doanac's nexus7 at home |
177 | 17 | NETWORK_FILE="${NETWORK_FILE-/home/ubuntu/magners-wifi}" | 18 | NETWORK_FILE="${NETWORK_FILE-/home/ubuntu/magners-wifi}" |
178 | 18 | 19 | ||
179 | 19 | # Image to use for flashing | ||
180 | 20 | # Syntax: TOUCH_IMAGE=--ubuntu-bootstrap | ||
181 | 21 | |||
182 | 22 | rm -rf clientlogs | 20 | rm -rf clientlogs |
183 | 23 | mkdir clientlogs | 21 | mkdir clientlogs |
184 | 24 | 22 | ||
185 | 25 | ${UTAH_PHABLET_CMD} -s ${ANDROID_SERIAL} --results-dir ${RESDIR} --network-file=${NETWORK_FILE} ${TOUCH_IMAGE} | 23 | ${UTAH_PHABLET_CMD} -s ${ANDROID_SERIAL} --results-dir ${RESDIR} --network-file=${NETWORK_FILE} ${TOUCH_IMAGE} |
186 | 24 | |||
187 | 25 | # get our target-based utilities into our PATH | ||
188 | 26 | adb -s ${ANDROID_SERIAL} push ${BASEDIR}/../utils/target /usr/local/bin/ | ||
189 | 26 | 27 | ||
190 | === removed file 'mediaplayer-app-autopilot/setup.sh' | |||
191 | --- mediaplayer-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
192 | +++ mediaplayer-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
193 | @@ -1,14 +0,0 @@ | |||
194 | 1 | #!/bin/sh | ||
195 | 2 | |||
196 | 3 | set -e | ||
197 | 4 | |||
198 | 5 | PKGS="autopilot-touch mediaplayer-app-autopilot" | ||
199 | 6 | |||
200 | 7 | MISSING=0 | ||
201 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
202 | 9 | if [ $MISSING -eq 1 ] ; then | ||
203 | 10 | apt-get install -yq --force-yes $PKGS | ||
204 | 11 | else | ||
205 | 12 | echo "setup not needed" | ||
206 | 13 | fi | ||
207 | 14 | /usr/local/bin/unlock_screen.sh | ||
208 | 15 | 0 | ||
209 | === modified file 'mediaplayer-app-autopilot/ts_control' | |||
210 | --- mediaplayer-app-autopilot/ts_control 2013-07-11 20:00:29 +0000 | |||
211 | +++ mediaplayer-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
212 | @@ -1,1 +1,1 @@ | |||
214 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=mediaplayer-app-autopilot prepare-autopilot-test.sh |
215 | 2 | 2 | ||
216 | === removed file 'memevent/setup.sh' | |||
217 | --- memevent/setup.sh 2013-08-07 16:52:12 +0000 | |||
218 | +++ memevent/setup.sh 1970-01-01 00:00:00 +0000 | |||
219 | @@ -1,14 +0,0 @@ | |||
220 | 1 | #!/bin/sh | ||
221 | 2 | |||
222 | 3 | set -e | ||
223 | 4 | |||
224 | 5 | PKGS="autopilot-touch camera-app-autopilot gallery-app-autopilot mediaplayer-app-autopilot webbrowser-app-autopilot" | ||
225 | 6 | |||
226 | 7 | MISSING=0 | ||
227 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
228 | 9 | if [ $MISSING -eq 1 ] ; then | ||
229 | 10 | apt-get install -yq --force-yes $PKGS | ||
230 | 11 | else | ||
231 | 12 | echo "setup not needed" | ||
232 | 13 | fi | ||
233 | 14 | /usr/local/bin/unlock_screen.sh | ||
234 | 15 | 0 | ||
235 | === modified file 'memevent/ts_control' | |||
236 | --- memevent/ts_control 2013-08-05 23:23:31 +0000 | |||
237 | +++ memevent/ts_control 2013-08-23 23:40:38 +0000 | |||
238 | @@ -1,2 +1,1 @@ | |||
241 | 1 | --- | 1 | ts_setup: PKGS="camera-app-autopilot gallery-app-autopilot mediaplayer-app-autopilot webbrowser-app-autopilot" prepare-autopilot-test.sh |
240 | 2 | ts_setup: ./setup.sh | ||
242 | 3 | 2 | ||
243 | === removed file 'music-app-autopilot/setup.sh' | |||
244 | --- music-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
245 | +++ music-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
246 | @@ -1,14 +0,0 @@ | |||
247 | 1 | #!/bin/sh | ||
248 | 2 | |||
249 | 3 | set -e | ||
250 | 4 | |||
251 | 5 | PKGS="autopilot-touch music-app-autopilot" | ||
252 | 6 | |||
253 | 7 | MISSING=0 | ||
254 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
255 | 9 | if [ $MISSING -eq 1 ] ; then | ||
256 | 10 | apt-get install -yq --force-yes $PKGS | ||
257 | 11 | else | ||
258 | 12 | echo "setup not needed" | ||
259 | 13 | fi | ||
260 | 14 | /usr/local/bin/unlock_screen.sh | ||
261 | 15 | 0 | ||
262 | === modified file 'music-app-autopilot/ts_control' | |||
263 | --- music-app-autopilot/ts_control 2013-08-01 22:15:03 +0000 | |||
264 | +++ music-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
265 | @@ -1,1 +1,1 @@ | |||
267 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=music-app-autopilot prepare-autopilot-test.sh |
268 | 2 | 2 | ||
269 | === removed file 'notes-app-autopilot/setup.sh' | |||
270 | --- notes-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
271 | +++ notes-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
272 | @@ -1,14 +0,0 @@ | |||
273 | 1 | #!/bin/sh | ||
274 | 2 | |||
275 | 3 | set -e | ||
276 | 4 | |||
277 | 5 | PKGS="autopilot-touch notes-app-autopilot" | ||
278 | 6 | |||
279 | 7 | MISSING=0 | ||
280 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
281 | 9 | if [ $MISSING -eq 1 ] ; then | ||
282 | 10 | apt-get install -yq --force-yes $PKGS | ||
283 | 11 | else | ||
284 | 12 | echo "setup not needed" | ||
285 | 13 | fi | ||
286 | 14 | /usr/local/bin/unlock_screen.sh | ||
287 | 15 | 0 | ||
288 | === modified file 'notes-app-autopilot/ts_control' | |||
289 | --- notes-app-autopilot/ts_control 2013-07-11 20:03:51 +0000 | |||
290 | +++ notes-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
291 | @@ -1,1 +1,1 @@ | |||
293 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=notes-app-autopilot prepare-autopilot-test.sh |
294 | 2 | 2 | ||
295 | === removed file 'phone-app-autopilot/setup.sh' | |||
296 | --- phone-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
297 | +++ phone-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
298 | @@ -1,14 +0,0 @@ | |||
299 | 1 | #!/bin/sh | ||
300 | 2 | |||
301 | 3 | set -e | ||
302 | 4 | |||
303 | 5 | PKGS="autopilot-touch phone-app-autopilot" | ||
304 | 6 | |||
305 | 7 | MISSING=0 | ||
306 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
307 | 9 | if [ $MISSING -eq 1 ] ; then | ||
308 | 10 | apt-get install -yq --force-yes $PKGS | ||
309 | 11 | else | ||
310 | 12 | echo "setup not needed" | ||
311 | 13 | fi | ||
312 | 14 | /usr/local/bin/unlock_screen.sh | ||
313 | 15 | 0 | ||
314 | === modified file 'phone-app-autopilot/ts_control' | |||
315 | --- phone-app-autopilot/ts_control 2013-07-11 20:22:43 +0000 | |||
316 | +++ phone-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
317 | @@ -1,1 +1,1 @@ | |||
319 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=phone-app-autopilot prepare-autopilot-test.sh |
320 | 2 | 2 | ||
321 | === modified file 'phone-app-connected-autopilot/setup.sh' | |||
322 | --- phone-app-connected-autopilot/setup.sh 2013-08-16 16:35:06 +0000 | |||
323 | +++ phone-app-connected-autopilot/setup.sh 2013-08-23 23:40:38 +0000 | |||
324 | @@ -2,15 +2,6 @@ | |||
325 | 2 | 2 | ||
326 | 3 | set -e | 3 | set -e |
327 | 4 | 4 | ||
328 | 5 | PKGS="autopilot-touch phone-app-connected-autopilot" | ||
329 | 6 | |||
330 | 7 | MISSING=0 | ||
331 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
332 | 9 | if [ $MISSING -eq 1 ] ; then | ||
333 | 10 | apt-get install -yq --force-yes $PKGS | ||
334 | 11 | else | ||
335 | 12 | echo "setup not needed" | ||
336 | 13 | fi | ||
337 | 14 | # we have issues with running the test too soon, this is a hack: | 5 | # we have issues with running the test too soon, this is a hack: |
338 | 15 | echo "SLEEPING 60 TO HELP MAKE SURE PHONE IS READY" | 6 | echo "SLEEPING 60 TO HELP MAKE SURE PHONE IS READY" |
339 | 16 | sleep 60 | 7 | sleep 60 |
340 | @@ -26,4 +17,4 @@ | |||
341 | 26 | ./sms_self.py | 17 | ./sms_self.py |
342 | 27 | sleep 10s # try and let the message come through | 18 | sleep 10s # try and let the message come through |
343 | 28 | 19 | ||
345 | 29 | /usr/local/bin/unlock_screen.sh | 20 | PKGS="phone-app-connected-autopilot" prepare-autopilot-test.sh |
346 | 30 | 21 | ||
347 | === removed file 'share-app-autopilot/setup.sh' | |||
348 | --- share-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
349 | +++ share-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
350 | @@ -1,14 +0,0 @@ | |||
351 | 1 | #!/bin/sh | ||
352 | 2 | |||
353 | 3 | set -e | ||
354 | 4 | |||
355 | 5 | PKGS="autopilot-touch share-app-autopilot" | ||
356 | 6 | |||
357 | 7 | MISSING=0 | ||
358 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
359 | 9 | if [ $MISSING -eq 1 ] ; then | ||
360 | 10 | apt-get install -yq --force-yes $PKGS | ||
361 | 11 | else | ||
362 | 12 | echo "setup not needed" | ||
363 | 13 | fi | ||
364 | 14 | /usr/local/bin/unlock_screen.sh | ||
365 | 15 | 0 | ||
366 | === modified file 'share-app-autopilot/ts_control' | |||
367 | --- share-app-autopilot/ts_control 2013-07-10 16:48:31 +0000 | |||
368 | +++ share-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
369 | @@ -1,1 +1,1 @@ | |||
371 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=share-app-autopilot prepare-autopilot-test.sh |
372 | 2 | 2 | ||
373 | === removed file 'ubuntu-calculator-app-autopilot/setup.sh' | |||
374 | --- ubuntu-calculator-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
375 | +++ ubuntu-calculator-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
376 | @@ -1,14 +0,0 @@ | |||
377 | 1 | #!/bin/sh | ||
378 | 2 | |||
379 | 3 | set -e | ||
380 | 4 | |||
381 | 5 | PKGS="autopilot-touch ubuntu-calculator-app-autopilot" | ||
382 | 6 | |||
383 | 7 | MISSING=0 | ||
384 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
385 | 9 | if [ $MISSING -eq 1 ] ; then | ||
386 | 10 | apt-get install -yq --force-yes $PKGS | ||
387 | 11 | else | ||
388 | 12 | echo "setup not needed" | ||
389 | 13 | fi | ||
390 | 14 | /usr/local/bin/unlock_screen.sh | ||
391 | 15 | 0 | ||
392 | === modified file 'ubuntu-calculator-app-autopilot/ts_control' | |||
393 | --- ubuntu-calculator-app-autopilot/ts_control 2013-08-01 22:15:03 +0000 | |||
394 | +++ ubuntu-calculator-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
395 | @@ -1,1 +1,1 @@ | |||
397 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=ubuntu-calculator-app-autopilot prepare-autopilot-test.sh |
398 | 2 | 2 | ||
399 | === removed file 'ubuntu-clock-app-autopilot/setup.sh' | |||
400 | --- ubuntu-clock-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
401 | +++ ubuntu-clock-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
402 | @@ -1,14 +0,0 @@ | |||
403 | 1 | #!/bin/sh | ||
404 | 2 | |||
405 | 3 | set -e | ||
406 | 4 | |||
407 | 5 | PKGS="autopilot-touch ubuntu-clock-app-autopilot" | ||
408 | 6 | |||
409 | 7 | MISSING=0 | ||
410 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
411 | 9 | if [ $MISSING -eq 1 ] ; then | ||
412 | 10 | apt-get install -yq --force-yes $PKGS | ||
413 | 11 | else | ||
414 | 12 | echo "setup not needed" | ||
415 | 13 | fi | ||
416 | 14 | /usr/local/bin/unlock_screen.sh | ||
417 | 15 | 0 | ||
418 | === modified file 'ubuntu-clock-app-autopilot/ts_control' | |||
419 | --- ubuntu-clock-app-autopilot/ts_control 2013-08-01 22:15:03 +0000 | |||
420 | +++ ubuntu-clock-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
421 | @@ -1,1 +1,1 @@ | |||
423 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=ubuntu-clock-app-autopilot prepare-autopilot-test.sh |
424 | 2 | 2 | ||
425 | === removed directory 'ubuntu-clock-autopilot' | |||
426 | === removed file 'ubuntu-clock-autopilot/master.run' | |||
427 | --- ubuntu-clock-autopilot/master.run 2013-08-16 19:34:15 +0000 | |||
428 | +++ ubuntu-clock-autopilot/master.run 1970-01-01 00:00:00 +0000 | |||
429 | @@ -1,15 +0,0 @@ | |||
430 | 1 | --- | ||
431 | 2 | testsuites: | ||
432 | 3 | - name: settle-before | ||
433 | 4 | fetch_method: dev | ||
434 | 5 | fetch_location: ../systemsettle | ||
435 | 6 | include_tests: | ||
436 | 7 | - systemsettle-before | ||
437 | 8 | - name: ubuntu-clock-autopilot | ||
438 | 9 | fetch_method: dev | ||
439 | 10 | fetch_location: ./ | ||
440 | 11 | - name: settle-after | ||
441 | 12 | fetch_method: dev | ||
442 | 13 | fetch_location: ../systemsettle | ||
443 | 14 | include_tests: | ||
444 | 15 | - systemsettle-after | ||
445 | 16 | 0 | ||
446 | === removed file 'ubuntu-clock-autopilot/setup.sh' | |||
447 | --- ubuntu-clock-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
448 | +++ ubuntu-clock-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
449 | @@ -1,14 +0,0 @@ | |||
450 | 1 | #!/bin/sh | ||
451 | 2 | |||
452 | 3 | set -e | ||
453 | 4 | |||
454 | 5 | PKGS="autopilot-touch ubuntu-clock-autopilot" | ||
455 | 6 | |||
456 | 7 | MISSING=0 | ||
457 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
458 | 9 | if [ $MISSING -eq 1 ] ; then | ||
459 | 10 | apt-get install -yq --force-yes $PKGS | ||
460 | 11 | else | ||
461 | 12 | echo "setup not needed" | ||
462 | 13 | fi | ||
463 | 14 | /usr/local/bin/unlock_screen.sh | ||
464 | 15 | 0 | ||
465 | === removed file 'ubuntu-clock-autopilot/ts_control' | |||
466 | --- ubuntu-clock-autopilot/ts_control 2013-08-01 22:15:03 +0000 | |||
467 | +++ ubuntu-clock-autopilot/ts_control 1970-01-01 00:00:00 +0000 | |||
468 | @@ -1,1 +0,0 @@ | |||
469 | 1 | ts_setup: sh ./setup.sh | ||
470 | 2 | 0 | ||
471 | === removed file 'ubuntu-clock-autopilot/tslist.auto' | |||
472 | --- ubuntu-clock-autopilot/tslist.auto 2013-08-01 22:15:03 +0000 | |||
473 | +++ ubuntu-clock-autopilot/tslist.auto 1970-01-01 00:00:00 +0000 | |||
474 | @@ -1,4 +0,0 @@ | |||
475 | 1 | - | ||
476 | 2 | discovery_cmd: "autopilot list ubuntu_clock | grep ubuntu_clock| sed -e 's/^.*ubuntu_clock.tests.//'" | ||
477 | 3 | test_cmd: utah-autopilot run -v ubuntu_clock.tests.{} | ||
478 | 4 | |||
479 | 5 | 0 | ||
480 | === removed file 'ubuntu-docviewer-app-autopilot/setup.sh' | |||
481 | --- ubuntu-docviewer-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
482 | +++ ubuntu-docviewer-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
483 | @@ -1,14 +0,0 @@ | |||
484 | 1 | #!/bin/sh | ||
485 | 2 | |||
486 | 3 | set -e | ||
487 | 4 | |||
488 | 5 | PKGS="autopilot-touch ubuntu-docviewer-app-autopilot" | ||
489 | 6 | |||
490 | 7 | MISSING=0 | ||
491 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
492 | 9 | if [ $MISSING -eq 1 ] ; then | ||
493 | 10 | apt-get install -yq --force-yes $PKGS | ||
494 | 11 | else | ||
495 | 12 | echo "setup not needed" | ||
496 | 13 | fi | ||
497 | 14 | /usr/local/bin/unlock_screen.sh | ||
498 | 15 | 0 | ||
499 | === modified file 'ubuntu-docviewer-app-autopilot/ts_control' | |||
500 | --- ubuntu-docviewer-app-autopilot/ts_control 2013-08-01 22:15:03 +0000 | |||
501 | +++ ubuntu-docviewer-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
502 | @@ -1,1 +1,1 @@ | |||
504 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=ubuntu-docviewer-app-autopilot prepare-autopilot-test.sh |
505 | 2 | 2 | ||
506 | === removed file 'ubuntu-filemanager-app-autopilot/setup.sh' | |||
507 | --- ubuntu-filemanager-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
508 | +++ ubuntu-filemanager-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
509 | @@ -1,14 +0,0 @@ | |||
510 | 1 | #!/bin/sh | ||
511 | 2 | |||
512 | 3 | set -e | ||
513 | 4 | |||
514 | 5 | PKGS="autopilot-touch ubuntu-filemanager-app-autopilot" | ||
515 | 6 | |||
516 | 7 | MISSING=0 | ||
517 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
518 | 9 | if [ $MISSING -eq 1 ] ; then | ||
519 | 10 | apt-get install -yq --force-yes $PKGS | ||
520 | 11 | else | ||
521 | 12 | echo "setup not needed" | ||
522 | 13 | fi | ||
523 | 14 | /usr/local/bin/unlock_screen.sh | ||
524 | 15 | 0 | ||
525 | === modified file 'ubuntu-filemanager-app-autopilot/ts_control' | |||
526 | --- ubuntu-filemanager-app-autopilot/ts_control 2013-08-01 22:15:03 +0000 | |||
527 | +++ ubuntu-filemanager-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
528 | @@ -1,1 +1,1 @@ | |||
530 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=ubuntu-filemanager-app-autopilot prepare-autopilot-test.sh |
531 | 2 | 2 | ||
532 | === removed file 'ubuntu-rssreader-app-autopilot/setup.sh' | |||
533 | --- ubuntu-rssreader-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
534 | +++ ubuntu-rssreader-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
535 | @@ -1,14 +0,0 @@ | |||
536 | 1 | #!/bin/sh | ||
537 | 2 | |||
538 | 3 | set -e | ||
539 | 4 | |||
540 | 5 | PKGS="autopilot-touch ubuntu-rssreader-app-autopilot" | ||
541 | 6 | |||
542 | 7 | MISSING=0 | ||
543 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
544 | 9 | if [ $MISSING -eq 1 ] ; then | ||
545 | 10 | apt-get install -yq --force-yes $PKGS | ||
546 | 11 | else | ||
547 | 12 | echo "setup not needed" | ||
548 | 13 | fi | ||
549 | 14 | /usr/local/bin/unlock_screen.sh | ||
550 | 15 | 0 | ||
551 | === modified file 'ubuntu-rssreader-app-autopilot/ts_control' | |||
552 | --- ubuntu-rssreader-app-autopilot/ts_control 2013-08-01 22:15:03 +0000 | |||
553 | +++ ubuntu-rssreader-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
554 | @@ -1,1 +1,1 @@ | |||
556 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=ubuntu-rssreader-app-autopilot prepare-autopilot-test.sh |
557 | 2 | 2 | ||
558 | === removed file 'ubuntu-terminal-app-autopilot/setup.sh' | |||
559 | --- ubuntu-terminal-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
560 | +++ ubuntu-terminal-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
561 | @@ -1,14 +0,0 @@ | |||
562 | 1 | #!/bin/sh | ||
563 | 2 | |||
564 | 3 | set -e | ||
565 | 4 | |||
566 | 5 | PKGS="autopilot-touch ubuntu-terminal-app-autopilot" | ||
567 | 6 | |||
568 | 7 | MISSING=0 | ||
569 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
570 | 9 | if [ $MISSING -eq 1 ] ; then | ||
571 | 10 | apt-get install -yq --force-yes $PKGS | ||
572 | 11 | else | ||
573 | 12 | echo "setup not needed" | ||
574 | 13 | fi | ||
575 | 14 | /usr/local/bin/unlock_screen.sh | ||
576 | 15 | 0 | ||
577 | === modified file 'ubuntu-terminal-app-autopilot/ts_control' | |||
578 | --- ubuntu-terminal-app-autopilot/ts_control 2013-08-01 22:15:03 +0000 | |||
579 | +++ ubuntu-terminal-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
580 | @@ -1,1 +1,1 @@ | |||
582 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=ubuntu-terminal-app-autopilot prepare-autopilot-test.sh |
583 | 2 | 2 | ||
584 | === removed file 'ubuntu-weather-app-autopilot/setup.sh' | |||
585 | --- ubuntu-weather-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
586 | +++ ubuntu-weather-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
587 | @@ -1,14 +0,0 @@ | |||
588 | 1 | #!/bin/sh | ||
589 | 2 | |||
590 | 3 | set -e | ||
591 | 4 | |||
592 | 5 | PKGS="autopilot-touch ubuntu-weather-app-autopilot" | ||
593 | 6 | |||
594 | 7 | MISSING=0 | ||
595 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
596 | 9 | if [ $MISSING -eq 1 ] ; then | ||
597 | 10 | apt-get install -yq --force-yes $PKGS | ||
598 | 11 | else | ||
599 | 12 | echo "setup not needed" | ||
600 | 13 | fi | ||
601 | 14 | /usr/local/bin/unlock_screen.sh | ||
602 | 15 | 0 | ||
603 | === modified file 'ubuntu-weather-app-autopilot/ts_control' | |||
604 | --- ubuntu-weather-app-autopilot/ts_control 2013-08-01 22:15:03 +0000 | |||
605 | +++ ubuntu-weather-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
606 | @@ -1,1 +1,1 @@ | |||
608 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=ubuntu-weather-app-autopilot prepare-autopilot-test.sh |
609 | 2 | 2 | ||
610 | === modified file 'unity8-autopilot/setup.sh' | |||
611 | --- unity8-autopilot/setup.sh 2013-08-02 20:02:27 +0000 | |||
612 | +++ unity8-autopilot/setup.sh 2013-08-23 23:40:38 +0000 | |||
613 | @@ -2,13 +2,5 @@ | |||
614 | 2 | 2 | ||
615 | 3 | set -e | 3 | set -e |
616 | 4 | 4 | ||
626 | 5 | PKGS="autopilot-touch unity8-autopilot" | 5 | NO_UNLOCK=1 PKGS="unity8-autopilot" prepare-autopilot-test.sh |
618 | 6 | |||
619 | 7 | MISSING=0 | ||
620 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
621 | 9 | if [ $MISSING -eq 1 ] ; then | ||
622 | 10 | apt-get install -yq --force-yes $PKGS | ||
623 | 11 | else | ||
624 | 12 | echo "setup not needed" | ||
625 | 13 | fi | ||
627 | 14 | sudo -i -u phablet bash -ic "stop unity8" | 6 | sudo -i -u phablet bash -ic "stop unity8" |
628 | 15 | 7 | ||
629 | === added directory 'utils' | |||
630 | === renamed directory 'unlock_screen' => 'utils/target' | |||
631 | === added file 'utils/target/prepare-autopilot-test.sh' | |||
632 | --- utils/target/prepare-autopilot-test.sh 1970-01-01 00:00:00 +0000 | |||
633 | +++ utils/target/prepare-autopilot-test.sh 2013-08-23 23:40:38 +0000 | |||
634 | @@ -0,0 +1,16 @@ | |||
635 | 1 | #!/bin/sh | ||
636 | 2 | |||
637 | 3 | set -e | ||
638 | 4 | |||
639 | 5 | #installs dependencies and unlocks screen so an autopilot test case can run | ||
640 | 6 | |||
641 | 7 | if [ -n "$PKGS" ] ; then | ||
642 | 8 | MISSING=0 | ||
643 | 9 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
644 | 10 | if [ $MISSING -eq 1 ] ; then | ||
645 | 11 | apt-get install -yq --force-yes $PKGS | ||
646 | 12 | else | ||
647 | 13 | echo "setup not needed" | ||
648 | 14 | fi | ||
649 | 15 | fi | ||
650 | 16 | [ -z $NO_UNLOCK ] && unlock_screen.sh | ||
651 | 0 | 17 | ||
652 | === modified file 'utils/target/unlock_screen.sh' | |||
653 | --- unlock_screen/unlock_screen.sh 2013-08-16 19:34:15 +0000 | |||
654 | +++ utils/target/unlock_screen.sh 2013-08-23 23:40:38 +0000 | |||
655 | @@ -1,11 +1,7 @@ | |||
656 | 1 | #!/bin/sh | 1 | #!/bin/sh |
657 | 2 | 2 | ||
663 | 3 | #temporary workaround till bug 1190792 is fixed | 3 | basedir=$(dirname $(readlink -f $0)) |
659 | 4 | ##Workaround removed for now, because we are trying to settle the system | ||
660 | 5 | ##and detect things like this first | ||
661 | 6 | #pkill ueventd | ||
662 | 7 | #sleep 3 | ||
664 | 8 | 4 | ||
665 | 9 | #temporary workaround for bug #1207386 | 5 | #temporary workaround for bug #1207386 |
666 | 10 | chmod 666 /dev/uinput | 6 | chmod 666 /dev/uinput |
668 | 11 | sudo -i -u phablet bash -ic "PYTHONPATH=$(pwd) /usr/local/bin/unlock_screen.py" | 7 | sudo -i -u phablet bash -ic "PYTHONPATH=$(pwd) ${basedir}/unlock_screen.py" |
669 | 12 | 8 | ||
670 | === removed file 'webbrowser-app-autopilot/setup.sh' | |||
671 | --- webbrowser-app-autopilot/setup.sh 2013-08-06 21:59:47 +0000 | |||
672 | +++ webbrowser-app-autopilot/setup.sh 1970-01-01 00:00:00 +0000 | |||
673 | @@ -1,14 +0,0 @@ | |||
674 | 1 | #!/bin/sh | ||
675 | 2 | |||
676 | 3 | set -e | ||
677 | 4 | |||
678 | 5 | PKGS="autopilot-touch webbrowser-app-autopilot" | ||
679 | 6 | |||
680 | 7 | MISSING=0 | ||
681 | 8 | dpkg -s $PKGS 2>/dev/null >/dev/null || MISSING=1 | ||
682 | 9 | if [ $MISSING -eq 1 ] ; then | ||
683 | 10 | apt-get install -yq --force-yes $PKGS | ||
684 | 11 | else | ||
685 | 12 | echo "setup not needed" | ||
686 | 13 | fi | ||
687 | 14 | /usr/local/bin/unlock_screen.sh | ||
688 | 15 | 0 | ||
689 | === modified file 'webbrowser-app-autopilot/ts_control' | |||
690 | --- webbrowser-app-autopilot/ts_control 2013-07-11 20:08:44 +0000 | |||
691 | +++ webbrowser-app-autopilot/ts_control 2013-08-23 23:40:38 +0000 | |||
692 | @@ -1,1 +1,1 @@ | |||
694 | 1 | ts_setup: sh ./setup.sh | 1 | ts_setup: PKGS=webbrowser-app-autopilot prepare-autopilot-test.sh |
The changes look good and the refactoring of the setup scripts was needed
indeed.
I have a few comments, but they're all related to issues that were already in
the code before this merge request, so I'm just writing them down here to see
what are your thoughts about them:
- Shell scripts arguments
While the scripts are documented and it's clear what environment variables have
to be set to use them, I'd prefer to have arguments, parse them and print the
usage string in case of failure.
- Spaces and tabs mixed
In jenkins.sh it seems that there are some lines that are indented using tabs
and other ones with spaces (and not always the same number of spaces). Being
python my main programming language, I'd prefer to use just 4 spaces per
indentation level.
- adb commands without serial number
Some adb commands don't have a serial number (one of them an adb push in this
merge proposal), neither had the old version of the script, so I guess it
works, but I feel like I'm mising something with regard to what device uses adb
by default.