[PATCH] D46676: [clangd] Remove LSP command-based #include insertion.

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov added inline comments.


================
Comment at: unittests/clangd/ClangdTests.cpp:941
 
-TEST_F(ClangdVFSTest, InsertIncludes) {
-  MockFSProvider FS;
----------------
Do we test the same thing somewhere else  (e.g. code completion) in one of the dependent changes?
Maybe it's worth noting in the commit description that this test was not removed completely, but instead moved to <new-test-name-here>?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D46676