[Libreoffice-commits] core.git: vcl/qa

Noel Grandin (via logerrit) logerrit at kemper.freedesktop.org
Sun Apr 21 15:51:36 UTC 2019


 vcl/qa/cppunit/pdfexport/pdfexport.cxx |    3 +++
 1 file changed, 3 insertions(+)

New commits:
commit 89b1a00e08d8938714dd493fa27f08ba8d9b618d
Author:     Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Sun Apr 21 16:53:37 2019 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Sun Apr 21 17:50:37 2019 +0200

    disable part of pdfexport test on windows
    
    has high failure rate on jenkins
    
    Change-Id: If429483face1c118ec3bdfed86cf9b84a23e7e0b
    Reviewed-on: https://gerrit.libreoffice.org/71040
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/vcl/qa/cppunit/pdfexport/pdfexport.cxx b/vcl/qa/cppunit/pdfexport/pdfexport.cxx
index ae28379472e7..bcc3732e9192 100644
--- a/vcl/qa/cppunit/pdfexport/pdfexport.cxx
+++ b/vcl/qa/cppunit/pdfexport/pdfexport.cxx
@@ -994,6 +994,8 @@ void PdfExportTest::testTdf118244_radioButtonGroup()
 // fail.
 void PdfExportTest::testTdf115117_1()
 {
+// keeps failing on the windows tinderboxen
+#if !defined _WIN32
     vcl::filter::PDFDocument aDocument;
     load("tdf115117-1.odt", aDocument);
 
@@ -1049,6 +1051,7 @@ void PdfExportTest::testTdf115117_1()
     const char* pEnd = pStart + aObjectStream.GetSize();
     auto it = std::search(pStart, pEnd, aCmap.getStr(), aCmap.getStr() + aCmap.getLength());
     CPPUNIT_ASSERT(it != pEnd);
+#endif // if !defined _WIN32
 }
 
 // This requires DejaVu Sans font, if it is missing the test will most likely


More information about the Libreoffice-commits mailing list