[Libreoffice-commits] online.git: test/httpwstest.cpp

[Miklos Vajna]

 test/httpwstest.cpp |    2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 43eabc09d12d666b1a60f7ae26fdf91864fd273b
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date:   Fri May 19 17:11:20 2017 +0200

    Fix HTTPWSTest::testInsertDelete() failure
    
    .uno:DeletePage deletes the "current" page, while the intent here is to
    delete all slides except the first one. Be explicit about this.
    
    Change-Id: Ia8a8a5bf907e3d79cc646d54803447525375ce72
    Reviewed-on: https://gerrit.libreoffice.org/37833
    Reviewed-by: Ashod Nakashian <ashnakash at gmail.com>
    Tested-by: Miklos Vajna <vmiklos at collabora.co.uk>

diff --git a/test/httpwstest.cpp b/test/httpwstest.cpp
index 4382158a..5d80f40e 100644
--- a/test/httpwstest.cpp
+++ b/test/httpwstest.cpp
@@ -1149,6 +1149,8 @@ void HTTPWSTest::testInsertDelete()
         std::cerr << "Deleting 10 slides." << std::endl;
         for (size_t it = 1; it <= 10; it++)
         {
+            // Explicitly delete the nth slide.
+            sendTextFrame(socket, "setclientpart part=" + std::to_string(it));
             sendTextFrame(socket, "uno .uno:DeletePage");
             response = getResponseString(socket, "status:");
             CPPUNIT_ASSERT_MESSAGE("did not receive a status: message as expected", !response.empty());