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

Unified Diff: build/android/test_package.py

Issue 10204001: Look for "END" marker while runnig APK tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/test_package.py
diff --git a/build/android/test_package.py b/build/android/test_package.py
index 0c04377bea1fdf6ead8f590c58312644feba3600..e4b10e17a6102669280ca14fa1c22dec41b73c75 100644
--- a/build/android/test_package.py
+++ b/build/android/test_package.py
@@ -132,16 +132,20 @@ class TestPackage(object):
timed_out = False
overall_fail = False
re_run = re.compile('\[ RUN \] ?(.*)\r\n')
+ # APK tests rely on the END tag.
+ re_end = re.compile('\[ END \] ?(.*)\r\n')
re_fail = re.compile('\[ FAILED \] ?(.*)\r\n')
re_runner_fail = re.compile('\[ RUNNER_FAILED \] ?(.*)\r\n')
re_ok = re.compile('\[ OK \] ?(.*)\r\n')
(io_stats_before, ready_to_continue) = self._BeginGetIOStats()
while ready_to_continue:
- found = p.expect([re_run, pexpect.EOF, re_runner_fail],
+ found = p.expect([re_run, pexpect.EOF, re_end, re_runner_fail],
timeout=self.timeout)
if found == 1: # matched pexpect.EOF
break
- if found == 2: # RUNNER_FAILED
+ if found == 2: # matched END.
+ break
+ if found == 3: # RUNNER_FAILED
logging.error('RUNNER_FAILED')
overall_fail = True
break
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698