Fix styles

Author: MGatner

rector.php

@@ -45,7 +45,7 @@
         SetList::DEAD_CODE,
         LevelSetList::UP_TO_PHP_74,
         PHPUnitSetList::PHPUNIT_CODE_QUALITY,
-        //PHPUnitSetList::PHPUNIT_100,
+        // PHPUnitSetList::PHPUNIT_100,
     ]);
 
     $rectorConfig->parallel();

tests/EntityTest.php

@@ -9,7 +9,7 @@
 final class EntityTest extends TestCase
 {
     /**
-     * @dataProvider ipAddressProvider
+     * @dataProvider provideIpAddressCasts
      *
      * @param mixed $input
      */
@@ -25,7 +25,7 @@ public function testIpAddressCasts($input, ?int $stored, ?string $retrieved): vo
         $this->assertSame($retrieved, $result);
     }
 
-    public function ipAddressProvider(): array
+    public static function provideIpAddressCasts(): iterable
     {
         return [
             ['127.0.0.1', 2_130_706_433, '127.0.0.1'],

tests/FilterTest.php

@@ -176,28 +176,28 @@ public function testShortCircuitsTransformers(): void
         $this->seeNumRecords(0, 'visits', []);
     }
 
-/*
-    Temporarily disabled because request handling doesn't work like this anymore
+    /*
+        Temporarily disabled because request handling doesn't work like this anymore
 
-    public function testRequiresIncomingRequest(): void
-    {
-        $this->expectException(RuntimeException::class);
-        $this->expectExceptionMessage(VisitsFilter::class . ' requires an IncomingRequest object.');
+        public function testRequiresIncomingRequest(): void
+        {
+            $this->expectException(RuntimeException::class);
+            $this->expectExceptionMessage(VisitsFilter::class . ' requires an IncomingRequest object.');
 
-        $this->request = service('clirequest');
+            $this->request = service('clirequest');
 
-        $this->call();
-    }
+            $this->call();
+        }
 
-    // Temporarily disabled because config injection isn't working anymore
-    public function testRequiresValidVisit(): void
-    {
-        $this->expectException(RuntimeException::class);
-        $this->expectExceptionMessage('Failed to create visit record: The host field is required.');
+        // Temporarily disabled because config injection isn't working anymore
+        public function testRequiresValidVisit(): void
+        {
+            $this->expectException(RuntimeException::class);
+            $this->expectExceptionMessage('Failed to create visit record: The host field is required.');
 
-        config('App')->baseURL = '0';
+            config('App')->baseURL = '0';
 
-        $this->call();
-    }
-*/
+            $this->call();
+        }
+    */
 }

tests/ModelTest.php

@@ -1,6 +1,5 @@
 <?php
 
-use CodeIgniter\Config\Services;
 use CodeIgniter\Test\DatabaseTestTrait;
 use Tatter\Visits\Entities\Visit;
 use Tatter\Visits\Models\VisitModel;
@@ -23,7 +22,7 @@ protected function setUp(): void
     }
 
     /**
-     * @dataProvider findSimilarNullProvider
+     * @dataProvider provideFindSimilarNull
      */
     public function testFindSimilarNull(string $field): void
     {
@@ -39,7 +38,7 @@ public function testFindSimilarNull(string $field): void
         $this->assertNull($result);
     }
 
-    public function findSimilarNullProvider(): array
+    public static function provideFindSimilarNull(): iterable
     {
         return [
             ['host'],
@@ -94,26 +93,26 @@ public function testMakeFromRequestUsesUserId(): void
         $this->assertSame(42, $result->user_id);
     }
 
-/*
-    Temporarily disabled because config injection isn't working anymore
+    /*
+        Temporarily disabled because config injection isn't working anymore
 
-    public function testMakeFromRequestRespectsBaseUrl(): void
-    {
-        $config = config('App');
+        public function testMakeFromRequestRespectsBaseUrl(): void
+        {
+            $config = config('App');
 
-        $config->baseURL   = 'http://foo.bar/folder/';
-        $config->indexPage = '';
+            $config->baseURL   = 'http://foo.bar/folder/';
+            $config->indexPage = '';
 
-        $request = service('request', $config)->setPath('fruits/banana#ripe');
+            $request = service('request', $config)->setPath('fruits/banana#ripe');
 
-        $result = $this->model->makeFromRequest($request);
+            $result = $this->model->makeFromRequest($request);
 
-        $this->assertSame('http', $result->scheme);
-        $this->assertSame('foo.bar', $result->host);
-        $this->assertSame('/folder/fruits/banana', $result->path);
-        $this->assertSame('ripe', $result->fragment);
-    }
-*/
+            $this->assertSame('http', $result->scheme);
+            $this->assertSame('foo.bar', $result->host);
+            $this->assertSame('/folder/fruits/banana', $result->path);
+            $this->assertSame('ripe', $result->fragment);
+        }
+    */
 
     public function testMakeFromRequestIgnoresPass(): void
     {