diff options
Diffstat (limited to 'clang-tools-extra/unittests/clangd')
-rw-r--r-- | clang-tools-extra/unittests/clangd/ClangdTests.cpp | 28 | ||||
-rw-r--r-- | clang-tools-extra/unittests/clangd/SyncAPI.cpp | 4 | ||||
-rw-r--r-- | clang-tools-extra/unittests/clangd/SyncAPI.h | 3 |
3 files changed, 8 insertions, 27 deletions
diff --git a/clang-tools-extra/unittests/clangd/ClangdTests.cpp b/clang-tools-extra/unittests/clangd/ClangdTests.cpp index a91b65ff2b7..ae47b60a53e 100644 --- a/clang-tools-extra/unittests/clangd/ClangdTests.cpp +++ b/clang-tools-extra/unittests/clangd/ClangdTests.cpp @@ -390,16 +390,7 @@ struct bar { T x; }; // Now switch to C++ mode. CDB.ExtraClangFlags = {"-xc++"}; - // By default addDocument does not check if CompileCommand has changed, so we - // expect to see the errors. - runAddDocument(Server, FooCpp, SourceContents1); - EXPECT_TRUE(DiagConsumer.hadErrorInLastDiags()); - runAddDocument(Server, FooCpp, SourceContents2); - EXPECT_TRUE(DiagConsumer.hadErrorInLastDiags()); - // Passing SkipCache=true will force addDocument to reparse the file with - // proper flags. - runAddDocument(Server, FooCpp, SourceContents2, WantDiagnostics::Auto, - /*SkipCache=*/true); + runAddDocument(Server, FooCpp, SourceContents2, WantDiagnostics::Auto); EXPECT_FALSE(DiagConsumer.hadErrorInLastDiags()); // Subsequent addDocument calls should finish without errors too. runAddDocument(Server, FooCpp, SourceContents1); @@ -431,14 +422,7 @@ int main() { return 0; } // Parse without the define, no errors should be produced. CDB.ExtraClangFlags = {}; - // By default addDocument does not check if CompileCommand has changed, so we - // expect to see the errors. - runAddDocument(Server, FooCpp, SourceContents); - EXPECT_TRUE(DiagConsumer.hadErrorInLastDiags()); - // Passing SkipCache=true will force addDocument to reparse the file with - // proper flags. - runAddDocument(Server, FooCpp, SourceContents, WantDiagnostics::Auto, - /*SkipCache=*/true); + runAddDocument(Server, FooCpp, SourceContents, WantDiagnostics::Auto); ASSERT_TRUE(Server.blockUntilIdleForTest()); EXPECT_FALSE(DiagConsumer.hadErrorInLastDiags()); // Subsequent addDocument call should finish without errors too. @@ -500,10 +484,8 @@ int hello; CDB.ExtraClangFlags.clear(); DiagConsumer.clear(); Server.removeDocument(BazCpp); - Server.addDocument(FooCpp, FooSource.code(), WantDiagnostics::Auto, - /*SkipCache=*/true); - Server.addDocument(BarCpp, BarSource.code(), WantDiagnostics::Auto, - /*SkipCache=*/true); + Server.addDocument(FooCpp, FooSource.code(), WantDiagnostics::Auto); + Server.addDocument(BarCpp, BarSource.code(), WantDiagnostics::Auto); ASSERT_TRUE(Server.blockUntilIdleForTest()); EXPECT_THAT(DiagConsumer.filesWithDiags(), @@ -708,7 +690,7 @@ int d; Server.addDocument(FilePaths[FileIndex], ShouldHaveErrors ? SourceContentsWithErrors : SourceContentsWithoutErrors, - WantDiagnostics::Auto, SkipCache); + WantDiagnostics::Auto); UpdateStatsOnAddDocument(FileIndex, ShouldHaveErrors); }; diff --git a/clang-tools-extra/unittests/clangd/SyncAPI.cpp b/clang-tools-extra/unittests/clangd/SyncAPI.cpp index 0a1c5988e7c..aa2a044f231 100644 --- a/clang-tools-extra/unittests/clangd/SyncAPI.cpp +++ b/clang-tools-extra/unittests/clangd/SyncAPI.cpp @@ -12,8 +12,8 @@ namespace clang { namespace clangd { void runAddDocument(ClangdServer &Server, PathRef File, StringRef Contents, - WantDiagnostics WantDiags, bool SkipCache) { - Server.addDocument(File, Contents, WantDiags, SkipCache); + WantDiagnostics WantDiags) { + Server.addDocument(File, Contents, WantDiags); if (!Server.blockUntilIdleForTest()) llvm_unreachable("not idle after addDocument"); } diff --git a/clang-tools-extra/unittests/clangd/SyncAPI.h b/clang-tools-extra/unittests/clangd/SyncAPI.h index d4d2ac8f901..0a140aa9db3 100644 --- a/clang-tools-extra/unittests/clangd/SyncAPI.h +++ b/clang-tools-extra/unittests/clangd/SyncAPI.h @@ -20,8 +20,7 @@ namespace clangd { // Calls addDocument and then blockUntilIdleForTest. void runAddDocument(ClangdServer &Server, PathRef File, StringRef Contents, - WantDiagnostics WantDiags = WantDiagnostics::Auto, - bool SkipCache = false); + WantDiagnostics WantDiags = WantDiagnostics::Auto); llvm::Expected<CompletionList> runCodeComplete(ClangdServer &Server, PathRef File, Position Pos, |