From d4ee3a8525d536f910e220bc47ff69dd95ea7bc6 Mon Sep 17 00:00:00 2001
From: Alexander Schranz <alexander@sulu.io>
Date: Fri, 5 Apr 2024 12:05:36 +0200
Subject: [PATCH] Fix phpunit installation

---
 .github/workflows/test-application.yaml |  9 +++++++++
 DependencyInjection/Configuration.php   |  2 +-
 Tests/Build/BuildRegistryTest.php       |  3 +++
 Tests/Build/BuilderContextTest.php      |  3 +++
 Tests/Command/BuildCommandTest.php      |  3 +++
 Tests/ProphecyTrait.php                 | 14 ++++++++++++++
 composer.json                           |  5 ++---
 phpunit.xml.dist                        |  1 +
 8 files changed, 36 insertions(+), 4 deletions(-)
 create mode 100644 Tests/ProphecyTrait.php

diff --git a/.github/workflows/test-application.yaml b/.github/workflows/test-application.yaml
index d0e2c64..ed51c45 100644
--- a/.github/workflows/test-application.yaml
+++ b/.github/workflows/test-application.yaml
@@ -19,10 +19,14 @@ jobs:
                     - php-version: '7.2'
                       dependency-versions: 'lowest'
                       tools: 'composer:v1'
+                      envs:
+                          SYMFONY_PHPUNIT_REQUIRE: phpspec/prophecy:^1.14
 
                     - php-version: '7.4'
                       dependency-versions: 'highest'
                       tools: 'composer:v2'
+                      envs:
+                          SYMFONY_PHPUNIT_REQUIRE: phpspec/prophecy:^1.14
 
                     - php-version: '8.0'
                       dependency-versions: 'highest'
@@ -60,6 +64,11 @@ jobs:
               with:
                   dependency-versions: ${{matrix.dependency-versions}}
 
+            - name: Execute test cases
+              run: vendor/bin/simple-phpunit install
+              env:
+                  SYMFONY_PHPUNIT_REQUIRE: ${{ matrix.envs.SYMFONY_PHPUNIT_REQUIRE }}
+
             - name: Execute test cases
               run: vendor/bin/simple-phpunit
 
diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
index dc377ac..ac18884 100644
--- a/DependencyInjection/Configuration.php
+++ b/DependencyInjection/Configuration.php
@@ -21,7 +21,7 @@
  */
 class Configuration implements ConfigurationInterface
 {
-    public function getConfigTreeBuilder()
+    public function getConfigTreeBuilder(): TreeBuilder
     {
         $treeBuilder = new TreeBuilder('massive_art_build');
         $rootNode = $treeBuilder->getRootNode();
diff --git a/Tests/Build/BuildRegistryTest.php b/Tests/Build/BuildRegistryTest.php
index b30769e..58e3c25 100644
--- a/Tests/Build/BuildRegistryTest.php
+++ b/Tests/Build/BuildRegistryTest.php
@@ -14,9 +14,12 @@
 use Massive\Bundle\BuildBundle\Build\BuilderInterface;
 use Massive\Bundle\BuildBundle\Build\BuildRegistry;
 use Massive\Bundle\BuildBundle\Tests\BaseTestCase;
+use Massive\Bundle\BuildBundle\Tests\ProphecyTrait;
 
 class BuildRegistryTest extends BaseTestCase
 {
+    use ProphecyTrait;
+
     /**
      * @var BuildRegistry
      */
diff --git a/Tests/Build/BuilderContextTest.php b/Tests/Build/BuilderContextTest.php
index 7301a6c..35cb1af 100644
--- a/Tests/Build/BuilderContextTest.php
+++ b/Tests/Build/BuilderContextTest.php
@@ -13,9 +13,12 @@
 
 use Massive\Bundle\BuildBundle\Build\BuilderContext;
 use Massive\Bundle\BuildBundle\Tests\BaseTestCase;
+use Massive\Bundle\BuildBundle\Tests\ProphecyTrait;
 
 class BuilderContextTest extends BaseTestCase
 {
+    use ProphecyTrait;
+
     protected $input;
     protected $output;
 
diff --git a/Tests/Command/BuildCommandTest.php b/Tests/Command/BuildCommandTest.php
index 0653bfc..2823f69 100644
--- a/Tests/Command/BuildCommandTest.php
+++ b/Tests/Command/BuildCommandTest.php
@@ -15,11 +15,14 @@
 use Massive\Bundle\BuildBundle\Command\BuildCommand;
 use Massive\Bundle\BuildBundle\ContainerAwareInterface;
 use Massive\Bundle\BuildBundle\Tests\BaseTestCase;
+use Massive\Bundle\BuildBundle\Tests\ProphecyTrait;
 use Prophecy\Argument;
 use Symfony\Component\Console\Tester\CommandTester;
 
 class BuildCommandTest extends BaseTestCase
 {
+    use ProphecyTrait;
+
     public function setUp()
     {
         parent::setUp();
diff --git a/Tests/ProphecyTrait.php b/Tests/ProphecyTrait.php
new file mode 100644
index 0000000..b8887af
--- /dev/null
+++ b/Tests/ProphecyTrait.php
@@ -0,0 +1,14 @@
+<?php
+
+namespace Massive\Bundle\BuildBundle\Tests;
+
+use Prophecy\PhpUnit\ProphecyTrait as ProphecyProphecyTrait;
+
+if (\trait_exists(ProphecyProphecyTrait::class)) {
+    \class_alias(ProphecyProphecyTrait::class, ProphecyTrait::class);
+} else {
+    trait ProphecyTrait
+    {
+        // provided by phpunit 8
+    }
+}
diff --git a/composer.json b/composer.json
index c10ff63..0ba301c 100644
--- a/composer.json
+++ b/composer.json
@@ -17,9 +17,8 @@
         "symfony/framework-bundle": "^4.3 || ^5.0 || ^6.0 || ^7.0"
     },
     "require-dev": {
-        "symfony/phpunit-bridge": "^5.0.4 || ^6.0 || ^7.0",
-        "matthiasnoback/symfony-dependency-injection-test": "^4.0 || ^5.0",
-        "phpspec/prophecy": "^1.14"
+        "symfony/phpunit-bridge": "^5.4.33|^6.3.10|^7.0.1",
+        "matthiasnoback/symfony-dependency-injection-test": "^4.0 || ^5.0"
     },
     "autoload": {
         "psr-4": {"Massive\\Bundle\\BuildBundle\\": "."},
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
index 8c3b0f3..be7c19f 100644
--- a/phpunit.xml.dist
+++ b/phpunit.xml.dist
@@ -19,5 +19,6 @@
         <env name="SYMFONY_DEPRECATIONS_HELPER" value="weak"/>
         <env name="SYMFONY_PHPUNIT_REMOVE_RETURN_TYPEHINT" value="1"/>
         <env name="SYMFONY_PHPUNIT_REMOVE" value="symfony/yaml"/>
+        <env name="SYMFONY_PHPUNIT_REQUIRE" value="phpspec/prophecy:^1.14 phpspec/prophecy-phpunit:^2.0"/>
     </php>
 </phpunit>