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

Unified Diff: scripts/slave/recipe_modules/chromium_tests/chromium_linux.py

Issue 1679703004: Remove build-side specification of instrumentation tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Rerun expectations. Created 4 years, 10 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 | scripts/slave/recipes/chromium.expected/chromium_linux_Android_Tests_logcat_upload_timeout.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/chromium_tests/chromium_linux.py
diff --git a/scripts/slave/recipe_modules/chromium_tests/chromium_linux.py b/scripts/slave/recipe_modules/chromium_tests/chromium_linux.py
index 3435aab76dc4d20bbe910a5fa739cef60b8d9988..698d04471ba3f2bb0e6e5c7774549218d8b7d1a1 100644
--- a/scripts/slave/recipe_modules/chromium_tests/chromium_linux.py
+++ b/scripts/slave/recipe_modules/chromium_tests/chromium_linux.py
@@ -249,10 +249,6 @@ SPEC = {
'android_config': 'main_builder',
'root_devices': True,
'tests': [
- steps.AndroidInstrumentationTest('AndroidWebViewTest'),
- steps.AndroidInstrumentationTest('ChromePublicTest'),
- steps.AndroidInstrumentationTest('ContentShellTest'),
- steps.AndroidInstrumentationTest('ChromeSyncShellTest'),
steps.AndroidJunitTest('base_junit_tests'),
steps.AndroidJunitTest('chrome_junit_tests'),
steps.AndroidJunitTest('components_junit_tests'),
@@ -262,8 +258,9 @@ SPEC = {
],
'test_generators': [
steps.generate_gtest,
- steps.generate_script,
+ steps.generate_instrumentation_test,
steps.generate_isolated_script,
+ steps.generate_script,
],
'testing': {
'platform': 'linux',
@@ -305,10 +302,6 @@ SPEC = {
'android_config': 'main_builder',
'root_devices': True,
'tests': [
- steps.AndroidInstrumentationTest('AndroidWebViewTest'),
- steps.AndroidInstrumentationTest('ChromePublicTest'),
- steps.AndroidInstrumentationTest('ContentShellTest'),
- steps.AndroidInstrumentationTest('ChromeSyncShellTest'),
steps.AndroidJunitTest('base_junit_tests'),
steps.AndroidJunitTest('chrome_junit_tests'),
steps.AndroidJunitTest('components_junit_tests'),
@@ -318,9 +311,9 @@ SPEC = {
],
'test_generators': [
steps.generate_gtest,
- steps.generate_script,
- steps.generate_isolated_script,
steps.generate_instrumentation_test,
+ steps.generate_isolated_script,
+ steps.generate_script,
],
'testing': {
'platform': 'linux',
« no previous file with comments | « no previous file | scripts/slave/recipes/chromium.expected/chromium_linux_Android_Tests_logcat_upload_timeout.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698