From 712f48c2176b4833cafb4972478939c8e8f7f04a Mon Sep 17 00:00:00 2001 From: Tim Murray Date: Tue, 19 Apr 2022 17:36:33 -0700 Subject: [PATCH] Fix WorkerThreadTest Queue capacity needs to be NUM_TASKS + 1 for the extra final cv.notify at the end. Test: atest --host --rerun-until-failure 1000 WorkerThreadTest Bug: b/229801892 Change-Id: I95eb07680fe80cfc3bfafaa2de81c48d61d14b4b --- biometrics/fingerprint/aidl/default/tests/WorkerThreadTest.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/biometrics/fingerprint/aidl/default/tests/WorkerThreadTest.cpp b/biometrics/fingerprint/aidl/default/tests/WorkerThreadTest.cpp index b2417bd242..902fb40c6b 100644 --- a/biometrics/fingerprint/aidl/default/tests/WorkerThreadTest.cpp +++ b/biometrics/fingerprint/aidl/default/tests/WorkerThreadTest.cpp @@ -71,7 +71,7 @@ TEST(WorkerThreadTest, ScheduleReturnsFalseWhenQueueIsFull) { TEST(WorkerThreadTest, TasksExecuteInOrder) { constexpr int NUM_TASKS = 10000; - WorkerThread worker(NUM_TASKS); + WorkerThread worker(NUM_TASKS + 1); std::mutex mut; std::condition_variable cv;