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

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

Issue 2319213002: DevTools: Add devtools for tryserver.chromium.linux (Closed)
Patch Set: Nits Created 4 years, 3 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/devtools.py
diff --git a/scripts/slave/recipes/devtools.py b/scripts/slave/recipes/devtools.py
index c4f32593eb027f8c548f0415a4cc4ac2e22b9aeb..e152e884a95af1e483b3ca2ee4609bfd3afbae7c 100644
--- a/scripts/slave/recipes/devtools.py
+++ b/scripts/slave/recipes/devtools.py
@@ -23,6 +23,10 @@ MASTERS = freeze({
'buildername': 'Chromium DevTools Linux',
'testname': 'devtools_fyi',
},
+ 'tryserver.chromium.linux': {
+ 'buildername': 'chromium_devtools',
+ 'testname': 'devtools_tryserver',
+ },
})
AFFECTED_PATHS = (
@@ -69,35 +73,33 @@ def RunSteps(api):
]
api.python('run eslint', node_path, eslint_args, cwd=devtools_path)
-
-def tryserver_properties(api, mastername, config):
- return api.properties.generic(
- buildername=config['buildername'],
- mastername=mastername,
- rietveld='https://rietveld.example.com',
- issue=1,
- patchset=2,
- )
-
def GenTests(api):
for mastername, config in MASTERS.iteritems():
- yield (
- api.test(config['testname'] + '_main') +
- api.properties.generic(
- buildername=config['buildername'],
- mastername=mastername,
+ if mastername.startswith('tryserver'):
+ yield (
+ api.test(config['testname'] + '_no_devtools') +
+ api.properties.tryserver(
+ buildername=config['buildername'],
+ mastername=mastername,
+ )
+ )
+ yield (
+ api.test(config['testname'] + '_with_devtools') +
+ api.properties.tryserver(
+ buildername=config['buildername'],
+ mastername=mastername,
+ ) +
+ api.override_step_data(
+ 'git diff to analyze patch',
+ api.raw_io.stream_output(
+ 'third_party/WebKit/Source/devtools/fake.js\n'),
+ )
)
- )
- yield (
- api.test(config['testname'] + '_tryserver_no_devtools') +
- tryserver_properties(api, mastername, config)
- )
- yield (
- api.test(config['testname'] + '_tryserver_with_devtools') +
- tryserver_properties(api, mastername, config) +
- api.override_step_data(
- 'git diff to analyze patch',
- api.raw_io.stream_output(
- 'third_party/WebKit/Source/devtools/fake.js\n')
+ else:
+ yield (
+ api.test(config['testname']) +
+ api.properties.generic(
+ buildername=config['buildername'],
+ mastername=mastername,
+ )
)
- )
« no previous file with comments | « masters/master.tryserver.chromium.linux/slaves.cfg ('k') | scripts/slave/recipes/devtools.expected/devtools_fyi.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698