Change visibility of setUp and tearDown to original protected

Author: paulbalandan

admin/module/tests/_support/DatabaseTestCase.php

@@ -42,14 +42,14 @@ class DatabaseTestCase extends CIUnitTestCase
      */
     protected $namespace = 'Tests\Support';
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
         // Extra code to run before each test
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         parent::tearDown();
 

admin/module/tests/_support/SessionTestCase.php

@@ -14,7 +14,7 @@ class SessionTestCase extends CIUnitTestCase
      */
     protected $session;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

admin/module/tests/database/ExampleDatabaseTest.php

@@ -4,7 +4,7 @@
 
 class ExampleDatabaseTest extends \Tests\Support\DatabaseTestCase
 {
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

admin/module/tests/session/ExampleSessionTest.php

@@ -2,7 +2,7 @@
 
 class ExampleSessionTest extends \Tests\Support\SessionTestCase
 {
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
     }

admin/module/tests/unit/ExampleTest.php

@@ -2,7 +2,7 @@
 
 class ExampleTest extends \CodeIgniter\Test\CIUnitTestCase
 {
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
     }

admin/starter/tests/_support/DatabaseTestCase.php

@@ -42,14 +42,14 @@ class DatabaseTestCase extends CIUnitTestCase
      */
     protected $namespace = 'Tests\Support';
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
         // Extra code to run before each test
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         parent::tearDown();
 

admin/starter/tests/_support/SessionTestCase.php

@@ -14,7 +14,7 @@ class SessionTestCase extends CIUnitTestCase
      */
     protected $session;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

admin/starter/tests/database/ExampleDatabaseTest.php

@@ -4,7 +4,7 @@
 
 class ExampleDatabaseTest extends \Tests\Support\DatabaseTestCase
 {
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

admin/starter/tests/session/ExampleSessionTest.php

@@ -2,7 +2,7 @@
 
 class ExampleSessionTest extends \Tests\Support\SessionTestCase
 {
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
     }

admin/starter/tests/unit/HealthTest.php

@@ -4,7 +4,7 @@
 
 class HealthTest extends \CodeIgniter\Test\CIUnitTestCase
 {
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
     }

tests/system/CLI/CLITest.php

@@ -18,7 +18,7 @@ protected function setUp(): void
         $this->stream_filter        = stream_filter_append(STDOUT, 'CITestStreamFilter');
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         stream_filter_remove($this->stream_filter);
     }

tests/system/CLI/CommandRunnerTest.php

@@ -42,7 +42,7 @@ protected function setUp(): void
         $this->streamFilter = stream_filter_append(STDERR, 'CITestStreamFilter');
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         stream_filter_remove($this->streamFilter);
     }

tests/system/CLI/ConsoleTest.php

@@ -39,7 +39,7 @@ protected function setUp(): void
         $this->app = new MockCodeIgniter(new MockCLIConfig());
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         stream_filter_remove($this->stream_filter);
     }

tests/system/Cache/CacheFactoryTest.php

@@ -23,7 +23,7 @@ protected function setUp(): void
         $this->config->storePath .= self::$directory;
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         if (is_dir($this->config->storePath)) {
             chmod($this->config->storePath, 0777);

tests/system/Cache/Handlers/FileHandlerTest.php

@@ -46,7 +46,7 @@ protected function setUp(): void
         $this->fileHandler->initialize();
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         parent::tearDown();
 

tests/system/Cache/Handlers/MemcachedHandlerTest.php

@@ -37,7 +37,7 @@ protected function setUp(): void
         $this->memcachedHandler->initialize();
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         foreach (self::getKeyArray() as $key) {
             $this->memcachedHandler->delete($key);

tests/system/Cache/Handlers/PredisHandlerTest.php

@@ -36,7 +36,7 @@ protected function setUp(): void
         $this->PredisHandler->initialize();
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         foreach (self::getKeyArray() as $key) {
             $this->PredisHandler->delete($key);

tests/system/Cache/Handlers/RedisHandlerTest.php

@@ -36,7 +36,7 @@ protected function setUp(): void
         $this->redisHandler->initialize();
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         foreach (self::getKeyArray() as $key) {
             $this->redisHandler->delete($key);

tests/system/CodeIgniterTest.php

@@ -35,7 +35,7 @@ protected function setUp(): void
         $this->codeigniter = new MockCodeIgniter($config);
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         parent::tearDown();
 

tests/system/Commands/ClearCacheTest.php

@@ -23,7 +23,7 @@ protected function setUp(): void
         Services::injectMock('cache', CacheFactory::getHandler(config('Cache')));
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         stream_filter_remove($this->streamFilter);
     }

tests/system/Commands/InfoCacheTest.php

@@ -23,7 +23,7 @@ protected function setUp(): void
         Services::injectMock('cache', CacheFactory::getHandler(config('Cache')));
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         stream_filter_remove($this->streamFilter);
 

tests/system/Commands/SessionsCommandsTest.php

@@ -19,7 +19,7 @@ protected function setUp(): void
         $this->streamFilter = stream_filter_append(STDERR, 'CITestStreamFilter');
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         stream_filter_remove($this->streamFilter);
 

tests/system/Config/ServicesTest.php

@@ -49,7 +49,7 @@ protected function setUp(): void
         $this->config   = new App();
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         $_SERVER = $this->original;
         Services::reset();

tests/system/Database/DatabaseTestCase/DatabaseTestCaseMigrationOnce1Test.php

@@ -43,7 +43,7 @@ class DatabaseTestCaseMigrationOnce1Test extends CIUnitTestCase
         'Tests\Support\MigrationTestMigrations',
     ];
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         Services::autoloader()->addNamespace('Tests\Support\MigrationTestMigrations', SUPPORTPATH . 'MigrationTestMigrations');
 

tests/system/Database/DatabaseTestCase/DatabaseTestCaseMigrationOnce2Test.php

@@ -42,7 +42,7 @@ class DatabaseTestCaseMigrationOnce2Test extends CIUnitTestCase
         'Tests\Support\MigrationTestMigrations',
     ];
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         Services::autoloader()->addNamespace('Tests\Support\MigrationTestMigrations', SUPPORTPATH . 'MigrationTestMigrations');
 

tests/system/Database/DatabaseTestCaseTest.php

@@ -47,7 +47,7 @@ class DatabaseTestCaseTest extends CIUnitTestCase
         'Tests\Support\MigrationTestMigrations',
     ];
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         if (! self::$loaded) {
             Services::autoloader()->addNamespace('Tests\Support\MigrationTestMigrations', SUPPORTPATH . 'MigrationTestMigrations');

tests/system/Database/Live/DEBugTest.php

@@ -28,7 +28,7 @@ public function testDBDebugFalse()
         $this->assertEquals(false, $result);
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         $this->setPrivateProperty($this->db, 'DBDebug', true);
         parent::tearDown();

tests/system/Database/Live/GetNumRowsTest.php

@@ -19,7 +19,7 @@ class GetNumRowsTest extends CIUnitTestCase
      *
      * @see \CodeIgniter\Test\CIDatabaseTestCase::setUp()
      */
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
     }
