[Uid] refactor MockUuidFactory to inherit from UuidFactory and keep API consistent

momito69 · momito69 · commit c307d8f08bd3 · 2025-09-23T17:18:48.000+02:00
diff --git a/src/Symfony/Component/Uid/CHANGELOG.md b/src/Symfony/Component/Uid/CHANGELOG.md
@@ -5,7 +5,7 @@ CHANGELOG
 ---
 
  * Add microsecond precision to UUIDv7
- * Add `MockUuidFactory` and `UuidFactoryInterface` to allow deterministic and mockable UUID generation for testing purposes
+ * Add `MockUuidFactory` to allow deterministic and mockable UUID generation for testing purposes
 
 7.3
 ---
diff --git a/src/Symfony/Component/Uid/Factory/MockUuidFactory.php b/src/Symfony/Component/Uid/Factory/MockUuidFactory.php
@@ -30,7 +30,9 @@ public function __construct(
         Uuid|string|null $nameBasedNamespace = null,
     ) {
         parent::__construct($defaultClass, $timeBasedClass, $nameBasedClass, $randomBasedClass, $timeBasedNode, $nameBasedNamespace);
-        $this->validateDefaultClass($defaultClass);
+        if (!is_subclass_of($this->defaultClass, Uuid::class)) {
+            throw new \InvalidArgumentException(\sprintf('The class "%s" must be a subclass of "%s".', $this->defaultClass, Uuid::class));
+        }
         $this->sequence = \is_array($uuids) ? new \ArrayIterator($uuids) : new \IteratorIterator($uuids);
     }
 
@@ -44,11 +46,4 @@ public function create(): Uuid
 
         return $this->defaultClass::fromString($uuid);
     }
-
-    private function validateDefaultClass(string $class): void
-    {
-        if (!is_subclass_of($class, Uuid::class)) {
-            throw new \InvalidArgumentException(\sprintf('The class "%s" must be a subclass of "%s".', $class, Uuid::class));
-        }
-    }
 }
