commit 7f1009ac70352bd2b7a468086dbbb499c5941810
author Ted Pudlik <tpudlik@google.com> Tue Nov 02 23:27:30 2021 +0000
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Nov 03 16:16:12 2021 +0000
tree 88c20ccb95d94d8a18acc47e99609bc9cafe4005
parent cd2ce40d3f3f76c9ca89866826e9d18f8569f0c3

pw_presubmit: Fix bug in sanitizer presubmit

Actually run (don't just build) the sanitizer tests, so that
`pw presubmit --step runtime_sanitizers` has a chance of catching
any bugs. Needless to say, this exposes a bunch of failures. I'm slowly
triaging them.

Bug: 514
No-Docs-Update-Reason: Minor bugfix
Change-Id: I67860b1eda68783b6b5504407698fa21fb30acf9
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/67080
Reviewed-by: Rob Mohr <mohrr@google.com>
Commit-Queue: Ted Pudlik <tpudlik@google.com>

BUILD.gn

diff --git a/BUILD.gn b/BUILD.gn
index 70da9d5..f94d769 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -161,10 +161,10 @@
 group("runtime_sanitizers") {
   if (host_os != "win") {
     deps = [
-      ":pw_module_tests($dir_pigweed/targets/host:host_clang_asan)",
-      ":pw_module_tests($dir_pigweed/targets/host:host_clang_msan)",
-      ":pw_module_tests($dir_pigweed/targets/host:host_clang_tsan)",
-      ":pw_module_tests($dir_pigweed/targets/host:host_clang_ubsan)",
+      ":pw_module_tests.run($dir_pigweed/targets/host:host_clang_asan)",
+      ":pw_module_tests.run($dir_pigweed/targets/host:host_clang_msan)",
+      ":pw_module_tests.run($dir_pigweed/targets/host:host_clang_tsan)",
+      ":pw_module_tests.run($dir_pigweed/targets/host:host_clang_ubsan)",
     ]
   }
 }