new phpstan errors fixed

Author: lonnieezell

src/Filters/GroupFilter.php

@@ -10,8 +10,6 @@
 use CodeIgniter\HTTP\Response;
 use CodeIgniter\HTTP\ResponseInterface;
 
-use CodeIgniter\Shield\Exceptions\GroupException;
-
 /**
  * Group Authorization Filter.
  */

src/Filters/PermissionFilter.php

@@ -10,8 +10,6 @@
 use CodeIgniter\HTTP\Response;
 use CodeIgniter\HTTP\ResponseInterface;
 
-use CodeIgniter\Shield\Exceptions\PermissionException;
-
 /**
  * Permission Authorization Filter.
  */

tests/Authentication/Filters/AbstractFilterTest.php

@@ -5,10 +5,10 @@
 namespace Tests\Authentication\Filters;
 
 use CodeIgniter\Config\Factories;
+use CodeIgniter\Shield\Test\AuthenticationTesting;
 use CodeIgniter\Test\FeatureTestTrait;
 use Config\Services;
 use Tests\Support\TestCase;
-use CodeIgniter\Shield\Test\AuthenticationTesting;
 
 /**
  * @internal

tests/Authentication/Filters/GroupFilterTest.php

@@ -9,12 +9,15 @@
 use CodeIgniter\Shield\Models\UserModel;
 use CodeIgniter\Test\DatabaseTestTrait;
 
+/**
+ * @internal
+ */
 final class GroupFilterTest extends AbstractFilterTest
 {
     use DatabaseTestTrait;
 
-    protected string $alias     = 'group';
-    protected string $classname = GroupFilter::class;
+    protected string $alias       = 'group';
+    protected string $classname   = GroupFilter::class;
     protected string $routeFilter = 'group:admin';
 
     public function testFilterNotAuthorized(): void
@@ -28,7 +31,7 @@ public function testFilterNotAuthorized(): void
         $result->assertSee('Open');
     }
 
-    public function testFilterSuccess()
+    public function testFilterSuccess(): void
     {
         /** @var User */
         $user = fake(UserModel::class);
@@ -45,7 +48,7 @@ public function testFilterSuccess()
         $this->assertSame($user->id, auth('session')->user()->id);
     }
 
-    public function testFilterIncorrectGroupNoPrevious()
+    public function testFilterIncorrectGroupNoPrevious(): void
     {
         /** @var User */
         $user = fake(UserModel::class);
@@ -61,7 +64,7 @@ public function testFilterIncorrectGroupNoPrevious()
         $result->assertSessionHas('error');
     }
 
-    public function testFilterIncorrectGroupWithPrevious()
+    public function testFilterIncorrectGroupWithPrevious(): void
     {
         /** @var User */
         $user = fake(UserModel::class);
@@ -77,5 +80,4 @@ public function testFilterIncorrectGroupWithPrevious()
 
         $result->assertSessionHas('error');
     }
-
 }

tests/Authentication/Filters/PermissionFilterTest.php

@@ -9,12 +9,15 @@
 use CodeIgniter\Shield\Models\UserModel;
 use CodeIgniter\Test\DatabaseTestTrait;
 
+/**
+ * @internal
+ */
 final class PermissionFilterTest extends AbstractFilterTest
 {
     use DatabaseTestTrait;
 
-    protected string $alias     = 'permission';
-    protected string $classname = PermissionFilter::class;
+    protected string $alias       = 'permission';
+    protected string $classname   = PermissionFilter::class;
     protected string $routeFilter = 'permission:admin.access';
 
     public function testFilterNotAuthorized(): void
@@ -28,7 +31,7 @@ public function testFilterNotAuthorized(): void
         $result->assertSee('Open');
     }
 
-    public function testFilterSuccess()
+    public function testFilterSuccess(): void
     {
         /** @var User */
         $user = fake(UserModel::class);
@@ -45,7 +48,7 @@ public function testFilterSuccess()
         $this->assertSame($user->id, auth('session')->user()->id);
     }
 
-    public function testFilterIncorrectGroupNoPrevious()
+    public function testFilterIncorrectGroupNoPrevious(): void
     {
         /** @var User */
         $user = fake(UserModel::class);
@@ -61,7 +64,7 @@ public function testFilterIncorrectGroupNoPrevious()
         $result->assertSessionHas('error');
     }
 
-    public function testFilterIncorrectGroupWithPrevious()
+    public function testFilterIncorrectGroupWithPrevious(): void
     {
         /** @var User */
         $user = fake(UserModel::class);
@@ -77,5 +80,4 @@ public function testFilterIncorrectGroupWithPrevious()
 
         $result->assertSessionHas('error');
     }
-
 }