fixed setUp()/tearDown() visibility

mirko-pagliai · Dec 18, 2024 · 2ecae98 · 2ecae98
1 parent 93bc828
commit 2ecae98
Show file tree

Hide file tree

Showing 10 changed files with 11 additions and 12 deletions.
diff --git a/src/TestSuite/DriverTestCase.php b/src/TestSuite/DriverTestCase.php
@@ -36,7 +36,7 @@ abstract class DriverTestCase extends TestCase
      * Called before every test method
      * @return void
      */
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

diff --git a/src/TestSuite/TestCase.php b/src/TestSuite/TestCase.php
@@ -28,10 +28,9 @@ abstract class TestCase extends BaseTestCase
     use BackupTrait;
 
     /**
-     * Called after every test method
-     * @return void
+     * @inheritDoc
      */
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         parent::tearDown();
 

diff --git a/tests/TestCase/BackupTraitTest.php b/tests/TestCase/BackupTraitTest.php
@@ -45,7 +45,7 @@ class BackupTraitTest extends TestCase
     /**
      * @inheritDoc
      */
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

diff --git a/tests/TestCase/Driver/MysqlTest.php b/tests/TestCase/Driver/MysqlTest.php
@@ -36,7 +36,7 @@ class MysqlTest extends DriverTestCase
      * {@inheritDoc}
      * @throws \PHPUnit\Framework\MockObject\Exception
      */
-    public function setUp(): void
+    protected function setUp(): void
     {
         if ($this->getConnection()->config()['scheme'] !== 'mysql') {
             $this->markTestSkipped('Skipping tests for mysql, current driver is `' . $this->getConnection()->config()['scheme'] . '`');

diff --git a/tests/TestCase/Driver/PostgresTest.php b/tests/TestCase/Driver/PostgresTest.php
@@ -27,7 +27,7 @@ class PostgresTest extends DriverTestCase
     /**
      * {@inheritDoc}
      */
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

diff --git a/tests/TestCase/Driver/SqliteTest.php b/tests/TestCase/Driver/SqliteTest.php
@@ -27,7 +27,7 @@ class SqliteTest extends DriverTestCase
     /**
      * {@inheritDoc}
      */
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

diff --git a/tests/TestCase/Utility/BackupExportAndImportTest.php b/tests/TestCase/Utility/BackupExportAndImportTest.php
@@ -69,7 +69,7 @@ protected function getAllRecords(): array
     /**
      * {@inheritDoc}
      */
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

diff --git a/tests/TestCase/Utility/BackupExportTest.php b/tests/TestCase/Utility/BackupExportTest.php
@@ -42,7 +42,7 @@ class BackupExportTest extends TestCase
      * {@inheritDoc}
      * @throws \ReflectionException
      */
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

diff --git a/tests/TestCase/Utility/BackupImportTest.php b/tests/TestCase/Utility/BackupImportTest.php
@@ -38,7 +38,7 @@ class BackupImportTest extends TestCase
      * {@inheritDoc}
      * @throws \ReflectionException
      */
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

diff --git a/tests/TestCase/Utility/BackupManagerTest.php b/tests/TestCase/Utility/BackupManagerTest.php
@@ -46,7 +46,7 @@ class BackupManagerTest extends TestCase
     /**
      * @inheritDoc
      */
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();