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

Unified Diff: scripts/slave/recipes/chromium_trybot.py

Issue 666563007: Revert of Use src-side launcher for telemetry_unittests in chromium_trybot recipe (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 6 years, 2 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
Index: scripts/slave/recipes/chromium_trybot.py
diff --git a/scripts/slave/recipes/chromium_trybot.py b/scripts/slave/recipes/chromium_trybot.py
index 0b29fc5ec3445f8698ecba2f84522a3c276c234a..f3524fdc26a9e678a3d397b20cacfeb874391c7f 100644
--- a/scripts/slave/recipes/chromium_trybot.py
+++ b/scripts/slave/recipes/chromium_trybot.py
@@ -794,18 +794,11 @@
if not api.platform.is_win and not bot_config.get('exclude_compile_all'):
compile_targets = ['all'] + compile_targets
- scripts_compile_targets = \
- api.chromium.get_compile_targets_for_scripts().json.output
-
# Tests that are only run if their compile_targets are going to be built.
conditional_tests = [api.chromium.steps.NaclIntegrationTest()]
if bot_config.get('add_telemetry_tests', True):
- conditional_tests += [
- api.chromium.steps.ScriptTest(
- 'telemetry_unittests', 'telemetry_unittests.py',
- scripts_compile_targets),
- api.chromium.steps.TelemetryPerfUnitTests()
- ]
+ conditional_tests += [api.chromium.steps.TelemetryUnitTests(),
+ api.chromium.steps.TelemetryPerfUnitTests()]
# See if the patch needs to compile on the current platform.
# Don't run analyze for other projects, such as blink, as there aren't that
@@ -826,8 +819,7 @@
tests = []
# TODO(phajdan.jr): Re-enable checkdeps on Windows when it works with git.
if not api.platform.is_win:
- tests.append(api.chromium.steps.ScriptTest(
- 'checkdeps', 'checkdeps.py', scripts_compile_targets))
+ tests.append(api.chromium.steps.ScriptTest('checkdeps', 'checkdeps.py'))
if api.platform.is_linux:
tests.extend([
api.chromium.steps.CheckpermsTest(),
@@ -836,7 +828,8 @@
conditional_tests = tests_in_compile_targets(
api, compile_targets, conditional_tests)
- tests.extend(find_test_named('telemetry_unittests', conditional_tests))
+ tests.extend(find_test_named(api.chromium.steps.TelemetryUnitTests.name,
+ conditional_tests))
tests.extend(find_test_named(api.chromium.steps.TelemetryPerfUnitTests.name,
conditional_tests))
tests.extend(gtest_tests)
« no previous file with comments | « scripts/slave/recipe_modules/chromium/steps.py ('k') | scripts/slave/recipes/chromium_trybot.expected/arm.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698