Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(613)

Unified Diff: build/android/buildbot_functions.sh

Issue 10387086: Run APK tests on the android_test trybot. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Updated SDK on bots. Ready for review. Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: build/android/buildbot_functions.sh
diff --git a/build/android/buildbot_functions.sh b/build/android/buildbot_functions.sh
index 2ea50f396be91a6be873e84564205a5b10480caf..093313b69a4d98fc5f67880c5211aaa276fe39af 100755
--- a/build/android/buildbot_functions.sh
+++ b/build/android/buildbot_functions.sh
@@ -239,3 +239,16 @@ function bb_run_tests {
echo "@@@BUILD_STEP Run Tests on actual hardware@@@"
build/android/run_tests.py --xvfb --verbose
}
+
+# Run APK tests on an actual device.
+function bb_run_apk_tests {
+ echo "@@@BUILD_STEP Run APK Tests on actual hardware@@@"
+ tempfile=/tmp/tempfile-$$.txt
+ # Filter out STEP_FAILURES, we don't want REDNESS on test failures for now.
+ build/android/run_tests.py --xvfb --verbose --apk=True \
+ | sed 's/@@@STEP_FAILURE@@@//g' | tee $tempfile
+ happy_failure=$(cat $tempfile | grep RUNNER_FAILED | wc -l)
+ if [[ $happy_failure -eq 0 ]] ; then
+ echo "@@@STEP_WARNINGS@@@"
+ fi
John Grabowski 2012/05/15 22:24:39 rm -f $tempfile but... refactor so we have a bb_r
nilesh 2012/05/15 22:59:25 I have not removed the run_tests.py step from bb_n
John Grabowski 2012/05/15 23:06:31 Yes. What I meant was, you did some cut and paste
+}
John Grabowski 2012/05/15 22:24:39 delete $tempfile when done
nilesh 2012/05/15 22:59:25 Done.

Powered by Google App Engine
This is Rietveld 408576698