Various PHPStan fix

Author: TavoNiievez

src/Codeception/Module/Symfony/ConsoleAssertionsTrait.php

@@ -101,6 +101,8 @@ private function configureOptions(array $parameters): array
 
     protected function grabKernelService(): KernelInterface
     {
-        return $this->grabService('kernel');
+        /** @var KernelInterface $kernel */
+        $kernel = $this->grabService(KernelInterface::class);
+        return $kernel;
     }
 }

src/Codeception/Module/Symfony/MailerAssertionsTrait.php

@@ -4,6 +4,7 @@
 
 namespace Codeception\Module\Symfony;
 
+use PHPUnit\Framework\Assert;
 use PHPUnit\Framework\Constraint\LogicalNot;
 use Symfony\Component\Mailer\Event\MessageEvent;
 use Symfony\Component\Mailer\Event\MessageEvents;
@@ -117,7 +118,7 @@ public function grabLastSentEmail(): ?Email
      * $emails = $I->grabSentEmails();
      * ```
      *
-     * @return \Symfony\Component\Mime\Email[]
+     * @return \Symfony\Component\Mime\RawMessage[]
      */
     public function grabSentEmails(): array
     {
@@ -171,6 +172,6 @@ protected function getMessageMailerEvents(): MessageEvents
             /** @var MessageLoggerListener $mailer */
             return $mailer->getEvents();
         }
-        $this->fail("Emails can't be tested without Symfony Mailer service.");
+        Assert::fail("Emails can't be tested without Symfony Mailer service.");
     }
 }

src/Codeception/Module/Symfony/ParameterAssertionsTrait.php

@@ -17,6 +17,8 @@ trait ParameterAssertionsTrait
      * $I->grabParameter('app.business_name');
      * ```
      * This only works for explicitly set parameters (just using `bind` for Symfony's dependency injection is not enough).
+     *
+     * @return array<array-key, mixed>|bool|string|int|float|UnitEnum|null
      */
     public function grabParameter(string $parameterName): array|bool|string|int|float|UnitEnum|null
     {
@@ -26,6 +28,8 @@ public function grabParameter(string $parameterName): array|bool|string|int|floa
 
     protected function grabParameterBagService(): ParameterBagInterface
     {
-        return $this->grabService('parameter_bag');
+        /** @var ParameterBagInterface $parameterBag */
+        $parameterBag = $this->grabService(ParameterBagInterface::class);
+        return $parameterBag;
     }
 }

src/Codeception/Module/Symfony/RouterAssertionsTrait.php

@@ -148,6 +148,8 @@ private function openRoute(string $routeName, array $params = []): void
 
     protected function grabRouterService(): RouterInterface
     {
-        return $this->grabService('router');
+        /** @var RouterInterface $router */
+        $router = $this->grabService('router');
+        return $router;
     }
 }

src/Codeception/Module/Symfony/ServicesAssertionsTrait.php

@@ -25,10 +25,12 @@ trait ServicesAssertionsTrait
     public function grabService(string $serviceId): object
     {
         if (!$service = $this->getService($serviceId)) {
-            Assert::fail("Service `{$serviceId}` is required by Codeception, but not loaded by Symfony. Possible solutions:\n
+            Assert::fail(
+                "Service `{$serviceId}` is required by Codeception, but not loaded by Symfony. Possible solutions:\n
             In your `config/packages/framework.php`/`.yaml`, set `test` to `true` (when in test environment), see https://symfony.com/doc/current/reference/configuration/framework.html#test\n
             If you're still getting this message, you're not using that service in your app, so Symfony isn't loading it at all.\n
-            Solution: Set it to `public` in your `config/services.php`/`.yaml`, see https://symfony.com/doc/current/service_container/alias_private.html#marking-services-as-public-private\n");
+            Solution: Set it to `public` in your `config/services.php`/`.yaml`, see https://symfony.com/doc/current/service_container/alias_private.html#marking-services-as-public-private\n"
+            );
         }
 
         return $service;

src/Codeception/Module/Symfony/ValidatorAssertionsTrait.php

@@ -101,6 +101,8 @@ protected function getViolationsForSubject(object $subject, ?string $propertyPat
 
     protected function getValidatorService(): ValidatorInterface
     {
-        return $this->grabService(ValidatorInterface::class);
+        /** @var ValidatorInterface $validator */
+        $validator = $this->grabService(ValidatorInterface::class);
+        return $validator;
     }
 }