From 3a74be2df6f03b9c775d46905eed9b7cb9900db8 Mon Sep 17 00:00:00 2001 From: David Edmundson Date: Tue, 14 Jul 2020 10:51:22 +0100 Subject: [PATCH] [autotests] Don't change screens during cleanup Screens are set during init, to do so at cleanup doesn't bring a lot. It leads to a potential awkward raciness with xwayland failing the test, whilst doing something that isn't part of what we're testing here or likely to happen in real life. --- autotests/integration/internal_window.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/autotests/integration/internal_window.cpp b/autotests/integration/internal_window.cpp index 4dd9e35204..caa5b96ed2 100644 --- a/autotests/integration/internal_window.cpp +++ b/autotests/integration/internal_window.cpp @@ -209,8 +209,6 @@ void InternalWindowTest::init() void InternalWindowTest::cleanup() { Test::destroyWaylandConnection(); - - QMetaObject::invokeMethod(kwinApp()->platform(), "setVirtualOutputs", Qt::DirectConnection, Q_ARG(int, 2)); } void InternalWindowTest::testEnterLeave()