[clang-tools-extra] 84cbd47 - [clangd] Fix a data race in RecordsLatencies test

Kadir Cetinkaya via cfe-commits cfe-commits at lists.llvm.org
Sat May 9 06:42:34 PDT 2020


Author: Kadir Cetinkaya
Date: 2020-05-09T15:42:21+02:00
New Revision: 84cbd472e59236bd8ec541bc764ababc6a10a878

URL: https://github.com/llvm/llvm-project/commit/84cbd472e59236bd8ec541bc764ababc6a10a878
DIFF: https://github.com/llvm/llvm-project/commit/84cbd472e59236bd8ec541bc764ababc6a10a878.diff

LOG: [clangd] Fix a data race in RecordsLatencies test

Added: 
    

Modified: 
    clang-tools-extra/clangd/unittests/ClangdLSPServerTests.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clangd/unittests/ClangdLSPServerTests.cpp b/clang-tools-extra/clangd/unittests/ClangdLSPServerTests.cpp
index c60b264baa9c..fe752821859d 100644
--- a/clang-tools-extra/clangd/unittests/ClangdLSPServerTests.cpp
+++ b/clang-tools-extra/clangd/unittests/ClangdLSPServerTests.cpp
@@ -156,7 +156,7 @@ TEST_F(LSPTest, RecordsLatencies) {
   llvm::StringLiteral MethodName = "method_name";
   EXPECT_THAT(Tracer.takeMetric("lsp_latency", MethodName), testing::SizeIs(0));
   llvm::consumeError(Client.call(MethodName, {}).take().takeError());
-  Client.sync();
+  stop();
   EXPECT_THAT(Tracer.takeMetric("lsp_latency", MethodName), testing::SizeIs(1));
 }
 } // namespace


        


More information about the cfe-commits mailing list