<div dir="ltr"><div dir="ltr">i was trying to provide a more generic "callback" mechanism, but you are right, it is not needed for this test.</div><div dir="ltr"><br></div><div>going to keep context prop logic though, as it might be necessary later on. SG?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, May 3, 2020 at 9:09 PM Sam McCall via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">sammccall added inline comments.<br>
<br>
<br>
================<br>
Comment at: clang-tools-extra/clangd/unittests/ClangdLSPServerTests.cpp:164<br>
+    WithContextValue Ctx(<br>
+        llvm::make_scope_exit([&CallFinished] { CallFinished.notify(); }));<br>
+    llvm::consumeError(Client.call(MethodName, {}).take().takeError());<br>
----------------<br>
Sorry, I didn't really put all the pieces together in my head the first time around.<br>
The context propagation seems OK, but it's too fiddly as a way to control sequencing - can't you just call Client.sync() and assert after that?<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D79302/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D79302/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D79302" rel="noreferrer" target="_blank">https://reviews.llvm.org/D79302</a><br>
<br>
<br>
<br>
</blockquote></div></div>