chore: support Symfony 8 (#184)

* chore: support Symfony 8

* mb_trim to trim

---------

Co-authored-by: Kevin Bond <kevinbond@gmail.com>

Author: nikophil

.gitattributes

@@ -1,2 +1,3 @@
 /.github export-ignore
 /tests export-ignore
+/stubs export-ignore

composer.json

@@ -12,25 +12,26 @@
         }
     ],
     "require": {
-        "php": ">=8.0",
+        "php": ">=8.1",
         "behat/mink": "^1.8",
-        "symfony/browser-kit": "^5.4|^6.0|^7.0",
-        "symfony/css-selector": "^5.4|^6.0|^7.0",
-        "symfony/dom-crawler": "^5.4|^6.0|^7.0",
-        "symfony/framework-bundle": "^5.4|^6.0|^7.0",
+        "symfony/browser-kit": "^6.4|^7.0|^8.0",
+        "symfony/css-selector": "^6.4|^7.0|^8.0",
+        "symfony/dom-crawler": "^6.4|^7.0|^8.0",
+        "symfony/framework-bundle": "^6.4|^7.0|^8.0",
         "zenstruck/assert": "^1.1",
         "zenstruck/callback": "^1.4.2"
     },
     "require-dev": {
         "dbrekelmans/bdi": "^1.0",
         "justinrainbow/json-schema": "^5.3",
         "mtdowling/jmespath.php": "^2.6",
-        "phpstan/phpstan": "^1.4",
+        "phpstan/phpstan": "^2.0",
         "phpunit/phpunit": "^9.6.21|^10.4",
-        "symfony/mime": "^5.4|^6.0|^7.0",
+        "symfony/console": "^6.4|^7.0|^8.0",
+        "symfony/mime": "^6.4|^7.0|^8.0",
         "symfony/panther": "^1.1|^2.0.1",
-        "symfony/phpunit-bridge": "^6.0|^7.0",
-        "symfony/security-bundle": "^5.4|^6.0|^7.0"
+        "symfony/phpunit-bridge": "^6.4|^7.0|^8.0",
+        "symfony/security-bundle": "^6.4|^7.0|^8.0"
     },
     "suggest": {
         "justinrainbow/json-schema": "Json schema validator. Needed to use Json::assertMatchesSchema().",

phpstan-baseline.neon

@@ -1,10 +1,8 @@
-includes:
-    - phpstan-baseline.neon
-
 parameters:
-    level: 8
+    level: 5
     paths:
         - src
+        - stubs
 
     excludePaths:
         # skip phpunit 10 missing classes

phpstan.neon

@@ -2,7 +2,7 @@
 <!-- https://phpunit.de/manual/current/en/appendixes.configuration.html -->
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd"
-         bootstrap="vendor/autoload.php"
+         bootstrap="tests/bootstrap.php"
          failOnRisky="true"
          failOnWarning="true"
          colors="true"

phpunit.xml.dist

@@ -15,8 +15,10 @@
 use Psr\Container\ContainerInterface;
 use Symfony\Component\BrowserKit\AbstractBrowser;
 use Symfony\Component\BrowserKit\CookieJar;
+use Symfony\Component\BrowserKit\Request;
 use Symfony\Component\DomCrawler\Crawler;
 use Symfony\Component\Filesystem\Filesystem;
+use Symfony\Component\HttpFoundation\Response;
 use Zenstruck\Browser\Assertion\SameUrlAssertion;
 use Zenstruck\Browser\Component;
 use Zenstruck\Browser\Session;
@@ -47,6 +49,9 @@ public function __construct(Driver $driver, array $options = [])
         $this->sourceDebug = $options['source_debug'] ?? false;
     }
 
+    /**
+     * @return AbstractBrowser<Request, Response>
+     */
     final public function client(): AbstractBrowser
     {
         return $this->session->client();
@@ -409,7 +414,7 @@ final public function use(callable $callback): self
     final public function saveSource(string $filename): self
     {
         if ($this->sourceDir) {
-            $filename = \sprintf('%s/%s', mb_rtrim($this->sourceDir, '/'), mb_ltrim($filename, '/'));
+            $filename = \sprintf('%s/%s', \rtrim($this->sourceDir, '/'), \ltrim($filename, '/'));
         }
 
         (new Filesystem())->dumpFile($this->savedSources[] = $filename, $this->session()->source($this->sourceDebug));

src/Browser.php

@@ -167,7 +167,7 @@ public function decoded()
             return $this->decoded;
         }
 
-        if ('' === mb_trim($this->source)) {
+        if ('' === \trim($this->source)) {
             return null;
         }
 

src/Browser/Json.php

@@ -43,7 +43,7 @@ final public function __construct(SymfonyKernelBrowser $client, array $options =
         $client->followRedirects((bool) ($options['follow_redirects'] ?? true));
         $client->catchExceptions((bool) ($options['catch_exceptions'] ?? true));
 
-        parent::__construct(new BrowserKitDriver($client), $options);
+        parent::__construct(new BrowserKitDriver($client), $options); // @phpstan-ignore argument.type
     }
 
     /**

src/Browser/KernelBrowser.php

@@ -149,7 +149,7 @@ final public function pause(): self
     final public function takeScreenshot(string $filename): self
     {
         if ($this->screenshotDir) {
-            $filename = \sprintf('%s/%s', mb_rtrim($this->screenshotDir, '/'), mb_ltrim($filename, '/'));
+            $filename = \sprintf('%s/%s', \rtrim($this->screenshotDir, '/'), \ltrim($filename, '/'));
         }
 
         $this->client()->takeScreenshot($this->savedScreenshots[] = $filename);
@@ -160,7 +160,7 @@ final public function takeScreenshot(string $filename): self
     final public function saveConsoleLog(string $filename): self
     {
         if ($this->consoleLogDir) {
-            $filename = \sprintf('%s/%s', mb_rtrim($this->consoleLogDir, '/'), mb_ltrim($filename, '/'));
+            $filename = \sprintf('%s/%s', \rtrim($this->consoleLogDir, '/'), \ltrim($filename, '/'));
         }
 
         $log = $this->client()->manage()->getLog('browser');

src/Browser/PantherBrowser.php

@@ -17,7 +17,9 @@
 use Behat\Mink\Session as MinkSession;
 use Behat\Mink\WebAssert;
 use Symfony\Component\BrowserKit\AbstractBrowser;
+use Symfony\Component\BrowserKit\Request;
 use Symfony\Component\DomCrawler\Crawler;
+use Symfony\Component\HttpFoundation\Response;
 use Zenstruck\Assert as ZenstruckAssert;
 use Zenstruck\Browser\Session\Assert;
 use Zenstruck\Browser\Session\Driver;
@@ -44,6 +46,9 @@ public static function varDump($what): void
         \function_exists('dump') ? dump($what) : \var_dump($what);
     }
 
+    /**
+     * @return AbstractBrowser<Request, Response>
+     */
     public function client(): AbstractBrowser
     {
         return $this->getDriver()->client();