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

[Xisco Fauli (via logerrit)]

 sc/qa/unit/data/xlsx/tdf107586.xlsx   |binary
 sc/qa/unit/subsequent_export-test.cxx |   17 +++++++++++++++++
 2 files changed, 17 insertions(+)

New commits:
commit 6c217cb062853989929ca2a6b7ee29cf68808f6b
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Tue Apr 13 16:17:50 2021 +0200
Commit:     Xisco Fauli <xiscofauli at libreoffice.org>
CommitDate: Tue Apr 13 19:37:33 2021 +0200

    tdf#107586: sc_subsequent_export: Add unittest
    
    Change-Id: I9f69b179e227ffeb56f9cc207618880cdd00331b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/114055
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofauli at libreoffice.org>

diff --git a/sc/qa/unit/data/xlsx/tdf107586.xlsx b/sc/qa/unit/data/xlsx/tdf107586.xlsx
new file mode 100644
index 000000000000..b5a32c4b8024
Binary files /dev/null and b/sc/qa/unit/data/xlsx/tdf107586.xlsx differ
diff --git a/sc/qa/unit/subsequent_export-test.cxx b/sc/qa/unit/subsequent_export-test.cxx
index be2339b7f25d..c7af15f9003e 100644
--- a/sc/qa/unit/subsequent_export-test.cxx
+++ b/sc/qa/unit/subsequent_export-test.cxx
@@ -205,6 +205,7 @@ public:
     void testTdf66668();
     void testTdf130108();
     void testTdf76949();
+    void testTdf107586();
     void testTdf55417();
     void testTdf129985();
     void testTdf73063();
@@ -391,6 +392,7 @@ public:
     CPPUNIT_TEST(testTdf66668);
     CPPUNIT_TEST(testTdf130108);
     CPPUNIT_TEST(testTdf76949);
+    CPPUNIT_TEST(testTdf107586);
     CPPUNIT_TEST(testTdf55417);
     CPPUNIT_TEST(testTdf129985);
     CPPUNIT_TEST(testTdf73063);
@@ -4431,6 +4433,21 @@ void ScExportTest::testTdf76949()
     xDocSh->DoClose();
 }
 
+void ScExportTest::testTdf107586()
+{
+    ScDocShellRef xDocSh = loadDoc(u"tdf107586.", FORMAT_XLSX);
+    CPPUNIT_ASSERT(xDocSh.is());
+
+    xmlDocUniquePtr pSheet = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, "xl/worksheets/sheet1.xml", FORMAT_XLSX);
+    CPPUNIT_ASSERT(pSheet);
+
+    // Without the fix in place, this test would haved failed with
+    // XPath '/x:worksheet/x:sheetPr/x:tabColor' number of nodes is incorrect
+    assertXPath(pSheet, "/x:worksheet/x:sheetPr/x:tabColor", "rgb", "FF9BBB59");
+
+    xDocSh->DoClose();
+}
+
 void ScExportTest::testTdf55417()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf55417.", FORMAT_XLSX);