refactor: by rector

Author: kenjis

tests/_support/Fakers/MonsterFaker.php

@@ -23,15 +23,15 @@ public function fake(Generator &$faker): Monster
     {
         return new Monster([
             'name'   => $faker->lastName(),
-            'health' => mt_rand(1, 50),
+            'health' => random_int(1, 50),
 
             /**
              * This is a special function of Fabricator that means "get how many items
              * have been generated for this table so far". It is handy for faking
              * related items, like the number of possible dungeons. But if none have
              * been made we will just pick a number.
              */
-            'dungeon_id' => mt_rand(1, Fabricator::getCount('dungeons') ?: 3),
+            'dungeon_id' => random_int(1, Fabricator::getCount('dungeons') ?: 3),
         ]);
     }
 }

tests/_support/Libraries/ConfigReader.php

@@ -38,14 +38,16 @@
 
 namespace Tests\Support\Libraries;
 
+use Config\App;
+
 /**
  * Class ConfigReader
  *
  * An extension of BaseConfig that prevents the constructor from
  * loading external values. Used to read actual local values from
  * a config file.
  */
-class ConfigReader extends \Config\App
+class ConfigReader extends App
 {
     public function __construct()
     {

tests/unit/HealthTest.php

@@ -1,9 +1,12 @@
 <?php
 
+use CodeIgniter\Test\CIUnitTestCase;
+use Tests\Support\Libraries\ConfigReader;
+
 /**
  * @internal
  */
-final class HealthTest extends \CodeIgniter\Test\CIUnitTestCase
+final class HealthTest extends CIUnitTestCase
 {
     protected function setUp(): void
     {
@@ -31,7 +34,7 @@ public function testBaseUrlHasBeenSet()
         }
 
         // Then check the actual config file
-        $reader = new \Tests\Support\Libraries\ConfigReader();
+        $reader = new ConfigReader();
         $config = ! empty($reader->baseUrl);
 
         $this->assertTrue($env || $config);