diff options
| author | 2022-10-05 01:29:00 +0000 | |
|---|---|---|
| committer | 2022-10-05 01:29:00 +0000 | |
| commit | 7f2d58d105cdd156a57376418635cd33c13dbaec (patch) | |
| tree | b29ae272a621a9326a788876b149c986cb22993f | |
| parent | e8151aac902961df1fbee87448e776ea87502b8e (diff) | |
| parent | b42efa0bcc6b1428fde7a93b55220e6a7c664495 (diff) | |
Merge "binderRpcTest: sleeps /= 10" am: 028d0454bc am: b42efa0bcc
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2212315
Change-Id: I755e79f076530a84fc3da23098d70689af8e722c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | libs/binder/tests/binderRpcTest.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/libs/binder/tests/binderRpcTest.cpp b/libs/binder/tests/binderRpcTest.cpp index 652da991a4..fbc83046e0 100644 --- a/libs/binder/tests/binderRpcTest.cpp +++ b/libs/binder/tests/binderRpcTest.cpp @@ -788,12 +788,12 @@ TEST_P(BinderRpc, ThreadPoolGreaterThanEqualRequested) { ts.push_back(std::thread([&] { proc.rootIface->lockUnlock(); })); } - usleep(100000); // give chance for calls on other threads + usleep(10000); // give chance for calls on other threads // other calls still work EXPECT_EQ(OK, proc.rootBinder->pingBinder()); - constexpr size_t blockTimeMs = 500; + constexpr size_t blockTimeMs = 50; size_t epochMsBefore = epochMillis(); // after this, we should never see a response within this time EXPECT_OK(proc.rootIface->unlockInMsAsync(blockTimeMs)); @@ -1085,7 +1085,7 @@ TEST_P(BinderRpc, SingleDeathRecipient) { } std::unique_lock<std::mutex> lock(dr->mMtx); - ASSERT_TRUE(dr->mCv.wait_for(lock, 1000ms, [&]() { return dr->dead; })); + ASSERT_TRUE(dr->mCv.wait_for(lock, 100ms, [&]() { 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)); @@ -1120,7 +1120,7 @@ TEST_P(BinderRpc, SingleDeathRecipientOnShutdown) { 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_EQ(std::cv_status::no_timeout, dr->mCv.wait_for(lock, 100ms)); } EXPECT_TRUE(dr->dead) << "Failed to receive the death notification."; @@ -1703,7 +1703,7 @@ TEST_P(BinderRpcServerOnly, Shutdown) { bool shutdown = false; for (int i = 0; i < 10 && !shutdown; i++) { - usleep(300 * 1000); // 300ms; total 3s + usleep(30 * 1000); // 30ms; total 300ms if (server->shutdown()) shutdown = true; } ASSERT_TRUE(shutdown) << "server->shutdown() never returns true"; |