sc/qa/unit/SparklineImportExportTest.cxx |    3 
 sc/qa/unit/bugfix-test.cxx               |    6 -
 sc/qa/unit/chart2dataprovider.cxx        |    1 
 sc/qa/unit/filters-test.cxx              |   13 ---
 sc/qa/unit/helper/qahelper.cxx           |   14 +++-
 sc/qa/unit/helper/qahelper.hxx           |    2 
 sc/qa/unit/jumbosheets-test.cxx          |    7 --
 sc/qa/unit/opencl-test.cxx               |    2 
 sc/qa/unit/pivottable_filters_test.cxx   |   48 --------------
 sc/qa/unit/subsequent_export_test.cxx    |   78 -----------------------
 sc/qa/unit/subsequent_export_test2.cxx   |  103 -------------------------------
 sc/qa/unit/subsequent_filters_test.cxx   |   96 +++-------------------------
 sc/qa/unit/subsequent_filters_test2.cxx  |  100 +-----------------------------
 13 files changed, 27 insertions(+), 446 deletions(-)

New commits:
commit 77a4b5b9b9a95ed4f9b4fccdb76d403086212fe2
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Wed Apr 6 16:39:34 2022 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Wed Apr 6 17:36:12 2022 +0200

    sc: qahelper: Check GetErrorCode returns none in loadDoc
    
    I found it while creating a unittest for tdf#121887.
    IMHO, loadDoc should fail if there is a warning
    While at it, remove redundant assertions
    
    Change-Id: Ib2b9572ee1f0f4da9ac23913e231eac1ad57cb49
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/132638
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/sc/qa/unit/SparklineImportExportTest.cxx 
b/sc/qa/unit/SparklineImportExportTest.cxx
index f6969831ee17..3ae19881f8aa 100644
--- a/sc/qa/unit/SparklineImportExportTest.cxx
+++ b/sc/qa/unit/SparklineImportExportTest.cxx
@@ -166,7 +166,6 @@ void checkSparklines(ScDocument& rDocument)
 void SparklineImportExportTest::testSparklinesRoundtripXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"Sparklines.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh);
 
     checkSparklines(xDocSh->GetDocument());
 
