sw/qa/extras/ooxmlexport/ooxmlexport18.cxx | 24 +++++++++++++++++++++++- 1 file changed, 23 insertions(+), 1 deletion(-)
New commits: commit ac29cbd07805adb702f3cc86e485dd5775b50043 Author: Xisco Fauli <[email protected]> AuthorDate: Fri Nov 21 15:54:44 2025 +0100 Commit: Xisco Fauli <[email protected]> CommitDate: Fri Nov 21 23:00:03 2025 +0100 sw_ooxmlexport18: do not skip DOCX export validation Change-Id: I44bc2c3998c3687a981b0a5465d2f9141ae48b4f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/194338 Tested-by: Jenkins Reviewed-by: Xisco Fauli <[email protected]> diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport18.cxx b/sw/qa/extras/ooxmlexport/ooxmlexport18.cxx index 6403780fd087..415000327e77 100644 --- a/sw/qa/extras/ooxmlexport/ooxmlexport18.cxx +++ b/sw/qa/extras/ooxmlexport/ooxmlexport18.cxx @@ -40,7 +40,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, testTdf150197_predefinedNumbering) @@ -123,6 +123,9 @@ DECLARE_OOXMLEXPORT_TEST(testTdf147646, "tdf147646_mergedCellNumbering.docx") DECLARE_OOXMLEXPORT_TEST(testTdf153526_commentInNumbering, "tdf153526_commentInNumbering.docx") { + //FIXME: validation error in OOXML export: Errors: 1 + skipValidation(); + // an exception was prematurely ending finishParagraph, losing numbering and CRs // so before the patch, this was 6. CPPUNIT_ASSERT_EQUAL(13, getParagraphs()); @@ -166,6 +169,10 @@ DECLARE_OOXMLEXPORT_TEST(testTdf154751_dualStrikethrough, "tdf154751_dualStriket CPPUNIT_TEST_FIXTURE(Test, testTdf154478) { createSwDoc("tdf154478.docx"); + + //FIXME: validation error in OOXML export: Errors: 4 + skipValidation(); + save(TestFilter::DOCX); xmlDocUniquePtr pXmlDoc = parseExport(u"word/comments.xml"_ustr); @@ -192,6 +199,9 @@ CPPUNIT_TEST_FIXTURE(Test, testTdf153592_columnBreaks) DECLARE_OOXMLEXPORT_TEST(testTdf104394_lostTextbox, "tdf104394_lostTextbox.docx") { + //FIXME: validation error in OOXML export: Errors: 4 + skipValidation(); + // This was only one page b/c the textbox was missing. CPPUNIT_ASSERT_EQUAL(2, getPages()); } @@ -301,6 +311,10 @@ CPPUNIT_TEST_FIXTURE(Test, testTdf154703_framePr2) createSwDoc("tdf154703_framePr2.rtf"); verify(); + + //FIXME: validation error in OOXML export: Errors: 1 + skipValidation(); + saveAndReload(TestFilter::DOCX); verify(/*bIsExport*/ true); @@ -762,6 +776,10 @@ CPPUNIT_TEST_FIXTURE(Test, testTdf152200) CPPUNIT_TEST_FIXTURE(Test, testTdf126477) { createSwDoc("embedded_chart.odt"); + + //FIXME: validation error in OOXML export: Errors: 2 + skipValidation(); + saveAndReload(TestFilter::DOCX); uno::Reference<text::XTextEmbeddedObjectsSupplier> xTEOSupplier(mxComponent, uno::UNO_QUERY); @@ -844,6 +862,10 @@ CPPUNIT_TEST_FIXTURE(Test, testExportingUnknownStyleInRedline) { // This must not fail assertions createSwDoc("UnknownStyleInRedline.docx"); + + //FIXME: validation error in OOXML export: Errors: 1 + skipValidation(); + saveAndReload(TestFilter::DOCX); // Check that the original unknown style name "UnknownStyle" is roundtripped // (maybe this is wrong, because Word does not do this).
