summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Steven Moreland <smoreland@google.com> 2024-10-02 02:52:59 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2024-10-02 02:52:59 +0000
commit22dfdb1ab88db4f28705be802cd8541f2d2a0588 (patch)
tree464cf7e039a1963232307b2264e697753d0683bd
parentcc077fa9125169288173909e17267dc22ec97f79 (diff)
parentcb7b0fa8b15cd696e5ec2e792eba1f9e6bff44f1 (diff)
Merge "Fix binderCacheUnitTest: Add 50ms wait with retry" into main am: cb7b0fa8b1
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3287676 Change-Id: I8a39f10e460ceddf293da1ee3649bdaca928258a Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r--libs/binder/tests/binderCacheUnitTest.cpp11
1 files changed, 10 insertions, 1 deletions
diff --git a/libs/binder/tests/binderCacheUnitTest.cpp b/libs/binder/tests/binderCacheUnitTest.cpp
index 482d197688..be5d559d8e 100644
--- a/libs/binder/tests/binderCacheUnitTest.cpp
+++ b/libs/binder/tests/binderCacheUnitTest.cpp
@@ -149,7 +149,16 @@ TEST_F(LibbinderCacheTest, RemoveFromCacheOnServerDeath) {
EXPECT_EQ(OK, mServiceManager->addService(kCachedServiceName, binder2));
// Confirm that new service is returned instead of old.
- sp<IBinder> result2 = mServiceManager->checkService(kCachedServiceName);
+ int retry_count = 5;
+ sp<IBinder> result2;
+ do {
+ std::this_thread::sleep_for(std::chrono::milliseconds(50));
+ if (retry_count-- == 0) {
+ break;
+ }
+ result2 = mServiceManager->checkService(kCachedServiceName);
+ } while (result2 != binder2);
+
ASSERT_EQ(binder2, result2);
}