From d594e608c3bba914b84be9e848a1040962b4df0c Mon Sep 17 00:00:00 2001 From: Jimmy Shiu Date: Wed, 20 Mar 2024 08:21:22 +0000 Subject: [PATCH] VTSHalPowerTarget.FMQAidl: Make sure session status OK before running test While PowerHal version >= 5 but devices didn't support HintSession. We should skip the test. Otherwise, it would fail. Merged-In: I0f757ab825020ded02c31231969e13f3375e47f3 Change-Id: I0f757ab825020ded02c31231969e13f3375e47f3 Test: atest VTSHalPowerTarget Bug: 328330332 --- power/aidl/vts/VtsHalPowerTargetTest.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/power/aidl/vts/VtsHalPowerTargetTest.cpp b/power/aidl/vts/VtsHalPowerTargetTest.cpp index 939deb2966..272674f194 100644 --- a/power/aidl/vts/VtsHalPowerTargetTest.cpp +++ b/power/aidl/vts/VtsHalPowerTargetTest.cpp @@ -159,7 +159,10 @@ class FMQAidl : public PowerAidl { auto status = power->createHintSessionWithConfig(getpid(), getuid(), kSelfTids, 16666666L, SessionTag::OTHER, &mSessionConfig, &mSession); - ASSERT_TRUE(status.isOk()); + mSessionSupport = status.isOk(); + if (!mSessionSupport) { + GTEST_SKIP() << "DEVICE not support Hint Session."; + } ASSERT_NE(nullptr, mSession); status = power->getSessionChannel(getpid(), getuid(), &mChannelConfig);