Added Psalm and Laminas coding standard

phpunit.xml.dist

@@ -3,20 +3,16 @@
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         bootstrap="./vendor/autoload.php"
         colors="true"
-        stopOnFailure="false"
-        processIsolation="false"
-        backupGlobals="false"
         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd"
         cacheDirectory=".phpunit.cache"
-        displayDetailsOnTestsThatTriggerDeprecations="false"
 >
   <testsuite name="LmcRbac tests">
     <directory>./test</directory>
   </testsuite>
   <logging/>
   <source>
     <include>
-      <directory suffix=".php">./src</directory>
+      <directory>./src</directory>
     </include>
   </source>
 </phpunit>

src/Service/RoleServiceFactory.php

@@ -23,6 +23,7 @@
 
 use Laminas\ServiceManager\Exception\ServiceNotCreatedException;
 use Lmc\Rbac\Options\ModuleOptions;
+use Lmc\Rbac\Role\RoleProviderInterface;
 use Psr\Container\ContainerInterface;
 
 /**
@@ -44,9 +45,13 @@ public function __invoke(ContainerInterface $container): RoleService
         }
 
         $roleProviderName = key($roleProvider);
+        $roleProvider = $container->get($roleProviderName);
+        if (! $roleProvider instanceof RoleProviderInterface) {
+            throw new ServiceNotCreatedException(sprintf('Class %s does not implement LmcRbac\Role\RoleProviderInterface', $roleProviderName));
+        }
 
         return new RoleService(
-            $container->get($roleProviderName),
+            $roleProvider,
             $moduleOptions->getGuestRole()
         );
     }

test/ConfigProviderTest.php

@@ -19,7 +19,7 @@
 
 declare(strict_types=1);
 
-namespace LmcRbacTest;
+namespace LmcTest;
 
 use Lmc\Rbac\ConfigProvider;
 use PHPUnit\Framework\Attributes\CoversClass;

test/ModuleTest.php

@@ -19,7 +19,7 @@
 
 declare(strict_types=1);
 
-namespace LmcRbacTest;
+namespace LmcTest;
 
 use Lmc\Rbac\ConfigProvider;
 use Lmc\Rbac\Module;

test/Service/RoleServiceFactoryTest.php

@@ -72,4 +72,26 @@ public function testThrowExceptionIfNoRoleProvider(): void
         $factory = new RoleServiceFactory();
         $factory($container);
     }
+
+    public function testThrowExceptionIfInvalidRoleProvider(): void
+    {
+        $this->expectException(\Laminas\ServiceManager\Exception\ServiceNotCreatedException::class);
+
+        $options = new ModuleOptions([
+            'guest_role' => 'guest',
+            'role_provider' => [
+                'InvalidRoleProvider' => [],
+            ],
+        ]);
+
+        $container = new ServiceManager(['services' => [
+            ModuleOptions::class => $options,
+            'InvalidRoleProvider' => function () {
+                return new \stdClass();
+            }
+        ]]);
+
+        $factory = new RoleServiceFactory();
+        $factory($container);
+    }
 }