[PATCH] D53946: [clangd] Test Context can be used for file status. NFC

2018-10-31 Thread Sam McCall via Phabricator via cfe-commits
sammccall added inline comments. Comment at: unittests/clangd/ClangdTests.cpp:1037 +TEST(ClangdTests, AddDocumentTracksDiagnostics) { + // We have clients relying on the fact that the Context passed to addDocument this tests a lot of things, and after 10 minut

[PATCH] D53946: [clangd] Test Context can be used for file status. NFC

2018-10-31 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov updated this revision to Diff 172008. ilya-biryukov added a comment. - Fix a typo Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53946 Files: unittests/clangd/ClangdTests.cpp Index: unittests/clangd/ClangdTests.cpp ==

[PATCH] D53946: [clangd] Test Context can be used for file status. NFC

2018-10-31 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov added a comment. Added a test that the context behaves the way we want it to. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53946 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/m