Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e0c4c497 authored by Emilian Peev's avatar Emilian Peev
Browse files

Camera: Add only physical ids during multi-camera configuration 

Per API specification "physicalCameraId" must not
contain any logical camera ids. The test case
should only use the physical ids it receives from the
camera static metadata.

Bug:109874630
Test:run vts --skip-all-system-status-check --skip-preconditions
--primary-abi-only --module VtsHalCameraProviderV2_4Target -l INFO

Change-Id: I82c5bf44f4572b3a6abdbb3101a11140907b1c9e
parent 0be95494

camera/provider/2.4/vts/functional/VtsHalCameraProviderV2_4TargetTest.cpp

+2 −3
Original line number Diff line number Diff line
@@ -3480,11 +3480,10 @@ TEST_F(CameraHidlTest, processMultiCaptureRequestPreview) { 
        ret = session->close();

        ASSERT_TRUE(ret.isOk());



        // Leave only 2 physical devices in the id set.

        auto it = physicalIds.begin();

        string physicalDeviceId = *it;

        // Leave only the first physical device in the id set and insert the logical device.

        string physicalDeviceId = *it; it++;

        physicalIds.erase(++it, physicalIds.end());

        physicalIds.emplace(deviceId);

        ASSERT_EQ(physicalIds.size(), 2u);



        V3_4::HalStreamConfiguration halStreamConfig;