Don't duplicate gtest-parallel flags in gtest-parallel-wrappers.
Also, move the --timeout=900 flag from the recipe to the mb.py script. That way it is passed as an arg to gtest-parallel and not to the test executable.
BUG=webrtc:7568
NOTRY=True
Review-Url: https://codereview.webrtc.org/2862803002
Cr-Commit-Position: refs/heads/master@{#18015}
diff --git a/tools-webrtc/mb/mb_unittest.py b/tools-webrtc/mb/mb_unittest.py
index 26ca190..2950015 100755
--- a/tools-webrtc/mb/mb_unittest.py
+++ b/tools-webrtc/mb/mb_unittest.py
@@ -448,8 +448,9 @@
self.assertEqual(command, [
'../../testing/test_env.py',
'../../tools-webrtc/gtest-parallel-wrapper.py',
- '--gtest_color=no',
'--output_dir=${ISOLATED_OUTDIR}/test_logs',
+ '--gtest_color=no',
+ '--timeout=900',
'./base_unittests',
'--workers=1',
'--',
@@ -494,8 +495,9 @@
self.assertEqual(command, [
'../../testing/xvfb.py',
'../../tools-webrtc/gtest-parallel-wrapper.py',
- '--gtest_color=no',
'--output_dir=${ISOLATED_OUTDIR}/test_logs',
+ '--gtest_color=no',
+ '--timeout=900',
'./base_unittests',
'--',
'--asan=0',
@@ -540,8 +542,9 @@
self.assertEqual(command, [
'../../testing/test_env.py',
'../../tools-webrtc/gtest-parallel-wrapper.py',
- '--gtest_color=no',
'--output_dir=${ISOLATED_OUTDIR}\\test_logs',
+ '--gtest_color=no',
+ '--timeout=900',
r'.\unittests.exe',
'--',
'--asan=0',
@@ -582,8 +585,9 @@
self.assertEqual(command, [
'../../testing/test_env.py',
'../../tools-webrtc/gtest-parallel-wrapper.py',
- '--gtest_color=no',
'--output_dir=${ISOLATED_OUTDIR}/test_logs',
+ '--gtest_color=no',
+ '--timeout=900',
'./base_unittests',
'--',
'--asan=0',