maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[clangd] Disable flaky test

This commit is contained in:
Sam McCall 2022-07-07 19:14:37 +02:00
parent 9432fbfe13
commit b15127d056
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
clang-tools-extra/clangd/unittests/TUSchedulerTests.cpp
View File

@ -1409,10 +1409,10 @@ TEST_F(TUSchedulerTests, PreambleThrottle) {
}
void release(RequestID ID) override {
Releases.push_back(ID);
Callback SatisfyNext;
{
std::lock_guard<std::mutex> Lock(Mu);
Releases.push_back(ID);
if (ID > 0 && Acquires.size() == NumRequests)
SatisfyNext = std::move(Callbacks[ID - 1]);
}
@ -1497,6 +1497,9 @@ TEST_F(TUSchedulerTests, PreambleThrottle) {
// We haven't released anything yet, we're still waiting.
EXPECT_THAT(Throttler.Releases, testing::IsEmpty());
// FIXME: This is flaky, becaues the request can be destroyed after shutdown
// if it hasn't been dequeued yet (stop() resets NextRequest).
#if 0
// Now close file A, which will shut down its AST worker.
S.remove(A);
// Request is destroyed after the queue shutdown, so release() has happened.
@ -1505,6 +1508,7 @@ TEST_F(TUSchedulerTests, PreambleThrottle) {
EXPECT_THAT(BuiltFilenames, testing::IsEmpty());
// But we've cancelled the request to build A (not sure which its ID is).
EXPECT_THAT(Throttler.Releases, ElementsAre(AnyOf(1, 0)));
#endif
// Now shut down the TU Scheduler.
}