sc/qa/unit/filters-test.cxx | 6 +++--- sc/qa/unit/functions_addin.cxx | 2 +- sc/qa/unit/functions_array.cxx | 2 +- sc/qa/unit/functions_database.cxx | 2 +- sc/qa/unit/functions_datetime.cxx | 2 +- sc/qa/unit/functions_financial.cxx | 2 +- sc/qa/unit/functions_information.cxx | 2 +- sc/qa/unit/functions_logical.cxx | 2 +- sc/qa/unit/functions_mathematical.cxx | 2 +- sc/qa/unit/functions_old.cxx | 2 +- sc/qa/unit/functions_spreadsheet.cxx | 2 +- sc/qa/unit/functions_statistical.cxx | 2 +- sc/qa/unit/functions_test.cxx | 4 ++-- sc/qa/unit/functions_test.hxx | 2 +- sc/qa/unit/functions_text.cxx | 2 +- sc/qa/unit/helper/qahelper.cxx | 4 ---- sc/qa/unit/helper/qahelper.hxx | 5 ----- 17 files changed, 18 insertions(+), 27 deletions(-)
New commits: commit e09823b549699a9f32b1b0d7621301117306cdc3 Author: Xisco Fauli <xiscofa...@libreoffice.org> AuthorDate: Thu Nov 24 13:31:16 2022 +0100 Commit: Xisco Fauli <xiscofa...@libreoffice.org> CommitDate: Thu Nov 24 15:39:29 2022 +0100 ScBootstrapFixture: no need to pass basestring anymore Change-Id: I58ce96af149eb2b7f4d357383b73ec4b1d1b51bb Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143224 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org> diff --git a/sc/qa/unit/filters-test.cxx b/sc/qa/unit/filters-test.cxx index 596be632595d..fcfbc786f334 100644 --- a/sc/qa/unit/filters-test.cxx +++ b/sc/qa/unit/filters-test.cxx @@ -69,11 +69,11 @@ private: void ScFiltersTest::createFileURL( std::u16string_view aFileBase, std::u16string_view aFileExtension, OUString& rFilePath) { - // m_aBaseString and aFileBase may contain multiple segments, so use + // aFileBase may contain multiple segments, so use // GetNewAbsURL instead of insertName for them: INetURLObject url(m_directories.getSrcRootURL()); url.setFinalSlash(); - url.GetNewAbsURL(m_aBaseString, &url); + url.GetNewAbsURL("sc/qa/unit/data", &url); url.insertName(aFileExtension, true); url.GetNewAbsURL(OUString::Concat(aFileBase) + aFileExtension, &url); rFilePath = url.GetMainURL(INetURLObject::DecodeMechanism::NONE); @@ -155,7 +155,7 @@ void ScFiltersTest::testTooManyColsRows() } ScFiltersTest::ScFiltersTest() - : ScBootstrapFixture( "sc/qa/unit/data" ) + : ScBootstrapFixture() { } diff --git a/sc/qa/unit/functions_addin.cxx b/sc/qa/unit/functions_addin.cxx index ce549a400014..4c9189de235d 100644 --- a/sc/qa/unit/functions_addin.cxx +++ b/sc/qa/unit/functions_addin.cxx @@ -22,7 +22,7 @@ void AddinFunctionsTest::testAddinFormulasFODS() } AddinFunctionsTest::AddinFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/addin/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_array.cxx b/sc/qa/unit/functions_array.cxx index 88a4b70d7c29..7584c4e0b309 100644 --- a/sc/qa/unit/functions_array.cxx +++ b/sc/qa/unit/functions_array.cxx @@ -38,7 +38,7 @@ void ArrayFunctionsTest::testDubiousArrayFormulasFODS() } ArrayFunctionsTest::ArrayFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/array/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_database.cxx b/sc/qa/unit/functions_database.cxx index 933d3dabd976..7bed3b5c956a 100644 --- a/sc/qa/unit/functions_database.cxx +++ b/sc/qa/unit/functions_database.cxx @@ -22,7 +22,7 @@ void DatabaseFunctionsTest::testDatabaseFormulasFODS() } DatabaseFunctionsTest::DatabaseFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/database/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_datetime.cxx b/sc/qa/unit/functions_datetime.cxx index e0c1a304956f..ba7f7ce2c56f 100644 --- a/sc/qa/unit/functions_datetime.cxx +++ b/sc/qa/unit/functions_datetime.cxx @@ -22,7 +22,7 @@ void DateTimeFunctionsTest::testDateTimeFormulasFODS() } DateTimeFunctionsTest::DateTimeFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/date_time/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_financial.cxx b/sc/qa/unit/functions_financial.cxx index 9188e75e0a05..b83b4cd667e6 100644 --- a/sc/qa/unit/functions_financial.cxx +++ b/sc/qa/unit/functions_financial.cxx @@ -22,7 +22,7 @@ void FinancialFunctionsTest::testFinancialFormulasFODS() } FinancialFunctionsTest::FinancialFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/financial/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_information.cxx b/sc/qa/unit/functions_information.cxx index 58aee20bbbea..9d6de422ec5b 100644 --- a/sc/qa/unit/functions_information.cxx +++ b/sc/qa/unit/functions_information.cxx @@ -22,7 +22,7 @@ void InformationFunctionsTest::testInformationFormulasFODS() } InformationFunctionsTest::InformationFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/information/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_logical.cxx b/sc/qa/unit/functions_logical.cxx index 15a4db6ed1fd..c501867a0e2c 100644 --- a/sc/qa/unit/functions_logical.cxx +++ b/sc/qa/unit/functions_logical.cxx @@ -22,7 +22,7 @@ void LogicalFunctionsTest::testLogicalFormulasFODS() } LogicalFunctionsTest::LogicalFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/logical/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_mathematical.cxx b/sc/qa/unit/functions_mathematical.cxx index 274a04db523e..9eb940df892c 100644 --- a/sc/qa/unit/functions_mathematical.cxx +++ b/sc/qa/unit/functions_mathematical.cxx @@ -22,7 +22,7 @@ void MathematicalFunctionsTest::testMathematicalFormulasFODS() } MathematicalFunctionsTest::MathematicalFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/mathematical/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_old.cxx b/sc/qa/unit/functions_old.cxx index e742f6b972fa..0a3663f988d9 100644 --- a/sc/qa/unit/functions_old.cxx +++ b/sc/qa/unit/functions_old.cxx @@ -14,7 +14,7 @@ public: }; FunctionsTestOld::FunctionsTestOld(): - FunctionsTest("sc/qq/unit/data/functions/fods") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_spreadsheet.cxx b/sc/qa/unit/functions_spreadsheet.cxx index 03c87732b604..865134fc31c1 100644 --- a/sc/qa/unit/functions_spreadsheet.cxx +++ b/sc/qa/unit/functions_spreadsheet.cxx @@ -22,7 +22,7 @@ void SpreadsheetFunctionsTest::testSpreadsheetFormulasFODS() } SpreadsheetFunctionsTest::SpreadsheetFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/spreadsheet/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_statistical.cxx b/sc/qa/unit/functions_statistical.cxx index 4d97d4cc1689..8f74aad999fb 100644 --- a/sc/qa/unit/functions_statistical.cxx +++ b/sc/qa/unit/functions_statistical.cxx @@ -22,7 +22,7 @@ void StatisticalFunctionsTest::testStatisticalFormulasFODS() } StatisticalFunctionsTest::StatisticalFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/statistical/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/functions_test.cxx b/sc/qa/unit/functions_test.cxx index 02c1cf1da4f4..6bc36540c2a4 100644 --- a/sc/qa/unit/functions_test.cxx +++ b/sc/qa/unit/functions_test.cxx @@ -13,8 +13,8 @@ #include <rtl/math.hxx> -FunctionsTest::FunctionsTest(const OUString& rPath): - ScBootstrapFixture(rPath) +FunctionsTest::FunctionsTest(): + ScBootstrapFixture() { } diff --git a/sc/qa/unit/functions_test.hxx b/sc/qa/unit/functions_test.hxx index 0a57987dfa01..52672bb8177b 100644 --- a/sc/qa/unit/functions_test.hxx +++ b/sc/qa/unit/functions_test.hxx @@ -19,7 +19,7 @@ class SCQAHELPER_DLLPUBLIC FunctionsTest : public ScBootstrapFixture, public tes { public: - FunctionsTest(const OUString& rPath); + FunctionsTest(); virtual bool load( const OUString &rFilter, diff --git a/sc/qa/unit/functions_text.cxx b/sc/qa/unit/functions_text.cxx index 96225593db05..a9bfb8fefbab 100644 --- a/sc/qa/unit/functions_text.cxx +++ b/sc/qa/unit/functions_text.cxx @@ -22,7 +22,7 @@ void TextFunctionsTest::testTextFormulasFODS() } TextFunctionsTest::TextFunctionsTest(): - FunctionsTest("sc/qa/unit/data/functions/text/fods/") + FunctionsTest() { } diff --git a/sc/qa/unit/helper/qahelper.cxx b/sc/qa/unit/helper/qahelper.cxx index cbeaf9e8ac90..bb263fcc9a44 100644 --- a/sc/qa/unit/helper/qahelper.cxx +++ b/sc/qa/unit/helper/qahelper.cxx @@ -21,7 +21,6 @@ #include <svx/svdpage.hxx> #include <svx/svdoole2.hxx> #include <tools/UnitConversion.hxx> -#include <tools/urlobj.hxx> #include <editeng/brushitem.hxx> #include <editeng/justifyitem.hxx> #include <formula/errorcodes.hxx> @@ -499,9 +498,6 @@ ScDocShellRef ScBootstrapFixture::load( return xDocShRef; } -ScBootstrapFixture::ScBootstrapFixture( const OUString& rsBaseString ) : m_aBaseString( rsBaseString ) {} -ScBootstrapFixture::~ScBootstrapFixture() {} - void ScBootstrapFixture::setUp() { test::BootstrapFixture::setUp(); diff --git a/sc/qa/unit/helper/qahelper.hxx b/sc/qa/unit/helper/qahelper.hxx index 139c4ff6e89b..3eac1c2c3a86 100644 --- a/sc/qa/unit/helper/qahelper.hxx +++ b/sc/qa/unit/helper/qahelper.hxx @@ -127,8 +127,6 @@ public: class SCQAHELPER_DLLPUBLIC ScBootstrapFixture : public test::BootstrapFixture { protected: - OUString m_aBaseString; - ScDocShellRef load( const OUString& rURL, const OUString& rFilter, const OUString &rUserData, const OUString& rTypeName, SfxFilterFlags nFilterFlags, SotClipboardFormatId nClipboardID, @@ -137,9 +135,6 @@ private: // reference to document interface that we are testing css::uno::Reference<css::uno::XInterface> m_xCalcComponent; public: - explicit ScBootstrapFixture( const OUString& rsBaseString ); - virtual ~ScBootstrapFixture() override; - virtual void setUp() override; virtual void tearDown() override; };