refactor: run rector

kenjis · kenjis · commit 917597d5749f · 2022-06-27T18:24:58.000+09:00
diff --git a/tests/system/CLI/ConsoleTest.php b/tests/system/CLI/ConsoleTest.php
@@ -24,6 +24,11 @@
  */
 final class ConsoleTest extends CIUnitTestCase
 {
+    /**
+     * @var DotEnv|mixed
+     */
+    public $env;
+
     private $stream_filter;
 
     protected function setUp(): void
diff --git a/tests/system/CommonFunctionsTest.php b/tests/system/CommonFunctionsTest.php
@@ -43,6 +43,13 @@
  */
 final class CommonFunctionsTest extends CIUnitTestCase
 {
+    /**
+     * @var App|mixed
+     */
+    public $config;
+
+    public MockIncomingRequest $request;
+
     protected function setUp(): void
     {
         unset($_ENV['foo'], $_SERVER['foo']);
diff --git a/tests/system/Config/DotEnvTest.php b/tests/system/Config/DotEnvTest.php
@@ -13,6 +13,7 @@
 
 use CodeIgniter\Test\CIUnitTestCase;
 use org\bovigo\vfs\vfsStream;
+use org\bovigo\vfs\vfsStreamDirectory;
 
 /**
  * @backupGlobals enabled
@@ -23,6 +24,12 @@
  */
 final class DotEnvTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|vfsStreamDirectory|null
+     */
+    public $root;
+
+    public string $path;
     private $fixturesFolder;
 
     protected function setUp(): void
diff --git a/tests/system/ControllerTest.php b/tests/system/ControllerTest.php
@@ -33,6 +33,11 @@
  */
 final class ControllerTest extends CIUnitTestCase
 {
+    /**
+     * @var App|mixed
+     */
+    public $config;
+
     /**
      * @var Controller
      */
diff --git a/tests/system/Database/Live/MetadataTest.php b/tests/system/Database/Live/MetadataTest.php
@@ -11,6 +11,7 @@
 
 namespace CodeIgniter\Database\Live;
 
+use CodeIgniter\Database\Forge;
 use CodeIgniter\Test\CIUnitTestCase;
 use CodeIgniter\Test\DatabaseTestTrait;
 use Config\Database;
@@ -25,6 +26,11 @@ final class MetadataTest extends CIUnitTestCase
 {
     use DatabaseTestTrait;
 
+    /**
+     * @var Forge|mixed
+     */
+    public $forge;
+
     protected $refresh = true;
     protected $seed    = CITestSeeder::class;
 
diff --git a/tests/system/HTTP/ContentSecurityPolicyTest.php b/tests/system/HTTP/ContentSecurityPolicyTest.php
@@ -27,6 +27,16 @@
  */
 final class ContentSecurityPolicyTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|Response
+     */
+    public $response;
+
+    /**
+     * @var \CodeIgniter\HTTP\ContentSecurityPolicy|mixed
+     */
+    public $csp;
+
     // Having this method as setUp() doesn't work - can't find Config\App !?
     protected function prepare(bool $CSPEnabled = true)
     {
diff --git a/tests/system/HTTP/Files/FileMovingTest.php b/tests/system/HTTP/Files/FileMovingTest.php
@@ -14,12 +14,30 @@
 use CodeIgniter\HTTP\Exceptions\HTTPException;
 use CodeIgniter\Test\CIUnitTestCase;
 use org\bovigo\vfs\vfsStream;
+use org\bovigo\vfs\vfsStreamDirectory;
 
 /**
  * @internal
  */
 final class FileMovingTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|vfsStreamDirectory|null
+     */
+    public $root;
+
+    public string $path;
+
+    /**
+     * @var mixed|string
+     */
+    public $start;
+
+    /**
+     * @var mixed|string
+     */
+    public $destination;
+
     protected function setUp(): void
     {
         parent::setUp();
diff --git a/tests/system/Helpers/CookieHelperTest.php b/tests/system/Helpers/CookieHelperTest.php
@@ -28,6 +28,7 @@
  */
 final class CookieHelperTest extends CIUnitTestCase
 {
+    public IncomingRequest $request;
     private $name;
     private string $value;
     private int $expire;
diff --git a/tests/system/Helpers/FilesystemHelperTest.php b/tests/system/Helpers/FilesystemHelperTest.php
@@ -20,6 +20,11 @@
  */
 final class FilesystemHelperTest extends CIUnitTestCase
 {
+    /**
+     * @var array<string, array<string, mixed[]>>|array<string, array<string, string>>|array<string, mixed>|array<string, mixed[]>|array<string, string>|mixed
+     */
+    public $structure;
+
     protected function setUp(): void
     {
         parent::setUp();
diff --git a/tests/system/Images/BaseHandlerTest.php b/tests/system/Images/BaseHandlerTest.php
@@ -17,6 +17,7 @@
 use CodeIgniter\Images\Handlers\BaseHandler;
 use CodeIgniter\Test\CIUnitTestCase;
 use org\bovigo\vfs\vfsStream;
+use org\bovigo\vfs\vfsStreamDirectory;
 
 /**
  * Test the common image processing functionality.
@@ -30,6 +31,26 @@
  */
 final class BaseHandlerTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|vfsStreamDirectory
+     */
+    public $root;
+
+    /**
+     * @var mixed|string
+     */
+    public $origin;
+
+    /**
+     * @var mixed|string
+     */
+    public $start;
+
+    /**
+     * @var mixed|string
+     */
+    public $path;
+
     protected function setUp(): void
     {
         if (! extension_loaded('gd')) {
diff --git a/tests/system/Images/GDHandlerTest.php b/tests/system/Images/GDHandlerTest.php
@@ -13,8 +13,10 @@
 
 use CodeIgniter\Config\Services;
 use CodeIgniter\Images\Exceptions\ImageException;
+use CodeIgniter\Images\Handlers\BaseHandler;
 use CodeIgniter\Test\CIUnitTestCase;
 use org\bovigo\vfs\vfsStream;
+use org\bovigo\vfs\vfsStreamDirectory;
 
 /**
  * Unit testing for the GD image handler.
@@ -29,6 +31,31 @@
  */
 final class GDHandlerTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|vfsStreamDirectory
+     */
+    public $root;
+
+    /**
+     * @var mixed|string
+     */
+    public $origin;
+
+    /**
+     * @var mixed|string
+     */
+    public $start;
+
+    /**
+     * @var mixed|string
+     */
+    public $path;
+
+    /**
+     * @var BaseHandler|mixed
+     */
+    public $handler;
+
     protected function setUp(): void
     {
         if (! extension_loaded('gd')) {
diff --git a/tests/system/Images/ImageMagickHandlerTest.php b/tests/system/Images/ImageMagickHandlerTest.php
@@ -13,6 +13,7 @@
 
 use CodeIgniter\Config\Services;
 use CodeIgniter\Images\Exceptions\ImageException;
+use CodeIgniter\Images\Handlers\BaseHandler;
 use CodeIgniter\Test\CIUnitTestCase;
 use Config\Images;
 use Imagick;
@@ -30,6 +31,26 @@
  */
 final class ImageMagickHandlerTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|string
+     */
+    public $root;
+
+    /**
+     * @var mixed|string
+     */
+    public $origin;
+
+    /**
+     * @var mixed|string
+     */
+    public $path;
+
+    /**
+     * @var BaseHandler|mixed
+     */
+    public $handler;
+
     protected function setUp(): void
     {
         if (! extension_loaded('imagick')) {
diff --git a/tests/system/Images/ImageTest.php b/tests/system/Images/ImageTest.php
@@ -14,12 +14,30 @@
 use CodeIgniter\Images\Exceptions\ImageException;
 use CodeIgniter\Test\CIUnitTestCase;
 use org\bovigo\vfs\vfsStream;
+use org\bovigo\vfs\vfsStreamDirectory;
 
 /**
  * @internal
  */
 final class ImageTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|vfsStreamDirectory
+     */
+    public $root;
+
+    public string $origin;
+
+    /**
+     * @var mixed|string
+     */
+    public $start;
+
+    /**
+     * @var Image|mixed
+     */
+    public $image;
+
     protected function setUp(): void
     {
         // create virtual file system
diff --git a/tests/system/Log/Handlers/FileHandlerTest.php b/tests/system/Log/Handlers/FileHandlerTest.php
@@ -15,13 +15,24 @@
 use CodeIgniter\Test\Mock\MockFileLogger;
 use CodeIgniter\Test\Mock\MockLogger as LoggerConfig;
 use org\bovigo\vfs\vfsStream;
+use org\bovigo\vfs\vfsStreamDirectory;
 use Tests\Support\Log\Handlers\TestHandler;
 
 /**
  * @internal
  */
 final class FileHandlerTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|vfsStreamDirectory
+     */
+    public $root;
+
+    /**
+     * @var mixed|string
+     */
+    public $start;
+
     protected function setUp(): void
     {
         parent::setUp();
diff --git a/tests/system/Pager/PagerRendererTest.php b/tests/system/Pager/PagerRendererTest.php
@@ -19,6 +19,11 @@
  */
 final class PagerRendererTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|string
+     */
+    public $expect;
+
     private URI $uri;
 
     protected function setUp(): void
diff --git a/tests/system/Test/TestCaseTest.php b/tests/system/Test/TestCaseTest.php
@@ -22,6 +22,11 @@
  */
 final class TestCaseTest extends CIUnitTestCase
 {
+    /**
+     * @var bool|mixed|resource
+     */
+    public $stream_filter;
+
     public function testGetPrivatePropertyWithObject()
     {
         $obj    = new __TestForReflectionHelper();
diff --git a/tests/system/Throttle/ThrottleTest.php b/tests/system/Throttle/ThrottleTest.php
@@ -19,6 +19,11 @@
  */
 final class ThrottleTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|MockCache
+     */
+    public $cache;
+
     protected function setUp(): void
     {
         parent::setUp();
diff --git a/tests/system/View/TableTest.php b/tests/system/View/TableTest.php
@@ -21,6 +21,11 @@
  */
 final class TableTest extends CIUnitTestCase
 {
+    /**
+     * @var mixed|MockTable
+     */
+    public $table;
+
     protected function setUp(): void
     {
         $this->table = new MockTable();

Original file line number	Diff line number	Diff line change
`@@ -43,6 +43,13 @@`
`43`	`43`	`*/`
`44`	`44`	`final class CommonFunctionsTest extends CIUnitTestCase`
`45`	`45`	`{`
	`46`	`+ /**`
	`47`	`+ * @var App\|mixed`
	`48`	`+ */`
	`49`	`+ public $config;`
	`50`	`+`
	`51`	`+ public MockIncomingRequest $request;`
	`52`	`+`
`46`	`53`	`protected function setUp(): void`
`47`	`54`	`{`
`48`	`55`	`unset($_ENV['foo'], $_SERVER['foo']);`
Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@`
`28`	`28`	`*/`
`29`	`29`	`final class CookieHelperTest extends CIUnitTestCase`
`30`	`30`	`{`
	`31`	`+ public IncomingRequest $request;`
`31`	`32`	`private $name;`
`32`	`33`	`private string $value;`
`33`	`34`	`private int $expire;`
Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,11 @@`
`20`	`20`	`*/`
`21`	`21`	`final class FilesystemHelperTest extends CIUnitTestCase`
`22`	`22`	`{`
	`23`	`+ /**`
	`24`	`+ * @var array<string, array<string, mixed[]>>\|array<string, array<string, string>>\|array<string, mixed>\|array<string, mixed[]>\|array<string, string>\|mixed`
	`25`	`+ */`
	`26`	`+ public $structure;`
	`27`	`+`
`23`	`28`	`protected function setUp(): void`
`24`	`29`	`{`
`25`	`30`	`parent::setUp();`