Android: Filter browsertests with MANUAL_ and PRE_ prefix.

build/android/pylib/browsertests/dispatch.py

 # Copyright (c) 2013 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
 
 from pylib import android_commands
 from pylib import cmd_helper
 from pylib import constants
@@ skipping 37 matching lines @@
         options.build_type,
         options.webkit,
         constants.BROWSERTEST_TEST_PACKAGE_NAME,
         constants.BROWSERTEST_TEST_ACTIVITY_NAME,
         constants.BROWSERTEST_COMMAND_LINE_FILE)
 
     # Get tests and split them up based on the number of devices.
   if options.gtest_filter:
     all_tests = [t for t in options.gtest_filter.split(':') if t]
   else:
-    all_tests = gtest_dispatch.GetAllEnabledTests(RunnerFactory,
-                                                  attached_devices)
+    all_enabled = gtest_dispatch.GetAllEnabledTests(RunnerFactory,
+                                                    attached_devices)
+    all_tests = _FilterTests(all_enabled)
 
   # Run tests.
   test_results = shard.ShardAndRunTests(RunnerFactory, attached_devices,
                                         all_tests, options.build_type)
   test_results.LogFull(
       test_type='Unit test',
       test_package=constants.BROWSERTEST_SUITE_NAME,
       build_type=options.build_type,
       flakiness_server=options.flakiness_dashboard_server)
   test_results.PrintAnnotation()
+
+def _FilterTests(all_enabled_tests):
+  """Filters out tests and fixtures starting with PRE_ and MANUAL_."""
+  return [t for t in all_enabled_tests if _ValidTest(t)]
+
+def _ValidTest(test):

[Yaron, 2013/03/18 20:41:31]

Instead of "Valid", call "Automatable"/Bot?

[nilesh, 2013/03/18 22:02:47]

Renamed to _ShouldRunOnBot

+  fixture, case = test.split('.', 1)
+  if _StartsWith(fixture, case, "PRE_"):
+    return False
+  if _StartsWith(fixture, case, "MANUAL_"):
+    return False
+  return True
+
+def _StartsWith(a, b, prefix):
+  return a.startswith(prefix) or b.startswith(prefix)