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

Commit 926a5fc9 authored by Lloyd Pique's avatar Lloyd Pique
Browse files

SF: Fix unit test getColorModes() expectation 

4b0e4ddf changed when the composer getColorMdoes() function is called,
so that it is only called if wide color displays are configured as
supported.

This patch simple adjusts the tests so that the call is no longer
expected.

Test: atest libsurfaceflinger_unittest
Bug: b/74827900
Change-Id: Id9f0c594d224648d6d30215744633cdab57c58f9
(cherry picked from commit 9654845d)
parent 17ea0116

services/surfaceflinger/tests/unittests/DisplayTransactionTest.cpp

+1 −3
Original line number Diff line number Diff line
@@ -165,9 +165,7 @@ TEST_F(DisplayTransactionTest, processDisplayChangesLockedProcessesPrimaryDispla 


    EXPECT_CALL(*mComposer, getActiveConfig(DisplayDevice::DISPLAY_PRIMARY, _))

            .WillOnce(DoAll(SetArgPointee<1>(0), Return(Error::NONE)));

    EXPECT_CALL(*mComposer, getColorModes(DisplayDevice::DISPLAY_PRIMARY, _))

            .WillOnce(DoAll(SetArgPointee<1>(std::vector<ColorMode>({ColorMode::NATIVE})),

                            Return(Error::NONE)));

    EXPECT_CALL(*mComposer, getColorModes(DisplayDevice::DISPLAY_PRIMARY, _)).Times(0);

    EXPECT_CALL(*mComposer, getHdrCapabilities(DisplayDevice::DISPLAY_PRIMARY, _, _, _, _))

            .WillOnce(DoAll(SetArgPointee<1>(std::vector<Hdr>()), Return(Error::NONE)));