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

Commit 3a5a3c61 authored by Emilian Peev's avatar Emilian Peev Committed by Eino-Ville Talvala
Browse files

Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP' 

Calls to 'cancelPicture' should be no-ops in case image capture
is not active.

Bug: 64569316
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO
Merged-In: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1
Change-Id: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1
parent 269d9ba0

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

+3 −3
Original line number Diff line number Diff line
@@ -1527,8 +1527,8 @@ TEST_F(CameraHidlTest, cancelPicture) { 
    }

}



// Image capture cancel should fail when image capture is not running.

TEST_F(CameraHidlTest, cancelPictureFail) {

// Image capture cancel is a no-op when image capture is not running.

TEST_F(CameraHidlTest, cancelPictureNOP) {

    for (auto provider : CameraHidlEnvironment::Instance()->mProviders) {

        hidl_vec<hidl_string> cameraDeviceNames = getCameraDeviceNames(

                provider.second);
@@ -1547,7 +1547,7 @@ TEST_F(CameraHidlTest, cancelPictureFail) { 


                Return<Status> returnStatus = device1->cancelPicture();

                ASSERT_TRUE(returnStatus.isOk());

                ASSERT_NE(Status::OK, returnStatus);

                ASSERT_EQ(Status::OK, returnStatus);



                stopPreviewAndClose(device1);

            }