diff options
author | Miklos Vajna <vmiklos@collabora.com> | 2019-10-31 09:39:38 +0100 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2019-10-31 09:39:45 +0100 |
commit | 3ba17824e356db841aae2e40d07dd5c29aba9128 (patch) | |
tree | 076f025b147391b328ec4785df0c7ff3bd0c49c4 /test/TileCacheTests.cpp | |
parent | 6ca47a7ab0f7affe02d1e3fe588e1182c91d28ca (diff) |
Fix TileCacheTests::testUnresponsiveClient()
This started to reliably fail for me recently.
I think the problem was that deleteAll() also listened for incoming
messages, so while waiting for textselection:, perhaps it processed (and
ignored) invalidatetiles:.
As a consequence, once deleteAll() returned and testUnresponsiveClient()
started to wait for invalidatetiles:, the test may or may not failed,
depending on the order of messages.
Fix the problem by just dispatching the SelectAll and Delete commands,
then waiting for invalidatetiles:.
Change-Id: I33f096e7dcb7f9daf1968dcb50d8a5f53824f5bd
Diffstat (limited to 'test/TileCacheTests.cpp')
-rw-r--r-- | test/TileCacheTests.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/test/TileCacheTests.cpp b/test/TileCacheTests.cpp index f9d94c6e1..0f0195b10 100644 --- a/test/TileCacheTests.cpp +++ b/test/TileCacheTests.cpp @@ -456,7 +456,8 @@ void TileCacheTests::testUnresponsiveClient() for (int x = 0; x < 8; ++x) { // Invalidate to force re-rendering. - deleteAll(socket2, testname); + sendTextFrame(socket2, "uno .uno:SelectAll", testname); + sendTextFrame(socket2, "uno .uno:Delete", testname); assertResponseString(socket2, "invalidatetiles:", testname + "2 "); sendTextFrame(socket2, "paste mimetype=text/html\n" + documentContents, testname + "2 "); assertResponseString(socket2, "invalidatetiles:", testname + "2 "); |