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

Commit 23ddfba4 authored by Ram Indani's avatar Ram Indani Committed by Android (Google) Code Review
Browse files

Merge "[Composer VTS] Check Vrr mode is the only mode in the Config Group" into main

parents 892e1fea 2b2ec50b
Loading
Loading
Loading
Loading
+9 −2
Original line number Diff line number Diff line
@@ -1231,13 +1231,20 @@ TEST_P(GraphicsComposerAidlV3Test, GetDisplayConfigurations) {
            EXPECT_NE(-1, displayConfig.vsyncPeriod);
            EXPECT_NE(-1, displayConfig.configGroup);
            if (displayConfig.dpi) {
                EXPECT_NE(-1, displayConfig.dpi->x);
                EXPECT_NE(-1, displayConfig.dpi->y);
                EXPECT_NE(-1.f, displayConfig.dpi->x);
                EXPECT_NE(-1.f, displayConfig.dpi->y);
            }
            if (displayConfig.vrrConfig) {
                const auto& vrrConfig = *displayConfig.vrrConfig;
                EXPECT_GE(vrrConfig.minFrameIntervalNs, displayConfig.vsyncPeriod);

                EXPECT_EQ(1, std::count_if(
                                     displayConfigurations.cbegin(), displayConfigurations.cend(),
                                     [displayConfig](const auto& config) {
                                         return config.configGroup == displayConfig.configGroup;
                                     }))
                        << "There should be only one VRR mode in one ConfigGroup";

                const auto verifyFrameIntervalIsDivisorOfVsync = [&](int32_t frameIntervalNs) {
                    constexpr auto kThreshold = 0.05f;  // 5%
                    const auto ratio =