Verify EVS extended information properly am: 9cf7c9f12b am: 85f140dc63 am: a6e3085e0d

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

Change-Id: Id86ceff9ed7bee51dcee2bb2f82da11a753d23a9
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:43:01 +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