binderSafeInterfaceTest: another quick flake fix

Steven Moreland · Steven Moreland · commit b754f1c33d63 · 2025-01-03T20:59:53.000Z
P.S. we should delete safe interface and move everything
to using AIDL-generated interfaces. It doesn't support
all the things AIDL does, and we don't want so many
ways of doing things.

Change-Id: I92c97e458314c0c85ec9ca8eab2c1552abec8ca6
Fixes: 387346410
Test: TH
diff --git a/libs/binder/tests/binderSafeInterfaceTest.cpp b/libs/binder/tests/binderSafeInterfaceTest.cpp
@@ -789,7 +789,7 @@ TEST_F(SafeInterfaceTest, TestCallMeBack) {
         std::optional<int32_t> waitForCallback() {
             std::unique_lock<decltype(mMutex)> lock(mMutex);
             bool success =
-                    mCondition.wait_for(lock, 100ms, [&]() { return static_cast<bool>(mValue); });
+                    mCondition.wait_for(lock, 1000ms, [&]() { return static_cast<bool>(mValue); });
             return success ? mValue : std::nullopt;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -789,7 +789,7 @@ TEST_F(SafeInterfaceTest, TestCallMeBack) {`
`789`	`789`	`std::optional<int32_t> waitForCallback() {`
`790`	`790`	`std::unique_lock<decltype(mMutex)> lock(mMutex);`
`791`	`791`	`bool success =`
`792`		`- mCondition.wait_for(lock, 100ms, [&]() { return static_cast<bool>(mValue); });`
	`792`	`+ mCondition.wait_for(lock, 1000ms, [&]() { return static_cast<bool>(mValue); });`
`793`	`793`	`return success ? mValue : std::nullopt;`
`794`	`794`	`}`
`795`	`795`