[PATCH] D40784: [clangd] Split code-completion tests out of ClangdTests. NFC.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 4 23:20:41 PST 2017


sammccall added inline comments.


================
Comment at: unittests/clangd/CMakeLists.txt:16
   JSONExprTests.cpp
+  TestFS.cpp
   TraceTests.cpp
----------------
hokein wrote:
> nit: maybe name it FSTests.cpp? to be consistent with other test file names.
Ah, this is deliberately inconsistent, though :-)
This isn't a test file, it's a library that's used by tests.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D40784





More information about the cfe-commits mailing list