[Turbo] Fix Turbo CI (PHPStan & Functional tests)

Author: Kocal

.github/workflows/code-quality.yaml

@@ -35,16 +35,27 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                package: ${{ fromJson(needs.php-packages-matrix.outputs.packages) }}
+                php-version: [ '8.1', '8.2', '8.3', '8.4']
+                dependency-version: ['']
+                symfony-version: ['']
+                minimum-stability: ['stable']
+                include:
+                    # lowest deps
+                    -   php-version: '8.1'
+                        dependency-version: 'lowest'
+                    # LTS version of Symfony
+                    -   php-version: '8.1'
+                        symfony-version: '6.4.*'
         steps:
             -   name: Checkout
                 uses: actions/checkout@v4
 
             - name: Configure environment
               run: |
                   echo COLUMNS=120 >> $GITHUB_ENV
-                  echo COMPOSER_MIN_STAB='composer config minimum-stability stable --ansi' >> $GITHUB_ENV
-                  echo COMPOSER_UP='composer update --no-progress --no-interaction --ansi' >> $GITHUB_ENV
+                  echo COMPOSER_MIN_STAB='composer config minimum-stability ${{ matrix.minimum-stability || 'stable' }} --ansi' >> $GITHUB_ENV
+                  echo COMPOSER_UP='composer update ${{ matrix.dependency-version == 'lowest' && '--prefer-lowest' || '' }} --no-progress --no-interaction --ansi' >> $GITHUB_ENV
+                  echo PHPUNIT_INSTALL='vendor/bin/simple-phpunit install' >> $GITHUB_ENV
                   echo PHPSTAN='vendor/bin/phpstan' >> $GITHUB_ENV
 
                   # TODO: Only Turbo has PHPStan configuration, let's improve this later :)
@@ -69,4 +80,4 @@ jobs:
               run: |
                   source .github/workflows/.utils.sh
 
-                  echo "$PACKAGES" | xargs -n1 | parallel -j +3 "_run_task {} '(cd src/{} && $COMPOSER_MIN_STAB && $COMPOSER_UP && $PHPSTAN)'"
+                  echo "$PACKAGES" | xargs -n1 | parallel -j +3 "_run_task {} '(cd src/{} && $COMPOSER_MIN_STAB && $COMPOSER_UP && $PHPUNIT_INSTALL && $PHPSTAN)'"

.github/workflows/functional-tests.yml

@@ -3,9 +3,11 @@ name: Functional Tests
 on:
     push:
         paths:
+            - '.github/workflows/functional-tests.yml'
             - 'src/Turbo/**'
     pull_request:
         paths:
+            - '.github/workflows/functional-tests.yml'
             - 'src/Turbo/**'
 
 jobs:

.github/workflows/unit-tests.yaml

@@ -38,7 +38,7 @@ jobs:
                         symfony-version: '6.4.*'
 
         env:
-            SYMFONY_REQUIRE: ${{ matrix.symfony-version }}
+            SYMFONY_REQUIRE: ${{ matrix.symfony-version || '>=5.4' }}
         steps:
             - uses: actions/checkout@v4
 

src/Turbo/.gitattributes

@@ -1,7 +1,7 @@
 /.git* export-ignore
 /.symfony.bundle.yaml export-ignore
 /phpunit.xml.dist export-ignore
-/phpstan.neon.dist export-ignore
+/phpstan.dist.neon export-ignore
 /assets/src export-ignore
 /assets/test export-ignore
 /assets/vitest.config.js export-ignore

src/Turbo/composer.json

@@ -40,7 +40,7 @@
     "require-dev": {
         "doctrine/doctrine-bundle": "^2.4.3",
         "doctrine/orm": "^2.8 | 3.0",
-        "phpstan/phpstan": "^1.10",
+        "phpstan/phpstan": "^2.1.17",
         "symfony/asset-mapper": "^6.4|^7.0",
         "symfony/debug-bundle": "^5.4|^6.0|^7.0",
         "symfony/form": "^5.4|^6.0|^7.0",
@@ -60,7 +60,6 @@
         "dbrekelmans/bdi": "dev-main"
     },
     "conflict": {
-        "symfony/flex": "<1.13"
     },
     "extra": {
         "thanks": {

src/Turbo/phpstan.neon.dist renamed to src/Turbo/phpstan.dist.neon

@@ -7,6 +7,8 @@ parameters:
 		- tests/app/var/*
 		# The Symfony Configuration API isn't good enough to be analysed
 		- src/DependencyInjection/Configuration.php
+	# to allow installing with various phsptan versions without reporting old errors here
+	reportUnmatchedIgnoredErrors: false
 	inferPrivatePropertyTypeFromConstructor: true
 	bootstrapFiles:
 		- vendor/bin/.phpunit/phpunit/vendor/autoload.php
@@ -36,16 +38,6 @@ parameters:
 			count: 1
 			path: src/Bridge/Mercure/TurboStreamListenRenderer.php
 
-		-
-			message: "#^Call to an undefined method Doctrine\\\\ORM\\\\Event\\\\OnFlushEventArgs\\:\\:getEntityManager\\(\\)\\.$#"
-			count: 1
-			path: src/Doctrine/BroadcastListener.php
-
-		-
-			message: "#^Call to an undefined method Doctrine\\\\ORM\\\\Event\\\\PostFlushEventArgs\\:\\:getEntityManager\\(\\)\\.$#"
-			count: 1
-			path: src/Doctrine/BroadcastListener.php
-
 		-
 			message: "#^Method Symfony\\\\UX\\\\Turbo\\\\TurboStreamResponse::__construct\\(\\) has parameter \\$headers with no value type specified in iterable type array\\.$#"
 			count: 1

src/Turbo/src/Attribute/Broadcast.php

@@ -42,6 +42,7 @@ final class Broadcast
      */
     public function __construct(...$options)
     {
+        // @phpstan-ignore function.alreadyNarrowedType
         if ([0] === array_keys($options) && \is_array($options[0]) && \is_string(key($options[0]))) {
             $options = $options[0];
         }

src/Turbo/src/Bridge/Mercure/Broadcaster.php

@@ -68,11 +68,6 @@ public function broadcast(object $entity, string $action, array $options): void
         $topics = [];
 
         foreach ((array) ($options['topics'] ?? []) as $topic) {
-            if (!\is_string($topic)) {
-                $topics[] = $topic;
-                continue;
-            }
-
             if (!str_starts_with($topic, '@=')) {
                 $topics[] = $topic;
                 continue;

src/Turbo/src/Doctrine/BroadcastListener.php

@@ -61,6 +61,7 @@ public function onFlush(EventArgs $eventArgs): void
             return;
         }
 
+        // @phpstan-ignore function.alreadyNarrowedType, method.notFound (`getEntityManager()` has been removed in Doctrine 3.0)
         $em = method_exists($eventArgs, 'getObjectManager') ? $eventArgs->getObjectManager() : $eventArgs->getEntityManager();
         $uow = $em->getUnitOfWork();
         foreach ($uow->getScheduledEntityInsertions() as $entity) {
@@ -85,6 +86,7 @@ public function postFlush(EventArgs $eventArgs): void
             return;
         }
 
+        // @phpstan-ignore function.alreadyNarrowedType, method.notFound (`getEntityManager()` has been removed in Doctrine 3.0)
         $em = method_exists($eventArgs, 'getObjectManager') ? $eventArgs->getObjectManager() : $eventArgs->getEntityManager();
 
         try {