From 38ac95c03b423a063d095aaa4e3254a43f97c49b Mon Sep 17 00:00:00 2001
From: David Edmundson <davidedmundson@kde.org>
Date: Tue, 7 May 2024 13:09:08 +0100
Subject: [PATCH] tests: Don't use auto and manual configure together

The tests always send an explicit manual configure. This is important
because we want to test the client's first buffer is correct and this
comes after the first configure event.

Having auto configure enabled means it's racey and flakey between the
auto response on the compositor thread and the client dispatching a
manual repsonse.

Change-Id: I92170a2d96c01d01c22983e43ff1f1281b1a48a2
Reviewed-by: David Redondo <qt@david-redondo.de>
---
 tests/auto/wayland/xdgdecorationv1/tst_xdgdecorationv1.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/auto/wayland/xdgdecorationv1/tst_xdgdecorationv1.cpp b/tests/auto/wayland/xdgdecorationv1/tst_xdgdecorationv1.cpp
index 54b15b3bf65..5ee85694406 100644
--- a/tests/auto/wayland/xdgdecorationv1/tst_xdgdecorationv1.cpp
+++ b/tests/auto/wayland/xdgdecorationv1/tst_xdgdecorationv1.cpp
@@ -111,7 +111,7 @@ public:
     explicit XdgDecorationCompositor()
     {
         exec([this] {
-            m_config.autoConfigure = true;
+            m_config.autoConfigure = false;
             add<XdgDecorationManagerV1>(xdgDecorationVersion);
         });
     }