@@ -30,7 +30,7 @@ public function setUp(): void
      *
      * @see \CodeIgniter\Test\CIDatabaseTestCase::tearDown()
      */
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         parent::tearDown();
     }

tests/system/Database/Live/PretendTest.php

@@ -13,7 +13,7 @@ class PretendTest extends CIUnitTestCase
 {
     use DatabaseTestTrait;
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         // We share `$this->db` in testing, so we need to restore the state.
         $this->db->pretend(false);

tests/system/Database/Live/SQLite/AlterTableTest.php

@@ -38,7 +38,7 @@ class AlterTableTest extends CIUnitTestCase
      */
     protected $forge;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

tests/system/Database/Migrations/MigrationRunnerTest.php

@@ -27,7 +27,7 @@ class MigrationRunnerTest extends CIUnitTestCase
     protected $start;
     protected $config;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

tests/system/Database/Migrations/MigrationTest.php

@@ -10,7 +10,7 @@ class MigrationTest extends CIUnitTestCase
 {
     use DatabaseTestTrait;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
     }

tests/system/Files/FileWithVfsTest.php

@@ -29,7 +29,7 @@ protected function setUp(): void
         $this->file  = new File($this->start . 'able/apple.php');
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         parent::tearDown();
 

tests/system/HTTP/DownloadResponseTest.php

@@ -10,7 +10,7 @@
 
 class DownloadResponseTest extends CIUnitTestCase
 {
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         if (isset($_SERVER['HTTP_USER_AGENT'])) {
             unset($_SERVER['HTTP_USER_AGENT']);

tests/system/HTTP/Files/FileMovingTest.php

@@ -25,7 +25,7 @@ protected function setUp(): void
         $_FILES = [];
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         parent::tearDown();
         $this->root = null;

tests/system/HTTP/MessageTest.php

@@ -22,7 +22,7 @@ protected function setUp(): void
 
     //--------------------------------------------------------------------
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         $this->message = null;
         unset($this->message);

tests/system/HTTP/NegotiateTest.php

@@ -30,7 +30,7 @@ protected function setUp(): void
 
     //--------------------------------------------------------------------
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         $this->request = $this->negotiate = null;
         unset($this->request, $this->negotiate);

tests/system/HTTP/ResponseTest.php

@@ -22,7 +22,7 @@ protected function setUp(): void
         $this->server = $_SERVER;
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         $_SERVER = $this->server;
         Factories::reset('config');

tests/system/HTTP/URITest.php

@@ -20,7 +20,7 @@ protected function setUp(): void
 
     //--------------------------------------------------------------------
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Factories::reset('config');
     }

tests/system/Helpers/URLHelper/CurrentUrlTest.php

@@ -47,7 +47,7 @@ protected function setUp(): void
         $_SERVER['SCRIPT_NAME'] = '/index.php';
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         parent::tearDown();
 

tests/system/Helpers/URLHelper/MiscUrlTest.php

@@ -39,7 +39,7 @@ protected function setUp(): void
         Factories::injectMock('config', 'App', $this->config);
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         parent::tearDown();