1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Client tests: Remove redundant cleanup testing

Browse Source 
We already have a QTRY_VERIFY(!compositor->surface()); in
tst_WaylandClient::cleanup() which is already run after every test.

The assertions from tests that are explicitly testing surface creation and
destruction have not been removed.

Change-Id: I21d574f5fe33d4e4c309dcfb2f50bde447238df2
Reviewed-by: Paul Olav Tvete <paul.tvete@qt.io>
This commit is contained in:
Johan Klokkhammer Helsing 2018-05-02 14:37:02 +02:00 committed by Johan Helsing
parent 72a5628a34
commit 52b2e5a346
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tests/auto/wayland/client/tst_client.cpp
View File

@ -263,9 +263,6 @@ void tst_WaylandClient::windowScreens()
compositor->sendRemoveOutput(secondOutput);
QTRY_COMPARE(QGuiApplication::screens().size(), 1);
QCOMPARE(window.screen(), primaryScreen);
window.destroy();
QTRY_VERIFY(!compositor->surface());
}
void tst_WaylandClient::removePrimaryScreen()
@ -297,9 +294,6 @@ void tst_WaylandClient::removePrimaryScreen()
QTRY_COMPARE(window.mousePressEventCount, 1);
compositor->sendMouseRelease(surface);
QTRY_COMPARE(window.mouseReleaseEventCount, 1);
window.destroy();
QTRY_VERIFY(!compositor->surface());
}
void tst_WaylandClient::createDestroyWindow()
@ -516,8 +510,6 @@ void tst_WaylandClient::dontCrashOnMultipleCommits()
}
delete window;
QTRY_VERIFY(!compositor->surface());
}
void tst_WaylandClient::hiddenTransientParent()