Add sanitizers to GitHub Actions CI

Run all testcases with ASan, UBSan and TSan during Linux CI
except some special testcases for ExitedWithCode and AllowLeak.

Introduces the following defines to indicate when built with a sanitizer,
which can be used to disable sanitizers on testcase level:
- GTEST_HAS_ADDRESS_SANITIZER
- GTEST_HAS_UNDEFINED_BEHAVIOR_SANITIZER
- GTEST_HAS_THREAD_SANITIZER

and the following bazel test tags to disable on test binary level:
no_asan, no_ubsan, no_tsan

Co-authored-by: tbonza <tylers.pile@gmail.com>

.bazelrc

@@ -0,0 +1,29 @@
+build:asan --strip=never
+build:asan --copt -fsanitize=address
+build:asan --copt -DGTEST_HAS_ADDRESS_SANITIZER
+build:asan --copt -O2
+build:asan --copt -g
+build:asan --copt -fno-omit-frame-pointer
+build:asan --copt -fno-sanitize-recover=all
+build:asan --linkopt -fsanitize=address
+build:asan --test_tag_filters=-no_asan
+
+build:ubsan --strip=never
+build:ubsan --copt -fsanitize=undefined
+build:ubsan --copt -DGTEST_HAS_UNDEFINED_BEHAVIOR_SANITIZER
+build:ubsan --copt -O2
+build:ubsan --copt -g
+build:ubsan --copt -fno-omit-frame-pointer
+build:ubsan --copt -fno-sanitize-recover=all
+build:ubsan --linkopt -fsanitize=undefined
+build:ubsan --test_tag_filters=-no_ubsan
+
+build:tsan --strip=never
+build:tsan --copt -fsanitize=thread
+build:tsan --copt -DGTEST_HAS_THREAD_SANITIZER
+build:tsan --copt -O2
+build:tsan --copt -g
+build:tsan --copt -fno-omit-frame-pointer
+build:tsan --copt -fno-sanitize-recover=all
+build:tsan --linkopt -fsanitize=thread
+build:tsan --test_tag_filters=-no_tsan

.github/workflows/gtest-ci.yml

@@ -19,6 +19,20 @@ jobs:
     - name: Tests
       run: bazel test --cxxopt=-std=c++14 --features=external_include_paths --test_output=errors ...
 
+  Linux_with_sanitizer:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        config: [asan, ubsan, tsan]
+
+    steps:
+    - uses: actions/checkout@v3
+      with:
+        fetch-depth: 0
+
+    - name: Tests with '${{ matrix.config }}'
+      run: bazel test -c dbg --config=${{ matrix.config }} --cxxopt=-std=c++14 --features=external_include_paths --test_output=errors ...
+
   macOS:
     runs-on: macos-latest
     steps:

BUILD.bazel

@@ -215,5 +215,9 @@ cc_test(
     name = "sample10_unittest",
     size = "small",
     srcs = ["googletest/samples/sample10_unittest.cc"],
+    tags = [
+        # A memory error is expected.
+        "no_asan",
+    ],
     deps = [":gtest"],
 )

googlemock/test/gmock-spec-builders_test.cc

@@ -2154,6 +2154,7 @@ TEST_F(GMockLogTest, DoesNotPrintWarningInternallyIfVerbosityIsError) {
   helper_.Foo(PrintMeNot());  // This should generate a warning.
 }
 
+#if !defined(GTEST_HAS_ADDRESS_SANITIZER)
 // Tests Mock::AllowLeak().
 
 TEST(AllowLeakTest, AllowsLeakingUnusedMockObject) {
@@ -2193,6 +2194,7 @@ TEST(AllowLeakTest, WorksWhenBothOnCallAndExpectCallArePresent) {
   EXPECT_CALL(*a, DoA(_)).Times(AnyNumber());
   Mock::AllowLeak(a);
 }
+#endif
 
 // Tests that we can verify and clear a mock object's expectations
 // when none of its methods has expectations.

googletest/test/googletest-death-test-test.cc

@@ -283,6 +283,8 @@ TEST(ExitStatusPredicateTest, ExitedWithCode) {
   EXPECT_FALSE(pred1(status42));
 }
 
+#if !defined(GTEST_HAS_ADDRESS_SANITIZER) && \
+    !defined(GTEST_HAS_THREAD_SANITIZER)
 // Tests the KilledBySignal predicate.
 TEST(ExitStatusPredicateTest, KilledBySignal) {
   const int status_segv = KilledExitStatus(SIGSEGV);
@@ -294,6 +296,7 @@ TEST(ExitStatusPredicateTest, KilledBySignal) {
   EXPECT_FALSE(pred_segv(status_kill));
   EXPECT_FALSE(pred_kill(status_segv));
 }
+#endif
 
 #endif  // GTEST_OS_WINDOWS || GTEST_OS_FUCHSIA