[Libreoffice-commits] core.git: sc/qa

Xisco Fauli (via logerrit) logerrit at kemper.freedesktop.org
Fri Feb 14 14:58:35 UTC 2020


 sc/qa/extras/anchor.cxx |   16 ++++++++++++++++
 1 file changed, 16 insertions(+)

New commits:
commit 2f9b7f4d7d35d0e2d85132792f8c74c54f05d6c1
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Fri Feb 14 14:02:16 2020 +0100
Commit:     Xisco Faulí <xiscofauli at libreoffice.org>
CommitDate: Fri Feb 14 15:58:00 2020 +0100

    tdf#121963: Add unittest (part 2)
    
    Forgot to add the .cxx file. Thansk to Miklos for noticing
    
    Change-Id: I922680fde98f51f4339fb0e188310b0d7cc635a7
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88708
    Tested-by: Jenkins
    Reviewed-by: Xisco Faulí <xiscofauli at libreoffice.org>

diff --git a/sc/qa/extras/anchor.cxx b/sc/qa/extras/anchor.cxx
index 18ea4f9792fe..5a9d35d18674 100644
--- a/sc/qa/extras/anchor.cxx
+++ b/sc/qa/extras/anchor.cxx
@@ -42,6 +42,7 @@ public:
     void testODFAnchorTypes();
     void testCopyColumnWithImages();
     void testCutWithImages();
+    void testTdf121963();
     void testTdf129552();
     void testTdf130556();
 
@@ -51,6 +52,7 @@ public:
     CPPUNIT_TEST(testODFAnchorTypes);
     CPPUNIT_TEST(testCopyColumnWithImages);
     CPPUNIT_TEST(testCutWithImages);
+    CPPUNIT_TEST(testTdf121963);
     CPPUNIT_TEST(testTdf129552);
     CPPUNIT_TEST(testTdf130556);
     CPPUNIT_TEST_SUITE_END();
@@ -372,6 +374,20 @@ void ScAnchorTest::testCutWithImages()
     pDocSh->DoClose();
 }
 
+void ScAnchorTest::testTdf121963()
+{
+    OUString aFileURL;
+    createFileURL("tdf121963.ods", aFileURL);
+    uno::Reference<css::lang::XComponent> xComponent = loadFromDesktop(aFileURL);
+    CPPUNIT_ASSERT(xComponent.is());
+
+    // Without the accompanying fix in place, this test would have never returned due to an infinite
+    // invalidation loop, where ScGridWindow::Paint() invalidated itself.
+    Scheduler::ProcessEventsToIdle();
+
+    xComponent->dispose();
+}
+
 void ScAnchorTest::testTdf129552()
 {
     OUString aFileURL;


More information about the Libreoffice-commits mailing list