@@ -181,7 +180,6 @@ void SparklineImportExportTest::testSparklinesExportODS()
 {
     // Load the document containing sparklines
     ScDocShellRef xDocSh = loadDoc(u"Sparklines.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh);
 
     // Save as ODS and check content.xml with XPath
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_ODS);
@@ -229,7 +227,6 @@ void SparklineImportExportTest::testSparklinesExportODS()
 void SparklineImportExportTest::testSparklinesRoundtripODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"Sparklines.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh);
 
     checkSparklines(xDocSh->GetDocument());
 
diff --git a/sc/qa/unit/bugfix-test.cxx b/sc/qa/unit/bugfix-test.cxx
index 3dac5e6df725..3b3cc27439df 100644
--- a/sc/qa/unit/bugfix-test.cxx
+++ b/sc/qa/unit/bugfix-test.cxx
@@ -146,7 +146,6 @@ void ScFiltersTest::testTdf137576_Measureline()
 
     // Get document
     ScDocShellRef xDocSh = loadDoc(u"tdf137576_Measureline.", FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load tdf137576_Measureline.ods", 
xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // Get shape
@@ -191,7 +190,6 @@ void ScFiltersTest::testTdf137216_HideCol()
 
     // Get document
     ScDocShellRef xDocSh = loadDoc(u"tdf137216_HideCol.", FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load tdf137216_HideCol.ods", 
xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // Get shape
@@ -216,7 +214,6 @@ void ScFiltersTest::testTdf137044_CoverHiddenRows()
 
     // Get document
     ScDocShellRef xDocSh = loadDoc(u"tdf137044_CoverHiddenRows.", FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load tdf137044_CoverHiddenRows.ods", 
xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // Get shape
@@ -263,7 +260,6 @@ void ScFiltersTest::testTdf137020_FlipVertical()
 {
     // Get document
     ScDocShellRef xDocSh = loadDoc(u"tdf137020_FlipVertical.", FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load tdf137020_FlipVertical.ods", 
xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // Get shape
@@ -580,7 +576,6 @@ void ScFiltersTest::testTdf129789()
 {
 
     ScDocShellRef xDocSh = loadDoc(u"tdf129789.", FORMAT_ODS, true);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     {
@@ -719,7 +714,6 @@ void ScFiltersTest::testTdf130725()
 void ScFiltersTest::testTdf104502_hiddenColsCountedInPageCount()
 {
     ScDocShellRef xShell = loadDoc(u"tdf104502_hiddenColsCountedInPageCount.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
 
diff --git a/sc/qa/unit/chart2dataprovider.cxx 
b/sc/qa/unit/chart2dataprovider.cxx
index 31af9ba6eef9..b8afa19564ec 100644
--- a/sc/qa/unit/chart2dataprovider.cxx
+++ b/sc/qa/unit/chart2dataprovider.cxx
@@ -78,7 +78,6 @@ static void lcl_createAndCheckDataProvider(ScDocument& rDoc, 
const OUString& cel
 void ScChart2DataProviderTest::testHeaderExpansion()
 {
     ScDocShellRef xDocSh = loadDoc(u"chart2dataprovider.", FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load ch.ods.", xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
 
diff --git a/sc/qa/unit/filters-test.cxx b/sc/qa/unit/filters-test.cxx
index 81d84520fff5..7025fdc52018 100644
--- a/sc/qa/unit/filters-test.cxx
+++ b/sc/qa/unit/filters-test.cxx
@@ -190,8 +190,6 @@ void ScFiltersTest::testRangeNameODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"named-ranges-global.", FORMAT_ODS);
 
-    CPPUNIT_ASSERT_MESSAGE("Failed to load named-ranges-global.*", 
xDocSh.is());
-
     xDocSh->DoHardRecalc();
 
     ScDocument& rDoc = xDocSh->GetDocument();
@@ -315,8 +313,6 @@ void ScFiltersTest::testContentDIF()
 {
     ScDocShellRef xDocSh = loadDoc(u"universal-content.", FORMAT_DIF);
 
-    CPPUNIT_ASSERT_MESSAGE("Failed to load universal-content.dif", 
xDocSh.is());
-
     xDocSh->DoClose();
 }
 
@@ -333,7 +329,6 @@ void ScFiltersTest::testContentXLSB()
 void ScFiltersTest::testContentXLS_XML()
 {
     ScDocShellRef xDocSh = loadDoc(u"universal-content.", FORMAT_XLS_XML);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
     testContentImpl(rDoc, FORMAT_XLS_XML);
@@ -343,7 +338,6 @@ void ScFiltersTest::testContentXLS_XML()
 void ScFiltersTest::testContentGnumeric()
 {
     ScDocShellRef xDocSh = loadDoc(u"universal-content.", FORMAT_GNUMERIC);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
     testContentImpl(rDoc, FORMAT_GNUMERIC);
@@ -353,7 +347,6 @@ void ScFiltersTest::testContentGnumeric()
 void ScFiltersTest::testSharedFormulaXLS()
 {
     ScDocShellRef xDocSh = loadDoc(u"shared-formula/basic.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     xDocSh->DoHardRecalc();
     // Check the results of formula cells in the shared formula range.
@@ -378,7 +371,6 @@ void ScFiltersTest::testSharedFormulaXLS()
     // to handle these wrong ranges that Excel stores.
 
     xDocSh = loadDoc(u"shared-formula/gap.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument& rDoc2 = xDocSh->GetDocument();
     rDoc2.CalcAll();
 
@@ -659,7 +651,6 @@ static void testEnhancedProtectionImpl( const ScDocument& 
rDoc )
 void ScFiltersTest::testEnhancedProtectionXLS()
 {
     ScDocShellRef xDocSh = loadDoc(u"enhanced-protection.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     testEnhancedProtectionImpl( rDoc);
@@ -670,7 +661,6 @@ void ScFiltersTest::testEnhancedProtectionXLS()
 void ScFiltersTest::testEnhancedProtectionXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"enhanced-protection.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     testEnhancedProtectionImpl( rDoc);
@@ -681,7 +671,6 @@ void ScFiltersTest::testEnhancedProtectionXLSX()
 void ScFiltersTest::testSortWithSharedFormulasODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"shared-formula/sort-crash.", FORMAT_ODS, 
true);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // E2:E10 should be shared.
@@ -733,7 +722,6 @@ void ScFiltersTest::testSortWithSharedFormulasODS()
 void ScFiltersTest::testSortWithSheetExternalReferencesODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"sort-with-sheet-external-references.", 
FORMAT_ODS, true);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     sc::AutoCalcSwitch aACSwitch(rDoc, true); // turn auto calc on.
     rDoc.CalcAll();
@@ -854,7 +842,6 @@ void 
ScFiltersTest::testSortWithSheetExternalReferencesODS_Impl( ScDocShellRef c
 void ScFiltersTest::testSortWithFormattingXLS()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf129127.", FORMAT_XLS, true);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // Set as an anonymous database range to sort.
diff --git a/sc/qa/unit/helper/qahelper.cxx b/sc/qa/unit/helper/qahelper.cxx
index a11d240b9b35..5c386b2db684 100644
--- a/sc/qa/unit/helper/qahelper.cxx
+++ b/sc/qa/unit/helper/qahelper.cxx
@@ -675,7 +675,6 @@ ScDocShellRef 
ScBootstrapFixture::loadDocAndSetupModelViewController(std::u16str
 
     // 1. Open the document
     ScDocShellRef xDocSh = loadDoc(rFileName, nFormat, true);
-    CPPUNIT_ASSERT_MESSAGE(OString("Failed to load " + 
OUStringToOString(rFileName, RTL_TEXTENCODING_UTF8)).getStr(), xDocSh.is());
 
     uno::Reference< frame::XModel2 > xModel2 = xDocSh->GetModel();
     CPPUNIT_ASSERT(xModel2.is());
@@ -694,13 +693,21 @@ ScDocShellRef 
ScBootstrapFixture::loadDocAndSetupModelViewController(std::u16str
 }
 
 ScDocShellRef ScBootstrapFixture::loadDoc(
-    std::u16string_view rFileName, sal_Int32 nFormat, bool bReadWrite )
+    std::u16string_view rFileName, sal_Int32 nFormat, bool bReadWrite, bool 
bCheckErrorCode )
 {
     OUString aFileExtension = OUString::fromUtf8(aFileFormats[nFormat].pName);
     OUString aFileName;
     createFileURL( rFileName, aFileExtension, aFileName );
 
-    return load(aFileName, nFormat, bReadWrite);
+    ScDocShellRef xDocShRef = load(aFileName, nFormat, bReadWrite);
+    CPPUNIT_ASSERT_MESSAGE(OString("Failed to load " + 
OUStringToOString(rFileName, RTL_TEXTENCODING_UTF8)).getStr(), xDocShRef.is());
+
+    if (bCheckErrorCode)
+    {
+        CPPUNIT_ASSERT(!xDocShRef->GetErrorCode());
+    }
+
+    return xDocShRef;
 }
 
 ScBootstrapFixture::ScBootstrapFixture( const OUString& rsBaseString ) : 
m_aBaseString( rsBaseString ) {}
@@ -871,7 +878,6 @@ void ScBootstrapFixture::miscRowHeightsTest( TestParam 
const * aTestValues, unsi
         int nImportType =  aTestValues[ index ].nImportType;
         int nExportType =  aTestValues[ index ].nExportType;
         ScDocShellRef xShell = loadDoc( sFileName, nImportType );
-        CPPUNIT_ASSERT(xShell.is());
 
         if ( nExportType != -1 )
             xShell = saveAndReload(*xShell, nExportType );
diff --git a/sc/qa/unit/helper/qahelper.hxx b/sc/qa/unit/helper/qahelper.hxx
index e1b1ee6c65f9..21b3c75f2869 100644
--- a/sc/qa/unit/helper/qahelper.hxx
+++ b/sc/qa/unit/helper/qahelper.hxx
@@ -205,7 +205,7 @@ protected:
     ScDocShellRef loadEmptyDocument(const 
com::sun::star::uno::Sequence<com::sun::star::beans::PropertyValue>& 
rPropertyValues = {});
     ScDocShellRef loadDocAndSetupModelViewController(std::u16string_view 
rFileName, sal_Int32 nFormat);
     ScDocShellRef loadDoc(
-        std::u16string_view rFileName, sal_Int32 nFormat, bool bReadWrite = 
false );
+        std::u16string_view rFileName, sal_Int32 nFormat, bool bReadWrite = 
false, bool bCheckErrorCode = true );
 
 public:
     static const FileFormat* getFileFormats() { return aFileFormats; }
diff --git a/sc/qa/unit/jumbosheets-test.cxx b/sc/qa/unit/jumbosheets-test.cxx
index deb516ae5b91..f21c27aafbda 100644
--- a/sc/qa/unit/jumbosheets-test.cxx
+++ b/sc/qa/unit/jumbosheets-test.cxx
@@ -88,7 +88,6 @@ void ScJumboSheetsTest::testRoundtripColumn2000Xlsx()
 void ScJumboSheetsTest::testRoundtripColumn2000(std::u16string_view name, int 
format)
 {
     ScDocShellRef xDocSh1 = loadDoc(name, format);
-    CPPUNIT_ASSERT(xDocSh1.is());
 
     {
         ScDocument& rDoc = xDocSh1->GetDocument();
@@ -120,7 +119,6 @@ void 
ScJumboSheetsTest::testRoundtripColumn2000(std::u16string_view name, int fo
 void ScJumboSheetsTest::testRoundtripColumnRange()
 {
     ScDocShellRef xDocSh1 = loadDoc(u"sum-whole-column-row.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh1.is());
 
     {
         ScDocument& rDoc = xDocSh1->GetDocument();
@@ -172,7 +170,6 @@ void ScJumboSheetsTest::testRoundtripColumnRange()
 void ScJumboSheetsTest::testRoundtripNamedRanges()
 {
     ScDocShellRef xDocSh1 = loadDoc(u"ranges-column-2000.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh1.is());
 
     std::pair<OUString, OUString> ranges[] = { { "CELLBXX1", "$Sheet1.$BXX$1" 
},
                                                { "CELLSA4_AMJ4", 
"$Sheet1.$A$4:$AMJ$4" },
@@ -229,7 +226,6 @@ void ScJumboSheetsTest::testNamedRangeNameConflict()
     // The document contains named ranges named 'num1' and 'num2', that should 
be still treated
     // as named references even though with 16k columns those are normally 
NUM1 and NUM2 cells.
     ScDocShellRef xDocSh = loadDoc(u"named-range-conflict.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     rDoc.CalcAll();
     CPPUNIT_ASSERT_EQUAL(0.0, rDoc.GetValue(10022, 0, 0)); // NUM1
@@ -251,7 +247,6 @@ void ScJumboSheetsTest::testNamedRangeNameConflict()
 void ScJumboSheetsTest::testTdf134553()
 {
     ScDocShellRef xDocSh = loadDocAndSetupModelViewController(u"tdf134553.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh);
 
     ScDocument& rDoc = xDocSh->GetDocument();
 
@@ -299,7 +294,6 @@ void ScJumboSheetsTest::testTdf134392()
 {
     // Without the fix in place, the file would have crashed
     ScDocShellRef xDocSh = loadDoc(u"tdf134392.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
     rDoc.CalcAll(); // perform hard re-calculation.
@@ -368,7 +362,6 @@ void ScJumboSheetsTest::testTdf109061()
 {
     // Without the fix in place, the file would have crashed
     ScDocShellRef xDocSh = loadDoc(u"tdf109061.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
     rDoc.CalcAll(); // perform hard re-calculation.
diff --git a/sc/qa/unit/opencl-test.cxx b/sc/qa/unit/opencl-test.cxx
index 3b258768952c..3b1132629ce0 100644
--- a/sc/qa/unit/opencl-test.cxx
+++ b/sc/qa/unit/opencl-test.cxx
@@ -517,11 +517,9 @@ bool ScOpenCLTest::initTestEnv(std::u16string_view 
fileName, sal_Int32 nFormat,
         return false;
 
     xDocSh = loadDoc(fileName, nFormat, bReadWrite);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load document.", xDocSh.is());
     enableOpenCL();
 
     xDocShRes = loadDoc(fileName, nFormat, bReadWrite);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load document.", xDocShRes.is());
 
     return true;
 }
diff --git a/sc/qa/unit/pivottable_filters_test.cxx 
b/sc/qa/unit/pivottable_filters_test.cxx
index 14f790e89b68..f0e5d5a8b830 100644
--- a/sc/qa/unit/pivottable_filters_test.cxx
+++ b/sc/qa/unit/pivottable_filters_test.cxx
@@ -207,7 +207,6 @@ bool hasDimension(const std::vector<const 
ScDPSaveDimension*>& rDims, const OUSt
 void ScPivotTableFiltersTest::testPivotTableBasicODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot-table-basic.", FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load pivot-table-basic.ods", 
xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
     CPPUNIT_ASSERT_EQUAL_MESSAGE("There should be exactly two sheets.", 
sal_Int16(2),
@@ -266,7 +265,6 @@ void ScPivotTableFiltersTest::testPivotTableBasicODS()
 void ScPivotTableFiltersTest::testPivotTableNamedRangeSourceODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot-table-named-range-source.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load 
pivot-table-named-range-source.ods", xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
 
@@ -320,7 +318,6 @@ bool checkVisiblePageFieldMember(const 
ScDPSaveDimension::MemberList& rMembers,
 void ScPivotTableFiltersTest::testPivotTableSharedCacheGroupODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot-table-shared-cache-with-group.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // Make sure that page field's visibility settings are loaded correctly.
@@ -455,7 +452,6 @@ void 
ScPivotTableFiltersTest::testPivotTableSharedCacheGroupODS()
 void ScPivotTableFiltersTest::testGetPivotDataXLS()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot-getpivotdata.", FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     rDoc.CalcAll();
 
@@ -472,7 +468,6 @@ void ScPivotTableFiltersTest::testGetPivotDataXLS()
 void ScPivotTableFiltersTest::testPivotTableSharedGroupXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot-table/shared-group-field.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // Check whether right group names are imported for both tables
@@ -609,7 +604,6 @@ void 
ScPivotTableFiltersTest::testPivotTableSharedNestedDateGroupXLSX()
 void ScPivotTableFiltersTest::testPivotTableSharedNumGroupXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot-table/shared-numgroup.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // Check whether right number groups are imported for both tables
@@ -647,7 +641,6 @@ void 
ScPivotTableFiltersTest::testPivotTableNoColumnsLayout()
 {
     // tdf#113268 - Pivot table: Missing popup button after opening a pivot 
table from ODS
     ScDocShellRef xDocSh = loadDoc(u"pivottable_no_columns_layout.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // There should be exactly 2 pivot tables
@@ -676,7 +669,6 @@ void 
ScPivotTableFiltersTest::testPivotTableNoColumnsLayout()
 void ScPivotTableFiltersTest::testTdf112501()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf112501.", FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // There should be exactly 2 pivot tables
@@ -746,7 +738,6 @@ void ScPivotTableFiltersTest::testPivotTableExportXLSX()
     // tdf#89139: pivot table definition needs to list items, including hidden
 
     ScDocShellRef xShell = loadDoc(u"tdf89139_pivot_table.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pTable
@@ -765,7 +756,6 @@ void ScPivotTableFiltersTest::testPivotTableExportXLSX()
 void ScPivotTableFiltersTest::testPivotTableExportXLSXSingleDataField()
 {
     ScDocShellRef xShell = loadDoc(u"tdf123421_1datafield.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pTable
@@ -788,7 +778,6 @@ void 
ScPivotTableFiltersTest::testPivotTableExportXLSXSingleDataField()
 void ScPivotTableFiltersTest::testPivotTableExportXLSXMultipleDataFields()
 {
     ScDocShellRef xShell = loadDoc(u"tdf123421_2datafields.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pTable
@@ -822,7 +811,6 @@ void ScPivotTableFiltersTest::testPivotCacheExportXLSX()
     // 
https://technet.microsoft.com/en-us/library/documentformat.openxml.spreadsheet.shareditems.aspx
 
     ScDocShellRef xShell = 
loadDoc(u"pivot-table/with-strings-integers-and-dates.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pCacheDef = XPathHelper::parseExport(pXPathFile, 
m_xSFactory,
@@ -1184,7 +1172,6 @@ void ScPivotTableFiltersTest::testPivotTableXLSX()
     } aTest;
 
     ScDocShellRef xDocSh = loadDoc(u"pivot-table/many-fields-in-cache.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument* pDoc = &xDocSh->GetDocument();
 
     // Initial check.
@@ -1308,7 +1295,6 @@ void 
ScPivotTableFiltersTest::testPivotTableTwoDataFieldsXLSX()
     } aTest;
 
     ScDocShellRef xDocSh = loadDoc(u"pivot-table/two-data-fields.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument* pDoc = &xDocSh->GetDocument();
 
     // Initial check.
@@ -1330,7 +1316,6 @@ void 
ScPivotTableFiltersTest::testPivotTableTwoDataFieldsXLSX()
 void ScPivotTableFiltersTest::testPivotTableMedianODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot-table-median.", FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load test document.", xDocSh.is());
 
     // Export the document and import again for a check
     ScDocShellRef xDocSh2 = saveAndReload(*xDocSh, FORMAT_ODS);
@@ -1368,7 +1353,6 @@ void ScPivotTableFiltersTest::testPivotTableMedianODS()
 void ScPivotTableFiltersTest::testPivotTableRowHeaderXLS()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot_row_header.", FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
 
     // There should be exactly 2 pivot tables
@@ -1438,7 +1422,6 @@ void ScPivotTableFiltersTest::testPivotTableRowHeaderXLS()
 void ScPivotTableFiltersTest::testPivotTableDoubleFieldFilter()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_double_field_filter.", 
FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(3), pDPs->GetCount());
@@ -1540,7 +1523,6 @@ void 
ScPivotTableFiltersTest::testPivotTableDoubleFieldFilter()
 void ScPivotTableFiltersTest::testPivotTableStringFieldFilter()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_string_field_filter.", 
FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -1576,7 +1558,6 @@ void 
ScPivotTableFiltersTest::testPivotTableStringFieldFilter()
 void ScPivotTableFiltersTest::testPivotTableDateFieldFilter()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_date_field_filter.", 
FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -1667,7 +1648,6 @@ void 
ScPivotTableFiltersTest::testPivotTableDateFieldFilter()
 void ScPivotTableFiltersTest::testPivotTableBoolFieldFilter()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_bool_field_filter.", 
FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -1703,7 +1683,6 @@ void 
ScPivotTableFiltersTest::testPivotTableBoolFieldFilter()
 void ScPivotTableFiltersTest::testPivotTableRowColPageFieldFilter()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_rowcolpage_field_filter.", 
FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -1813,7 +1792,6 @@ void 
ScPivotTableFiltersTest::testPivotTableRowColPageFieldFilter()
 void ScPivotTableFiltersTest::testPivotTableEmptyItem()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_empty_item.", FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -1853,7 +1831,6 @@ void ScPivotTableFiltersTest::testPivotTableEmptyItem()
 void ScPivotTableFiltersTest::testPivotTablePageFieldFilter()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_page_field_filter.", 
FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -1960,7 +1937,6 @@ void 
ScPivotTableFiltersTest::testPivotTableFirstHeaderRowXLSX()
     // tdf#112733: We have different tables here, but have the same value as 
firstHeaderRow
     // The documentation is not clear about what firstHeaderRow actually 
means, but MS Excel works on this way
     ScDocShellRef xShell = loadDoc(u"pivot_table_first_header_row.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pTable
@@ -1982,7 +1958,6 @@ void 
ScPivotTableFiltersTest::testPivotTableFirstHeaderRowXLSX()
 void ScPivotTableFiltersTest::testPivotTableDoubleFieldFilterXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_double_field_filter.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(3), pDPs->GetCount());
@@ -2084,7 +2059,6 @@ void 
ScPivotTableFiltersTest::testPivotTableDoubleFieldFilterXLSX()
 void ScPivotTableFiltersTest::testPivotTableStringFieldFilterXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_string_field_filter.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -2120,7 +2094,6 @@ void 
ScPivotTableFiltersTest::testPivotTableStringFieldFilterXLSX()
 void ScPivotTableFiltersTest::testPivotTableDateFieldFilterXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_date_field_filter.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -2196,7 +2169,6 @@ void 
ScPivotTableFiltersTest::testPivotTableDateFieldFilterXLSX()
 void ScPivotTableFiltersTest::testPivotTableBoolFieldFilterXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_bool_field_filter.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -2233,7 +2205,6 @@ void 
ScPivotTableFiltersTest::testPivotTableBoolFieldFilterXLSX()
 void ScPivotTableFiltersTest::testPivotTableRowColPageFieldFilterXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_rowcolpage_field_filter.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -2341,7 +2312,6 @@ void 
ScPivotTableFiltersTest::testPivotTableRowColPageFieldFilterXLSX()
 void ScPivotTableFiltersTest::testPivotTableErrorItemFilterXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_error_item_filter.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -2372,7 +2342,6 @@ void 
ScPivotTableFiltersTest::testPivotTableErrorItemFilterXLSX()
 void ScPivotTableFiltersTest::testPivotTableErrorItemFilterXLSB()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_error_item_filter.", 
FORMAT_XLSB);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -2396,7 +2365,6 @@ void 
ScPivotTableFiltersTest::testPivotTableErrorItemFilterXLSB()
 void ScPivotTableFiltersTest::testPivotTableErrorItem2FilterXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf122471.", FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -2424,7 +2392,6 @@ void 
ScPivotTableFiltersTest::testPivotTableErrorItem2FilterXLSX()
 void ScPivotTableFiltersTest::testPivotTableOutlineModeXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"pivottable_outline_mode.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pTable
@@ -2444,7 +2411,6 @@ void 
ScPivotTableFiltersTest::testPivotTableOutlineModeXLSX()
 void ScPivotTableFiltersTest::testPivotTableDuplicatedMemberFilterXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"pivottable_duplicated_member_filter.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pTable
@@ -2463,7 +2429,6 @@ void 
ScPivotTableFiltersTest::testPivotTableDuplicatedMemberFilterXLSX()
 void ScPivotTableFiltersTest::testPivotTableTabularModeXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"pivottable_tabular_mode.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pTable
@@ -2484,7 +2449,6 @@ void 
ScPivotTableFiltersTest::testPivotTableTabularModeXLSX()
 void ScPivotTableFiltersTest::testPivotTableDuplicateFields()
 {
     ScDocShellRef xShell = loadDoc(u"caseinsensitive-duplicate-fields.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pCacheDef = XPathHelper::parseExport(pXPathFile, 
m_xSFactory,
@@ -2509,7 +2473,6 @@ void 
ScPivotTableFiltersTest::testPivotTableDuplicateFields()
 void ScPivotTableFiltersTest::testTdf112106()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf112106.", FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     ScDocument& rDoc = xDocSh->GetDocument();
     ScDPCollection* pDPs = rDoc.GetDPCollection();
     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1), pDPs->GetCount());
@@ -2540,7 +2503,6 @@ void ScPivotTableFiltersTest::testTdf123923()
     // tdf#123923: Excel fails when it finds "Err:504" instead of "#REF!" in 
pivot table cache
 
     ScDocShellRef xShell = loadDoc(u"pivot-table-err-in-cache.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pTable = XPathHelper::parseExport(pXPathFile, m_xSFactory,
@@ -2556,7 +2518,6 @@ void ScPivotTableFiltersTest::testTdf123939()
     // tdf#123939: Excel warns on containsMixedTypes="1" if sharedItems has 
only strings and errors
 
     ScDocShellRef xShell = loadDoc(u"pivot-table-str-and-err-in-data.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pTable = XPathHelper::parseExport(pXPathFile, m_xSFactory,
@@ -2580,7 +2541,6 @@ void ScPivotTableFiltersTest::testTdf123939()
 void ScPivotTableFiltersTest::testTdf124651()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf124651_simplePivotTable.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/pivotTables/pivotTable1.xml", FORMAT_XLSX);
@@ -2594,7 +2554,6 @@ void ScPivotTableFiltersTest::testTdf124651()
 void ScPivotTableFiltersTest::testTdf124736()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot-table/shared-dategroup.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
     xDocSh->DoClose();
@@ -2647,7 +2606,6 @@ void ScPivotTableFiltersTest::testTdf124736()
 void ScPivotTableFiltersTest::tesTtdf124772NumFmt()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot-table-num-fmt.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
     xDocSh->DoClose();
@@ -2673,7 +2631,6 @@ void ScPivotTableFiltersTest::testTdf124810()
     {
         // First, test that we roundtrip existing pivot table style 
information from XLSX.
         ScDocShellRef xDocSh = loadDoc(u"pivot_dark1.", FORMAT_XLSX);
-        CPPUNIT_ASSERT(xDocSh.is());
 
         xmlDocUniquePtr pTable = XPathHelper::parseExport2(
             *this, *xDocSh, m_xSFactory, "xl/pivotTables/pivotTable1.xml", 
FORMAT_XLSX);
@@ -2695,7 +2652,6 @@ void ScPivotTableFiltersTest::testTdf124810()
         // original document. Just use some ODS as source. This might be 
changed when we start
         // exporting better pivot table style information.
         ScDocShellRef xDocSh = loadDoc(u"tdf124651_simplePivotTable.", 
FORMAT_ODS);
-        CPPUNIT_ASSERT(xDocSh.is());
 
         xmlDocUniquePtr pTable = XPathHelper::parseExport2(
             *this, *xDocSh, m_xSFactory, "xl/pivotTables/pivotTable1.xml", 
FORMAT_XLSX);
@@ -2716,7 +2672,6 @@ void ScPivotTableFiltersTest::testTdf124810()
 void ScPivotTableFiltersTest::testTdf124883()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivot-table/two-data-fields.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pTable = XPathHelper::parseExport2(
         *this, *xDocSh, m_xSFactory, "xl/pivotTables/pivotTable1.xml", 
FORMAT_XLSX);
@@ -2733,7 +2688,6 @@ void ScPivotTableFiltersTest::testTdf124883()
 void ScPivotTableFiltersTest::testTdf125046()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_long_text.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(
         *this, *xDocSh, m_xSFactory, 
"xl/pivotCache/pivotCacheDefinition1.xml", FORMAT_XLSX);
@@ -2745,7 +2699,6 @@ void ScPivotTableFiltersTest::testTdf125046()
 void ScPivotTableFiltersTest::testTdf125055()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_1s_difference.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(
         *this, *xDocSh, m_xSFactory, 
"xl/pivotCache/pivotCacheDefinition1.xml", FORMAT_XLSX);
@@ -2781,7 +2734,6 @@ void ScPivotTableFiltersTest::testTdf125055()
 void ScPivotTableFiltersTest::testTdf125086()
 {
     ScDocShellRef xDocSh = loadDoc(u"pivottable_fieldInRowsAndData.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/pivotTables/pivotTable1.xml", FORMAT_XLSX);
diff --git a/sc/qa/unit/subsequent_export_test.cxx 
b/sc/qa/unit/subsequent_export_test.cxx
index 93250fe199a2..d543cd692af0 100644
--- a/sc/qa/unit/subsequent_export_test.cxx
+++ b/sc/qa/unit/subsequent_export_test.cxx
@@ -347,7 +347,6 @@ void ScExportTest::testDefaultFontHeight()
 void ScExportTest::testTdf139167()
 {
     ScDocShellRef xShell = loadDoc(u"tdf139167.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -366,7 +365,6 @@ void ScExportTest::testTdf139167()
 void ScExportTest::testTdf113271()
 {
     ScDocShellRef xShell = loadDoc(u"tdf113271.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -389,7 +387,6 @@ void ScExportTest::testTdf113271()
 void ScExportTest::testTdf139394()
 {
     ScDocShellRef xShell = loadDoc(u"tdf139394.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -421,7 +418,6 @@ void ScExportTest::testTdf139394()
 void ScExportTest::testExtCondFormatXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"tdf139021.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -468,7 +464,6 @@ void ScExportTest::testExtCondFormatXLSX()
 void ScExportTest::testTdf90104()
 {
     ScDocShellRef xShell = loadDoc(u"tdf90104.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -494,7 +489,6 @@ void ScExportTest::testTdf111876()
     // Document with relative path hyperlink
 
     ScDocShellRef xShell = loadDoc(u"tdf111876.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -534,7 +528,6 @@ void ScExportTest::testPasswordExportODS()
 void ScExportTest::testTdf134332()
 {
     ScDocShellRef xShell = loadDoc(u"tdf134332.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
 
@@ -557,7 +550,6 @@ void ScExportTest::testTdf134332()
 void ScExportTest::testConditionalFormatExportODS()
 {
     ScDocShellRef xShell = loadDoc(u"new_cond_format_test_export.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_ODS);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -572,7 +564,6 @@ void ScExportTest::testConditionalFormatExportODS()
 void ScExportTest::testCondFormatExportCellIs()
 {
     ScDocShellRef xShell = loadDoc(u"condFormat_cellis.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
 
@@ -608,7 +599,6 @@ void ScExportTest::testCondFormatExportCellIs()
 void ScExportTest::testConditionalFormatExportXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"new_cond_format_test_export.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -630,7 +620,6 @@ void ScExportTest::testConditionalFormatExportXLSX()
 void ScExportTest::testTdf99856_dataValidationTest()
 {
     ScDocShellRef xShell = loadDoc(u"tdf99856_dataValidationTest.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load doc", xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT_MESSAGE("Failed to reload doc", xDocSh.is());
@@ -653,7 +642,6 @@ void ScExportTest::testProtectionKeyODS_UTF16LErtlSHA1()
     OUString const 
password("1012345678901234567890123456789012345678901234567890");
 
     ScDocShellRef xShell = loadDoc(u"protection-key1.", FORMAT_FODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load doc", xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
     ScDocProtection* const pDocProt(rDoc.GetDocProtection());
@@ -682,7 +670,6 @@ void ScExportTest::testProtectionKeyODS_UTF8SHA1()
     OUString const 
password("1012345678901234567890123456789012345678901234567890");
 
     ScDocShellRef xShell = loadDoc(u"protection-key2.", FORMAT_FODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load doc", xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
     ScDocProtection* const pDocProt(rDoc.GetDocProtection());
@@ -711,7 +698,6 @@ void ScExportTest::testProtectionKeyODS_UTF8SHA256ODF12()
     OUString const 
password("1012345678901234567890123456789012345678901234567890");
 
     ScDocShellRef xShell = loadDoc(u"protection-key3.", FORMAT_FODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load doc", xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
     ScDocProtection* const pDocProt(rDoc.GetDocProtection());
@@ -742,7 +728,6 @@ void ScExportTest::testProtectionKeyODS_UTF8SHA256W3C()
     OUString const 
password("1012345678901234567890123456789012345678901234567890");
 
     ScDocShellRef xShell = loadDoc(u"protection-key4.", FORMAT_FODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load doc", xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
     ScDocProtection* const pDocProt(rDoc.GetDocProtection());
@@ -773,7 +758,6 @@ void ScExportTest::testProtectionKeyODS_XL_SHA1()
     OUString const 
password("1012345678901234567890123456789012345678901234567890");
 
     ScDocShellRef xShell = loadDoc(u"protection-key5.", FORMAT_FODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load doc", xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
     ScDocProtection* const pDocProt(rDoc.GetDocProtection());
@@ -806,7 +790,6 @@ void ScExportTest::testProtectionKeyODS_XL_SHA1()
 void ScExportTest::testColorScaleExportODS()
 {
     ScDocShellRef xShell = loadDoc(u"colorscale.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_ODS);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -822,7 +805,6 @@ void ScExportTest::testColorScaleExportODS()
 void ScExportTest::testColorScaleExportXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"colorscale.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -838,7 +820,6 @@ void ScExportTest::testColorScaleExportXLSX()
 void ScExportTest::testDataBarExportODS()
 {
     ScDocShellRef xShell = loadDoc(u"databar.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_ODS);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -853,7 +834,6 @@ void ScExportTest::testDataBarExportODS()
 void ScExportTest::testFormatExportODS()
 {
     ScDocShellRef xShell = loadDoc(u"formats.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_ODS);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -869,7 +849,6 @@ void ScExportTest::testCommentExportXLSX()
 {
     //tdf#104729 FILESAVE OpenOffice do not save author of the comment during 
export to .xlsx
     ScDocShellRef xShell = loadDoc(u"comment.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pComments
@@ -906,7 +885,6 @@ void ScExportTest::testCommentExportXLSX_2_XLSX()
 {
     //tdf#117287 FILESAVE XLSX: Comments always disappear after opening the 
exported XLSX file with Excel
     ScDocShellRef xShell = loadDoc(u"tdf117287_comment.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
     ScAddress aPosC9(2, 8, 0);
@@ -939,7 +917,6 @@ void ScExportTest::testCustomColumnWidthExportXLSX()
 {
     //tdf#100946 FILESAVE Excel on macOS ignored column widths in XLSX last 
saved by LO
     ScDocShellRef xShell = loadDoc(u"custom_column_width.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -1015,7 +992,6 @@ void ScExportTest::testXfDefaultValuesXLSX()
 {
     //tdf#70565 FORMATTING: User Defined Custom Formatting is not applied 
during importing XLSX documents
     ScDocShellRef xShell = loadDoc(u"xf_default_values.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pSheet = XPathHelper::parseExport(pXPathFile, m_xSFactory, 
"xl/styles.xml");
@@ -1194,7 +1170,6 @@ void ScExportTest::testColumnWidthResaveXLSX()
     // tdf#91475 FILESAVE: Column width is not preserved in XLSX / after round 
trip.
     // Test if after resave .xlsx file, columns width is identical with 
previous one
     ScDocShellRef xShell = loadDoc(u"different-column-width-excel2010.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -1235,8 +1210,6 @@ void ScExportTest::testColumnWidthExportFromODStoXLSX()
 
     ScDocShellRef xShell = loadDoc(u"different-column-width.", FORMAT_ODS);
 
-    CPPUNIT_ASSERT(xShell.is());
-
     ScDocument& rOdsDoc = xShell->GetDocument();
 
     // Col 1, Tab 0 (Column width 2.00 in)
@@ -1302,7 +1275,6 @@ void ScExportTest::testOutlineExportXLSX()
     //tdf#100347 FILESAVE FILEOPEN after exporting to .xlsx format grouping 
are lost
     //tdf#51524  FILESAVE .xlsx and.xls looses width information for 
hidden/collapsed grouped columns
     ScDocShellRef xShell = loadDoc(u"outline.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -1450,7 +1422,6 @@ void ScExportTest::testOutlineExportXLSX()
 void ScExportTest::testAllRowsHiddenXLSX()
 {
     ScDocShellRef xOrigDocSh = loadDoc(u"tdf105840_allRowsHidden.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xOrigDocSh.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile
         = ScBootstrapFixture::exportTo(*xOrigDocSh, FORMAT_XLSX);
@@ -1467,7 +1438,6 @@ void ScExportTest::testHiddenEmptyRowsXLSX()
 {
     //tdf#98106 FILESAVE: Hidden and empty rows became visible when export to 
.XLSX
     ScDocShellRef xShell = loadDoc(u"hidden-empty-rows.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -1487,7 +1457,6 @@ void ScExportTest::testHiddenEmptyColsODS()
 {
     //tdf#98106 FILESAVE: Hidden and empty rows became visible when export to 
.XLSX
     ScDocShellRef xShell = loadDoc(u"tdf128895_emptyHiddenCols.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_ODS);
     xmlDocUniquePtr pSheet = XPathHelper::parseExport(pXPathFile, m_xSFactory, 
"content.xml");
@@ -1502,7 +1471,6 @@ void ScExportTest::testLandscapeOrientationXLSX()
 {
     //tdf#48767 - Landscape page orientation is not loaded from .xlsx format 
with MS Excel, after export with Libre Office
     ScDocShellRef xShell = loadDoc(u"hidden-empty-rows.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -1519,7 +1487,6 @@ void ScExportTest::testLandscapeOrientationXLSX()
 void ScExportTest::testDataBarExportXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"databar.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -1662,7 +1629,6 @@ void setEscapement(ScFieldEditEngine& rEE, sal_Int32 
nPara, sal_Int32 nStart, sa
 void ScExportTest::testNamedRangeBugfdo62729()
 {
     ScDocShellRef xShell = loadDoc(u"fdo62729.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
     ScDocument& rDoc = xShell->GetDocument();
 
     ScRangeName* pNames = rDoc.GetRangeName();
@@ -1687,7 +1653,6 @@ void ScExportTest::testNamedRangeBugfdo62729()
 void ScExportTest::testBuiltinRangesXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"built-in_ranges.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
     xShell->DoClose();
@@ -2222,7 +2187,6 @@ void ScExportTest::testRichTextExportODS()
 void ScExportTest::testRichTextCellFormatXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"cellformat.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -2455,7 +2419,6 @@ void checkMatrixRange(ScDocument& rDoc, const ScRange& 
rRange)
 void ScExportTest::testInlineArrayXLS()
 {
     ScDocShellRef xShell = loadDoc(u"inline-array.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLS);
     xShell->DoClose();
@@ -2478,7 +2441,6 @@ void ScExportTest::testInlineArrayXLS()
 void ScExportTest::testEmbeddedChartODS()
 {
     ScDocShellRef xShell = loadDoc(u"embedded-chart.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> 
pTempFile(ScBootstrapFixture::exportTo(*xShell, FORMAT_ODS));
 
@@ -2496,7 +2458,6 @@ void ScExportTest::testEmbeddedChartODS()
 void ScExportTest::testEmbeddedChartXLS()
 {
     ScDocShellRef xShell = loadDoc(u"embedded-chart.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLS);
     xShell->DoClose();
@@ -2525,7 +2486,6 @@ void ScExportTest::testEmbeddedChartXLS()
 void ScExportTest::testCellAnchoredGroupXLS()
 {
     ScDocShellRef xDocSh_in = loadDoc(u"cell-anchored-group.", FORMAT_XLS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load cell-anchored-group.xls", 
xDocSh_in.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xDocSh_in, FORMAT_ODS);
     CPPUNIT_ASSERT_MESSAGE("Failed to save and reload 
cell-anchored-group.ods", xDocSh.is());
@@ -2551,7 +2511,6 @@ void ScExportTest::testCellAnchoredGroupXLS()
 void ScExportTest::testFormulaReferenceXLS()
 {
     ScDocShellRef xShell = loadDoc(u"formula-reference.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLS);
     xShell->DoClose();
@@ -2576,7 +2535,6 @@ void ScExportTest::testFormulaReferenceXLS()
 void ScExportTest::testSheetProtectionXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"ProtecteSheet1234Pass.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2600,7 +2558,6 @@ void ScExportTest::testSheetProtectionXLSX()
 void ScExportTest::testSheetProtectionXLSB()
 {
     ScDocShellRef xShell = loadDoc(u"tdf108017_calcProtection.", FORMAT_XLSB);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2663,7 +2620,6 @@ void ScExportTest::testExcelCellBorders(sal_uLong 
nFormatType)
     };
 
     ScDocShellRef xDocSh = loadDoc(u"cell-borders.", nFormatType);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load file", xDocSh.is());
     {
         ScDocument& rDoc = xDocSh->GetDocument();
 
@@ -2872,7 +2828,6 @@ void ScExportTest::testBordersExchangeXLSX()
 
     ScDocShellRef xShell
         = loadDoc(u"test_borders_export.", FORMAT_ODS); // load the ods with 
our Borders
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh
         = saveAndReload(*xShell, FORMAT_XLSX); // save the ods to xlsx and 
load xlsx
@@ -3122,7 +3077,6 @@ void ScExportTest::testTrackChangesSimpleXLSX()
     // First, test the xls variant.
 
     ScDocShellRef xDocSh = loadDoc(u"track-changes/simple-cell-changes.", 
FORMAT_XLS);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument* pDoc = &xDocSh->GetDocument();
     bool bGood = aTest.check(*pDoc);
     CPPUNIT_ASSERT_MESSAGE("Initial check failed (xls).", bGood);
@@ -3149,7 +3103,6 @@ void ScExportTest::testTrackChangesSimpleXLSX()
     // Now, test the xlsx variant the same way.
 
     xDocSh = loadDoc(u"track-changes/simple-cell-changes.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     pDoc = &xDocSh->GetDocument();
     aTest.check(*pDoc);
     CPPUNIT_ASSERT_MESSAGE("Initial check failed (xlsx).", bGood);
@@ -3216,7 +3169,6 @@ void ScExportTest::testSheetTabColorsXLSX()
 
     ScDocShellRef xDocSh = loadDoc(u"sheet-tab-color.", FORMAT_XLSX);
     {
-        CPPUNIT_ASSERT_MESSAGE("Failed to load file.", xDocSh.is());
         ScDocument& rDoc = xDocSh->GetDocument();
         bool bRes = aTest.checkContent(rDoc);
         CPPUNIT_ASSERT_MESSAGE("Failed on the initial content check.", bRes);
@@ -3235,7 +3187,6 @@ void ScExportTest::testSheetTabColorsXLSX()
 void ScExportTest::testTdf133487()
 {
     ScDocShellRef xShell = loadDoc(u"shapes_foreground_background.", 
FORMAT_FODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load the document.", xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_ODS);
     xmlDocUniquePtr pXmlDoc = XPathHelper::parseExport(pXPathFile, 
m_xSFactory, "content.xml");
@@ -3379,7 +3330,6 @@ void ScExportTest::testSharedFormulaExportXLS()
 
     ScDocShellRef xDocSh = loadDoc(u"shared-formula/3d-reference.", 
FORMAT_ODS);
     {
-        CPPUNIT_ASSERT_MESSAGE("Failed to load file.", xDocSh.is());
         ScDocument& rDoc = xDocSh->GetDocument();
 
         // Check the content of the original.
@@ -3475,7 +3425,6 @@ void ScExportTest::testSharedFormulaExportXLSX()
 
     ScDocShellRef xDocSh = loadDoc(u"shared-formula/3d-reference.", 
FORMAT_XLSX);
     {
-        CPPUNIT_ASSERT_MESSAGE("Failed to load file.", xDocSh.is());
         ScDocument& rDoc = xDocSh->GetDocument();
 
         bool bRes = aTest.checkContent(rDoc);
@@ -3547,7 +3496,6 @@ void 
ScExportTest::testSharedFormulaStringResultExportXLSX()
 
     ScDocShellRef xDocSh = loadDoc(u"shared-formula/text-results.", 
FORMAT_XLSX);
     {
-        CPPUNIT_ASSERT_MESSAGE("Failed to load file.", xDocSh.is());
         ScDocument& rDoc = xDocSh->GetDocument();
 
         // Check content without re-calculation, to test cached formula 
results.
@@ -3574,7 +3522,6 @@ void 
ScExportTest::testSharedFormulaStringResultExportXLSX()
 void ScExportTest::testFunctionsExcel2010(sal_uLong nFormatType)
 {
     ScDocShellRef xShell = loadDoc(u"functions-excel-2010.", FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load the document.", xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, nFormatType);
     ScDocument& rDoc = xDocSh->GetDocument();
@@ -3592,7 +3539,6 @@ void ScExportTest::testFunctionsExcel2010XLS() { 
testFunctionsExcel2010(FORMAT_X
 void ScExportTest::testCeilingFloor(sal_uLong nFormatType)
 {
     ScDocShellRef xShell = loadDoc(u"ceiling-floor.", FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load the document.", xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, nFormatType);
     ScDocument& rDoc = xDocSh->GetDocument();
@@ -3609,7 +3555,6 @@ void ScExportTest::testCeilingFloorODSToXLSX()
 {
     // tdf#100011 - Cannot open sheet containing FLOOR/CEILING functions by MS 
Excel, after export to .xlsx
     ScDocShellRef xShell = loadDoc(u"ceiling-floor.", FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load the document.", xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pSheet = XPathHelper::parseExport(pXPathFile, m_xSFactory, 
"xl/workbook.xml");
@@ -3629,7 +3574,6 @@ void ScExportTest::testCustomXml()
 {
     // Load document and export it to a temporary file
     ScDocShellRef xShell = loadDoc(u"customxml.", FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load the document.", xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pXmlDoc
@@ -3665,7 +3609,6 @@ static sal_Unicode lcl_getWindowsDrive(const OUString& 
aURL)
 void ScExportTest::testRelativePathsODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"fdo79305.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     std::shared_ptr<utl::TempFile> pTempFile = exportTo(*xDocSh, FORMAT_ODS);
     xmlDocUniquePtr pDoc = XPathHelper::parseExport(pTempFile, m_xSFactory, 
"content.xml");
@@ -3709,7 +3652,6 @@ void testSheetProtection_Impl(const ScDocument& rDoc)
 void ScExportTest::testSheetProtectionODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"sheet-protection.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     {
         ScDocument& rDoc = xDocSh->GetDocument();
@@ -3752,7 +3694,6 @@ void ScExportTest::testSwappedOutImageExport()
 
         const OString sFailedMessage
             = OString::Concat("Failed on filter: ") + aFilterNames[nFilter];
-        CPPUNIT_ASSERT_MESSAGE(sFailedMessage.getStr(), xDocSh.is());
 
         // Export the document and import again for a check
         ScDocShellRef xDocSh2 = saveAndReload(*xDocSh, nFilter);
@@ -3833,7 +3774,6 @@ void ScExportTest::tearDown()
 void ScExportTest::testSupBookVirtualPathXLS()
 {
     ScDocShellRef xShell = loadDoc(u"external-ref.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLS);
     xShell->DoClose();
@@ -3924,7 +3864,6 @@ void ScExportTest::testImageWithSpecialID()
 
         const OString sFailedMessage
             = OString::Concat("Failed on filter: ") + aFilterNames[nFilter];
-        CPPUNIT_ASSERT_MESSAGE(sFailedMessage.getStr(), xDocSh.is());
 
         // Export the document and import again for a check
         ScDocShellRef xDocSh2 = saveAndReload(*xDocSh, nFilter);
@@ -4078,7 +4017,6 @@ void ScExportTest::testRelativeNamedExpressionsXLS()
 void ScExportTest::testSheetTextBoxHyperlinkXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"textbox-hyperlink.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -4098,7 +4036,6 @@ void ScExportTest::testSheetTextBoxHyperlinkXLSX()
 void ScExportTest::testFontSizeXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"fontSize.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/drawings/drawing1.xml", FORMAT_XLSX);
@@ -4114,7 +4051,6 @@ void ScExportTest::testFontSizeXLSX()
 void ScExportTest::testSheetCharacterKerningSpaceXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"textbox-CharKerningSpace.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -4136,7 +4072,6 @@ void ScExportTest::testSheetCharacterKerningSpaceXLSX()
 void ScExportTest::testSheetCondensedCharacterSpaceXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"textbox-CondensedCharacterSpace.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -4158,7 +4093,6 @@ void ScExportTest::testSheetCondensedCharacterSpaceXLSX()
 void ScExportTest::testTextUnderlineColorXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"underlineColor.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/drawings/drawing1.xml", FORMAT_XLSX);
@@ -4192,7 +4126,6 @@ void ScExportTest::testTextUnderlineColorXLSX()
 void ScExportTest::testSheetRunParagraphPropertyXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"TextColor.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -4238,7 +4171,6 @@ void ScExportTest::testPreserveTextWhitespace2XLSX()
 void ScExportTest::testHiddenShapeXLS()
 {
     ScDocShellRef xDocSh = loadDoc(u"hiddenShape.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
     CPPUNIT_ASSERT(rDoc.GetTableCount() > 0);
@@ -4255,7 +4187,6 @@ void ScExportTest::testHiddenShapeXLS()
 void ScExportTest::testHiddenShapeXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"hiddenShape.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
     CPPUNIT_ASSERT(rDoc.GetTableCount() > 0);
@@ -4277,7 +4208,6 @@ void ScExportTest::testHiddenShapeXLSX()
 void ScExportTest::testShapeAutofitXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"testShapeAutofit.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/drawings/drawing1.xml", FORMAT_XLSX);
@@ -4294,7 +4224,6 @@ void ScExportTest::testShapeAutofitXLSX()
 void ScExportTest::testHyperlinkXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"hyperlink.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(
         *this, *xDocSh, m_xSFactory, "xl/drawings/_rels/drawing1.xml.rels", 
FORMAT_XLSX);
@@ -4307,7 +4236,6 @@ void ScExportTest::testHyperlinkXLSX()
 void ScExportTest::testMoveCellAnchoredShapesODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"move-cell-anchored-shapes.", FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load move-cell-anchored-shapes.ods", 
xDocSh.is());
 
     // There are two cell-anchored objects on the first sheet.
     ScDocument& rDoc = xDocSh->GetDocument();
@@ -4480,7 +4408,6 @@ void ScExportTest::testMoveCellAnchoredShapesODS()
 void ScExportTest::testConditionalFormatRangeListXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"conditionalformat_rangelist.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
     CPPUNIT_ASSERT(pDoc);
@@ -4491,7 +4418,6 @@ void ScExportTest::testConditionalFormatRangeListXLSX()
 void ScExportTest::testConditionalFormatContainsTextXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"conditionalformat_containstext.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
     CPPUNIT_ASSERT(pDoc);
@@ -4503,7 +4429,6 @@ void ScExportTest::testConditionalFormatContainsTextXLSX()
 void ScExportTest::testConditionalFormatPriorityCheckXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"conditional_fmt_checkpriority.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
     CPPUNIT_ASSERT(pDoc);
@@ -4552,7 +4477,6 @@ void 
ScExportTest::testConditionalFormatPriorityCheckXLSX()
 void ScExportTest::testConditionalFormatOriginXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"conditional_fmt_origin.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
     CPPUNIT_ASSERT(pDoc);
@@ -4566,7 +4490,6 @@ void ScExportTest::testConditionalFormatOriginXLSX()
 void ScExportTest::testTdf113646()
 {
     ScDocShellRef xShell = loadDoc(u"tdf113646.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     auto pXPathFile = ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pSheet = XPathHelper::parseExport(pXPathFile, m_xSFactory, 
"xl/styles.xml");
@@ -4581,7 +4504,6 @@ void ScExportTest::testDateStandardfilterXLSX()
 {
     // XLSX Roundtripping standard filter with date
     ScDocShellRef xDocSh = loadDoc(u"tdf142607.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
diff --git a/sc/qa/unit/subsequent_export_test2.cxx 
b/sc/qa/unit/subsequent_export_test2.cxx
index b0ffb138c51f..77b8858fcd97 100644
--- a/sc/qa/unit/subsequent_export_test2.cxx
+++ b/sc/qa/unit/subsequent_export_test2.cxx
@@ -348,7 +348,6 @@ void ScExportTest2::registerNamespaces(xmlXPathContextPtr& 
pXmlXPathCtx)
 void ScExportTest2::testMatrixMultiplicationXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"matrix-multiplication.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -373,7 +372,6 @@ void ScExportTest2::testMatrixMultiplicationXLSX()
 void ScExportTest2::testRefStringXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"ref_string.", FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to open doc", xDocSh.is());
 
     //make sure ref syntax gets saved for MSO-produced docs
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX);
@@ -390,7 +388,6 @@ void ScExportTest2::testRefStringConfigXLSX()
 {
     // this doc is configured with CalcA1 ref syntax
     ScDocShellRef xDocSh = loadDoc(u"empty.", FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to open doc", xDocSh.is());
 
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX);
     CPPUNIT_ASSERT_MESSAGE("Failed to reload doc", xDocSh.is());
@@ -516,7 +513,6 @@ void ScExportTest2::testHeaderFooterContentODS()
 void ScExportTest2::testTextDirectionXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"writingMode.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc
         = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, 
"xl/styles.xml", FORMAT_XLSX);
@@ -531,7 +527,6 @@ void ScExportTest2::testTextDirectionXLSX()
 void ScExportTest2::testTdf121260()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf121260.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
     // change formula syntax (i.e. not string ref syntax) to ExcelA1
@@ -557,7 +552,6 @@ void ScExportTest2::testTdf121260()
 void ScExportTest2::testTdf120168()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf120168.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc
         = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, 
"xl/styles.xml", FORMAT_XLSX);
@@ -576,7 +570,6 @@ void ScExportTest2::testTdf66668()
 {
     // Would hang on exporting without the fix in place
     ScDocShellRef xDocSh = loadDoc(u"tdf66668.", FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to open doc", xDocSh.is());
 
     xmlDocUniquePtr pDoc
         = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, 
"xl/styles.xml", FORMAT_XLSX);
@@ -587,7 +580,6 @@ void ScExportTest2::testTdf66668()
 void ScExportTest2::testTdf130108()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf130108.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc
         = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, 
"xl/styles.xml", FORMAT_XLSX);
@@ -606,7 +598,6 @@ void ScExportTest2::testTdf130108()
 void ScExportTest2::testTdf76949()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf76949.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pSheet = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                        
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
@@ -620,7 +611,6 @@ void ScExportTest2::testTdf76949()
 void ScExportTest2::testTdf107586()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf107586.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pSheet = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                        
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
@@ -636,7 +626,6 @@ void ScExportTest2::testTdf107586()
 void ScExportTest2::testTdf55417()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf55417.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc
         = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, 
"xl/styles.xml", FORMAT_XLSX);
@@ -650,7 +639,6 @@ void ScExportTest2::testTdf55417()
 void ScExportTest2::testTdf129985()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf129985.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc
         = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, 
"xl/styles.xml", FORMAT_XLSX);
@@ -664,7 +652,6 @@ void ScExportTest2::testTdf129985()
 void ScExportTest2::testTdf73063()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf73063.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc
         = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, 
"xl/styles.xml", FORMAT_XLSX);
@@ -680,7 +667,6 @@ xmlDocUniquePtr 
ScExportTest2::testTdf95640(std::u16string_view rFileName, sal_I
                                             sal_Int32 nDestFormat)
 {
     ScDocShellRef xShell = loadDoc(rFileName, nSourceFormat);
-    CPPUNIT_ASSERT(xShell);
 
     auto pXPathFile = ScBootstrapFixture::exportTo(*xShell, nDestFormat);
     xShell->DoClose();
@@ -734,7 +720,6 @@ void ScExportTest2::testDateAutofilterXLSX()
 {
     // XLSX Roundtripping autofilter with date list
     ScDocShellRef xDocSh = loadDoc(u"dateAutofilter.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
@@ -759,7 +744,6 @@ void ScExportTest2::testDateAutofilterXLSX()
 void ScExportTest2::testDateAutofilterODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf142231.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc
         = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, 
"content.xml", FORMAT_ODS);
@@ -773,7 +757,6 @@ void ScExportTest2::testDateAutofilterODS()
 void ScExportTest2::testAutofilterColorsODF()
 {
     ScDocShellRef xDocSh = loadDoc(u"autofilter-colors.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc
         = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, 
"content.xml", FORMAT_ODS);
@@ -813,7 +796,6 @@ void ScExportTest2::testAutofilterColorsOOXML()
 {
     {
         ScDocShellRef xDocSh = loadDoc(u"autofilter-colors.", FORMAT_XLSX);
-        CPPUNIT_ASSERT(xDocSh.is());
         std::shared_ptr<utl::TempFile> pXPathFile
             = ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
         xmlDocUniquePtr pTable1
@@ -835,7 +817,6 @@ void ScExportTest2::testAutofilterColorsOOXML()
 
     {
         ScDocShellRef xDocSh = loadDoc(u"autofilter-colors-fg.", FORMAT_XLSX);
-        CPPUNIT_ASSERT(xDocSh.is());
         std::shared_ptr<utl::TempFile> pXPathFile
             = ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
         xmlDocUniquePtr pTable1
@@ -859,7 +840,6 @@ void ScExportTest2::testAutofilterColorsOOXML()
 void ScExportTest2::testAutofilterTop10XLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf143068_top10filter.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
@@ -873,7 +853,6 @@ void ScExportTest2::testAutofilterTop10XLSX()
 void ScExportTest2::testTdf88657ODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf88657.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc
         = XPathHelper::parseExport2(*this, *xDocSh, m_xSFactory, "styles.xml", 
FORMAT_ODS);
@@ -887,7 +866,6 @@ void ScExportTest2::testTdf88657ODS()
 void ScExportTest2::testTdf41722()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf41722.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
@@ -903,7 +881,6 @@ void ScExportTest2::testTdf41722()
 void ScExportTest2::testTdf113621()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf113621.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
@@ -917,7 +894,6 @@ void ScExportTest2::testTdf113621()
 void ScExportTest2::testEscapeCharInNumberFormatXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf81939.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
 
@@ -950,7 +926,6 @@ void ScExportTest2::testEscapeCharInNumberFormatXLSX()
 void ScExportTest2::testNatNumInNumberFormatXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf79398_NatNum5.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX); // Convert [NatNum5] to 
[DBNum2] in Chinese
     CPPUNIT_ASSERT(xDocSh.is());
 
@@ -967,7 +942,6 @@ void ScExportTest2::testNatNumInNumberFormatXLSX()
 void ScExportTest2::testExponentWithoutSignFormatXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf102370_ExponentWithoutSign.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
 
@@ -988,7 +962,6 @@ void ScExportTest2::testExponentWithoutSignFormatXLSX()
 void ScExportTest2::testExtendedLCIDXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf36038_ExtendedLCID.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
 
@@ -1064,7 +1037,6 @@ void ScExportTest2::testHiddenRepeatedRowsODS()
 void ScExportTest2::testHyperlinkTargetFrameODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"hyperlink_frame.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
     const EditTextObject* pEditText = rDoc.GetEditText(ScAddress(2, 5, 0));
@@ -1105,7 +1077,6 @@ void ScExportTest2::testOpenDocumentAsReadOnly()
 void ScExportTest2::testKeepSettingsOfBlankRows()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf41425.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -1121,7 +1092,6 @@ void ScExportTest2::testKeepSettingsOfBlankRows()
 void ScExportTest2::testTdf133595()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf133595.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -1137,7 +1107,6 @@ void ScExportTest2::testTdf133595()
 void ScExportTest2::testTdf134769()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf134769.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -1160,7 +1129,6 @@ void ScExportTest2::testTdf134769()
 void ScExportTest2::testTdf106181()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf106181.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -1199,7 +1167,6 @@ void ScExportTest2::testTdf106181()
 void ScExportTest2::testTdf145057()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf145057.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX);
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
@@ -1212,7 +1179,6 @@ void ScExportTest2::testTdf145057()
 void ScExportTest2::testTdf105272()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf105272.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX);
     ScDocument& rDoc = xDocSh->GetDocument();
     //without the fix in place,it would fail
@@ -1227,7 +1193,6 @@ void ScExportTest2::testTdf105272()
 void ScExportTest2::testTdf118990()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf118990.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX);
     ScDocument& rDoc = xDocSh->GetDocument();
 
@@ -1250,7 +1215,6 @@ void ScExportTest2::testTdf118990()
 void ScExportTest2::testTdf121612()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf121612.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX);
 
     ScDocument& rDoc = xDocSh->GetDocument();
@@ -1269,7 +1233,6 @@ void ScExportTest2::testTdf121612()
 void ScExportTest2::testTdf112936()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf112936.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(
         *this, *xDocSh, m_xSFactory, 
"xl/pivotCache/pivotCacheDefinition1.xml", FORMAT_XLSX);
@@ -1301,7 +1264,6 @@ void ScExportTest2::testXltxExport()
 void ScExportTest2::testPivotCacheAfterExportXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"numgroup_example.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     // export only
     std::shared_ptr<utl::TempFile> pTemp = exportTo(*xDocSh, FORMAT_XLSX);
@@ -1330,7 +1292,6 @@ void ScExportTest2::testPivotCacheAfterExportXLSX()
 void ScExportTest2::testTdf114969XLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"sheet_name_with_dots.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
@@ -1344,7 +1305,6 @@ void ScExportTest2::testTdf114969XLSX()
 void ScExportTest2::testTdf115192XLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"test_115192.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(
         *this, *xDocSh, m_xSFactory, "xl/drawings/_rels/drawing1.xml.rels", 
FORMAT_XLSX);
@@ -1361,7 +1321,6 @@ void ScExportTest2::testTdf115192XLSX()
 void ScExportTest2::testTdf142764()
 {
     ScDocShellRef xShell = loadDoc(u"tdf142764.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell);
 
     auto pXPathFile = ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
 
@@ -1378,7 +1337,6 @@ void ScExportTest2::testTdf142764()
 void ScExportTest2::testTdf91634XLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"image_hyperlink.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
 
     xmlDocUniquePtr pDoc
@@ -1400,7 +1358,6 @@ void ScExportTest2::testTdf91634XLSX()
 void ScExportTest2::testValidationCopyPaste()
 {
     ScDocShellRef xDocSh = loadDoc(u"validation-copypaste.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     ScDocument& rSrcDoc = xDocSh->GetDocument();
 
     // Copy B1 from src doc to clip
@@ -1435,7 +1392,6 @@ void ScExportTest2::testValidationCopyPaste()
 void ScExportTest2::testTdf115159()
 {
     ScDocShellRef xShell = loadDoc(u"tdf115159.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
     xShell->DoClose();
@@ -1463,7 +1419,6 @@ void ScExportTest2::testTdf112567()
     });
 
     ScDocShellRef xShell = loadDoc(u"tdf112567.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
     xShell->DoClose();
@@ -1491,7 +1446,6 @@ void ScExportTest2::testTdf122191()
     });
 
     ScDocShellRef xShell = loadDoc(u"tdf122191.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
     CPPUNIT_ASSERT_EQUAL(OUString("IGAZ"), rDoc.GetString(0, 0, 0));
@@ -1512,7 +1466,6 @@ void ScExportTest2::testTdf122191()
 void ScExportTest2::testTdf142881()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf142881.", FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to load tdf142881.xlsx", xDocSh.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
     xmlDocUniquePtr pDrawing1
@@ -1556,7 +1509,6 @@ void ScExportTest2::testTdf112567b()
     });
 
     ScDocShellRef xShell = loadDoc(u"tdf112567.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
     xShell->DoClose();
@@ -1578,7 +1530,6 @@ void ScExportTest2::testTdf112567b()
 void ScExportTest2::testTdf123645XLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"chart_hyperlink.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
 
     xmlDocUniquePtr pDoc
@@ -1619,7 +1570,6 @@ void ScExportTest2::testTdf123645XLSX()
 void ScExportTest2::testTdf125173XLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"text_box_hyperlink.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
 
     xmlDocUniquePtr pDoc
@@ -1641,7 +1591,6 @@ void ScExportTest2::testTdf125173XLSX()
 void ScExportTest2::testTdf79972XLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf79972.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
 
     xmlDocUniquePtr pDoc
@@ -1662,7 +1611,6 @@ void ScExportTest2::testTdf79972XLSX()
 void ScExportTest2::testTdf126024XLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"hyperlink_formula.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
 
     xmlDocUniquePtr pDoc
@@ -1683,7 +1631,6 @@ void ScExportTest2::testTdf126024XLSX()
 void ScExportTest2::testTdf126177XLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"hyperlink_export.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
 
     xmlDocUniquePtr pDoc
@@ -1705,7 +1652,6 @@ void ScExportTest2::testCommentTextVAlignment()
 {
     // Testing comment text alignments.
     ScDocShellRef xShell = loadDoc(u"CommentTextVAlign.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
 
@@ -1722,7 +1668,6 @@ void ScExportTest2::testCommentTextHAlignment()
 {
     // Testing comment text alignments.
     ScDocShellRef xShell = loadDoc(u"CommentTextHAlign.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
 
@@ -1744,7 +1689,6 @@ void ScExportTest2::testRotatedImageODS()
     xGlobalSheetSettings->setMetric(static_cast<sal_Int16>(FieldUnit::MM));
 
     ScDocShellRef xDocSh = loadDoc(u"tdf103092_RotatedImage.", FORMAT_ODS, 
true);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     std::shared_ptr<utl::TempFile> pTemp = exportTo(*xDocSh, FORMAT_ODS);
     CPPUNIT_ASSERT(pTemp);
@@ -1789,7 +1733,6 @@ void ScExportTest2::testTdf85553()
 void ScExportTest2::testTdf128976()
 {
     ScDocShellRef xShell = loadDoc(u"tdf128976.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLS);
     xShell->DoClose();
@@ -1871,7 +1814,6 @@ void ScExportTest2::testTdf120502()
 void ScExportTest2::testTdf131372()
 {
     ScDocShellRef xShell = loadDoc(u"tdf131372.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell);
 
     auto pXPathFile = ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
 
@@ -1887,7 +1829,6 @@ void ScExportTest2::testTdf131372()
 void ScExportTest2::testTdf81470()
 {
     ScDocShellRef xShell = loadDoc(u"tdf81470.", FORMAT_XLS);
-    CPPUNIT_ASSERT(xShell);
 
     //without the fix in place, it would have crashed at export time
     auto pXPathFile = ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
@@ -1910,7 +1851,6 @@ void ScExportTest2::testTdf81470()
 void ScExportTest2::testTdf122331()
 {
     ScDocShellRef xShell = loadDoc(u"tdf122331.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell);
 
     auto pXPathFile = ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
 
@@ -1929,7 +1869,6 @@ void ScExportTest2::testTdf83779()
 {
     // Roundtripping TRUE/FALSE constants (not functions) must convert them to 
functions
     ScDocShellRef xShell = loadDoc(u"tdf83779.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell);
 
     auto pXPathFile = ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
 
@@ -1947,7 +1886,6 @@ void 
ScExportTest2::testTdf121715_FirstPageHeaderFooterXLSX()
 {
     // Check if first page header and footer are exported properly
     ScDocShellRef xShell = loadDoc(u"tdf121715.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pDoc
@@ -1968,7 +1906,6 @@ void 
ScExportTest2::testTdf121716_ExportEvenHeaderFooterXLSX()
     //  then the footer will be duplicated to have the same footer separately 
for even/odd pages
 
     ScDocShellRef xShell = loadDoc(u"tdf121716_EvenHeaderFooter.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2002,7 +1939,6 @@ void ScExportTest2::testTdf134459_HeaderFooterColorXLSX()
 {
     // Colors in header and footer should be exported, and imported properly
     ScDocShellRef xShell = loadDoc(u"tdf134459_HeaderFooterColor.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2023,7 +1959,6 @@ void 
ScExportTest2::testTdf134817_HeaderFooterTextWith2SectionXLSX()
 {
     // Header/footer text with multiple selection should be exported, and 
imported properly
     ScDocShellRef xShell = loadDoc(u"tdf134817_HeaderFooterTextWith2Section.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2044,7 +1979,6 @@ void ScExportTest2::testTdf121718_UseFirstPageNumberXLSX()
 {
     // If "First page number" is not checked then useFirstPageNumb, and 
firstPageNumber should not be exported.
     ScDocShellRef xShell = loadDoc(u"tdf121718_UseFirstPageNumber.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2069,7 +2003,6 @@ void ScExportTest2::testTdf121718_UseFirstPageNumberXLSX()
 void ScExportTest2::testHeaderFontStyleXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"tdf134826.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
     SfxStyleSheetBase* pStyleSheet
@@ -2107,7 +2040,6 @@ void ScExportTest2::testTdf135828_Shape_Rect()
     // tdf#123613 Check the positioning, and allow massive rounding errors 
because of the back and
     // forth conversion between emu and hmm.
     ScDocShellRef xShell = loadDoc(u"tdf135828_Shape_Rect.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2140,7 +2072,6 @@ void ScExportTest2::testTdf135828_Shape_Rect()
 void ScExportTest2::testTdf123139XLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf123139_applyAlignment.", FORMAT_XLSX);
-    CPPUNIT_ASSERT_MESSAGE("Failed to open doc", xDocSh.is());
 
     ScDocument& rDoc = xDocSh->GetDocument();
     const ScPatternAttr* pAttr = rDoc.GetPattern(0, 0, 0); //A1
@@ -2198,7 +2129,6 @@ void ScExportTest2::testTdf123139XLSX()
 void ScExportTest2::testTdf123353()
 {
     ScDocShellRef xShell = loadDoc(u"tdf123353.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2217,7 +2147,6 @@ void ScExportTest2::testTdf123353()
 void ScExportTest2::testTdf140098()
 {
     ScDocShellRef xShell = loadDoc(u"tdf140098.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2237,7 +2166,6 @@ void ScExportTest2::testTdf133688_precedents()
 {
     // tdf#133688 Check that we do not export detective shapes.
     ScDocShellRef xShell = loadDoc(u"tdf133688_dont_save_precedents_to_xlsx.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pDrawing
@@ -2253,7 +2181,6 @@ void 
ScExportTest2::testTdf91251_missingOverflowRoundtrip()
     // tdf#91251 check whether textBox overflow property (horzOverflow and 
vertOverflow) is
     // getting preserved after roundtrip
     ScDocShellRef xShell = loadDoc(u"tdf91251_missingOverflowRoundtrip.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2277,7 +2204,6 @@ void ScExportTest2::testTdf137000_handle_upright()
     // (Upright is an xml attribute of xdr:txBody/a:bodyPr. It is set when
     // in a textbox menu we choose: do not rotate this element.)
     ScDocShellRef xShell = loadDoc(u"tdf137000_export_upright.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pDrawing
@@ -2291,7 +2217,6 @@ void ScExportTest2::testTdf137000_handle_upright()
 void ScExportTest2::testTdf126305_DataValidatyErrorAlert()
 {
     ScDocShellRef xShell = loadDoc(u"tdf126305.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2313,7 +2238,6 @@ void ScExportTest2::testTdf126305_DataValidatyErrorAlert()
 void ScExportTest2::testTdf76047_externalLink()
 {
     ScDocShellRef pShell = loadDoc(u"tdf76047_externalLink.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(pShell.is());
 
     // load data from external links. (tdf76047_externalLinkSource.ods)
     // that file has to be in the same directory as tdf76047_externalLink.xlsx
@@ -2338,7 +2262,6 @@ void ScExportTest2::testTdf76047_externalLink()
 void ScExportTest2::testTdf87973_externalLinkSkipUnuseds()
 {
     ScDocShellRef pShell = loadDoc(u"tdf87973_externalLinkSkipUnuseds.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(pShell.is());
 
     // try to load data from external link: tdf132105_external.ods
     // that file has to be in the same directory as 
tdf87973_externalLinkSkipUnuseds.ods
@@ -2389,7 +2312,6 @@ void ScExportTest2::testTdf87973_externalLinkSkipUnuseds()
 void ScExportTest2::testTdf51022_lostPrintRange()
 {
     ScDocShellRef pShell = loadDoc(u"tdf87973_externalLinkSkipUnuseds.", 
FORMAT_ODS);
-    CPPUNIT_ASSERT(pShell.is());
 
     pShell->ReloadAllLinks();
     ScDocument& rDoc = pShell->GetDocument();
@@ -2416,7 +2338,6 @@ void ScExportTest2::testTdf51022_lostPrintRange()
 void ScExportTest2::testTdf138741_externalLinkSkipUnusedsCrash()
 {
     ScDocShellRef xShell = loadDoc(u"tdf138741_externalLinkSkipUnusedsCrash.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell);
 
     //without the fix in place, it would have crashed at export time
     ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
@@ -2427,7 +2348,6 @@ void 
ScExportTest2::testTdf138741_externalLinkSkipUnusedsCrash()
 void ScExportTest2::testTdf138824_linkToParentDirectory()
 {
     ScDocShellRef xShell = 
loadDoc(u"childDir/tdf138824_linkToParentDirectory.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocument& rDoc = xShell->GetDocument();
 
@@ -2465,7 +2385,6 @@ void ScExportTest2::testTdf138824_linkToParentDirectory()
 void ScExportTest2::testTdf129969()
 {
     ScDocShellRef xShell = loadDoc(u"external_hyperlink.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2482,7 +2401,6 @@ void ScExportTest2::testTdf129969()
 void ScExportTest2::testTdf147088()
 {
     ScDocShellRef xShell = loadDoc(u"tdf147088.", FORMAT_FODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     xShell->DoClose();
@@ -2501,7 +2419,6 @@ void ScExportTest2::testTdf147088()
 void ScExportTest2::testTdf84874()
 {
     ScDocShellRef xShell = loadDoc(u"tdf84874.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     xShell->DoClose();
@@ -2532,7 +2449,6 @@ void ScExportTest2::testTdf84874()
 void ScExportTest2::testTdf136721_paper_size()
 {
     ScDocShellRef xShell = loadDoc(u"tdf136721_letter_sized_paper.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
     xmlDocUniquePtr pDoc
@@ -2546,7 +2462,6 @@ void ScExportTest2::testTdf139258_rotated_image()
 {
     // Check that the topleft position of the image is correct.
     ScDocShellRef xShell = loadDoc(u"tdf139258_rotated_image.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xShell.is());
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
 
@@ -2582,7 +2497,6 @@ void ScExportTest2::testTdf144642_RowHeightRounding()
     // Calc simulates this roundings but only if the xlsx file was saved in MS 
Excel.
 
     ScDocShellRef xShell = loadDoc(u"tdf144642_RowHeight_10mm_SavedByCalc.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
     ScDocument& rDoc = xShell->GetDocument();
     // 10mm == 567 twips == 28.35pt
     CPPUNIT_ASSERT_EQUAL(sal_uInt16(567), rDoc.GetRowHeight(0, 0));
@@ -2590,7 +2504,6 @@ void ScExportTest2::testTdf144642_RowHeightRounding()
     xShell->DoClose();
 
     xShell = loadDoc(u"tdf144642_RowHeight_28.35pt_SavedByExcel.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
     ScDocument& rDoc2 = xShell->GetDocument();
     // 555twips == 27.75pt == 9.79mm
     CPPUNIT_ASSERT_EQUAL(sal_uInt16(555), rDoc2.GetRowHeight(0, 0));
@@ -2605,7 +2518,6 @@ void 
ScExportTest2::testTdf145129_DefaultRowHeightRounding()
 
     ScDocShellRef xShell
         = loadDoc(u"tdf145129_DefaultRowHeight_28.35pt_SavedByExcel.", 
FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
     ScDocument& rDoc = xShell->GetDocument();
     // 555twips == 27.75pt == 9.79mm
     CPPUNIT_ASSERT_EQUAL(sal_uInt16(555), rDoc.GetRowHeight(0, 0));
@@ -2616,7 +2528,6 @@ void 
ScExportTest2::testTdf145129_DefaultRowHeightRounding()
 void ScExportTest2::testTdf140431()
 {
     ScDocShellRef xShell = loadDoc(u"129969-min.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
@@ -2636,7 +2547,6 @@ void ScExportTest2::testCheckboxFormControlXlsxExport()
         return;
     // Given a document that has a checkbox form control:
     ScDocShellRef xShell = loadDoc(u"checkbox-form-control.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     // When exporting to XLSX:
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
@@ -2653,7 +2563,6 @@ void ScExportTest2::testButtonFormControlXlsxExport()
 {
     // Given a document that has a checkbox form control:
     ScDocShellRef xShell = loadDoc(u"button-form-control.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
 
     // When exporting to XLSX:
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xShell, FORMAT_XLSX);
@@ -2681,7 +2590,6 @@ void ScExportTest2::testTdf142929_filterLessThanXLSX()
 {
     // Document contains a standard filter with '<' condition.
     ScDocShellRef xDocSh = loadDoc(u"tdf142929.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
@@ -2697,7 +2605,6 @@ void ScExportTest2::testInvalidNamedRange()
     // Given a document which has a named range (myname) that refers to the 
"1" external link, but
     // the link's type is xlPathMissing, when importing that document:
     ScDocShellRef xDocSh = loadDoc(u"invalid-named-range.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     // Then make sure that named range is ignored, as "1" can't be resolved, 
and exporting it back
     // to XLSX (without the xlPathMissing link) would corrupt the document:
@@ -2712,7 +2619,6 @@ void ScExportTest2::testInvalidNamedRange()
 void ScExportTest2::testExternalDefinedNameXLSX()
 {
     ScDocShellRef xShell = loadDoc(u"tdf144397.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xShell.is());
     ScDocShellRef xDocSh = saveAndReload(*xShell, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
 
@@ -2772,7 +2678,6 @@ void ScExportTest2::testExternalDefinedNameXLSX()
 void ScExportTest2::testHyperlinkLocationXLSX()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf143220.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
 
     xmlDocUniquePtr pDoc = XPathHelper::parseExport2(*this, *xDocSh, 
m_xSFactory,
                                                      
"xl/worksheets/sheet1.xml", FORMAT_XLSX);
@@ -2796,7 +2701,6 @@ void ScExportTest2::testTdf142264ManyChartsToXLSX()
                         
officecfg::Office::Common::Cache::DrawingEngine::OLE_Objects::get());
 
     ScDocShellRef xDocSh = loadDoc(u"many_charts.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     xDocSh = saveAndReload(*xDocSh, FORMAT_XLSX);
     CPPUNIT_ASSERT(xDocSh.is());
 
@@ -2872,7 +2776,6 @@ void ScExportTest2::testTdf142264ManyChartsToXLSX()
 void ScExportTest2::testTdf143929MultiColumnToODS()
 {
     ScDocShellRef xDocSh = loadDoc(u"two-col-shape.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh);
 
     {
         css::uno::Reference<css::drawing::XDrawPagesSupplier> 
xSupplier(xDocSh->GetModel(),
@@ -2943,7 +2846,6 @@ void ScExportTest2::testTdf143929MultiColumnToODS()
 void ScExportTest2::testTdf142578()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf142578.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh);
 
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
     xmlDocUniquePtr pSheet
@@ -2981,7 +2883,6 @@ void ScExportTest2::testTdf142578()
 void ScExportTest2::testTdf145059()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf145059.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh);
 
     // Export to xlsx.
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
@@ -3009,7 +2910,6 @@ void ScExportTest2::testTdf145059()
 void ScExportTest2::testTdf130104_XLSXIndent()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf130104_indent.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh);
 
     // Resave the xlsx file without any modification.
     std::shared_ptr<utl::TempFile> pXPathFile = 
ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
@@ -3097,7 +2997,6 @@ void ScExportTest2::testTdf130104_XLSXIndent()
 void ScExportTest2::testWholeRowBold()
 {
     ScDocShellRef xDocSh1 = loadDoc(u"blank.", FORMAT_ODS);
-    CPPUNIT_ASSERT_MESSAGE("Failed to open empty doc", xDocSh1.is());
     ScDocument* pDoc = &xDocSh1->GetDocument();
 
     // Make entire second row bold.
@@ -3128,7 +3027,6 @@ void ScExportTest2::testWholeRowBold()
 void ScExportTest2::testXlsxRowsOrder()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf58243.", FORMAT_XLSX);
-    CPPUNIT_ASSERT(xDocSh.is());
     // Make sure code in SheetDataBuffer doesn't assert columns/rows sorting.
     ScBootstrapFixture::exportTo(*xDocSh, FORMAT_XLSX);
     xDocSh->DoClose();
@@ -3137,7 +3035,6 @@ void ScExportTest2::testXlsxRowsOrder()
 void ScExportTest2::testTdf91286()
 {
     ScDocShellRef xDocSh = loadDoc(u"tdf91286.", FORMAT_ODS);
-    CPPUNIT_ASSERT(xDocSh.is());
     std::shared_ptr<utl::TempFile> pTemp = exportTo(*xDocSh, FORMAT_XLSX);
     xDocSh->DoClose();
 
diff --git a/sc/qa/unit/subsequent_filters_test.cxx 
b/sc/qa/unit/subsequent_filters_test.cxx
index f0a749e810dd..d2cdaee3da54 100644

... etc. - the rest is truncated

Reply via email to