~aleteoryx/muditaos

2000c3c2d15cd46c35378dc8ccd0552c558c7de3 — Lukasz Mastalerz 2 years ago 310befd
[BH-1764] Fix GridLayoutTesting Border Callback Test

Fix not seting focus to false when all items where removed from parent
2 files changed, 3 insertions(+), 2 deletions(-)

M module-gui/gui/widgets/Item.cpp
M module-gui/test/test-google/test-gui-gridlayout.cpp
M module-gui/gui/widgets/Item.cpp => module-gui/gui/widgets/Item.cpp +1 -0
@@ 93,6 93,7 @@ namespace gui
        }
        if (item == focusItem) {
            focusItem = nullptr;
            focus     = false;
        }

        auto fi = std::find(children.begin(), children.end(), item);

M module-gui/test/test-google/test-gui-gridlayout.cpp => module-gui/test/test-google/test-gui-gridlayout.cpp +2 -2
@@ 1,4 1,4 @@
// Copyright (c) 2017-2021, Mudita Sp. z.o.o. All rights reserved.
// Copyright (c) 2017-2023, Mudita Sp. z.o.o. All rights reserved.
// For licensing, see https://github.com/mudita/MuditaOS/LICENSE.md

#include "gtest/gtest.h"


@@ 267,7 267,7 @@ TEST_F(GridLayoutTesting, Navigate_Test_ActiveItems_2_BorderCallback)
}

///> TODO: Enable this test when issue with setFocus will be resolved
TEST_F(GridLayoutTesting, DISABLED_Border_Callback_Test)
TEST_F(GridLayoutTesting, Border_Callback_Test)
{
    ///> Test for grid layout with 46 elements
    ///> | 1  | 2  | 3  | 4  | 5  | 6  | 7  | 8  | 9  | 10 | 11 | 12 |