Remove @param Something&MockObject annotations (see contao#1936)

Description
-----------

Because they will no longer trigger `type MockObject is not within the annotated types` warnings in PhpStorm (see contao#1924 (comment)).

Commits
-------

2584d3d Remove @param Something&MockObject annotations

core-bundle/tests/Cache/ContaoCacheWarmerTest.php

@@ -133,10 +133,6 @@ public function testDoesNotCreateTheCacheFolderIfTheInstallationIsIncomplete():
         $this->assertFileNotExists(Path::join($this->getTempDir(), 'var/cache/contao'));
     }
 
-    /**
-     * @param Connection&MockObject      $connection
-     * @param ContaoFramework&MockObject $framework
-     */
     private function getCacheWarmer(Connection $connection = null, ContaoFramework $framework = null, string $bundle = 'test-bundle'): ContaoCacheWarmer
     {
         if (null === $connection) {

core-bundle/tests/Command/MigrateCommandTest.php

@@ -18,7 +18,6 @@
 use Contao\CoreBundle\Migration\MigrationResult;
 use Contao\CoreBundle\Tests\TestCase;
 use Contao\InstallationBundle\Database\Installer;
-use PHPUnit\Framework\MockObject\MockObject;
 use Symfony\Component\Config\FileLocator;
 use Symfony\Component\Console\Tester\CommandTester;
 use Symfony\Component\Filesystem\Filesystem;
@@ -179,7 +178,6 @@ public function testDoesNotAbortIfMigrationFails(): void
      * @param array<array<string>>          $pendingMigrations
      * @param array<array<MigrationResult>> $migrationResults
      * @param array<array<string>>          $runonceFiles
-     * @param Installer&MockObject          $installer
      */
     private function getCommand(array $pendingMigrations = [], array $migrationResults = [], array $runonceFiles = [], Installer $installer = null): MigrateCommand
     {

core-bundle/tests/Command/ResizeImagesCommandTest.php

@@ -19,7 +19,6 @@
 use Contao\Image\DeferredImageStorageInterface;
 use Contao\Image\DeferredResizerInterface;
 use Contao\Image\ImageInterface;
-use PHPUnit\Framework\MockObject\MockObject;
 use Symfony\Bridge\PhpUnit\ClockMock;
 use Symfony\Component\Console\Tester\CommandTester;
 use Symfony\Component\Filesystem\Filesystem;
@@ -131,11 +130,6 @@ function () {
         $this->assertNotRegExp('/image2.jpg/', $display);
     }
 
-    /**
-     * @param ImageFactoryInterface&MockObject         $factory
-     * @param DeferredResizerInterface&MockObject      $resizer
-     * @param DeferredImageStorageInterface&MockObject $storage
-     */
     private function getCommand(ImageFactoryInterface $factory = null, DeferredResizerInterface $resizer = null, DeferredImageStorageInterface $storage = null): ResizeImagesCommand
     {
         return new ResizeImagesCommand(

core-bundle/tests/Command/UserPasswordCommandTest.php

@@ -235,9 +235,6 @@ public function testResetsPasswordWithRequiredChangeOnNextLogin(): void
         (new CommandTester($command))->execute($input, ['interactive' => false]);
     }
 
-    /**
-     * @param Connection&MockObject $connection
-     */
     private function getCommand(Connection $connection = null, string $password = null): UserPasswordCommand
     {
         if (null === $connection) {

core-bundle/tests/EventListener/PrettyErrorScreenListenerTest.php

@@ -23,7 +23,6 @@
 use Contao\CoreBundle\Fixtures\Exception\PageErrorResponseException;
 use Contao\CoreBundle\Tests\TestCase;
 use Lexik\Bundle\MaintenanceBundle\Exception\ServiceUnavailableException;
-use PHPUnit\Framework\MockObject\MockObject;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpKernel\Event\ExceptionEvent;
 use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException;
@@ -308,9 +307,6 @@ public function testDoesNotLogUnloggableExceptions(): void
         $this->assertSame(500, $event->getResponse()->getStatusCode());
     }
 
-    /**
-     * @param Environment&MockObject $twig
-     */
     private function getListener(bool $isBackendUser = false, bool $expectLogging = false, Environment $twig = null): PrettyErrorScreenListener
     {
         if (null === $twig) {

core-bundle/tests/EventListener/UserSessionListenerTest.php

@@ -335,13 +335,6 @@ public function testFailsToStoreTheSessionIfThereIsNoSession(): void
         $listener->write($this->getResponseEvent($request));
     }
 
-    /**
-     * Mocks a session listener.
-     *
-     * @param Connection&MockObject               $connection
-     * @param Security&MockObject                 $security
-     * @param EventDispatcherInterface&MockObject $eventDispatcher
-     */
     private function getListener(Connection $connection = null, Security $security = null, EventDispatcherInterface $eventDispatcher = null): UserSessionListener
     {
         if (null === $connection) {

core-bundle/tests/Fragment/FragmentHandlerTest.php

@@ -206,9 +206,6 @@ public function testConvertsExceptionsToResponseExceptions(): void
         $fragmentHandler->render($uri);
     }
 
-    /**
-     * @param BaseFragmentHandler&MockObject $fragmentHandler
-     */
     private function getFragmentHandler(FragmentRegistry $registry = null, ServiceLocator $renderers = null, ServiceLocator $preHandlers = null, Request $request = null, BaseFragmentHandler $fragmentHandler = null): FragmentHandler
     {
         if (null === $registry) {

core-bundle/tests/Framework/ContaoFrameworkTest.php

@@ -690,9 +690,6 @@ public function testDelegatesTheResetCalls(): void
         $this->assertCount(0, $registry);
     }
 
-    /**
-     * @param TokenChecker&MockObject $tokenChecker
-     */
     private function mockFramework(Request $request = null, ScopeMatcher $scopeMatcher = null, TokenChecker $tokenChecker = null): ContaoFramework
     {
         $requestStack = new RequestStack();

core-bundle/tests/Image/ImageFactoryTest.php

@@ -839,12 +839,6 @@ public static function emptyHookCallback(): void
     {
     }
 
-    /**
-     * @param ResizerInterface&MockObject $resizer
-     * @param ImagineInterface&MockObject $imagine
-     * @param ImagineInterface&MockObject $imagineSvg
-     * @param ContaoFramework&MockObject  $framework
-     */
     private function getImageFactory(ResizerInterface $resizer = null, ImagineInterface $imagine = null, ImagineInterface $imagineSvg = null, Filesystem $filesystem = null, ContaoFramework $framework = null, bool $bypassCache = null, array $imagineOptions = null, array $validExtensions = null, string $uploadDir = null): ImageFactory
     {
         if (null === $resizer) {

core-bundle/tests/Image/PictureFactoryTest.php

@@ -722,11 +722,6 @@ public function getAspectRatios(): \Generator
         yield [false, 20, 100, 22, 100];
     }
 
-    /**
-     * @param PictureGeneratorInterface&MockObject $pictureGenerator
-     * @param ImageFactoryInterface&MockObject     $imageFactory
-     * @param ContaoFramework&MockObject           $framework
-     */
     private function getPictureFactory(PictureGeneratorInterface $pictureGenerator = null, ImageFactoryInterface $imageFactory = null, ContaoFramework $framework = null, bool $bypassCache = null, array $imagineOptions = null): PictureFactory
     {
         if (null === $pictureGenerator) {

core-bundle/tests/Monolog/ContaoTableProcessorTest.php

@@ -17,7 +17,6 @@
 use Contao\CoreBundle\Monolog\ContaoTableProcessor;
 use Contao\CoreBundle\Tests\TestCase;
 use Monolog\Logger;
-use PHPUnit\Framework\MockObject\MockObject;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorage;
@@ -232,10 +231,6 @@ public function sourceProvider(): \Generator
         yield [ContaoCoreBundle::SCOPE_BACKEND, null, 'BE'];
     }
 
-    /**
-     * @param RequestStack&MockObject          $requestStack
-     * @param TokenStorageInterface&MockObject $tokenStorage
-     */
     private function getContaoTableProcessor(RequestStack $requestStack = null, TokenStorageInterface $tokenStorage = null): ContaoTableProcessor
     {
         if (null === $requestStack) {

core-bundle/tests/OptIn/OptInTokenTest.php

@@ -322,9 +322,6 @@ public function testDoesNotRequireSubjectAndTextToResendToken(): void
         $this->assertTrue($token->hasBeenSent());
     }
 
-    /**
-     * @param ContaoFramework&MockObject $framework
-     */
     private function getToken(OptInModel $model, ContaoFramework $framework = null): OptInTokenInterface
     {
         if (null === $framework) {

core-bundle/tests/Routing/Matcher/LanguageFilterTest.php

@@ -15,16 +15,13 @@
 use Contao\CoreBundle\Routing\Matcher\LanguageFilter;
 use Contao\CoreBundle\Tests\TestCase;
 use Contao\PageModel;
-use PHPUnit\Framework\MockObject\MockObject;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\Routing\Route;
 use Symfony\Component\Routing\RouteCollection;
 
 class LanguageFilterTest extends TestCase
 {
     /**
-     * @param PageModel&MockObject $page
-     *
      * @dataProvider getRoutesAndLanguages
      */
     public function testRemovesARouteIfTheAcceptedLanguagesDoNotMatch(string $name, ?PageModel $page, string $acceptLanguage, bool $expectPageModel, bool $expectRemoval, bool $prependLocale = false): void

core-bundle/tests/Routing/RouteProviderTest.php

@@ -774,9 +774,6 @@ private function createRootPage(string $language, string $alias, bool $fallback
         return $page;
     }
 
-    /**
-     * @param ContaoFramework&MockObject $framework
-     */
     private function getRouteProvider(ContaoFramework $framework = null, string $urlSuffix = '.html', bool $prependLocale = false): RouteProvider
     {
         if (null === $framework) {

core-bundle/tests/Security/Authentication/AuthenticationSuccessHandlerTest.php

@@ -453,10 +453,6 @@ public function testRemovesTheTargetPathInTheSessionOnLogin(): void
         $this->getHandler()->onAuthenticationSuccess($request, $token);
     }
 
-    /**
-     * @param ContaoFramework&MockObject $framework
-     * @param LoggerInterface&MockObject $logger
-     */
     private function getHandler(ContaoFramework $framework = null, LoggerInterface $logger = null): AuthenticationSuccessHandler
     {
         if (null === $framework) {

core-bundle/tests/Security/Authentication/FrontendPreviewAuthenticatorTest.php

@@ -250,12 +250,6 @@ public function testDoesNotRemoveTheAuthenticationIfTheSessionDoesNotContainATok
         $this->assertFalse($authenticator->removeFrontendAuthentication());
     }
 
-    /**
-     * @param Security&MockObject              $security
-     * @param SessionInterface&MockObject      $session
-     * @param UserProviderInterface&MockObject $userProvider
-     * @param LoggerInterface&MockObject       $logger
-     */
     private function getAuthenticator(Security $security = null, SessionInterface $session = null, UserProviderInterface $userProvider = null, LoggerInterface $logger = null): FrontendPreviewAuthenticator
     {
         if (null === $security) {

core-bundle/tests/Security/Authentication/Provider/AuthenticationProviderTest.php

@@ -523,11 +523,6 @@ public function onInvalidCredentials($username): bool
         return false;
     }
 
-    /**
-     * @param ContaoFramework&MockObject                $framework
-     * @param AuthenticationHandlerInterface&MockObject $twoFactorHandler
-     * @param TrustedDeviceManagerInterface&MockObject  $trustedDeviceManager
-     */
     private function createUsernamePasswordProvider(ContaoFramework $framework = null, AuthenticationHandlerInterface $twoFactorHandler = null, TrustedDeviceManagerInterface $trustedDeviceManager = null): AuthenticationProvider
     {
         $userProvider = $this->createMock(UserProviderInterface::class);

core-bundle/tests/Security/Authentication/Token/TokenCheckerTest.php

@@ -339,8 +339,6 @@ private function mockUser(string $class): User
     }
 
     /**
-     * @param Request&MockObject $request
-     *
      * @return RequestStack&MockObject
      */
     private function mockRequestStack(Request $request = null): RequestStack

core-bundle/tests/Security/User/ContaoUserProviderTest.php

@@ -282,9 +282,6 @@ public function onPostAuthenticate(): void
         // Dummy method to test the postAuthenticate hook
     }
 
-    /**
-     * @param ContaoFramework&MockObject $framework
-     */
     private function getProvider(ContaoFramework $framework = null, string $userClass = BackendUser::class): ContaoUserProvider
     {
         if (null === $framework) {

core-bundle/tests/Twig/Extension/ContaoTemplateExtensionTest.php

@@ -16,7 +16,6 @@
 use Contao\CoreBundle\Framework\ContaoFramework;
 use Contao\CoreBundle\Tests\TestCase;
 use Contao\CoreBundle\Twig\Extension\ContaoTemplateExtension;
-use PHPUnit\Framework\MockObject\MockObject;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\HttpFoundation\Response;
@@ -75,9 +74,6 @@ public function testDoesNotRenderTheBackEndTemplateIfNotInBackEndScope(): void
         $this->assertEmpty($this->getExtension(null, 'frontend')->renderContaoBackendTemplate());
     }
 
-    /**
-     * @param ContaoFramework&MockObject $framework
-     */
     private function getExtension(ContaoFramework $framework = null, string $scope = 'backend'): ContaoTemplateExtension
     {
         $request = new Request();

core-bundle/tests/Util/SimpleTokenParserTest.php

@@ -524,11 +524,9 @@ public function parseSimpleTokensInvalidComparison(): \Generator
     public function testParseSimpleTokenWithCustomExtensionProvider(): void
     {
         $stringExtensionProvider = new class() implements ExpressionFunctionProviderInterface {
-            public function getFunctions()
+            public function getFunctions(): array
             {
-                return [
-                    ExpressionFunction::fromPhp('strtoupper'),
-                ];
+                return [ExpressionFunction::fromPhp('strtoupper')];
             }
         };