From 4370ecf4e74cfbe6dee9edd2cfef00ba2c37430d Mon Sep 17 00:00:00 2001 From: Tim Lin Date: Fri, 8 Apr 2022 22:54:48 +0800 Subject: [PATCH] wait for response before running next TC All TCs store RadioResponseInfo in a shared variable and read it to verify the result. So there is race condition problem if we don't wait for responses. Bug: 228593077 Test: run vts -m VtsHalRadioTargetTest Change-Id: I45e60f324926588c55abd0a719fd55352767a3eb --- radio/aidl/vts/radio_config_test.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/radio/aidl/vts/radio_config_test.cpp b/radio/aidl/vts/radio_config_test.cpp index 2309f9ef7e..81d87d23c4 100644 --- a/radio/aidl/vts/radio_config_test.cpp +++ b/radio/aidl/vts/radio_config_test.cpp @@ -59,6 +59,7 @@ TEST_P(RadioConfigTest, getHalDeviceCapabilities) { serial = GetRandomSerialNumber(); ndk::ScopedAStatus res = radio_config->getHalDeviceCapabilities(serial); ASSERT_OK(res); + EXPECT_EQ(std::cv_status::no_timeout, wait()); ALOGI("getHalDeviceCapabilities, rspInfo.error = %s\n", toString(radioRsp_config->rspInfo.error).c_str()); } @@ -70,6 +71,7 @@ TEST_P(RadioConfigTest, getSimSlotsStatus) { serial = GetRandomSerialNumber(); ndk::ScopedAStatus res = radio_config->getSimSlotsStatus(serial); ASSERT_OK(res); + EXPECT_EQ(std::cv_status::no_timeout, wait()); ALOGI("getSimSlotsStatus, rspInfo.error = %s\n", toString(radioRsp_config->rspInfo.error).c_str()); }