sw/qa/extras/htmlexport/htmlexport.cxx |   34 +++++++++------------------------
 1 file changed, 10 insertions(+), 24 deletions(-)

New commits:
commit d7974d46c8e3d6dc55b3ad11bb6e71e25552cf99
Author:     Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Tue Feb 27 16:36:16 2024 +0600
Commit:     Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Tue Feb 27 15:43:28 2024 +0100

    Simplify a bit
    
    Change-Id: Ib13dab2b57a5ca0d477783887c637d4cabbebcf7
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164029
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>

diff --git a/sw/qa/extras/htmlexport/htmlexport.cxx 
b/sw/qa/extras/htmlexport/htmlexport.cxx
index 6a2a0f9c1785..fcb60925488c 100644
--- a/sw/qa/extras/htmlexport/htmlexport.cxx
+++ b/sw/qa/extras/htmlexport/htmlexport.cxx
@@ -302,7 +302,7 @@ CPPUNIT_TEST_FIXTURE(HtmlExportTest, testFdo62336)
     // The problem was essentially a crash during table export as docx/rtf/html
     // If calc-layout is enabled, the crash does not occur, that's why 
loadFromFile/save is used
     loadFromFile(u"fdo62336.docx");
-    save("HTML (StarWriter)");
+    save(mpFilter);
 }
 
 CPPUNIT_TEST_FIXTURE(HtmlExportTest, testFdo86857)
@@ -681,7 +681,7 @@ CPPUNIT_TEST_FIXTURE(HtmlExportTest, testReqIfOleData)
     createSwDoc("reqif-ole-data.xhtml");
     verify();
     setFilterOptions("xhtmlns=reqif-xhtml");
-    saveAndReload("HTML (StarWriter)");
+    saveAndReload(mpFilter);
     verify();
 }
 
@@ -737,7 +737,7 @@ CPPUNIT_TEST_FIXTURE(HtmlExportTest, testReqIfOleImg)
     createSwDoc("reqif-ole-img.xhtml");
     verify();
     setFilterOptions("xhtmlns=reqif-xhtml");
-    saveAndReload("HTML (StarWriter)");
+    saveAndReload(mpFilter);
     verify();
 }
 
@@ -1127,8 +1127,7 @@ CPPUNIT_TEST_FIXTURE(SwHtmlDomExportTest, 
testBlockQuoteReqIf)
     xParagraph->setPropertyValue("ParaStyleName", 
uno::Any(OUString("Quotations")));
 
     // Export it.
-    setFilterOptions("xhtmlns=reqif-xhtml");
-    save("HTML (StarWriter)");
+    ExportToReqif();
     xmlDocUniquePtr pDoc = WrapReqifFromTempFile();
 
     // Without the accompanying fix in place, this test would have failed with:
@@ -1588,9 +1587,7 @@ CPPUNIT_TEST_FIXTURE(SwHtmlDomExportTest, 
testPartiallyNumberedListHTML)
     // When exporting to HTML:
     ExportToHTML();
 
-    SvMemoryStream aStream;
-    WrapFromTempFile(aStream);
-    xmlDocUniquePtr pXmlDoc = parseXmlStream(&aStream);
+    xmlDocUniquePtr pXmlDoc = parseXml(maTempFile);
     CPPUNIT_ASSERT(pXmlDoc); // if we have missing closing marks - parse error
 
     // Without the accompanying fix in place, this test would have failed:
@@ -2089,11 +2086,7 @@ CPPUNIT_TEST_FIXTURE(SwHtmlDomExportTest, 
testOleEmfPreviewToHtml)
     createSwDoc("ole2.odt");
 
     // When exporting to HTML:
-    uno::Reference<frame::XStorable> xStorable(mxComponent, uno::UNO_QUERY);
-    uno::Sequence<beans::PropertyValue> aStoreProperties = {
-        comphelper::makePropertyValue("FilterName", OUString("HTML 
(StarWriter)")),
-    };
-    xStorable->storeToURL(maTempFile.GetURL(), aStoreProperties);
+    ExportToHTML();
 
     // Then make sure the <img> tag has matching file extension and data:
     htmlDocUniquePtr pDoc = parseHtml(maTempFile);
@@ -2275,7 +2268,7 @@ CPPUNIT_TEST_FIXTURE(HtmlExportTest, testClearingBreak)
     createSwWebDoc("clearing-break.html");
     // Then make sure that the clear property of the break is not ignored:
     verify();
-    saveAndReload("HTML (StarWriter)");
+    saveAndReload(mpFilter);
     // Make sure that the clear property of the break is not ignored during 
export:
     verify();
 }
@@ -2340,11 +2333,7 @@ CPPUNIT_TEST_FIXTURE(HtmlExportTest, testImageKeepRatio)
     xBodyText->insertTextContent(xCursor, xTextContent, false);
 
     // When exporting to HTML:
-    uno::Reference<frame::XStorable> xStorable(mxComponent, uno::UNO_QUERY);
-    uno::Sequence<beans::PropertyValue> aStoreProperties = {
-        comphelper::makePropertyValue("FilterName", OUString("HTML 
(StarWriter)")),
-    };
-    xStorable->storeToURL(maTempFile.GetURL(), aStoreProperties);
+    save(mpFilter);
 
     // Then make sure that the width is not a fixed size, that would break on 
resizing the browser
     // window:
@@ -2417,10 +2406,7 @@ CPPUNIT_TEST_FIXTURE(HtmlExportTest, testTdf114769)
     xRun->setPropertyValue("HyperLinkURL", 
uno::Any(OUString(".\another.odt")));
 
     // Export
-    uno::Reference<frame::XStorable> xStorable(mxComponent, uno::UNO_QUERY);
-    uno::Sequence<beans::PropertyValue> aStoreProperties
-        = { comphelper::makePropertyValue("FilterName", OUString("HTML 
(StarWriter)")) };
-    xStorable->storeToURL(maTempFile.GetURL(), aStoreProperties);
+    save(mpFilter);
 
     htmlDocUniquePtr pHtmlDoc = parseHtml(maTempFile);
     CPPUNIT_ASSERT(pHtmlDoc);
@@ -2440,7 +2426,7 @@ CPPUNIT_TEST_FIXTURE(HtmlExportTest, testTdf114769)
 CPPUNIT_TEST_FIXTURE(HtmlExportTest, testTdf153923)
 {
     createSwDoc("TableWithIndent.fodt");
-    save("HTML (StarWriter)");
+    save(mpFilter);
 
     // Parse it as XML (strict!)
     xmlDocUniquePtr pDoc = parseXml(maTempFile);

Reply via email to