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

Side by Side Diff: mojo/tools/get_test_list.py

Issue 774943007: Two tweaks for asan builds (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: review feedback Created 6 years 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 unified diff | Download patch
« no previous file with comments | « no previous file | mojo/tools/mojob.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2014 The Chromium Authors. All rights reserved. 2 # Copyright 2014 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Central list of tests to run (as appropriate for a given config). Add tests 6 """Central list of tests to run (as appropriate for a given config). Add tests
7 to run by modifying this file. 7 to run by modifying this file.
8 8
9 Note that this file is both imported (by mojob.py) and run directly (via a 9 Note that this file is both imported (by mojob.py) and run directly (via a
10 recipe).""" 10 recipe)."""
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 os.path.join("mojo", "tools", "data", "apptests"), 89 os.path.join("mojo", "tools", "data", "apptests"),
90 build_dir]) 90 build_dir])
91 91
92 # Python unit tests: 92 # Python unit tests:
93 if ShouldRunTest(Config.TEST_TYPE_DEFAULT, Config.TEST_TYPE_UNIT, "python"): 93 if ShouldRunTest(Config.TEST_TYPE_DEFAULT, Config.TEST_TYPE_UNIT, "python"):
94 AddEntry("Python unit tests", 94 AddEntry("Python unit tests",
95 ["python", os.path.join("mojo", "tools", 95 ["python", os.path.join("mojo", "tools",
96 "run_mojo_python_tests.py")]) 96 "run_mojo_python_tests.py")])
97 97
98 # Python bindings tests (Linux-only): 98 # Python bindings tests (Linux-only):
99 if target_os == Config.OS_LINUX and ShouldRunTest(Config.TEST_TYPE_DEFAULT, 99 # See http://crbug.com/438781 for details on asan exclusion.
100 Config.TEST_TYPE_UNIT, 100 if (target_os == Config.OS_LINUX and
101 "python"): 101 ShouldRunTest(Config.TEST_TYPE_DEFAULT, Config.TEST_TYPE_UNIT,
102 "python") and
103 config.sanitizer != Config.SANITIZER_ASAN):
102 AddEntry("Python bindings tests", 104 AddEntry("Python bindings tests",
103 ["python", 105 ["python",
104 os.path.join("mojo", "tools", 106 os.path.join("mojo", "tools",
105 "run_mojo_python_bindings_tests.py"), 107 "run_mojo_python_bindings_tests.py"),
106 "--build-dir=" + build_dir]) 108 "--build-dir=" + build_dir])
107 109
108 # Sky tests (Linux-only): 110 # Sky tests (Linux-only):
109 if target_os == Config.OS_LINUX and ShouldRunTest(Config.TEST_TYPE_DEFAULT, 111 if target_os == Config.OS_LINUX and ShouldRunTest(Config.TEST_TYPE_DEFAULT,
110 "sky"): 112 "sky"):
111 # TODO(vtl): Temporary hack: Sky tests are currently really unstable, so 113 # TODO(vtl): Temporary hack: Sky tests are currently really unstable, so
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
166 config = Config(**json.load(args.config_file)) 168 config = Config(**json.load(args.config_file))
167 test_list = GetTestList(config) 169 test_list = GetTestList(config)
168 json.dump(test_list, args.test_list_file, indent=2) 170 json.dump(test_list, args.test_list_file, indent=2)
169 args.test_list_file.write("\n") 171 args.test_list_file.write("\n")
170 172
171 return 0 173 return 0
172 174
173 175
174 if __name__ == "__main__": 176 if __name__ == "__main__":
175 sys.exit(main()) 177 sys.exit(main())
OLDNEW
« 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