Created separate shards for suspicious testcases.

build/android/pylib/gtest/gtest_test_instance.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import logging
 import os
 import re
 import tempfile
 
 from devil.android import apk_helper
@@ skipping 68 matching lines @@
 EXTRA_SHARD_NANO_TIMEOUT = (
     'org.chromium.native_test.NativeTestInstrumentationTestRunner.'
         'ShardNanoTimeout')
 _EXTRA_SHARD_SIZE_LIMIT = (
     'org.chromium.native_test.NativeTestInstrumentationTestRunner.'
         'ShardSizeLimit')
 
 # TODO(jbudorick): Remove these once we're no longer parsing stdout to generate
 # results.
 _RE_TEST_STATUS = re.compile(
-    r'\[ +((?:RUN)|(?:FAILED)|(?:OK)) +\] ?([^ ]+)(?: \((\d+) ms\))?$')
+    r'\[ +((?:RUN)|(?:FAILED)|(?:OK)|(?:CRASHED)) +\]'
+    r' ?([^ ]+)?(?: \((\d+) ms\))?$')
 _RE_TEST_RUN_STATUS = re.compile(
     r'\[ +(PASSED|RUNNER_FAILED|CRASHED) \] ?[^ ]+')
-
+# Crash detection constants.
+_RE_TEST_ERROR = re.compile(r'FAILURES!!! Tests run: \d+,'
+                                    r' Failures: \d+, Errors: 1')
+_RE_TEST_CURRENTLY_RUNNING = re.compile(r'\[ERROR:.*?\]'
+                                    r' Currently running: (.*)')
 
 # TODO(jbudorick): Make this a class method of GtestTestInstance once
 # test_package_apk and test_package_exe are gone.
 def ParseGTestListTests(raw_list):
   """Parses a raw test list as provided by --gtest_list_tests.
 
   Args:
     raw_list: The raw test listing with the following format:
 
     IPCChannelTest.
@@ skipping 96 matching lines @@
         self._app_data_file_dir = args.app_data_file_dir
       else:
         self._app_data_file_dir = tempfile.mkdtemp()
         logging.critical('Saving app files to %s', self._app_data_file_dir)
     else:
       self._app_data_files = None
       self._app_data_file_dir = None
 
     self._test_arguments = args.test_arguments
 
+

[jbudorick, 2016/03/01 23:19:15]

nit: remove these two blank lines

+
   @property
   def activity(self):
     return self._apk_helper and self._apk_helper.GetActivityName()
 
   @property
   def apk(self):
     return self._apk_helper and self._apk_helper.path
 
   @property
   def apk_helper(self):
@@ skipping 129 matching lines @@
     """Parses raw gtest output and returns a list of results.
 
     Args:
       output: A list of output lines.
     Returns:
       A list of base_test_result.BaseTestResults.
     """
     log = []
     result_type = None
     results = []
+    test_name = None
     for l in output:
       logging.info(l)
       matcher = _RE_TEST_STATUS.match(l)
       if matcher:
+        # Be aware that test name and status might not appear on same line.
+        test_name = matcher.group(2) if matcher.group(2) else test_name
+        duration = int(matcher.group(3)) if matcher.group(3) else 0
         if matcher.group(1) == 'RUN':
           log = []
         elif matcher.group(1) == 'OK':
           result_type = base_test_result.ResultType.PASS
         elif matcher.group(1) == 'FAILED':
           result_type = base_test_result.ResultType.FAIL
+        elif matcher.group(1) == 'CRASHED':
+          result_type = base_test_result.ResultType.CRASH
+
+      # Needs another matcher here to match crashess, like those of DCHECK.

[jbudorick, 2016/03/01 23:19:15]

nit: s/crashess/crashes/

+      matcher = _RE_TEST_CURRENTLY_RUNNING.match(l)
+      if matcher:
+        test_name = matcher.group(1)
+        result_type = base_test_result.ResultType.CRASH
+        duration = 0 # Don't know.
 
       if log is not None:
         log.append(l)
 
       if result_type:
-        test_name = matcher.group(2)
-        duration = int(matcher.group(3)) if matcher.group(3) else 0
         results.append(base_test_result.BaseTestResult(
             test_name, result_type, duration,
             log=('\n'.join(log) if log else '')))
         log = None
         result_type = None
 
     return results
 
   #override
   def TearDown(self):
     """Clear the mappings created by SetUp."""
     if self._isolate_delegate:
       self._isolate_delegate.Clear()