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

[Xisco Fauli (via logerrit)]

 vcl/qa/cppunit/pdfexport/data/tdf135346.ods |binary
 vcl/qa/cppunit/pdfexport/pdfexport.cxx      |   17 +++++++++++++++++
 2 files changed, 17 insertions(+)

New commits:
commit b28de9d32016a904e4ba457a9a6c62098416c729
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Fri Jun 10 11:02:06 2022 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Fri Jun 10 12:27:43 2022 +0200

    tdf#135346: vcl_pdfexport: Add unittest
    
    Change-Id: I11ad776419ea3c0eb5babd0f1ee2c7c70b7e7485
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135580
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/vcl/qa/cppunit/pdfexport/data/tdf135346.ods 
b/vcl/qa/cppunit/pdfexport/data/tdf135346.ods
new file mode 100644
index 000000000000..5f696e550471
Binary files /dev/null and b/vcl/qa/cppunit/pdfexport/data/tdf135346.ods differ
diff --git a/vcl/qa/cppunit/pdfexport/pdfexport.cxx 
b/vcl/qa/cppunit/pdfexport/pdfexport.cxx
index 711aaa17742f..2531dab1a3f4 100644
--- a/vcl/qa/cppunit/pdfexport/pdfexport.cxx
+++ b/vcl/qa/cppunit/pdfexport/pdfexport.cxx
@@ -2165,6 +2165,23 @@ CPPUNIT_TEST_FIXTURE(PdfExportTest, testTdf147027)
     CPPUNIT_ASSERT_EQUAL(778, pPdfPage->getObjectCount());
 }
 
+CPPUNIT_TEST_FIXTURE(PdfExportTest, testTdf135346)
+{
+    // Load the Calc document.
+    aMediaDescriptor["FilterName"] <<= OUString("calc_pdf_Export");
+    saveAsPDF(u"tdf135346.ods");
+    std::unique_ptr<vcl::pdf::PDFiumDocument> pPdfDocument = parseExport();
+    CPPUNIT_ASSERT(pPdfDocument);
+    CPPUNIT_ASSERT_EQUAL(1, pPdfDocument->getPageCount());
+    std::unique_ptr<vcl::pdf::PDFiumPage> pPdfPage = 
pPdfDocument->openPage(/*nIndex=*/0);
+    CPPUNIT_ASSERT(pPdfPage);
+
+    // Without the fix in place, this test would have failed with
+    // - Expected: 56
+    // - Actual  : 0
+    CPPUNIT_ASSERT_EQUAL(56, pPdfPage->getObjectCount());
+}
+
 CPPUNIT_TEST_FIXTURE(PdfExportTest, testTdf147164)
 {
     aMediaDescriptor["FilterName"] <<= OUString("impress_pdf_Export");