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

Tamás Zolnai (via logerrit) logerrit at kemper.freedesktop.org
Tue Oct 1 11:08:38 UTC 2019


 test/TileCacheTests.cpp |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 10a0ee1cfe9c3a90e13b8fbf7fb8c1591418c2ad
Author:     Tamás Zolnai <tamas.zolnai at collabora.com>
AuthorDate: Mon Sep 30 13:42:43 2019 +0200
Commit:     Tamás Zolnai <tamas.zolnai at collabora.com>
CommitDate: Tue Oct 1 13:03:31 2019 +0200

    Fix up TileCacheTests::testTileInvalidateWriterPage() test
    
    The part number in case of writer has no actual meaning, since
    for writer we have only one part (the whole document).
    It seems the code is not consistent in that it is 0 or 1.
    
    Change-Id: I9e56079f7861f7e0d5794e6f711045cd9448dfa7
    Reviewed-on: https://gerrit.libreoffice.org/79851
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
    Reviewed-by: Tamás Zolnai <tamas.zolnai at collabora.com>

diff --git a/test/TileCacheTests.cpp b/test/TileCacheTests.cpp
index 673d9580c..12bd324e7 100644
--- a/test/TileCacheTests.cpp
+++ b/test/TileCacheTests.cpp
@@ -830,7 +830,7 @@ void TileCacheTests::testTileInvalidateWriterPage()
     int part = -1;
     CPPUNIT_ASSERT_MESSAGE("No part# in invalidatetiles message.",
                            LOOLProtocol::getTokenIntegerFromMessage(res, "part", part));
-    CPPUNIT_ASSERT_EQUAL(1, part);
+    CPPUNIT_ASSERT_EQUAL(0, part);
 }
 
 // This isn't yet used


More information about the Libreoffice-commits mailing list