================
@@ -98,7 +98,6 @@ TEST_F(CodeCompletionTest, TwoDecls) {
 
 TEST_F(CodeCompletionTest, CompFunDeclsNoError) {
   auto Err = llvm::Error::success();
-  auto comps = runComp("void app(", Err);
----------------
AaronBallman wrote:

This just removed the interesting part of the test. I think this should add an 
`EXPECT_EQ` on `comps.size()` as other tests do.

(In fact, I think this unit test will now fail because it sets `Err` to success 
and then expects it to be a failure?)

https://github.com/llvm/llvm-project/pull/204994
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to