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

Commit 721f1728 authored by Igor Murashkin's avatar Igor Murashkin
Browse files

ProCameraTest: Drop frames test move processing delay between lock/unlock 

* This makes it more likely to find races with the buffer being unlocked
  by itself

Change-Id: I1fa7dbfb73f60664a02a35678b58bcc9b06893a2
parent 7fa43f37

camera/tests/ProCameraTests.cpp

+5 −5
Original line number Diff line number Diff line
@@ -1001,11 +1001,6 @@ TEST_F(ProCameraTest, WaitForSingleStreamBufferAndDropFrames) { 


    // Consume a couple of results

    for (int i = 0; i < NUM_REQUESTS; ++i) {

        // Process at 10fps, stream is at 15fps.

        // This means we will definitely fill up the buffer queue with

        // extra buffers and need to drop them.

        usleep(TEST_FRAME_PROCESSING_DELAY_US);



        int numFrames;

        EXPECT_TRUE((numFrames = mCamera->waitForFrameBuffer(streamId)) > 0);
@@ -1025,6 +1020,11 @@ TEST_F(ProCameraTest, WaitForSingleStreamBufferAndDropFrames) { 
                ", dataPtr = " << (void*)buf.data <<

                ", timestamp = " << buf.timestamp << std::endl;



        // Process at 10fps, stream is at 15fps.

        // This means we will definitely fill up the buffer queue with

        // extra buffers and need to drop them.

        usleep(TEST_FRAME_PROCESSING_DELAY_US);



        EXPECT_OK(consumer->unlockBuffer(buf));

    }