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

Side by Side Diff: build/android/pylib/browsertests/dispatch.py

Issue 19040002: [Android] Convert more test targets to use isolate. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add maruel/csharp as owners for isolate files Created 7 years, 5 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « breakpad/breakpad_unittests.isolate ('k') | build/android/pylib/gtest/dispatch.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """Dispatches content_browsertests.""" 5 """Dispatches content_browsertests."""
6 6
7 import logging 7 import logging
8 import os 8 import os
9 import sys 9 import sys
10 10
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
48 # to dispatch any tests. 48 # to dispatch any tests.
49 if not ports.ResetTestServerPortAllocation(): 49 if not ports.ResetTestServerPortAllocation():
50 raise Exception('Failed to reset test server port.') 50 raise Exception('Failed to reset test server port.')
51 51
52 test_suite_dir = os.path.join(cmd_helper.OutDirectory.get(), 52 test_suite_dir = os.path.join(cmd_helper.OutDirectory.get(),
53 options.build_type) 53 options.build_type)
54 options.test_suite = os.path.join(test_suite_dir, 54 options.test_suite = os.path.join(test_suite_dir,
55 'apks', 55 'apks',
56 constants.BROWSERTEST_SUITE_NAME + '.apk') 56 constants.BROWSERTEST_SUITE_NAME + '.apk')
57 57
58 deps_dir = gtest_dispatch._GenerateDepsDirUsingIsolate(
59 constants.BROWSERTEST_SUITE_NAME, options.build_type)
60
58 # Constructs a new TestRunner with the current options. 61 # Constructs a new TestRunner with the current options.
59 def RunnerFactory(device, shard_index): 62 def RunnerFactory(device, shard_index):
60 return test_runner.TestRunner( 63 return test_runner.TestRunner(
61 device, 64 device,
62 options.test_suite, 65 options.test_suite,
63 options.test_arguments, 66 options.test_arguments,
64 options.timeout, 67 options.timeout,
65 options.cleanup_test_files, 68 options.cleanup_test_files,
66 options.tool, 69 options.tool,
67 options.build_type, 70 options.build_type,
68 options.webkit, 71 options.webkit,
69 options.push_deps, 72 options.push_deps,
70 constants.BROWSERTEST_TEST_PACKAGE_NAME, 73 constants.BROWSERTEST_TEST_PACKAGE_NAME,
71 constants.BROWSERTEST_TEST_ACTIVITY_NAME, 74 constants.BROWSERTEST_TEST_ACTIVITY_NAME,
72 constants.BROWSERTEST_COMMAND_LINE_FILE) 75 constants.BROWSERTEST_COMMAND_LINE_FILE,
76 deps_dir=deps_dir)
73 77
74 # Get tests and split them up based on the number of devices. 78 # Get tests and split them up based on the number of devices.
75 all_enabled = gtest_dispatch.GetAllEnabledTests(RunnerFactory, 79 all_enabled = gtest_dispatch.GetAllEnabledTests(RunnerFactory,
76 attached_devices) 80 attached_devices)
77 if options.test_filter: 81 if options.test_filter:
78 all_tests = unittest_util.FilterTestNames(all_enabled, 82 all_tests = unittest_util.FilterTestNames(all_enabled,
79 options.test_filter) 83 options.test_filter)
80 else: 84 else:
81 all_tests = _FilterTests(all_enabled) 85 all_tests = _FilterTests(all_enabled)
82 86
(...skipping 24 matching lines...) Expand all
107 fixture, case = test.split('.', 1) 111 fixture, case = test.split('.', 1)
108 if _StartsWith(fixture, case, 'PRE_'): 112 if _StartsWith(fixture, case, 'PRE_'):
109 return False 113 return False
110 if _StartsWith(fixture, case, 'MANUAL_'): 114 if _StartsWith(fixture, case, 'MANUAL_'):
111 return False 115 return False
112 return True 116 return True
113 117
114 118
115 def _StartsWith(a, b, prefix): 119 def _StartsWith(a, b, prefix):
116 return a.startswith(prefix) or b.startswith(prefix) 120 return a.startswith(prefix) or b.startswith(prefix)
OLDNEW
« no previous file with comments | « breakpad/breakpad_unittests.isolate ('k') | build/android/pylib/gtest/dispatch.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698