diff options
Diffstat (limited to 'dbaccess/qa/extras/empty-stdlib-save.cxx')
-rw-r--r-- | dbaccess/qa/extras/empty-stdlib-save.cxx | 28 |
1 files changed, 6 insertions, 22 deletions
diff --git a/dbaccess/qa/extras/empty-stdlib-save.cxx b/dbaccess/qa/extras/empty-stdlib-save.cxx index aa7dd293004f..5c3b5d028d8c 100644 --- a/dbaccess/qa/extras/empty-stdlib-save.cxx +++ b/dbaccess/qa/extras/empty-stdlib-save.cxx @@ -9,16 +9,12 @@ #include <sal/config.h> #include <test/unoapi_test.hxx> -#include <com/sun/star/lang/XComponent.hpp> #include <com/sun/star/frame/XStorable.hpp> #include <com/sun/star/document/XEmbeddedScripts.hpp> #include <com/sun/star/script/XStorageBasedLibraryContainer.hpp> #include <com/sun/star/script/XLibraryContainer.hpp> -#include <com/sun/star/util/XCloseable.hpp> #include <com/sun/star/container/XHierarchicalNameAccess.hpp> -#include <vcl/svapp.hxx> - using namespace ::com::sun::star; using namespace ::com::sun::star::uno; @@ -47,25 +43,18 @@ DialogSaveTest::DialogSaveTest() void DialogSaveTest::test() { - // UnoApiTest::setUp (via InitVCL) puts each test under a locked SolarMutex, - // but at least the below xDocCloseable->close call could lead to a deadlock - // then, and it looks like none of the code here requires the SolarMutex to - // be locked anyway: - SolarMutexReleaser rel; - const OUString aFileName(m_directories.getURLFromWorkdir(u"CppunitTest/testEmptyStdlibSave.odb")); { - uno::Reference< lang::XComponent > xComponent = loadFromDesktop(aFileName); - CPPUNIT_ASSERT(xComponent.is()); - uno::Reference< frame::XStorable > xDocStorable(xComponent, UNO_QUERY_THROW); - uno::Reference< document::XEmbeddedScripts > xDocScr(xComponent, UNO_QUERY_THROW); + mxComponent = loadFromDesktop(aFileName); + uno::Reference< frame::XStorable > xDocStorable(mxComponent, UNO_QUERY_THROW); + uno::Reference< document::XEmbeddedScripts > xDocScr(mxComponent, UNO_QUERY_THROW); uno::Reference< script::XStorageBasedLibraryContainer > xStorBasLib(xDocScr->getBasicLibraries()); CPPUNIT_ASSERT(xStorBasLib.is()); uno::Reference< script::XLibraryContainer > xBasLib(xStorBasLib, UNO_QUERY_THROW); uno::Reference< script::XStorageBasedLibraryContainer > xStorDlgLib(xDocScr->getDialogLibraries()); CPPUNIT_ASSERT(xStorDlgLib.is()); uno::Reference< script::XLibraryContainer > xDlgLib(xStorDlgLib, UNO_QUERY_THROW); - const OUString sStandard("Standard"); + static constexpr OUString sStandard(u"Standard"_ustr); xBasLib->loadLibrary(sStandard); xDlgLib->loadLibrary(sStandard); CPPUNIT_ASSERT(xBasLib->isLibraryLoaded(sStandard)); @@ -90,7 +79,7 @@ void DialogSaveTest::test() // uno::Reference< util::XModifiable > xScrMod(xDocScr, UNO_QUERY_THROW); // xScrMod->setModified(sal_True); - // uno::Reference< util::XModifiable > xDocMod(xComponent, UNO_QUERY_THROW); + // uno::Reference< util::XModifiable > xDocMod(mxComponent, UNO_QUERY_THROW); // std::cerr << "** Modified: " << static_cast<bool>(xDocMod->isModified()) << std::endl; // xDocMod->setModified(sal_True); // std::cerr << "** Modified: " << static_cast<bool>(xDocMod->isModified()) << std::endl; @@ -100,15 +89,10 @@ void DialogSaveTest::test() // do *not* change to store(As|To|URL)! xDocStorable->store(); - // close - uno::Reference< util::XCloseable > xDocCloseable(xComponent, UNO_QUERY_THROW); - xDocCloseable->close(false); - // All our uno::References are (should?) be invalid now -> let them go out of scope } { - uno::Sequence<uno::Any> args(1); - args[0] <<= aFileName; + uno::Sequence<uno::Any> args{ uno::Any(aFileName) }; Reference<container::XHierarchicalNameAccess> xHNA(getMultiServiceFactory()->createInstanceWithArguments("com.sun.star.packages.Package", args), UNO_QUERY_THROW); CPPUNIT_ASSERT(!xHNA->hasByHierarchicalName("Basic/Standard")); CPPUNIT_ASSERT(!xHNA->hasByHierarchicalName("Dialogs/Standard")); |