diff options
author | Tim Lin <timtmlin@google.com> | 2022-04-08 22:54:48 +0800 |
---|---|---|
committer | Tim Lin <timtmlin@google.com> | 2022-04-08 22:54:51 +0800 |
commit | 4370ecf4e74cfbe6dee9edd2cfef00ba2c37430d (patch) | |
tree | fde8c776ad2aa8361d9d07e047be0d473e2ff419 /radio | |
parent | c19a0bb019493a3d868ae0bc8ff1da87132bcf8e (diff) |
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
Diffstat (limited to 'radio')
-rw-r--r-- | radio/aidl/vts/radio_config_test.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
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()); } |