diff options
Diffstat (limited to 'dbaccess/qa/unit')
-rw-r--r-- | dbaccess/qa/unit/data/firebird_empty_le.odb | bin | 1733 -> 0 bytes | |||
-rw-r--r-- | dbaccess/qa/unit/data/firebird_integer_le.odb | bin | 43990 -> 0 bytes | |||
-rw-r--r-- | dbaccess/qa/unit/data/firebird_integer_le_ods12.odb | bin | 75701 -> 0 bytes | |||
-rw-r--r-- | dbaccess/qa/unit/data/tdf132924.odb | bin | 0 -> 3148 bytes | |||
-rw-r--r-- | dbaccess/qa/unit/dbtest_base.cxx | 36 | ||||
-rw-r--r-- | dbaccess/qa/unit/embeddeddb_performancetest.cxx | 49 | ||||
-rw-r--r-- | dbaccess/qa/unit/firebird-regression.cxx | 106 | ||||
-rw-r--r-- | dbaccess/qa/unit/firebird.cxx | 45 | ||||
-rw-r--r-- | dbaccess/qa/unit/hsql_binary_import.cxx | 6 | ||||
-rw-r--r-- | dbaccess/qa/unit/hsqldb.cxx | 6 | ||||
-rw-r--r-- | dbaccess/qa/unit/tdf119625.cxx | 6 | ||||
-rw-r--r-- | dbaccess/qa/unit/tdf126268.cxx | 6 |
12 files changed, 76 insertions, 184 deletions
diff --git a/dbaccess/qa/unit/data/firebird_empty_le.odb b/dbaccess/qa/unit/data/firebird_empty_le.odb Binary files differdeleted file mode 100644 index 766c17db2ec9..000000000000 --- a/dbaccess/qa/unit/data/firebird_empty_le.odb +++ /dev/null diff --git a/dbaccess/qa/unit/data/firebird_integer_le.odb b/dbaccess/qa/unit/data/firebird_integer_le.odb Binary files differdeleted file mode 100644 index da2ec1499198..000000000000 --- a/dbaccess/qa/unit/data/firebird_integer_le.odb +++ /dev/null diff --git a/dbaccess/qa/unit/data/firebird_integer_le_ods12.odb b/dbaccess/qa/unit/data/firebird_integer_le_ods12.odb Binary files differdeleted file mode 100644 index 95691ed20585..000000000000 --- a/dbaccess/qa/unit/data/firebird_integer_le_ods12.odb +++ /dev/null diff --git a/dbaccess/qa/unit/data/tdf132924.odb b/dbaccess/qa/unit/data/tdf132924.odb Binary files differnew file mode 100644 index 000000000000..8cee7bcbabfd --- /dev/null +++ b/dbaccess/qa/unit/data/tdf132924.odb diff --git a/dbaccess/qa/unit/dbtest_base.cxx b/dbaccess/qa/unit/dbtest_base.cxx index caee25fa70ac..88da4b469742 100644 --- a/dbaccess/qa/unit/dbtest_base.cxx +++ b/dbaccess/qa/unit/dbtest_base.cxx @@ -30,11 +30,6 @@ class DBTestBase public: DBTestBase() : UnoApiTest("dbaccess/qa/unit/data") {}; - utl::TempFile createTempCopy(std::u16string_view pathname); - - uno::Reference< XOfficeDatabaseDocument > - getDocumentForFileName(std::u16string_view sFileName); - uno::Reference<XOfficeDatabaseDocument> getDocumentForUrl(OUString const & url); uno::Reference< XConnection > @@ -42,36 +37,9 @@ public: uno::Reference< XOfficeDatabaseDocument > const & xDocument); }; -utl::TempFile DBTestBase::createTempCopy(std::u16string_view pathname) { - OUString url; - createFileURL(pathname, url); - utl::TempFile tmp; - tmp.EnableKillingFile(); - auto const e = osl::File::copy(url, tmp.GetURL()); - CPPUNIT_ASSERT_EQUAL_MESSAGE( - (OString( - "<" + OUStringToOString(url, RTL_TEXTENCODING_UTF8) + "> -> <" - + OUStringToOString(tmp.GetURL(), RTL_TEXTENCODING_UTF8) + ">") - .getStr()), - osl::FileBase::E_None, e); - return tmp; -} - -uno::Reference< XOfficeDatabaseDocument > - DBTestBase::getDocumentForFileName(std::u16string_view sFileName) -{ - OUString sFilePath; - createFileURL(sFileName, sFilePath); - return getDocumentForUrl(sFilePath); -} - uno::Reference<XOfficeDatabaseDocument> DBTestBase::getDocumentForUrl(OUString const & url) { - uno::Reference< lang::XComponent > xComponent (loadFromDesktop(url)); - CPPUNIT_ASSERT(xComponent.is()); - - uno::Reference< XOfficeDatabaseDocument > xDocument(xComponent, UNO_QUERY); - CPPUNIT_ASSERT(xDocument.is()); - + mxComponent = loadFromDesktop(url); + uno::Reference< XOfficeDatabaseDocument > xDocument(mxComponent, UNO_QUERY_THROW); return xDocument; } diff --git a/dbaccess/qa/unit/embeddeddb_performancetest.cxx b/dbaccess/qa/unit/embeddeddb_performancetest.cxx index e4119804d3b1..184ef0831ad1 100644 --- a/dbaccess/qa/unit/embeddeddb_performancetest.cxx +++ b/dbaccess/qa/unit/embeddeddb_performancetest.cxx @@ -82,7 +82,7 @@ class EmbeddedDBPerformanceTest : public DBTestBase { private: - static constexpr OUStringLiteral our_sEnableTestEnvVar = u"DBA_PERFTEST"; + static constexpr OUString our_sEnableTestEnvVar = u"DBA_PERFTEST"_ustr; // We store the results and print them at the end due to the amount of warning @@ -92,7 +92,7 @@ private: void printTimes(const TimeValue* pTime1, const TimeValue* pTime2, const TimeValue* pTime3); void doPerformanceTestOnODB(const OUString& rDriverURL, - const OUString& rDBName, + std::u16string_view rDBName, const bool bUsePreparedStatement); void setupTestTable(uno::Reference< XConnection > const & xConnection); @@ -102,13 +102,13 @@ private: // Individual Tests void performPreparedStatementInsertTest( uno::Reference< XConnection > const & xConnection, - const OUString& rDBName); + std::u16string_view rDBName); void performStatementInsertTest( uno::Reference< XConnection > const & xConnection, - const OUString& rDBName); + std::u16string_view rDBName); void performReadTest( uno::Reference< XConnection > const & xConnection, - const OUString& rDBName); + std::u16string_view rDBName); // Perform all tests on a given DB. void testFirebird(); @@ -124,8 +124,7 @@ public: SvFileStream* EmbeddedDBPerformanceTest::getWordListStream() { - OUString wlPath; - createFileURL(u"wordlist", wlPath); + OUString wlPath = createFileURL(u"wordlist"); return new SvFileStream(wlPath, StreamMode::READ); } @@ -134,11 +133,11 @@ void EmbeddedDBPerformanceTest::printTimes( const TimeValue* pTime2, const TimeValue* pTime3) { - m_aOutputBuffer - .append(getPrintableTimeValue(pTime1)).append("\t") - .append(getPrintableTimeValue(pTime2)).append("\t") - .append(getPrintableTimeValue(pTime3)).append("\t") - .append("\n"); + m_aOutputBuffer.append( + getPrintableTimeValue(pTime1) + "\t" + + getPrintableTimeValue(pTime2) + "\t" + + getPrintableTimeValue(pTime3) + "\t" + "\n"); } // TODO: we probably should create a document from scratch instead? @@ -146,7 +145,7 @@ void EmbeddedDBPerformanceTest::printTimes( void EmbeddedDBPerformanceTest::testPerformance() { OUString sEnabled; - osl_getEnvironment(OUString(our_sEnableTestEnvVar).pData, &sEnabled.pData); + osl_getEnvironment(our_sEnableTestEnvVar.pData, &sEnabled.pData); if (sEnabled.isEmpty()) return; @@ -173,17 +172,17 @@ void EmbeddedDBPerformanceTest::testFirebird() { m_aOutputBuffer.append("Standard Insert\n"); - doPerformanceTestOnODB("sdbc:embedded:firebird", "Firebird", false); + doPerformanceTestOnODB("sdbc:embedded:firebird", u"Firebird", false); m_aOutputBuffer.append("PreparedStatement Insert\n"); - doPerformanceTestOnODB("sdbc:embedded:firebird", "Firebird", true); + doPerformanceTestOnODB("sdbc:embedded:firebird", u"Firebird", true); } void EmbeddedDBPerformanceTest::testHSQLDB() { m_aOutputBuffer.append("Standard Insert\n"); - doPerformanceTestOnODB("sdbc:embedded:hsqldb", "HSQLDB", false); + doPerformanceTestOnODB("sdbc:embedded:hsqldb", u"HSQLDB", false); m_aOutputBuffer.append("PreparedStatement Insert\n"); - doPerformanceTestOnODB("sdbc:embedded:hsqldb", "HSQLDB", true); + doPerformanceTestOnODB("sdbc:embedded:hsqldb", u"HSQLDB", true); } /** @@ -192,10 +191,10 @@ void EmbeddedDBPerformanceTest::testHSQLDB() */ void EmbeddedDBPerformanceTest::doPerformanceTestOnODB( const OUString& rDriverURL, - const OUString& rDBName, + std::u16string_view rDBName, const bool bUsePreparedStatement) { - ::utl::TempFile aFile; + ::utl::TempFileNamed aFile; aFile.EnableKillingFile(); { @@ -244,7 +243,7 @@ void EmbeddedDBPerformanceTest::setupTestTable( void EmbeddedDBPerformanceTest::performPreparedStatementInsertTest( uno::Reference< XConnection > const & xConnection, - const OUString& rDBName) + std::u16string_view rDBName) { uno::Reference< XPreparedStatement > xPreparedStatement = xConnection->prepareStatement( @@ -278,7 +277,7 @@ void EmbeddedDBPerformanceTest::performPreparedStatementInsertTest( getTimeDifference(&aStart, &aMiddle, &aTimeInsert); getTimeDifference(&aMiddle, &aEnd, &aTimeCommit); getTimeDifference(&aStart, &aEnd, &aTimeTotal); - m_aOutputBuffer.append("Insert: ").append(rDBName).append("\n"); + m_aOutputBuffer.append(OUString::Concat("Insert: ") + rDBName + "\n"); printTimes(&aTimeInsert, &aTimeCommit, &aTimeTotal); pFile->Close(); @@ -286,7 +285,7 @@ void EmbeddedDBPerformanceTest::performPreparedStatementInsertTest( void EmbeddedDBPerformanceTest::performStatementInsertTest( uno::Reference< XConnection > const & xConnection, - const OUString& rDBName) + std::u16string_view rDBName) { uno::Reference< XStatement > xStatement = xConnection->createStatement(); @@ -316,7 +315,7 @@ void EmbeddedDBPerformanceTest::performStatementInsertTest( getTimeDifference(&aStart, &aMiddle, &aTimeInsert); getTimeDifference(&aMiddle, &aEnd, &aTimeCommit); getTimeDifference(&aStart, &aEnd, &aTimeTotal); - m_aOutputBuffer.append("Insert: ").append(rDBName).append("\n"); + m_aOutputBuffer.append(OUString::Concat("Insert: ") + rDBName + "\n"); printTimes(&aTimeInsert, &aTimeCommit, &aTimeTotal); pFile->Close(); @@ -324,7 +323,7 @@ void EmbeddedDBPerformanceTest::performStatementInsertTest( void EmbeddedDBPerformanceTest::performReadTest( uno::Reference< XConnection > const & xConnection, - const OUString& rDBName) + std::u16string_view rDBName) { uno::Reference< XStatement > xStatement = xConnection->createStatement(); @@ -347,7 +346,7 @@ void EmbeddedDBPerformanceTest::performReadTest( getTimeDifference(&aStart, &aMiddle, &aTimeSelect); getTimeDifference(&aMiddle, &aEnd, &aTimeIterate); getTimeDifference(&aStart, &aEnd, &aTimeTotal); - m_aOutputBuffer.append("Read from: ").append(rDBName).append("\n"); + m_aOutputBuffer.append(OUString::Concat("Read from: ") + rDBName + "\n"); printTimes(&aTimeSelect, &aTimeIterate, &aTimeTotal); } diff --git a/dbaccess/qa/unit/firebird-regression.cxx b/dbaccess/qa/unit/firebird-regression.cxx deleted file mode 100644 index d5c53b93a64d..000000000000 --- a/dbaccess/qa/unit/firebird-regression.cxx +++ /dev/null @@ -1,106 +0,0 @@ -/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */ -/* - * This file is part of the LibreOffice project. - * - * This Source Code Form is subject to the terms of the Mozilla Public - * License, v. 2.0. If a copy of the MPL was not distributed with this - * file, You can obtain one at http://mozilla.org/MPL/2.0/. - */ - -#include "dbtest_base.cxx" - -#include <com/sun/star/sdb/XOfficeDatabaseDocument.hpp> -#include <com/sun/star/sdbc/XColumnLocate.hpp> -#include <com/sun/star/sdbc/XConnection.hpp> -#include <com/sun/star/sdbc/XResultSet.hpp> -#include <com/sun/star/sdbc/XRow.hpp> -#include <com/sun/star/sdbc/XStatement.hpp> - -using namespace ::com::sun::star; -using namespace ::com::sun::star::sdb; -using namespace ::com::sun::star::sdbc; -using namespace ::com::sun::star::uno; - -class FirebirdTest - : public DBTestBase -{ -public: - void testEmptyDBConnection(); - void testIntegerDatabase(); - - CPPUNIT_TEST_SUITE(FirebirdTest); - CPPUNIT_TEST(testEmptyDBConnection); - CPPUNIT_TEST(testIntegerDatabase); - CPPUNIT_TEST_SUITE_END(); -}; - -/** - * Test the loading of an "empty" file, i.e. the embedded database has not yet - * been initialised (as occurs when a new .odb is created and opened by base). - */ -void FirebirdTest::testEmptyDBConnection() -{ -#ifdef OSL_BIGENDIAN - auto const tmp = createTempCopy("firebird_empty_be.odb"); -#else - auto const tmp = createTempCopy(u"firebird_empty_le.odb"); -#endif - uno::Reference< XOfficeDatabaseDocument > xDocument = - getDocumentForUrl(tmp.GetURL()); - - getConnectionForDocument(xDocument); - - closeDocument(uno::Reference<lang::XComponent>(xDocument, uno::UNO_QUERY)); -} - -/** - * Test reading of integers from a known .odb to verify that the data - * can still be read on all systems. - */ -void FirebirdTest::testIntegerDatabase() -{ -#ifdef OSL_BIGENDIAN - uno::Reference< XOfficeDatabaseDocument > xDocument = - getDocumentForFileName("firebird_integer_be_ods12.odb"); -#else - uno::Reference< XOfficeDatabaseDocument > xDocument = - getDocumentForFileName(u"firebird_integer_le_ods12.odb"); -#endif - - uno::Reference< XConnection > xConnection = - getConnectionForDocument(xDocument); - - uno::Reference< XStatement > xStatement = xConnection->createStatement(); - CPPUNIT_ASSERT(xStatement.is()); - - uno::Reference< XResultSet > xResultSet = xStatement->executeQuery( - "SELECT * FROM TESTTABLE"); - CPPUNIT_ASSERT(xResultSet.is()); - CPPUNIT_ASSERT(xResultSet->next()); - - uno::Reference< XRow > xRow(xResultSet, UNO_QUERY); - CPPUNIT_ASSERT(xRow.is()); - uno::Reference< XColumnLocate > xColumnLocate(xRow, UNO_QUERY); - CPPUNIT_ASSERT(xColumnLocate.is()); - - CPPUNIT_ASSERT_EQUAL(sal_Int16(-30000), - xRow->getShort(xColumnLocate->findColumn("_SMALLINT"))); - CPPUNIT_ASSERT_EQUAL(sal_Int32(-2100000000), - xRow->getInt(xColumnLocate->findColumn("_INT"))); - CPPUNIT_ASSERT_EQUAL(SAL_CONST_INT64(-9000000000000000000), - xRow->getLong(xColumnLocate->findColumn("_BIGINT"))); - CPPUNIT_ASSERT_EQUAL(OUString("5"), - xRow->getString(xColumnLocate->findColumn("_CHAR"))); - CPPUNIT_ASSERT_EQUAL(OUString("5"), - xRow->getString(xColumnLocate->findColumn("_VARCHAR"))); - - CPPUNIT_ASSERT(!xResultSet->next()); // Should only be one row - - closeDocument(uno::Reference<lang::XComponent>(xDocument, uno::UNO_QUERY)); -} - -CPPUNIT_TEST_SUITE_REGISTRATION(FirebirdTest); - -CPPUNIT_PLUGIN_IMPLEMENT(); - -/* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/dbaccess/qa/unit/firebird.cxx b/dbaccess/qa/unit/firebird.cxx index e67a4d10c097..1b6b7172fdbd 100644 --- a/dbaccess/qa/unit/firebird.cxx +++ b/dbaccess/qa/unit/firebird.cxx @@ -15,6 +15,7 @@ #include <com/sun/star/sdbc/XResultSet.hpp> #include <com/sun/star/sdbc/XRow.hpp> #include <com/sun/star/sdbc/XStatement.hpp> +#include <com/sun/star/util/XCloseable.hpp> using namespace ::com::sun::star; using namespace ::com::sun::star::sdb; @@ -27,10 +28,12 @@ class FirebirdTest public: void testEmptyDBConnection(); void testIntegerDatabase(); + void testTdf132924(); CPPUNIT_TEST_SUITE(FirebirdTest); CPPUNIT_TEST(testEmptyDBConnection); CPPUNIT_TEST(testIntegerDatabase); + CPPUNIT_TEST(testTdf132924); CPPUNIT_TEST_SUITE_END(); }; @@ -40,13 +43,14 @@ public: */ void FirebirdTest::testEmptyDBConnection() { - auto const tmp = createTempCopy(u"firebird_empty.odb"); + createTempCopy(u"firebird_empty.odb"); uno::Reference< XOfficeDatabaseDocument > xDocument = - getDocumentForUrl(tmp.GetURL()); + getDocumentForUrl(maTempFile.GetURL()); getConnectionForDocument(xDocument); - closeDocument(uno::Reference<lang::XComponent>(xDocument, uno::UNO_QUERY)); + css::uno::Reference<util::XCloseable> xCloseable(mxComponent, css::uno::UNO_QUERY_THROW); + xCloseable->close(false); } /** @@ -55,8 +59,8 @@ void FirebirdTest::testEmptyDBConnection() */ void FirebirdTest::testIntegerDatabase() { - uno::Reference< XOfficeDatabaseDocument > xDocument = - getDocumentForFileName(u"firebird_integer_ods12.odb"); + loadFromFile(u"firebird_integer_ods12.odb"); + uno::Reference< XOfficeDatabaseDocument > xDocument(mxComponent, UNO_QUERY_THROW); uno::Reference< XConnection > xConnection = getConnectionForDocument(xDocument); @@ -87,7 +91,36 @@ void FirebirdTest::testIntegerDatabase() CPPUNIT_ASSERT(!xResultSet->next()); // Should only be one row - closeDocument(uno::Reference<lang::XComponent>(xDocument, uno::UNO_QUERY)); + css::uno::Reference<util::XCloseable> xCloseable(mxComponent, css::uno::UNO_QUERY_THROW); + xCloseable->close(false); +} + +void FirebirdTest::testTdf132924() +{ + loadFromFile(u"tdf132924.odb"); + uno::Reference< XOfficeDatabaseDocument > xDocument(mxComponent, UNO_QUERY_THROW); + uno::Reference<XConnection> xConnection = getConnectionForDocument(xDocument); + + uno::Reference<XStatement> xStatement = xConnection->createStatement(); + CPPUNIT_ASSERT(xStatement.is()); + + uno::Reference<XResultSet> xResultSet = xStatement->executeQuery("SELECT * FROM AliasTest"); + CPPUNIT_ASSERT(xResultSet.is()); + CPPUNIT_ASSERT(xResultSet->next()); + + uno::Reference<XRow> xRow(xResultSet, UNO_QUERY); + CPPUNIT_ASSERT(xRow.is()); + uno::Reference<XColumnLocate> xColumnLocate(xRow, UNO_QUERY); + CPPUNIT_ASSERT(xColumnLocate.is()); + + // Without the fix in place, this test would have failed with: + // - Expected: 1 + // - Actual : The column name 'TestId' is not valid + CPPUNIT_ASSERT_EQUAL(sal_Int16(1), xRow->getShort(xColumnLocate->findColumn("TestId"))); + CPPUNIT_ASSERT_EQUAL(OUString("TestName"), xRow->getString(xColumnLocate->findColumn("TestName"))); + + css::uno::Reference<util::XCloseable> xCloseable(mxComponent, css::uno::UNO_QUERY_THROW); + xCloseable->close(false); } CPPUNIT_TEST_SUITE_REGISTRATION(FirebirdTest); diff --git a/dbaccess/qa/unit/hsql_binary_import.cxx b/dbaccess/qa/unit/hsql_binary_import.cxx index 4eac0cdc4130..569463e7e4ef 100644 --- a/dbaccess/qa/unit/hsql_binary_import.cxx +++ b/dbaccess/qa/unit/hsql_binary_import.cxx @@ -46,8 +46,9 @@ void HsqlBinaryImportTest::testBinaryImport() } // the migration requires the file to be writable - utl::TempFile const temp(createTempCopy(u"hsqldb_migration_test.odb")); - uno::Reference<XOfficeDatabaseDocument> const xDocument = getDocumentForUrl(temp.GetURL()); + createTempCopy(u"hsqldb_migration_test.odb"); + uno::Reference<XOfficeDatabaseDocument> const xDocument + = getDocumentForUrl(maTempFile.GetURL()); uno::Reference<XConnection> xConnection = getConnectionForDocument(xDocument); // at this point migration is already done @@ -86,7 +87,6 @@ void HsqlBinaryImportTest::testBinaryImport() CPPUNIT_ASSERT_EQUAL(sal_uInt16{ 2 }, date.Month); CPPUNIT_ASSERT_EQUAL(sal_Int16{ 1998 }, date.Year); - closeDocument(uno::Reference<lang::XComponent>(xDocument, uno::UNO_QUERY)); if (!oldValue) { std::shared_ptr<comphelper::ConfigurationChanges> xChanges( diff --git a/dbaccess/qa/unit/hsqldb.cxx b/dbaccess/qa/unit/hsqldb.cxx index 5ee011af276e..eb553eac756e 100644 --- a/dbaccess/qa/unit/hsqldb.cxx +++ b/dbaccess/qa/unit/hsqldb.cxx @@ -32,12 +32,10 @@ public: */ void HSQLDBTest::testEmptyDBConnection() { - auto const file = createTempCopy(u"hsqldb_empty.odb"); - uno::Reference<XOfficeDatabaseDocument> xDocument = getDocumentForUrl(file.GetURL()); + createTempCopy(u"hsqldb_empty.odb"); + uno::Reference<XOfficeDatabaseDocument> xDocument = getDocumentForUrl(maTempFile.GetURL()); getConnectionForDocument(xDocument); - - css::uno::Reference<css::lang::XComponent>(xDocument, css::uno::UNO_QUERY_THROW)->dispose(); } CPPUNIT_TEST_SUITE_REGISTRATION(HSQLDBTest); diff --git a/dbaccess/qa/unit/tdf119625.cxx b/dbaccess/qa/unit/tdf119625.cxx index 22fb5849305e..ba0c7b2ce345 100644 --- a/dbaccess/qa/unit/tdf119625.cxx +++ b/dbaccess/qa/unit/tdf119625.cxx @@ -66,8 +66,9 @@ void Tdf119625Test::testTime() } // the migration requires the file to be writable - utl::TempFile const temp(createTempCopy(u"tdf119625.odb")); - uno::Reference<XOfficeDatabaseDocument> const xDocument = getDocumentForUrl(temp.GetURL()); + createTempCopy(u"tdf119625.odb"); + uno::Reference<XOfficeDatabaseDocument> const xDocument + = getDocumentForUrl(maTempFile.GetURL()); uno::Reference<XConnection> xConnection = getConnectionForDocument(xDocument); // at this point migration is already done @@ -106,7 +107,6 @@ void Tdf119625Test::testTime() } CPPUNIT_ASSERT(!xRes->next()); - closeDocument(uno::Reference<lang::XComponent>(xDocument, uno::UNO_QUERY)); if (!oldValue) { std::shared_ptr<comphelper::ConfigurationChanges> xChanges( diff --git a/dbaccess/qa/unit/tdf126268.cxx b/dbaccess/qa/unit/tdf126268.cxx index a4ed9dc3b868..c06fdead79c7 100644 --- a/dbaccess/qa/unit/tdf126268.cxx +++ b/dbaccess/qa/unit/tdf126268.cxx @@ -60,8 +60,9 @@ void Tdf126268Test::testNumbers() } // the migration requires the file to be writable - utl::TempFile const temp(createTempCopy(u"tdf126268.odb")); - uno::Reference<XOfficeDatabaseDocument> const xDocument = getDocumentForUrl(temp.GetURL()); + createTempCopy(u"tdf126268.odb"); + uno::Reference<XOfficeDatabaseDocument> const xDocument + = getDocumentForUrl(maTempFile.GetURL()); uno::Reference<XConnection> xConnection = getConnectionForDocument(xDocument); @@ -82,7 +83,6 @@ void Tdf126268Test::testNumbers() } CPPUNIT_ASSERT(!xRes->next()); - closeDocument(uno::Reference<lang::XComponent>(xDocument, uno::UNO_QUERY)); if (!oldValue) { std::shared_ptr<comphelper::ConfigurationChanges> xChanges( |