minkphp · aik099 · Jan 13, 2025 · Jan 5, 2025 · Jan 5, 2025 · Jan 6, 2025
diff --git a/http-kernel-fixtures/cookie_page2.php b/http-kernel-fixtures/cookie_page2.php
@@ -9,7 +9,7 @@
 <body>
     Previous cookie: <?php
         /** @var \Symfony\Component\HttpFoundation\Request $request */
-        echo $request->cookies->has('srvr_cookie') ? html_escape_value($request->cookies->get('srvr_cookie')) : 'NO';
+        echo $request->cookies->has('srvr_cookie') ? html_escape_value($request->cookies->get('srvr_cookie') ?? '') : 'NO';
     ?>
 </body>
 </html>
diff --git a/http-kernel-fixtures/sub-folder/cookie_page1.php b/http-kernel-fixtures/sub-folder/cookie_page1.php
@@ -1,6 +1,7 @@
 <?php
     /** @var \Symfony\Component\HttpFoundation\Request $request */
     $requestUri = $request->server->get('REQUEST_URI');
+    assert(is_string($requestUri));
     $resp = new Symfony\Component\HttpFoundation\Response();
     $cook = Symfony\Component\HttpFoundation\Cookie::create('srvr_cookie', 'srv_var_is_set_sub_folder', 0, dirname($requestUri));
     $resp->headers->setCookie($cook);

diff --git a/http-kernel-fixtures/sub-folder/cookie_page4.php b/http-kernel-fixtures/sub-folder/cookie_page4.php
@@ -1,7 +1,10 @@
 <?php
     /** @var \Symfony\Component\HttpFoundation\Request $request */
     $resp = new Symfony\Component\HttpFoundation\Response();
-    $cookiePath = dirname($request->server->get('REQUEST_URI')) . '/';
+    $requestUri = $request->server->get('REQUEST_URI');
+    assert(is_string($requestUri));
+    $cookiePath = dirname($requestUri) . '/';
+    assert(is_string($cookiePath));
     $cookie = Symfony\Component\HttpFoundation\Cookie::create('srvr_cookie', 'srv_var_is_set', 0, $cookiePath);
     $resp->headers->setCookie($cookie);
 ?>

diff --git a/phpstan.dist.neon b/phpstan.dist.neon
@@ -1,11 +1,10 @@
 parameters:
-    level: 8
+    level: 9
     paths:
         - src
         - tests
         - http-kernel-fixtures
         - web-fixtures
-    checkMissingIterableValueType: false
 
 includes:
     - vendor/phpstan/phpstan-phpunit/extension.neon

diff --git a/src/FixturesKernel.php b/src/FixturesKernel.php
@@ -2,6 +2,7 @@
 
 namespace Behat\Mink\Tests\Driver\Util;
 
+use Behat\Mink\Tests\Driver\TestCase;
 use Symfony\Component\HttpFoundation\Cookie;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
