test: Fix the Composer orchestrator integration tests (#1143)

In #1082 the test cases where incorrectly refactored making them share the same condition.
box-project · Nov 4, 2023 · 0c76068 · 0c76068
1 parent e0e31f2
commit 0c76068
Show file tree

Hide file tree

Showing 4 changed files with 44 additions and 11 deletions.
diff --git a/tests/Composer/BaseComposerOrchestratorComposerTestCase.php b/tests/Composer/BaseComposerOrchestratorComposerTestCase.php
@@ -21,39 +21,46 @@
 use function file_exists;
 use function iterator_to_array;
 use function sprintf;
-use function version_compare;
 
 abstract class BaseComposerOrchestratorComposerTestCase extends FileSystemTestCase
 {
     protected const FIXTURES = __DIR__.'/../../fixtures/composer-dump';
     protected const COMPOSER_AUTOLOADER_NAME = 'ComposerAutoloaderInit80c62b20a4a44fb21e8e102ccb92ff05';
 
     protected ComposerOrchestrator $composerOrchestrator;
-    protected string $composerVersion;
+
     protected bool $skip;
+    protected string $skipReason;
 
     protected function setUp(): void
     {
         $this->composerOrchestrator = ComposerOrchestrator::create();
 
-        if (!isset($this->skip)) {
-            $this->composerVersion = $this->composerOrchestrator->getVersion();
+        if (!isset($this->skip, $this->skipReason)) {
+            $composerVersion = $this->composerOrchestrator->getVersion();
+
+            [$skip, $supportedConstraint] = $this->shouldSkip($composerVersion);
 
-            $this->skip = version_compare($this->composerVersion, '2.3.0', '>=');
+            $this->skip = $skip;
+            $this->skipReason = sprintf(
+                'Can only be executed with Composer %s. Got "%s".',
+                $supportedConstraint,
+                $composerVersion,
+            );
         }
 
         if ($this->skip) {
-            self::markTestSkipped(
-                sprintf(
-                    'Can only be executed with Composer ~2.2.0. Got "%s".',
-                    $this->composerVersion,
-                ),
-            );
+            self::markTestSkipped($this->skipReason);
         }
 
         parent::setUp();
     }
 
+    /**
+     * @return array{bool, string}
+     */
+    abstract protected function shouldSkip(string $composerVersion): array;
+
     /**
      * @param array<array{string, string}> $recordedClasses
      * @param array<array{string, string}> $recordedFunctions

diff --git a/tests/Composer/ComposerOrchestratorComposer22TestCase.php b/tests/Composer/ComposerOrchestratorComposer22TestCase.php
@@ -20,6 +20,7 @@
 use RuntimeException;
 use function file_get_contents;
 use function preg_replace;
+use function version_compare;
 
 /**
  * @covers \KevinGH\Box\Composer\AutoloadDumper
@@ -30,6 +31,14 @@
  */
 class ComposerOrchestratorComposer22TestCase extends BaseComposerOrchestratorComposerTestCase
 {
+    protected function shouldSkip(string $composerVersion): array
+    {
+        return [
+            version_compare($composerVersion, '2.3.0', '>='),
+            '~2.2.0',
+        ];
+    }
+
     /**
      * @dataProvider composerAutoloadProvider
      */

diff --git a/tests/Composer/ComposerOrchestratorComposer23TestCase.php b/tests/Composer/ComposerOrchestratorComposer23TestCase.php
@@ -30,6 +30,15 @@
  */
 class ComposerOrchestratorComposer23TestCase extends BaseComposerOrchestratorComposerTestCase
 {
+    protected function shouldSkip(string $composerVersion): array
+    {
+        return [
+            version_compare($composerVersion, '2.3.0', '<')
+                || version_compare($composerVersion, '2.4.0', '>='),
+            '~2.3.0',
+        ];
+    }
+
     /**
      * @dataProvider composerAutoloadProvider
      */

diff --git a/tests/Composer/ComposerOrchestratorComposer24TestCase.php b/tests/Composer/ComposerOrchestratorComposer24TestCase.php
@@ -30,6 +30,14 @@
  */
 class ComposerOrchestratorComposer24TestCase extends BaseComposerOrchestratorComposerTestCase
 {
+    protected function shouldSkip(string $composerVersion): array
+    {
+        return [
+            version_compare($composerVersion, '2.4.0', '<'),
+            '>=2.4.0',
+        ];
+    }
+
     /**
      * @dataProvider composerAutoloadProvider
      */