binderUtilsHostTest: prevent flake

Steven Moreland · Steven Moreland · commit 54e5f257a93b · 2024-12-30T13:34:03.000-08:00
When the child process here takes too long to shutdown, but not 100s that the final sleep takes, this is flaking.

Change-Id: Ic9d5d9020a7abbd37824afc94106cb4b5704163c
Fixes: 383262768
Test: N/A
diff --git a/libs/binder/tests/binderUtilsHostTest.cpp b/libs/binder/tests/binderUtilsHostTest.cpp
@@ -89,8 +89,8 @@ TEST(UtilsHost, ExecuteLongRunning2) {
     }
 
     // ~CommandResult() called, child process is killed.
-    // Assert that the second sleep does not finish.
-    EXPECT_LT(millisSince(start), 6000);
+    // Assert that the last sleep does not finish.
+    EXPECT_LT(millisSince(start), 8000);
 }
 
 TEST(UtilsHost, KillWithSigKill) {

Original file line number	Diff line number	Diff line change
`@@ -89,8 +89,8 @@ TEST(UtilsHost, ExecuteLongRunning2) {`
`89`	`89`	`}`
`90`	`90`
`91`	`91`	`// ~CommandResult() called, child process is killed.`
`92`		`- // Assert that the second sleep does not finish.`
`93`		`- EXPECT_LT(millisSince(start), 6000);`
	`92`	`+ // Assert that the last sleep does not finish.`
	`93`	`+ EXPECT_LT(millisSince(start), 8000);`
`94`	`94`	`}`
`95`	`95`
`96`	`96`	`TEST(UtilsHost, KillWithSigKill) {`