Fix: Detect PHPUnit version based on series instead of identifier

Co-authored-by: Andreas Möller <[email protected]>
Co-authored-by: Michael Voříšek <[email protected]>

Author: localheinz

CHANGELOG.md

@@ -14,6 +14,10 @@ For a full diff see [`2.4.0...main`][2.4.0...main].
 - Added support for PHP 8.0 ([#375]), by [@localheinz] and [@mvorisek]
 - Added support for PHP 7.4 ([#390]), by [@localheinz] and [@mvorisek]
 
+### Changed
+
+- Improved detection of PHPUnit version ([#393]), by [@localheinz] and [@mvorisek]
+
 ## [`2.4.0`][2.4.0]
 
 For a full diff see [`2.3.2...2.4.0`][2.3.2...2.4.0].
@@ -204,6 +208,7 @@ For a full diff see [`7afa59c...1.0.0`][7afa59c...1.0.0].
 [#367]: https://github.com/ergebnis/phpunit-slow-test-detector/pull/367
 [#375]: https://github.com/ergebnis/phpunit-slow-test-detector/pull/375
 [#390]: https://github.com/ergebnis/phpunit-slow-test-detector/pull/390
+[#393]: https://github.com/ergebnis/phpunit-slow-test-detector/pull/393
 
 [@HypeMC]: https://github.com/HypeMC
 [@localheinz]: https://github.com/localheinz

src/Extension.php

@@ -18,15 +18,15 @@
 use PHPUnit\Util;
 
 try {
-    $phpUnitVersion = Version\Version::fromString(Runner\Version::id());
+    $phpUnitVersionSeries = Version\Series::fromString(Runner\Version::series());
 } catch (\InvalidArgumentException $exception) {
     throw new \RuntimeException(\sprintf(
-        'Unable to determine PHPUnit version from version identifier "%s".',
-        Runner\Version::id(),
+        'Unable to determine PHPUnit version from version series "%s".',
+        Runner\Version::series(),
     ));
 }
 
-if ($phpUnitVersion->major()->equals(Version\Major::fromInt(9))) {
+if ($phpUnitVersionSeries->major()->equals(Version\Major::fromInt(9))) {
     /**
      * @internal
      */
@@ -170,7 +170,7 @@ private function resolveMaximumDuration(string $test): Duration
     return;
 }
 
-if ($phpUnitVersion->major()->equals(Version\Major::fromInt(10))) {
+if ($phpUnitVersionSeries->major()->equals(Version\Major::fromInt(10))) {
     /**
      * @internal
      */
@@ -224,6 +224,6 @@ public function bootstrap(
 }
 
     throw new \RuntimeException(\sprintf(
-        'Unable to select extension for PHPUnit version with version identifier "%s".',
-        Runner\Version::id(),
+        'Unable to select extension for PHPUnit version with version series "%s".',
+        Runner\Version::series(),
     ));

src/Version/Minor.php

@@ -0,0 +1,54 @@
+<?php
+
+declare(strict_types=1);
+
+/**
+ * Copyright (c) 2021-2023 Andreas Möller
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE.md file that was distributed with this source code.
+ *
+ * @see https://github.com/ergebnis/phpunit-slow-test-detector
+ */
+
+namespace Ergebnis\PHPUnit\SlowTestDetector\Version;
+
+/**
+ * @internal
+ */
+final class Series
+{
+    private Major $major;
+
+    private function __construct(Major $major)
+    {
+        $this->major = $major;
+    }
+
+    public static function create(Major $major): self
+    {
+        return new self($major);
+    }
+
+    /**
+     * @throws \InvalidArgumentException
+     */
+    public static function fromString(string $value): self
+    {
+        if (0 === \preg_match('/^(?P<major>(0|[1-9]\d*))\.(?P<minor>(0|[1-9]\d*))?$/', $value, $matches)) {
+            throw new \InvalidArgumentException(\sprintf(
+                'Value "%s" does not appear to be a valid value for a semantic version.',
+                $value,
+            ));
+        }
+
+        $major = Major::fromInt((int) $matches['major']);
+
+        return self::create($major);
+    }
+
+    public function major(): Major
+    {
+        return $this->major;
+    }
+}