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

Unified Diff: mojo/tools/get_test_list.py

Issue 1147443002: Only run the NaCl tests if the test type is DEFAULT, UNIT, or app. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Remove accidentally modified file. Created 5 years, 7 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
« no previous file with comments | « no previous file | mojo/tools/mojob.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/tools/get_test_list.py
diff --git a/mojo/tools/get_test_list.py b/mojo/tools/get_test_list.py
index dedba5d8db2af10d2b24115187b8237debd2cf36..13cee6bc22fd5b70646667a2d7a611aa976409b3 100755
--- a/mojo/tools/get_test_list.py
+++ b/mojo/tools/get_test_list.py
@@ -114,7 +114,9 @@ def GetTestList(config, verbose_count=0):
"--build-dir=" + build_dir])
# NaCl tests (Linux only):
- if target_os == Config.OS_LINUX and config.sanitizer != Config.SANITIZER_ASAN:
+ if (target_os == Config.OS_LINUX and
+ config.sanitizer != Config.SANITIZER_ASAN and
+ ShouldRunTest(Config.TEST_TYPE_DEFAULT, Config.TEST_TYPE_UNIT, "app") ):
viettrungluu 2015/05/14 22:49:38 nit: no space before second )
AddEntry("NaCl tests",
[os.path.join(build_dir, "monacl_shell"),
os.path.join(build_dir, "irt_" + config.target_cpu,
« no previous file with comments | « no previous file | mojo/tools/mojob.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698