@@ -25,8 +26,8 @@ public function handle(Request $request, $type = 1 /* self::MAIN_REQUEST */ , $c
 
     private function handleFixtureRequest(Request $request): Response
     {
-        $fixturesDir = realpath(__DIR__ . '/../web-fixtures');
-        $overwriteDir = realpath(__DIR__ . '/../http-kernel-fixtures');
+        $fixturesDir = realpath(TestCase::WEB_FIXTURES_DIR);
+        $overwriteDir = realpath(TestCase::KERNEL_FIXTURES_DIR);
 
         require_once $fixturesDir . '/utils.php';
 
@@ -60,8 +61,9 @@ private function prepareSession(Request $request): void
 
         $cookies = $request->cookies;
 
-        if ($cookies->has($session->getName())) {
-            $session->setId($cookies->get($session->getName()));
+        $value = $cookies->get($session->getName());
+        if ($value !== null) {
+            $session->setId($value);
         } else {
             $session->migrate(false);
         }

diff --git a/tests/AbstractConfig.php b/tests/AbstractConfig.php
@@ -22,7 +22,7 @@ abstract public function createDriver();
      */
     public function mapRemoteFilePath($file)
     {
-        if (!isset($_SERVER['TEST_MACHINE_BASE_PATH']) || !isset($_SERVER['DRIVER_MACHINE_BASE_PATH'])) {
+        if (!isset($_SERVER['TEST_MACHINE_BASE_PATH'], $_SERVER['DRIVER_MACHINE_BASE_PATH'])) {
             return $file;
         }
 

diff --git a/tests/Basic/BasicAuthTest.php b/tests/Basic/BasicAuthTest.php
@@ -20,12 +20,13 @@ public function testSetBasicAuth(string $user, string $pass, string $pageText):
         $this->assertStringContainsString($pageText, $session->getPage()->getContent());
     }
 
+    /**
+     * @return iterable<string, array{string, string, string}>
+     */
     public static function setBasicAuthDataProvider(): iterable
     {
-        return [
-            ['mink-user', 'mink-password', 'is authenticated'],
-            ['', '', 'is not authenticated'],
-        ];
+        yield 'valid credentials' => ['mink-user', 'mink-password', 'is authenticated'];
+        yield 'no credentials' => ['', '', 'is not authenticated'];
     }
 
     public function testBasicAuthInUrl(): void

diff --git a/tests/Basic/BestPracticesTest.php b/tests/Basic/BestPracticesTest.php
@@ -24,9 +24,9 @@ public function testImplementFindXpath(): void
     {
         $driver = $this->createDriver();
 
-        $this->assertNotImplementMethod('find', $driver, 'The driver should overwrite `findElementXpaths` rather than `find` for forward compatibility with Mink 2.');
-        $this->assertImplementMethod('findElementXpaths', $driver, 'The driver must be able to find elements.');
-        $this->assertNotImplementMethod('setSession', $driver, 'The driver should not deal with the Session directly for forward compatibility with Mink 2.');
+        $this->assertMethodIsNotImplemented('find', $driver, 'The driver should overwrite `findElementXpaths` rather than `find` for forward compatibility with Mink 2.');
+        $this->assertMethodIsImplemented('findElementXpaths', $driver, 'The driver must be able to find elements.');
+        $this->assertMethodIsNotImplemented('setSession', $driver, 'The driver should not deal with the Session directly for forward compatibility with Mink 2.');
     }
 
     /**
@@ -36,9 +36,12 @@ public function testImplementBasicApi(string $method): void
     {
         $driver = $this->createDriver();
 
-        $this->assertImplementMethod($method, $driver, 'The driver is unusable when this method is not implemented.');
+        $this->assertMethodIsImplemented($method, $driver, 'The driver is unusable when this method is not implemented.');
     }
 
+    /**
+     * @return iterable<array{string}>
+     */
     public static function provideRequiredMethods(): iterable
     {
         return [
@@ -53,7 +56,7 @@ public static function provideRequiredMethods(): iterable
         ];
     }
 
-    private function assertImplementMethod(string $method, object $object, string $reason = ''): void
+    private function assertMethodIsImplemented(string $method, object $object, string $reason = ''): void
     {
         $ref = new \ReflectionClass(get_class($object));
         $refMethod = $ref->getMethod($method);
@@ -67,7 +70,7 @@ private function assertImplementMethod(string $method, object $object, string $r
         $this->assertNotSame(CoreDriver::class, $refMethod->getDeclaringClass()->name, $message);
     }
 
-    private function assertNotImplementMethod(string $method, object $object, string $reason = ''): void
+    private function assertMethodIsNotImplemented(string $method, object $object, string $reason = ''): void
     {
         $ref = new \ReflectionClass(get_class($object));
         $refMethod = $ref->getMethod($method);

diff --git a/tests/Basic/ContentTest.php b/tests/Basic/ContentTest.php
@@ -61,6 +61,9 @@ public function testGetAttribute(string $attributeName, ?string $attributeValue)
         $this->assertSame($attributeValue, $element->getAttribute($attributeName));
     }
 
+    /**
+     * @return iterable<array{string, mixed}>
+     */
     public static function getAttributeDataProvider(): iterable
     {
         return [

diff --git a/tests/Basic/CookieTest.php b/tests/Basic/CookieTest.php
@@ -4,6 +4,9 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
+/**
+ * @phpstan-type TCookieRemovalMode 'session_reset'|'cookie_delete'
+ */
 final class CookieTest extends TestCase
 {
     /**
@@ -87,6 +90,9 @@ public function testCookieWithPaths(string $cookieRemovalMode): void
         $this->assertStringContainsString('Previous cookie: NO', $session->getPage()->getText());
     }
 
+    /**
+     * @phpstan-return iterable<array{TCookieRemovalMode}>
+     */
     public static function cookieWithPathsDataProvider(): iterable
     {
         return [
@@ -96,6 +102,7 @@ public static function cookieWithPathsDataProvider(): iterable
     }
 
     /**
+     * @phpstan-param TCookieRemovalMode $cookieRemovalMode
      * @dataProvider cookieWithPathsDataProvider
      */
     public function testCookieInSubPath(string $cookieRemovalMode): void

diff --git a/tests/Basic/IFrameTest.php b/tests/Basic/IFrameTest.php
@@ -29,13 +29,11 @@ public function testIFrame(string $iframeIdentifier, string $elementSelector, st
     }
 
     /**
-     * @return array
+     * @return iterable<string, array{string, string, string}>
      */
-    public static function iFrameDataProvider()
+    public static function iFrameDataProvider(): iterable
     {
-        return array(
-            'by name' => array('subframe_by_name', '#text', 'iFrame div text'),
-            'by id' => array('subframe_by_id', '#foobar', 'Some accentués characters'),
-        );
+        yield 'by name' => ['subframe_by_name', '#text', 'iFrame div text'];
+        yield 'by id' => ['subframe_by_id', '#foobar', 'Some accentués characters'];
     }
 }
diff --git a/tests/Form/GeneralTest.php b/tests/Form/GeneralTest.php
@@ -4,6 +4,7 @@
 
 use Behat\Mink\Exception\DriverException;
 use Behat\Mink\Tests\Driver\TestCase;
+use Behat\Mink\Tests\Driver\Util\FixturesKernel;
 
 final class GeneralTest extends TestCase
 {
@@ -77,14 +78,15 @@ public function testFormSubmitWays(string $submitVia): void
         }
     }
 
+    /**
+     * @return iterable<array{string}>
+     */
     public static function formSubmitWaysDataProvider(): iterable
     {
-        return [
-            ['Save'],
-            ['input-type-image'],
-            ['button-without-type'],
-            ['button-type-submit'],
-        ];
+        yield ['Save'];
+        yield ['input-type-image'];
+        yield ['button-without-type'];
+        yield ['button-type-submit'];
     }
 
     public function testFormSubmit(): void
@@ -189,7 +191,7 @@ public function testAdvancedForm(): void
         $notes->setValue('new notes');
         $this->assertEquals('new notes', $notes->getValue());
 
-        $about->attachFile($this->mapRemoteFilePath(__DIR__ . '/../../web-fixtures/some_file.txt'));
+        $about->attachFile($this->mapRemoteFilePath(self::WEB_FIXTURES_DIR . '/some_file.txt'));
 
         $button = $page->findButton('Register');
         $this->assertNotNull($button);
@@ -352,7 +354,7 @@ public function testSubmitEmptyTextarea(): void
     /**
      * @dataProvider provideInvalidValues
      *
-     * @param mixed $value
+     * @param array<array-key, mixed>|bool|string $value
      */
     public function testSetInvalidValueInField(string $field, $value): void
     {
@@ -366,6 +368,9 @@ public function testSetInvalidValueInField(string $field, $value): void
         $color->setValue($value);
     }
 
+    /**
+     * @return iterable<string, array{string, mixed}>
+     */
     public static function provideInvalidValues(): iterable
     {
         $trueValue = ['true', true];

diff --git a/tests/Form/Html5Test.php b/tests/Form/Html5Test.php
@@ -163,7 +163,7 @@ public function testHtml5FormMethod(): void
     /**
      * @dataProvider provideInvalidValues
      *
-     * @param mixed $value
+     * @param array<array-key, mixed>|bool|string $value
      */
     public function testSetInvalidValueInField(string $field, $value): void
     {
@@ -177,6 +177,9 @@ public function testSetInvalidValueInField(string $field, $value): void
         $color->setValue($value);
     }
 
+    /**
+     * @return iterable<string, array{string, mixed}>
+     */
     public static function provideInvalidValues(): iterable
     {
         $trueValue = ['true', true];

diff --git a/tests/Form/RadioTest.php b/tests/Form/RadioTest.php
@@ -112,6 +112,9 @@ public function testSetBooleanValue(bool $value): void
         $option->setValue($value);
     }
 
+    /**
+     * @return iterable<array{bool}>
+     */
     public static function provideBooleanValues(): iterable
     {
         yield [true];

diff --git a/tests/Form/SelectTest.php b/tests/Form/SelectTest.php
@@ -78,6 +78,9 @@ public function testElementSelectedStateCheck(string $selectName, string $option
         $this->assertTrue($option->isSelected());
     }
 
+    /**
+     * @return iterable<array{string, string, string}>
+     */
     public static function elementSelectedStateCheckDataProvider(): iterable
     {
         return [
@@ -119,6 +122,9 @@ public function testSetBooleanValue(bool $value): void
         $select->setValue($value);
     }
 
+    /**
+     * @return iterable<array{bool}>
+     */
     public static function provideBooleanValues(): iterable
     {
         yield [true];

diff --git a/tests/Js/ChangeEventTest.php b/tests/Js/ChangeEventTest.php
@@ -3,6 +3,7 @@
 namespace Behat\Mink\Tests\Driver\Js;
 
 use Behat\Mink\Tests\Driver\TestCase;
+use Behat\Mink\Tests\Driver\Util\FixturesKernel;
 
 /**
  * @group slow
@@ -63,20 +64,18 @@ public function testSetValueChangeEvent(string $elementId, string $valueForEmpty
         }
     }
 
+    /**
+     * @return iterable<string, array{string, string, string}>
+     */
     public static function setValueChangeEventDataProvider(): iterable
     {
-        $file1 = __DIR__ . '/../../web-fixtures/file1.txt';
-        $file2 = __DIR__ . '/../../web-fixtures/file2.txt';
-
-        return [
-            'input default' => ['the-input-default', 'from empty', 'from existing'],
-            'input text' => ['the-input-text', 'from empty', 'from existing'],
-            'input email' => ['the-email', 'from empty', 'from existing'],
-            'textarea' => ['the-textarea', 'from empty', 'from existing'],
-            'file' => ['the-file', $file1, $file2],
-            'select' => ['the-select', '30'],
-            'radio' => ['the-radio-m', 'm'],
-        ];
+        yield 'input default' => ['the-input-default', 'from empty', 'from existing'];
+        yield 'input text' => ['the-input-text', 'from empty', 'from existing'];
+        yield 'input email' => ['the-email', 'from empty', 'from existing'];
+        yield 'textarea' => ['the-textarea', 'from empty', 'from existing'];
+        yield 'file' => ['the-file', self::WEB_FIXTURES_DIR . '/file1.txt', self::WEB_FIXTURES_DIR . '/file2.txt'];
+        yield 'select' => ['the-select', '30', ''];
+        yield 'radio' => ['the-radio-m', 'm', ''];
     }
 
     /**
@@ -95,12 +94,13 @@ public function testSelectOptionChangeEvent(string $elementId, string $elementVa
         $this->assertElementChangeCount($elementId);
     }
 
+    /**
+     * @return iterable<string, array{string, string}>
+     */
     public static function selectOptionChangeEventDataProvider(): iterable
     {
-        return [
-            'select' => ['the-select', '30'],
-            'radio' => ['the-radio-m', 'm'],
-        ];
+        yield 'select' => ['the-select', '30'];
+        yield 'radio' => ['the-radio-m', 'm'];
     }
 
     /**
@@ -145,12 +145,13 @@ public function testUncheckChangeEvent(bool $useSetValue): void
         $this->assertElementChangeCount('the-checked-checkbox');
     }
 
+    /**
+     * @return iterable<array{mixed}>
+     */
     public static function checkboxTestWayDataProvider(): iterable
     {
-        return [
-            [true],
-            [false],
-        ];
+        yield [true];
+        yield [false];
     }
 
     private function assertElementChangeCount(string $elementId, string $message = ''): void