diff options
| author | 2023-05-31 21:26:41 +0000 | |
|---|---|---|
| committer | 2023-05-31 21:27:14 +0000 | |
| commit | 9c2032247e3ba71394247432b776c4991d10cc7b (patch) | |
| tree | a90f32425fe61f5ff939de4f8bacf33dec9cb8c8 | |
| parent | 4afa2c395765c7d944505c94f46c4172f6476975 (diff) | |
RPC binder: one more pool test flake fix
Fixes: 285094756
Test: binderRpcTest
Change-Id: Ie36371803c03ab4e25f7bee0a26842c5f2dc92e4
| -rw-r--r-- | libs/binder/tests/binderRpcTest.cpp | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/libs/binder/tests/binderRpcTest.cpp b/libs/binder/tests/binderRpcTest.cpp index 505f30fc41..f88cfd6965 100644 --- a/libs/binder/tests/binderRpcTest.cpp +++ b/libs/binder/tests/binderRpcTest.cpp @@ -461,8 +461,11 @@ static void testThreadPoolOverSaturated(sp<IBinderRpcTest> iface, size_t numCall EXPECT_GE(epochMsAfter, epochMsBefore + 2 * sleepMs); - // Potential flake, but make sure calls are handled in parallel. - EXPECT_LE(epochMsAfter, epochMsBefore + 4 * sleepMs); + // Potential flake, but make sure calls are handled in parallel. Due + // to past flakes, this only checks that the amount of time taken has + // some parallelism. Other tests such as ThreadPoolGreaterThanEqualRequested + // check this more exactly. + EXPECT_LE(epochMsAfter, epochMsBefore + (numCalls - 1) * sleepMs); } TEST_P(BinderRpc, ThreadPoolOverSaturated) { |