sw/qa/extras/ooxmlexport/ooxmlexport6.cxx | 33 +++++++++++++++++++++++++++++- 1 file changed, 32 insertions(+), 1 deletion(-)
New commits: commit f75fc5c454db33da938b7c7050f49bfa46baca95 Author: Xisco Fauli <[email protected]> AuthorDate: Thu Nov 20 14:01:47 2025 +0100 Commit: Xisco Fauli <[email protected]> CommitDate: Thu Nov 20 22:49:28 2025 +0100 sw_ooxmlexport6: do not skip DOCX export validation Change-Id: I2ecf22820deed02af772ef2b9cf89715d1d69246 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/194270 Reviewed-by: Xisco Fauli <[email protected]> Tested-by: Jenkins diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport6.cxx b/sw/qa/extras/ooxmlexport/ooxmlexport6.cxx index 2eef94530bc8..f179b63c3ea3 100644 --- a/sw/qa/extras/ooxmlexport/ooxmlexport6.cxx +++ b/sw/qa/extras/ooxmlexport/ooxmlexport6.cxx @@ -30,7 +30,7 @@ class Test : public SwModelTestBase { public: - Test() : SwModelTestBase(u"/sw/qa/extras/ooxmlexport/data/"_ustr, /*bSkipValidation*/ true) {} + Test() : SwModelTestBase(u"/sw/qa/extras/ooxmlexport/data/"_ustr) {} }; CPPUNIT_TEST_FIXTURE(Test, testTdf133701) @@ -82,6 +82,10 @@ CPPUNIT_TEST_FIXTURE(Test, testDmlPictureInTextframe) CPPUNIT_TEST_FIXTURE(Test, testDmlGroupshapeRelsize) { createSwDoc("dml-groupshape-relsize.docx"); + + // FIXME: validation error in OOXML export: Errors: 1 + skipValidation(); + save(TestFilter::DOCX); xmlDocUniquePtr pXmlDoc = parseExport(u"word/document.xml"_ustr); // Relative size wasn't imported. @@ -503,6 +507,10 @@ CPPUNIT_TEST_FIXTURE(Test, testTableFloatingMargins) CPPUNIT_TEST_FIXTURE(Test, testTdf127814) { createSwDoc("tdf127814.docx"); + + // FIXME: validation error in OOXML export: Errors: 136 + skipValidation(); + save(TestFilter::DOCX); // Paragraph top margin was 0 in a table started on a new page xmlDocUniquePtr pXmlDoc = parseExport(u"word/document.xml"_ustr); @@ -552,6 +560,9 @@ CPPUNIT_TEST_FIXTURE(Test, testTdf132514) DECLARE_OOXMLEXPORT_TEST(testTdf153891, "tdf153891.docx") { + // FIXME: validation error in OOXML export: Errors: 1 + skipValidation(); + // This document simply crashed the importer. } @@ -733,6 +744,10 @@ CPPUNIT_TEST_FIXTURE(Test, testFdo73215) CPPUNIT_TEST_FIXTURE(Test, testBehinddoc) { createSwDoc("behinddoc.docx"); + + // FIXME: validation error in OOXML export: Errors: 1 + skipValidation(); + save(TestFilter::DOCX); xmlDocUniquePtr pXmlDoc = parseExport(u"word/document.xml"_ustr); // This was "0", shape was in the foreground. @@ -783,6 +798,10 @@ CPPUNIT_TEST_FIXTURE(Test, testFdo73247) CPPUNIT_TEST_FIXTURE(Test, testFdo70942) { createSwDoc("fdo70942.docx"); + + // FIXME: validation error in OOXML export: Errors: 2 + skipValidation(); + save(TestFilter::DOCX); xmlDocUniquePtr pXmlDoc = parseExport(u"word/document.xml"_ustr); assertXPath(pXmlDoc, "/w:document/w:body/w:p[1]/w:r[1]/mc:AlternateContent/mc:Choice/w:drawing/wp:anchor/a:graphic/a:graphicData/wps:wsp/wps:spPr/a:prstGeom", @@ -918,6 +937,10 @@ CPPUNIT_TEST_FIXTURE(Test, testShapeThemePreservation) CPPUNIT_TEST_FIXTURE(Test, testFDO73546) { createSwDoc("FDO73546.docx"); + + // FIXME: validation error in OOXML export: Errors: 8 + skipValidation(); + save(TestFilter::DOCX); xmlDocUniquePtr pXmlDoc = parseExport(u"word/header2.xml"_ustr); assertXPath(pXmlDoc, "/w:hdr/w:p[1]/w:r[3]/mc:AlternateContent/mc:Choice/w:drawing/wp:anchor", "distL",u"0"); @@ -1032,6 +1055,10 @@ CPPUNIT_TEST_FIXTURE(Test, testFdo74110) CPPUNIT_TEST_FIXTURE(Test, testOuterShdw) { createSwDoc("testOuterShdw.docx"); + + // FIXME: validation error in OOXML export: Errors: 3 + skipValidation(); + saveAndReload(TestFilter::DOCX); xmlDocUniquePtr pXmlDoc = parseExport(u"word/document.xml"_ustr); assertXPath(pXmlDoc, "//mc:AlternateContent[1]/mc:Choice[1]/w:drawing[1]/wp:anchor[1]/a:graphic[1]/a:graphicData[1]/wps:wsp[1]/wps:spPr[1]/a:effectLst[1]/a:outerShdw[1]", "dist", u"1041400"); @@ -1040,6 +1067,10 @@ CPPUNIT_TEST_FIXTURE(Test, testOuterShdw) CPPUNIT_TEST_FIXTURE(Test, testExtentValue) { createSwDoc("fdo74605.docx"); + + // FIXME: validation error in OOXML export: Errors: 45 + skipValidation(); + save(TestFilter::DOCX); xmlDocUniquePtr pXmlDoc = parseExport(u"word/document.xml"_ustr); sal_Int32 nX = getXPath(pXmlDoc, "/w:document/w:body/w:p[2]/w:r[1]/mc:AlternateContent[1]/mc:Choice[1]/w:drawing[1]/wp:anchor[1]/wp:extent", "cx").toInt32();
