diff options
author | Tomaž Vajngerl <tomaz.vajngerl@collabora.co.uk> | 2023-10-11 22:41:02 +0900 |
---|---|---|
committer | Tomaž Vajngerl <quikee@gmail.com> | 2023-10-12 05:08:27 +0200 |
commit | 4fd6b125751897923428523336a768310a58b36e (patch) | |
tree | 4d4adcc4f469ba30550fc499dcb718b6565c07bc | |
parent | e3d09d2e525ef6d5b93c6646e3cadc3f01297ea0 (diff) |
editeng: move the methods into Test class definition
This makes makes all the Test class methods contined in the
class definition body, so more readable and less noise for the
actual test definitions.
Change-Id: I3ea177f27aa868a396e537b1682ce91d93c1208e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/157837
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
-rw-r--r-- | editeng/qa/unit/core-test.cxx | 33 |
1 files changed, 13 insertions, 20 deletions
diff --git a/editeng/qa/unit/core-test.cxx b/editeng/qa/unit/core-test.cxx index d3f92ce60c2c..12099ea0a719 100644 --- a/editeng/qa/unit/core-test.cxx +++ b/editeng/qa/unit/core-test.cxx @@ -45,10 +45,20 @@ namespace { class Test : public test::BootstrapFixture { public: - Test(); + Test() {} - virtual void setUp() override; - virtual void tearDown() override; + void setUp() override + { + test::BootstrapFixture::setUp(); + mpItemPool = new EditEngineItemPool(); + SfxApplication::GetOrCreate(); + } + + void tearDown() override + { + mpItemPool.clear(); + test::BootstrapFixture::tearDown(); + } #if HAVE_MORE_FONTS /// Test text portions position when percentage line spacing is set @@ -135,23 +145,6 @@ private: rtl::Reference<EditEngineItemPool> mpItemPool; }; -Test::Test() {} - -void Test::setUp() -{ - test::BootstrapFixture::setUp(); - - mpItemPool = new EditEngineItemPool(); - - SfxApplication::GetOrCreate(); -} - -void Test::tearDown() -{ - mpItemPool.clear(); - test::BootstrapFixture::tearDown(); -} - #if HAVE_MORE_FONTS void Test::testLineSpacing() { |