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

Unified Diff: tools/mb/mb.py

Issue 2503503002: MB: Run test with gtest-parallel on swarming. (Closed)
Patch Set: Seems to work now :) Created 4 years, 1 month 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: tools/mb/mb.py
diff --git a/tools/mb/mb.py b/tools/mb/mb.py
index 038217e072b032b3b2243cd7a38ca70b008f5f21..798c54ea3761e46a88eaf540613ca976a7fff60b 100755
--- a/tools/mb/mb.py
+++ b/tools/mb/mb.py
@@ -1077,6 +1077,7 @@ class MetaBuildWrapper(object):
executable = isolate_map[target].get('executable', target)
executable_suffix = '.exe' if self.platform == 'win32' else ''
+ executable_preffix = '.\\' if self.platform == 'win32' else './'
cmdline = []
extra_files = []
@@ -1108,26 +1109,30 @@ class MetaBuildWrapper(object):
'xdisplaycheck',
'../../testing/test_env.py',
'../../testing/xvfb.py',
+ '../../third_party/gtest-parallel/gtest-parallel',
]
cmdline = [
'../../testing/xvfb.py',
'.',
- './' + str(executable) + executable_suffix,
- '--brave-new-test-launcher',
- '--test-launcher-bot-mode',
+ 'python',
+ '../../third_party/gtest-parallel/gtest-parallel',
+ executable_preffix + str(executable) + executable_suffix,
+ '--',
'--asan=%d' % asan,
'--msan=%d' % msan,
'--tsan=%d' % tsan,
]
elif test_type in ('windowed_test_launcher', 'console_test_launcher'):
extra_files = [
- '../../testing/test_env.py'
+ '../../testing/test_env.py',
+ '../../third_party/gtest-parallel/gtest-parallel',
]
cmdline = [
'../../testing/test_env.py',
- './' + str(executable) + executable_suffix,
- '--brave-new-test-launcher',
- '--test-launcher-bot-mode',
+ 'python',
+ '../../third_party/gtest-parallel/gtest-parallel',
+ executable_preffix + str(executable) + executable_suffix,
+ '--',
'--asan=%d' % asan,
'--msan=%d' % msan,
'--tsan=%d' % tsan,
« third_party/gtest-parallel/gtest-parallel ('K') | « third_party/gtest-parallel/gtest-parallel ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698