diff options
| author | 2022-08-23 02:34:35 +0000 | |
|---|---|---|
| committer | 2022-08-23 02:34:35 +0000 | |
| commit | 12528b183683e9d1f08dc38bd86757638f56ba09 (patch) | |
| tree | fac3e70f1fd0c5d449d9cfe3c485899f3b7207b7 /libs | |
| parent | e3f13500b4260abb91b652ec2a34d6336431eb33 (diff) | |
| parent | 29d6ddfca51b5b5562cbd0c87287e05835f392c4 (diff) | |
Merge "binderRpcTest: assert on wait_for timeout" am: 453334dab3 am: 9132781781 am: c1438662aa am: 29d6ddfca5
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2190036
Change-Id: Ie7b45ebaab8e7ae3d2b936e7a8afa4a6d128f0a4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
Diffstat (limited to 'libs')
| -rw-r--r-- | libs/binder/tests/binderRpcTest.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/libs/binder/tests/binderRpcTest.cpp b/libs/binder/tests/binderRpcTest.cpp index 0ee15a1498..b59308c156 100644 --- a/libs/binder/tests/binderRpcTest.cpp +++ b/libs/binder/tests/binderRpcTest.cpp @@ -1095,10 +1095,7 @@ TEST_P(BinderRpc, SingleDeathRecipient) { } std::unique_lock<std::mutex> lock(dr->mMtx); - if (!dr->dead) { - EXPECT_EQ(std::cv_status::no_timeout, dr->mCv.wait_for(lock, 1000ms)); - } - EXPECT_TRUE(dr->dead) << "Failed to receive the death notification."; + ASSERT_TRUE(dr->mCv.wait_for(lock, 1000ms, [&]() { return dr->dead; })); // need to wait for the session to shutdown so we don't "Leak session" EXPECT_TRUE(proc.proc.sessions.at(0).session->shutdownAndWait(true)); |