phpstan fixes

Author: rochamarcelo

src/Authentication/AuthenticationService.php

@@ -14,11 +14,13 @@
 namespace CakeDC\Auth\Authentication;
 
 use Authentication\AuthenticationService as BaseService;
+use Authentication\Authenticator\Result;
 use Authentication\Authenticator\ResultInterface;
 use Authentication\Authenticator\StatelessInterface;
 use Cake\Datasource\EntityInterface;
 use Psr\Http\Message\ServerRequestInterface;
 use RuntimeException;
+use UnexpectedValueException;
 
 class AuthenticationService extends BaseService
 {
@@ -71,6 +73,9 @@ public function authenticate(ServerRequestInterface $request): ResultInterface
         foreach ($this->authenticators() as $authenticator) {
             $result = $authenticator->authenticate($request);
             if ($result->isValid()) {
+                if (!method_exists($authenticator, 'getConfig')) {
+                    throw new UnexpectedValueException('Authenticators must implement method `getConfig` to check when 2fa is required');
+                }
                 $skipTwoFactorVerify = $authenticator->getConfig('skipTwoFactorVerify');
                 $userData = $result->getData();
                 if ($userData instanceof EntityInterface) {

src/Authenticator/OneTimeTokenAuthenticator.php

@@ -10,6 +10,7 @@
 use Cake\Core\Configure;
 use Cake\ORM\TableRegistry;
 use Psr\Http\Message\ServerRequestInterface;
+use UnexpectedValueException;
 
 class OneTimeTokenAuthenticator extends AbstractAuthenticator implements AuthenticatorInterface
 {
@@ -29,6 +30,13 @@ public function authenticate(ServerRequestInterface $request): ResultInterface
         }
 
         $usersTable = TableRegistry::getTableLocator()->get(Configure::read('Users.table'));
+        if (!method_exists($usersTable, 'loginWithToken')) {
+            throw new UnexpectedValueException(sprintf(
+                '%s requires to implement `%s::loginWithToken`',
+                self::class,
+                get_class($usersTable)
+            ));
+        }
 
         $user = $usersTable->loginWithToken($token);
 

src/Policy/SuperuserPolicy.php

@@ -48,11 +48,7 @@ public function __construct(array $config = [])
     }
 
     /**
-     * Check permission
-     *
-     * @param \Authorization\IdentityInterface|null $identity user identity
-     * @param \Psr\Http\Message\ServerRequestInterface $resource server request
-     * @return bool
+     * @inheritDoc
      */
     public function canAccess(?IdentityInterface $identity, ServerRequestInterface $resource): ResultInterface
     {