Extract MatrixInterface::lowestAndHighest into static class Versions

Author: tangrufus

src/Console/Command.php

@@ -12,6 +12,7 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
+use TypistTech\PhpMatrix\Versions;
 
 #[AsCommand(
     name: 'php-matrix',
@@ -118,14 +119,12 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             );
         }
 
-        [$lowest, $highest] = $matrix->lowestAndHighest(...$versions);
-
         $result = json_encode(
             (object) [
                 self::CONSTRAINT_ARGUMENT_NAME => $constraint,
-                'versions' => $versions,
-                'lowest' => $lowest,
-                'highest' => $highest,
+                'versions' => Versions::sort(...$versions),
+                'lowest' => Versions::lowest(...$versions),
+                'highest' => Versions::highest(...$versions),
             ],
             JSON_THROW_ON_ERROR | JSON_PRETTY_PRINT
         );

src/Matrix.php

@@ -22,16 +22,4 @@ public function satisfiedBy(string $constraint): array
             $constraint
         );
     }
-
-    public function lowestAndHighest(string $version, string ...$versions): array
-    {
-        if (empty($versions)) {
-            return [$version, $version];
-        }
-
-        $sorted = Semver::sort([$version, ...$versions]);
-        $count = count($sorted);
-
-        return [$sorted[0], $sorted[$count - 1]];
-    }
 }

src/MatrixInterface.php

@@ -10,9 +10,4 @@ interface MatrixInterface
      * @return string[]
      */
     public function satisfiedBy(string $constraint): array;
-
-    /**
-     * @return string[]
-     */
-    public function lowestAndHighest(string $version, string ...$versions): array;
 }

src/Versions.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace TypistTech\PhpMatrix;
+
+use Composer\Semver\Semver;
+use UnexpectedValueException;
+
+readonly class Versions
+{
+    public static function sort(string ...$versions): array
+    {
+        if (empty($versions)) {
+            throw new UnexpectedValueException('Argument #1 ($versions) must not be empty');
+        }
+
+        return Semver::sort($versions);
+    }
+
+    public static function lowest(string ...$versions): string
+    {
+        $sorted = self::sort(...$versions);
+
+        return $sorted[0];
+    }
+
+    public static function highest(string ...$versions): string
+    {
+        $sorted = self::sort(...$versions);
+
+        return $sorted[array_key_last($sorted)];
+    }
+}

tests/.pest/snapshots/E2E/Console/CommandTest/_TypistTech_PhpMatrix_Console_Command__→_it_fails_with_data_set_____4_______auto_______full_______4____auto____full__.snap

@@ -1,5 +1,5 @@
 
-In Command.php line 116:
+In Command.php line 117:
 
   Error! No PHP versions could satisfy the constraint '^4'.  
 

tests/.pest/snapshots/E2E/Console/CommandTest/_TypistTech_PhpMatrix_Console_Command__→_it_fails_with_data_set_____4_______auto_______minor_only_______4____auto____minor_only__.snap

@@ -1,5 +1,5 @@
 
-In Command.php line 116:
+In Command.php line 117:
 
   Error! No PHP versions could satisfy the constraint '^4'.  
 

tests/.pest/snapshots/E2E/Console/CommandTest/_TypistTech_PhpMatrix_Console_Command__→_it_fails_with_data_set_____4_______auto_______not_a_mode_______4____auto____not_a_mode__.snap

@@ -1,5 +1,5 @@
 
-In Command.php line 99:
+In Command.php line 100:
 
   Error! Invalid mode 'not-a-mode'. Available modes: [full, minor-only]  
 

tests/.pest/snapshots/E2E/Console/CommandTest/_TypistTech_PhpMatrix_Console_Command__→_it_fails_with_data_set_____4_______auto______null______4____auto___null_.snap

@@ -1,5 +1,5 @@
 
-In Command.php line 116:
+In Command.php line 117:
 
   Error! No PHP versions could satisfy the constraint '^4'.  
 

tests/.pest/snapshots/E2E/Console/CommandTest/_TypistTech_PhpMatrix_Console_Command__→_it_fails_with_data_set_____4_______not_a_source_______full_______4____not_a_source____full__.snap

@@ -1,5 +1,5 @@
 
-In Command.php line 86:
+In Command.php line 87:
 
   Error! Invalid source 'not-a-source'. Available sources: [auto, php.net, of  
   fline]                                                                       

tests/.pest/snapshots/E2E/Console/CommandTest/_TypistTech_PhpMatrix_Console_Command__→_it_fails_with_data_set_____4_______not_a_source_______minor_only_______4____not_a_source____minor_only__.snap

@@ -1,5 +1,5 @@
 
-In Command.php line 86:
+In Command.php line 87:
 
   Error! Invalid source 'not-a-source'. Available sources: [auto, php.net, of  
   fline]