[PATCH] D146655: [clang-tidy] Ignore DISABLED_ in test suite name in google-avoid-underscore-in-googletest-name
Piotr Zegar via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 22 14:34:51 PDT 2023
PiotrZSL added inline comments.
================
Comment at: clang-tools-extra/test/clang-tidy/checkers/google/avoid-underscore-in-googletest-name.cpp:90
// Underscores are allowed to disable a test with the DISABLED_ prefix.
-// https://github.com/google/googletest/blob/master/googletest/docs/faq.md#why-should-test-suite-names-and-test-names-not-contain-underscore
+// https://google.github.io/googletest/faq.html#why-should-test-suite-names-and-test-names-not-contain-underscore
TEST(TestCaseName, TestName) {}
----------------
Would be good if this link would exist also in documentation for this check.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146655/new/
https://reviews.llvm.org/D146655
More information about the cfe-commits
mailing list