From 3f13a493c8def6978f596f75c22fa7121dc896ab Mon Sep 17 00:00:00 2001 From: Vlad Zagorodniy Date: Mon, 7 Jan 2019 14:35:44 +0200 Subject: [PATCH] [autotests] Destroy main window in ToplevelOpenCloseAnimationTest::testDontAnimatePopups Summary: Be nice and destroy all created windows in testDontAnimatePopups. Test Plan: The test still passes. Reviewers: #kwin, davidedmundson Reviewed By: #kwin, davidedmundson Subscribers: kwin Tags: #kwin Differential Revision: https://phabricator.kde.org/D18037 --- .../effects/toplevel_open_close_animation_test.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/autotests/integration/effects/toplevel_open_close_animation_test.cpp b/autotests/integration/effects/toplevel_open_close_animation_test.cpp index 87dfe295be..0fe95c65c7 100644 --- a/autotests/integration/effects/toplevel_open_close_animation_test.cpp +++ b/autotests/integration/effects/toplevel_open_close_animation_test.cpp @@ -208,6 +208,10 @@ void ToplevelOpenCloseAnimationTest::testDontAnimatePopups() popupSurface.reset(); QVERIFY(popupClosedSpy.wait()); QVERIFY(!effect->isActive()); + + // Destroy the main window. + mainWindowSurface.reset(); + QVERIFY(Test::waitForWindowDestroyed(mainWindow)); } WAYLANDTEST_MAIN(ToplevelOpenCloseAnimationTest)