sw/qa/extras/ooxmlexport/ooxmlexport3.cxx |    6 ++++++
 1 file changed, 6 insertions(+)

New commits:
commit c50315c55dbb5daf86ccb2a91468c05a53e926e9
Author:     Justin Luth <[email protected]>
AuthorDate: Mon May 18 12:21:04 2020 +0300
Commit:     Justin Luth <[email protected]>
CommitDate: Mon May 18 19:24:10 2020 +0200

    tdf#95495 ooxmlexport3: less implementation-specific unit test
    
    Although the original document completely defines the numbering
    in (inherited) paragraph styles, the unit test was checking
    whether the paragaph itself had the exported numbering.
    That tests the current implementation where style information
    is copied to the paragrah, but in a correct world that information
    would stay in the styles.
    
    This unit test uses the EXTREME example where a "chapter numbering"
    is inherited by a sub-style - something LO completely does not
    support, so this needs serious emulation. Thus I am keeping
    the implementation-specific check, but clearly documenting
    that fact.
    
    Change-Id: I5b95646498c4d936eceb9be1841c654a5bd81275
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94401
    Tested-by: Jenkins
    Reviewed-by: Justin Luth <[email protected]>

diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport3.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport3.cxx
index 48b2cef55586..2876bb47929a 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport3.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport3.cxx
@@ -1085,7 +1085,13 @@ DECLARE_OOXMLEXPORT_TEST(testNumberingLevels, 
"tdf95495.docx")
         return;
 
     // tdf#95495: set list level of the custom style based on the setting of 
the parent style
+    // [this assertXPath is not a very good test, since the numbering 
definition is not set on the paragraph itself,
+    //  but in a style. This just tests the current copy-to-paragraph 
implementation. But leaving it for now,
+    //  since this example is very much a corner case, so anyone trespassing 
here should double-check everything...]
     assertXPath(pXmlDocument, "/w:document/w:body/w:p[2]/w:pPr/w:numPr/w:ilvl 
[@w:val = '1']", 1);
+
+    xmlDocUniquePtr pXmlDoc = parseLayoutDump();
+    assertXPath(pXmlDoc, "//body/txt[5]/LineBreak", "Line", "A.2.1 
.DESCRIPTION");
 }
 
 DECLARE_OOXMLEXPORT_TEST(testVerticalBorders, "calendar3.docx")
_______________________________________________
Libreoffice-commits mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to