summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorXisco Fauli <xiscofauli@libreoffice.org>2021-04-16 11:19:31 +0200
committerXisco Fauli <xiscofauli@libreoffice.org>2021-04-16 12:28:24 +0200
commit691aaa442f0c41cedf230d71fc10802c0c5508b8 (patch)
treeb8fbe81271cb6717308e406a35f3d15dfef17dc5
parent63401011324485bf14c824f7a4c9fc7db7444810 (diff)
tdf#120168: sc_subsequent_export: Add unittest
Change-Id: I8a8cf9355fd4be0d7f22dbef3d701ede4b209623 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/114200 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
-rw-r--r--sc/qa/unit/data/xlsx/tdf120168.xlsxbin0 -> 15192 bytes
-rw-r--r--sc/qa/unit/subsequent_export-test.cxx19
2 files changed, 19 insertions, 0 deletions
diff --git a/sc/qa/unit/data/xlsx/tdf120168.xlsx b/sc/qa/unit/data/xlsx/tdf120168.xlsx
new file mode 100644
index 000000000000..362b30595460
--- /dev/null
+++ b/sc/qa/unit/data/xlsx/tdf120168.xlsx
Binary files differ
diff --git a/sc/qa/unit/subsequent_export-test.cxx b/sc/qa/unit/subsequent_export-test.cxx
index 1c17b2f527c1..138555180f41 100644
--- a/sc/qa/unit/subsequent_export-test.cxx
+++ b/sc/qa/unit/subsequent_export-test.cxx
@@ -202,6 +202,7 @@ public:
void testPreserveTextWhitespaceXLSX();
void testPreserveTextWhitespace2XLSX();
void testTextDirectionXLSX();
+ void testTdf120168();
void testTdf66668();
void testTdf130108();
void testTdf76949();
@@ -389,6 +390,7 @@ public:
CPPUNIT_TEST(testMoveCellAnchoredShapesODS);
CPPUNIT_TEST(testMatrixMultiplicationXLSX);
CPPUNIT_TEST(testTextDirectionXLSX);
+ CPPUNIT_TEST(testTdf120168);
CPPUNIT_TEST(testTdf66668);
CPPUNIT_TEST(testTdf130108);
CPPUNIT_TEST(testTdf76949);
@@ -4392,6 +4394,23 @@ void ScExportTest::testTextDirectionXLSX()
xDocSh->DoClose();
}
+void ScExportTest::testTdf120168()
+{
+ ScDocShellRef xDocSh = loadDoc(u"tdf120168.", FORMAT_XLSX);
+ CPPUNIT_ASSERT(xDocSh.is());
+
+ xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, "xl/styles.xml", FORMAT_XLSX);
+ CPPUNIT_ASSERT(pDoc);
+
+ // Without the fix in place, this test would have failed with
+ // - Expected: left
+ // - Actual : general
+ assertXPath(pDoc, "/x:styleSheet/x:cellXfs/x:xf[2]/x:alignment", "horizontal", "left");
+ assertXPath(pDoc, "/x:styleSheet/x:cellXfs/x:xf[3]/x:alignment", "horizontal", "right");
+
+ xDocSh->DoClose();
+}
+
void ScExportTest::testTdf66668()
{
// Would hang on exporting without the fix in place