[PATCH] D74299: [clang-tidy] extend tests of run-clang-tidy

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 5 08:18:55 PDT 2020


aaron.ballman added a comment.

Generally looks good to me, but I had a question about test coverage.



================
Comment at: clang-tools-extra/test/clang-tidy/infrastructure/run-clang-tidy_config-file.cpp:1
-// RUN: %run_clang_tidy --help
 // RUN: rm -rf %t
----------------
Is there a reason to remove this test coverage? I believe this is testing that we can access the help without crashing.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74299/new/

https://reviews.llvm.org/D74299





More information about the cfe-commits mailing list