test/source/unoapi_test.cxx | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-)
New commits: commit a89f887d16443af53bfbec20158c65e65bf7f1f3 Author: Xisco Fauli <[email protected]> AuthorDate: Mon Feb 16 22:44:21 2026 +0100 Commit: Xisco Fauli <[email protected]> CommitDate: Tue Feb 17 08:34:57 2026 +0100 UnoApiTest: check no document is open before opening a new one Change-Id: I8abe65f88b63446dd8bfe2465c9a8d2d4aff31ea Reviewed-on: https://gerrit.libreoffice.org/c/core/+/199508 Reviewed-by: Xisco Fauli <[email protected]> Tested-by: Jenkins diff --git a/test/source/unoapi_test.cxx b/test/source/unoapi_test.cxx index 79eef61ab8a8..543fb9e898b7 100644 --- a/test/source/unoapi_test.cxx +++ b/test/source/unoapi_test.cxx @@ -58,10 +58,7 @@ void UnoApiTest::tearDown() } if (mxComponent.is()) - { - mxComponent->dispose(); - mxComponent.clear(); - } + dispose(); test::BootstrapFixture::tearDown(); } @@ -262,11 +259,7 @@ void UnoApiTest::loadFromURL(OUString const& rURL, if (meImportFilterName != TestFilter::NONE) aMediaDescriptor[u"FilterName"_ustr] <<= TestFilterNames.at(meImportFilterName); - if (mxComponent.is()) - { - mxComponent->dispose(); - mxComponent.clear(); - } + CPPUNIT_ASSERT_MESSAGE("A document is already open!", !mxComponent.is()); mxComponent = loadFromDesktop(rURL, OUString(), aMediaDescriptor.getAsConstPropertyValueList()); CPPUNIT_ASSERT(mxComponent); @@ -359,6 +352,7 @@ void UnoApiTest::saveAndReload(TestFilter eFilter, const char* pPassword) { save(eFilter, rParams, pPassword); + dispose(); loadFromURL(maTempFile.GetURL(), rParams, pPassword); }
