test/TileCacheTests.cpp | 37 +++++++++++++++++++++++-------------- 1 file changed, 23 insertions(+), 14 deletions(-)
New commits: commit 0106ab24f646ff28e463679bf2c540c6bcfacebf Author: Ashod Nakashian <ashod.nakash...@collabora.co.uk> AuthorDate: Sat Oct 26 10:11:35 2019 -0400 Commit: Ashod Nakashian <ashnak...@gmail.com> CommitDate: Tue Oct 29 02:40:34 2019 +0100 test: improve TileCacheTests::testUnresponsiveClient Change-Id: Ib08c4675c9477cb925b5d42545cdc2cdae780fbe Reviewed-on: https://gerrit.libreoffice.org/81579 Reviewed-by: Ashod Nakashian <ashnak...@gmail.com> Tested-by: Ashod Nakashian <ashnak...@gmail.com> diff --git a/test/TileCacheTests.cpp b/test/TileCacheTests.cpp index 913d777fc..f9d94c6e1 100644 --- a/test/TileCacheTests.cpp +++ b/test/TileCacheTests.cpp @@ -430,20 +430,22 @@ void TileCacheTests::testDisconnectMultiView() void TileCacheTests::testUnresponsiveClient() { - const char* testname = "testUnresponsiveClient"; + const std::string testname = "unresponsiveClient-"; + std::string documentPath, documentURL; - getDocumentPathAndURL("hello.odt", documentPath, documentURL, "unresponsiveClient "); + getDocumentPathAndURL("hello.odt", documentPath, documentURL, testname); TST_LOG("Connecting first client."); - std::shared_ptr<LOOLWebSocket> socket1 = loadDocAndGetSocket(_uri, documentURL, "unresponsiveClient-1 "); + std::shared_ptr<LOOLWebSocket> socket1 + = loadDocAndGetSocket(_uri, documentURL, testname + "1 "); TST_LOG("Connecting second client."); - std::shared_ptr<LOOLWebSocket> socket2 = loadDocAndGetSocket(_uri, documentURL, "unresponsiveClient-2 "); + std::shared_ptr<LOOLWebSocket> socket2 + = loadDocAndGetSocket(_uri, documentURL, testname + "2 "); // Pathologically request tiles and fail to read (say slow connection). // Meanwhile, verify that others can get all tiles fine. // TODO: Track memory consumption to verify we don't buffer too much. - std::ostringstream oss; for (int i = 0; i < 1000; ++i) { @@ -454,24 +456,31 @@ void TileCacheTests::testUnresponsiveClient() for (int x = 0; x < 8; ++x) { // Invalidate to force re-rendering. - sendTextFrame(socket2, "uno .uno:SelectAll"); - sendTextFrame(socket2, "uno .uno:Delete"); - assertResponseString(socket2, "invalidatetiles:", "client2 "); - sendTextFrame(socket2, "paste mimetype=text/html\n" + documentContents); - assertResponseString(socket2, "invalidatetiles:", "client2 "); + deleteAll(socket2, testname); + assertResponseString(socket2, "invalidatetiles:", testname + "2 "); + sendTextFrame(socket2, "paste mimetype=text/html\n" + documentContents, testname + "2 "); + assertResponseString(socket2, "invalidatetiles:", testname + "2 "); // Ask for tiles and don't read! - sendTextFrame(socket1, "tilecombine nviewid=0 part=0 width=256 height=256 tileposx=0,3840,7680,11520,0,3840,7680,11520 tileposy=0,0,0,0,3840,3840,3840,3840 tilewidth=3840 tileheight=3840"); + sendTextFrame(socket1, "tilecombine nviewid=0 part=0 width=256 height=256 " + "tileposx=0,3840,7680,11520,0,3840,7680,11520 " + "tileposy=0,0,0,0,3840,3840,3840,3840 tilewidth=3840 " + "tileheight=3840", + testname + "1 "); // Verify that we get all 8 tiles. - sendTextFrame(socket2, "tilecombine nviewid=0 part=0 width=256 height=256 tileposx=0,3840,7680,11520,0,3840,7680,11520 tileposy=0,0,0,0,3840,3840,3840,3840 tilewidth=3840 tileheight=3840"); + sendTextFrame(socket2, "tilecombine nviewid=0 part=0 width=256 height=256 " + "tileposx=0,3840,7680,11520,0,3840,7680,11520 " + "tileposy=0,0,0,0,3840,3840,3840,3840 tilewidth=3840 " + "tileheight=3840", + testname + "2 "); for (int i = 0; i < 8; ++i) { - std::vector<char> tile = getResponseMessage(socket2, "tile:", "client2 "); + std::vector<char> tile = getResponseMessage(socket2, "tile:", testname + "2 "); CPPUNIT_ASSERT_MESSAGE("Did not receive tile #" + std::to_string(i+1) + " of 8: message as expected", !tile.empty()); } /// Send canceltiles message to clear tiles-on-fly list, otherwise wsd waits for tileprocessed messages - sendTextFrame(socket2, "canceltiles"); + sendTextFrame(socket2, "canceltiles", testname + "2 "); } } _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits