summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/unittests/clangd/TUSchedulerTests.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'clang-tools-extra/unittests/clangd/TUSchedulerTests.cpp')
-rw-r--r--clang-tools-extra/unittests/clangd/TUSchedulerTests.cpp14
1 files changed, 5 insertions, 9 deletions
diff --git a/clang-tools-extra/unittests/clangd/TUSchedulerTests.cpp b/clang-tools-extra/unittests/clangd/TUSchedulerTests.cpp
index 34b6f25ce06..b18f04ed39c 100644
--- a/clang-tools-extra/unittests/clangd/TUSchedulerTests.cpp
+++ b/clang-tools-extra/unittests/clangd/TUSchedulerTests.cpp
@@ -65,10 +65,8 @@ TEST_F(TUSchedulerTests, MissingFiles) {
ASSERT_FALSE(bool(Preamble));
ignoreError(Preamble.takeError());
});
- S.remove(Missing, [&](llvm::Error Err) {
- EXPECT_TRUE(bool(Err));
- ignoreError(std::move(Err));
- });
+ // remove() shouldn't crash on missing files.
+ S.remove(Missing);
// Assert there aren't any errors for added file.
S.runWithAST(
@@ -76,7 +74,7 @@ TEST_F(TUSchedulerTests, MissingFiles) {
S.runWithPreamble(Added, [&](llvm::Expected<InputsAndPreamble> Preamble) {
EXPECT_TRUE(bool(Preamble));
});
- S.remove(Added, [&](llvm::Error Err) { EXPECT_FALSE(bool(Err)); });
+ S.remove(Added);
// Assert that all operations fail after removing the file.
S.runWithAST(Added, [&](llvm::Expected<InputsAndAST> AST) {
@@ -87,10 +85,8 @@ TEST_F(TUSchedulerTests, MissingFiles) {
ASSERT_FALSE(bool(Preamble));
ignoreError(Preamble.takeError());
});
- S.remove(Added, [&](llvm::Error Err) {
- EXPECT_TRUE(bool(Err));
- ignoreError(std::move(Err));
- });
+ // remove() shouldn't crash on missing files.
+ S.remove(Added);
}
TEST_F(TUSchedulerTests, ManyUpdates) {
OpenPOWER on IntegriCloud