diff options
| author | 2024-10-02 02:52:59 +0000 | |
|---|---|---|
| committer | 2024-10-02 02:52:59 +0000 | |
| commit | 22dfdb1ab88db4f28705be802cd8541f2d2a0588 (patch) | |
| tree | 464cf7e039a1963232307b2264e697753d0683bd | |
| parent | cc077fa9125169288173909e17267dc22ec97f79 (diff) | |
| parent | cb7b0fa8b15cd696e5ec2e792eba1f9e6bff44f1 (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.cpp | 11 |
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); } |