Evolution-x/hardware_interfaces
1
0
Fork 0
mirror of https://github.com/Evolution-X/hardware_interfaces synced 2026-02-01 11:36:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "[DO NOT MERGE] vts: Fix for setProp in VtsHalAutomotiveVehicleV2_0TargetTest" into android12L-tests-dev

Browse Source
This commit is contained in:
Treehugger Robot
2024-02-06 05:25:19 +00:00
committed by Gerrit Code Review
parent 198bcf3a6d 29a9a9ccfd
commit 1f3bee1191
1 changed files with 23 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
automotive/vehicle/2.0/vts/functional/VtsHalAutomotiveVehicleV2_0TargetTest.cpp
View File

@@ -24,6 +24,10 @@
#include <gtest/gtest.h> #include <gtest/gtest.h>
#include <hidl/GtestPrinter.h> #include <hidl/GtestPrinter.h>
#include <hidl/ServiceManagement.h> #include <hidl/ServiceManagement.h>
#include <utils/SystemClock.h>
#include <chrono>
#include <thread>
using namespace android::hardware::automotive::vehicle::V2_0; using namespace android::hardware::automotive::vehicle::V2_0;
using ::android::sp; using ::android::sp;
@@ -32,6 +36,7 @@ using ::android::hardware::Return;
constexpr auto kTimeout = std::chrono::milliseconds(500); constexpr auto kTimeout = std::chrono::milliseconds(500);
constexpr auto kInvalidProp = 0x31600207; constexpr auto kInvalidProp = 0x31600207;
static constexpr auto kPropSetDelayMillis = std::chrono::milliseconds(10000);
class VtsVehicleCallback : public IVehicleCallback { class VtsVehicleCallback : public IVehicleCallback {
private: private:
@@ -240,8 +245,25 @@ TEST_P(VehicleHalHidlTest, setProp) {
// check set success // check set success
invokeGet(cfg.prop, 0); invokeGet(cfg.prop, 0);
ASSERT_EQ(StatusCode::OK, mActualStatusCode); // Retry getting the value until we pass the timeout. getValue might not return
// the expected value immediately since setValue is async.
auto propSetTimeoutMillis = ::android::uptimeMillis() + kPropSetDelayMillis.count();
while (true) {
invokeGet(cfg.prop, 0);
if (mActualStatusCode == StatusCode::OK &&
mActualValue.status == VehiclePropertyStatus::AVAILABLE &&
mActualValue.value.int32Values[0] == setValue) {
break;
}
if (::android::uptimeMillis() >= propSetTimeoutMillis) {
// Reach timeout, the following assert should fail.
break;
}
// Sleep for 100ms between each invokeGet retry.
std::this_thread::sleep_for(std::chrono::milliseconds(100));
}
ASSERT_EQ(StatusCode::OK, mActualStatusCode);
// If the property isn't available, it doesn't make sense to check // If the property isn't available, it doesn't make sense to check
// the returned value. // the returned value.
if (mActualValue.status == VehiclePropertyStatus::AVAILABLE) { if (mActualValue.status == VehiclePropertyStatus::AVAILABLE) {