================ @@ -124,7 +124,9 @@ TEST_F(MainLoopTest, NoSpuriousPipeReads) { // pipe is empty, and we should not be called anymore. char X; size_t len = sizeof(X); - EXPECT_THAT_EXPECTED(r->Read(&X, len).ToError(), llvm::HasValue(1)); + ASSERT_THAT_ERROR(r->Read(&X, len).ToError(), llvm::Succeeded()); + EXPECT_EQ(len, sizeof(X)); + EXPECT_EQ(X, 'X'); ---------------- labath wrote:
My bad. I was looking at the interface for the pipe class. We really ought to rectify the two :/ https://github.com/llvm/llvm-project/pull/145621 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits