Avoid skipping test if it's supported.

Previously the test will be skipped regardless whether setThreads is
supported, this patch moves the check below and check the returned
status code to determine whether the test should be skipped.

Bug: b/287282833
Test: atest VtsHalPowerTargetTest
Change-Id: I9ce6b4ead5e245478247903a75922ece74f38ee7
This commit is contained in:
Peiyong Lin
2023-06-14 18:12:39 +00:00
parent 51d7c99586
commit 9fb755319b

View File

@@ -255,11 +255,10 @@ TEST_P(PowerAidl, setThreads) {
}
ASSERT_TRUE(status.isOk());
if (mApiLevel < kCompatibilityMatrix8ApiLevel) {
status = session->setThreads(kEmptyTids);
if (mApiLevel < kCompatibilityMatrix8ApiLevel && isUnknownOrUnsupported(status)) {
GTEST_SKIP() << "DEVICE not launching with Android 14 and beyond.";
}
status = session->setThreads(kEmptyTids);
ASSERT_FALSE(status.isOk());
ASSERT_EQ(EX_ILLEGAL_ARGUMENT, status.getExceptionCode());