summaryrefslogtreecommitdiff
path: root/test/TileCacheTests.cpp
diff options
context:
space:
mode:
authorMiklos Vajna <vmiklos@collabora.com>2019-10-31 09:39:38 +0100
committerMiklos Vajna <vmiklos@collabora.com>2019-10-31 09:39:45 +0100
commit3ba17824e356db841aae2e40d07dd5c29aba9128 (patch)
tree076f025b147391b328ec4785df0c7ff3bd0c49c4 /test/TileCacheTests.cpp
parent6ca47a7ab0f7affe02d1e3fe588e1182c91d28ca (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.cpp3
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 ");