Verify EVS extended information properly am: 9cf7c9f12b

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3013053

Change-Id: I4d11e02f4302fd0fba26455349eb0d0c8e1079ba
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Changyeon Jo
2024-07-11 19:07:40 +00:00
committed by Automerger Merge Worker

View File

@@ -317,12 +317,20 @@ TEST_P(EvsAidlTest, CameraOpenClean) {
// Verify methods for extended info
const auto id = 0xFFFFFFFF; // meaningless id
std::vector<uint8_t> values;
bool isSupported = false;
auto status = pCam->setExtendedInfo(id, values);
if (isLogicalCam) {
EXPECT_TRUE(!status.isOk() && status.getServiceSpecificError() ==
static_cast<int>(EvsResult::NOT_SUPPORTED));
} else {
EXPECT_TRUE(status.isOk());
if (status.isOk()) {
// 0xFFFFFFFF is valid for EVS HAL implementation under
// test.
isSupported = true;
} else {
EXPECT_TRUE(status.getServiceSpecificError() ==
static_cast<int>(EvsResult::INVALID_ARG));
}
}
status = pCam->getExtendedInfo(id, &values);
@@ -330,7 +338,12 @@ TEST_P(EvsAidlTest, CameraOpenClean) {
EXPECT_TRUE(!status.isOk() && status.getServiceSpecificError() ==
static_cast<int>(EvsResult::NOT_SUPPORTED));
} else {
EXPECT_TRUE(status.isOk());
if (isSupported) {
EXPECT_TRUE(status.isOk());
} else {
EXPECT_TRUE(!status.isOk() && status.getServiceSpecificError() ==
static_cast<int>(EvsResult::INVALID_ARG));
}
}
// Explicitly close the camera so resources are released right away