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

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

Issue 1577983006: Support official Chrome builders. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 11 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/recipe_modules/chromium_tests/chromium_fyi.py
diff --git a/scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py b/scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py
index 3ba0f8a8cac2081f33c1895d353dd86e93103d2d..ba9ebd3308295bd8a4e377221ce88736dbe0cfe2 100644
--- a/scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py
+++ b/scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py
@@ -368,6 +368,34 @@ SPEC = {
'platform': 'linux',
},
},
+ 'ChromePracticeFullTester': {
Sergiy Byelozyorov 2016/01/12 09:31:04 Why was this added? Is this a real config or just
Ken Russell (switch to Gerrit) 2016/01/12 18:29:07 It's a testing config used to get to 100% code cov
+ 'chromium_config': 'chromium',
+ 'chromium_apply_config': ['ninja_confirm_noop'],
+ 'gclient_config': 'chromium',
+ 'chromium_config_kwargs': {
+ 'BUILD_CONFIG': 'Release',
+ 'TARGET_BITS': 64,
+ },
+ 'bot_type': 'builder_tester',
+ 'compile_targets': [
+ 'chromium_swarm_tests',
+ ],
+ 'tests': [
+ steps.GTestTest('base_unittests'),
+ ],
+ 'test_generators': [
+ steps.generate_gtest,
+ steps.generate_script,
+ steps.generate_isolated_script,
+ ],
+ # The official builders specify the test_spec here as opposed to
+ # reading it from a file.
+ 'test_spec': {
Sergiy Byelozyorov 2016/01/12 09:41:34 nit: keep } on the same line, i.e. 'test_spec': {}
Ken Russell (switch to Gerrit) 2016/01/12 18:29:07 I'm going to leave this as is. It's the same way i
+ },
+ 'testing': {
+ 'platform': 'linux',
+ },
+ },
'ChromiumPractice': {
'chromium_config': 'chromium',
'gclient_config': 'blink_merged',

Powered by Google App Engine
This is Rietveld 408576698