diff options
author | Paul Olav Tvete <paul.tvete@qt.io> | 2023-09-15 16:04:05 +0200 |
---|---|---|
committer | Paul Olav Tvete <paul.tvete@qt.io> | 2023-09-18 09:45:25 +0200 |
commit | 3b58b13d9b209c9e9b1a99d0eea224c15b3b3e04 (patch) | |
tree | d623578d6b56c5a9a4e37784b8fd81812bddb4ba /tests/auto/compositor/compositor | |
parent | d8d3d6097afeac62f1b0285e3d5365c7cb580547 (diff) |
Autotest: don't access object that has gone out of scope
Fixes: QTBUG-117069
Change-Id: Iafa603ce9b31eb66beebf18d62661c4daaf2632b
Reviewed-by: David Edmundson <davidedmundson@kde.org>
Diffstat (limited to 'tests/auto/compositor/compositor')
-rw-r--r-- | tests/auto/compositor/compositor/tst_compositor.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/tests/auto/compositor/compositor/tst_compositor.cpp b/tests/auto/compositor/compositor/tst_compositor.cpp index ed7490b36..86b56326b 100644 --- a/tests/auto/compositor/compositor/tst_compositor.cpp +++ b/tests/auto/compositor/compositor/tst_compositor.cpp @@ -1199,9 +1199,9 @@ void tst_WaylandCompositor::emitsErrorOnSameIviId() QTRY_VERIFY(&firstClient.iviApplication); QWaylandIviSurface *firstIviSurface = nullptr; - QObject::connect(&compositor.iviApplication, &QWaylandIviApplication::iviSurfaceCreated, [&](QWaylandIviSurface *s) { - firstIviSurface = s; - }); + auto connection = QObject::connect(&compositor.iviApplication, + &QWaylandIviApplication::iviSurfaceCreated, + [&](QWaylandIviSurface *s) { firstIviSurface = s; }); firstClient.createIviSurface(firstClient.createSurface(), 123); QTRY_VERIFY(firstIviSurface); @@ -1220,6 +1220,7 @@ void tst_WaylandCompositor::emitsErrorOnSameIviId() QTRY_COMPARE(static_cast<ivi_application_error>(secondClient.protocolError.code), IVI_APPLICATION_ERROR_IVI_ID); QTRY_COMPARE(compositor.clients().size(), 1); } + QObject::disconnect(connection); } // The other clients have passed out of scope and have been destroyed, |