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

Commit 9654845d 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: None
Change-Id: Id9f0c594d224648d6d30215744633cdab57c58f9
parent bad8c0a7

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)));