Extract class

sebastianbergmann · Oct 13, 2024 · 85b07dc · 85b07dc
1 parent a244f58
commit 85b07dc
Show file tree

Hide file tree

Showing 4 changed files with 103 additions and 80 deletions.
diff --git a/src/Framework/MockObject/Generator/Generator.php b/src/Framework/MockObject/Generator/Generator.php
@@ -891,7 +891,10 @@ private function generateCodeForTestDoubleClass(string $type, bool $mockObject,
                 'use_statements'  => $useStatements,
                 'mock_class_name' => $_mockClassName['className'],
                 'methods'         => $mockedMethods,
-                'property_hooks'  => $this->codeForPropertyHooks($propertiesWithHooks, $_mockClassName['className']),
+                'property_hooks'  => (new HookedPropertyGenerator)->generate(
+                    $_mockClassName['className'],
+                    $propertiesWithHooks,
+                ),
             ],
         );
 
@@ -1162,7 +1165,7 @@ private function interfaceMethods(string $interfaceName, bool $cloneArguments):
     }
 
     /**
-     * @param list<Property> $propertiesWithHooks
+     * @param list<HookedProperty> $propertiesWithHooks
      *
      * @return list<ConfigurableMethod>
      */
@@ -1211,7 +1214,7 @@ private function configurableMethods(MockMethodSet $methods, array $propertiesWi
     /**
      * @param ?ReflectionClass<object> $class
      *
-     * @return list<Property>
+     * @return list<HookedProperty>
      */
     private function properties(?ReflectionClass $class): array
     {
@@ -1264,7 +1267,7 @@ private function properties(?ReflectionClass $class): array
                 continue;
             }
 
-            $properties[] = new Property(
+            $properties[] = new HookedProperty(
                 $property->getName(),
                 $mapper->fromPropertyType($property),
                 $hasGetHook,
@@ -1274,72 +1277,4 @@ private function properties(?ReflectionClass $class): array
 
         return $properties;
     }
-
-    /**
-     * @param list<Property> $propertiesWithHooks
-     * @param class-string   $className
-     *
-     * @return non-empty-string
-     */
-    private function codeForPropertyHooks(array $propertiesWithHooks, string $className): string
-    {
-        $propertyHooks = '';
-
-        foreach ($propertiesWithHooks as $property) {
-            $propertyHooks .= sprintf(
-                <<<'EOT'
-
-    public %s $%s {
-EOT,
-                $property->type()->asString(),
-                $property->name(),
-            );
-
-            if ($property->hasGetHook()) {
-                $propertyHooks .= sprintf(
-                    <<<'EOT'
-
-        get {
-            return $this->__phpunit_getInvocationHandler()->invoke(
-                new \PHPUnit\Framework\MockObject\Invocation(
-                    '%s', '$%s::get', [], '%s', $this, false
-                )
-            );
-        }
-
-EOT,
-                    $className,
-                    $property->name(),
-                    $property->type()->asString(),
-                );
-            }
-
-            if ($property->hasSetHook()) {
-                $propertyHooks .= sprintf(
-                    <<<'EOT'
-
-        set (%s $value) {
-            $this->__phpunit_getInvocationHandler()->invoke(
-                new \PHPUnit\Framework\MockObject\Invocation(
-                    '%s', '$%s::set', [$value], 'void', $this, false
-                )
-            );
-        }
-
-EOT,
-                    $property->type()->asString(),
-                    $className,
-                    $property->name(),
-                );
-            }
-
-            $propertyHooks .= <<<'EOT'
-    }
-
-EOT;
-
-        }
-
-        return $propertyHooks;
-    }
 }
diff --git a/...amework/MockObject/Generator/Property.php → ...k/MockObject/Generator/HookedProperty.php b/...amework/MockObject/Generator/Property.php → ...k/MockObject/Generator/HookedProperty.php
@@ -16,7 +16,7 @@
  *
  * @internal This class is not covered by the backward compatibility promise for PHPUnit
  */
-final readonly class Property
+final readonly class HookedProperty
 {
     /**
      * @var non-empty-string

diff --git a/src/Framework/MockObject/Generator/HookedPropertyGenerator.php b/src/Framework/MockObject/Generator/HookedPropertyGenerator.php
@@ -0,0 +1,88 @@
+<?php declare(strict_types=1);
+/*
+ * This file is part of PHPUnit.
+ *
+ * (c) Sebastian Bergmann <sebastian@phpunit.de>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+namespace PHPUnit\Framework\MockObject\Generator;
+
+use function sprintf;
+
+/**
+ * @no-named-arguments Parameter names are not covered by the backward compatibility promise for PHPUnit
+ *
+ * @internal This class is not covered by the backward compatibility promise for PHPUnit
+ */
+final class HookedPropertyGenerator
+{
+    /**
+     * @param class-string         $className
+     * @param list<HookedProperty> $properties
+     *
+     * @return non-empty-string
+     */
+    public function generate(string $className, array $properties): string
+    {
+        $code = '';
+
+        foreach ($properties as $property) {
+            $code .= sprintf(
+                <<<'EOT'
+
+    public %s $%s {
+EOT,
+                $property->type()->asString(),
+                $property->name(),
+            );
+
+            if ($property->hasGetHook()) {
+                $code .= sprintf(
+                    <<<'EOT'
+
+        get {
+            return $this->__phpunit_getInvocationHandler()->invoke(
+                new \PHPUnit\Framework\MockObject\Invocation(
+                    '%s', '$%s::get', [], '%s', $this, false
+                )
+            );
+        }
+
+EOT,
+                    $className,
+                    $property->name(),
+                    $property->type()->asString(),
+                );
+            }
+
+            if ($property->hasSetHook()) {
+                $code .= sprintf(
+                    <<<'EOT'
+
+        set (%s $value) {
+            $this->__phpunit_getInvocationHandler()->invoke(
+                new \PHPUnit\Framework\MockObject\Invocation(
+                    '%s', '$%s::set', [$value], 'void', $this, false
+                )
+            );
+        }
+
+EOT,
+                    $property->type()->asString(),
+                    $className,
+                    $property->name(),
+                );
+            }
+
+            $code .= <<<'EOT'
+    }
+
+EOT;
+
+        }
+
+        return $code;
+    }
+}
diff --git a/tests/unit/Framework/MockObject/Generator/PropertyTest.php b/tests/unit/Framework/MockObject/Generator/PropertyTest.php
@@ -15,7 +15,7 @@
 use PHPUnit\Framework\TestCase;
 use SebastianBergmann\Type\Type;
 
-#[CoversClass(Property::class)]
+#[CoversClass(HookedProperty::class)]
 #[Group('test-doubles')]
 #[Small]
 final class PropertyTest extends TestCase
@@ -24,7 +24,7 @@ public function testHasName(): void
     {
         $name = 'property-name';
 
-        $property = new Property($name, Type::fromName('string', false), false, false);
+        $property = new HookedProperty($name, Type::fromName('string', false), false, false);
 
         $this->assertSame($name, $property->name());
     }
@@ -33,35 +33,35 @@ public function testHasType(): void
     {
         $type = Type::fromName('string', false);
 
-        $property = new Property('property-name', $type, false, false);
+        $property = new HookedProperty('property-name', $type, false, false);
 
         $this->assertSame($type, $property->type());
     }
 
     public function testMayHaveGetHook(): void
     {
-        $property = new Property('property-name', Type::fromName('string', false), true, false);
+        $property = new HookedProperty('property-name', Type::fromName('string', false), true, false);
 
         $this->assertTrue($property->hasGetHook());
     }
 
     public function testMayNotHaveGetHook(): void
     {
-        $property = new Property('property-name', Type::fromName('string', false), false, false);
+        $property = new HookedProperty('property-name', Type::fromName('string', false), false, false);
 
         $this->assertFalse($property->hasGetHook());
     }
 
     public function testMayHaveSetHook(): void
     {
-        $property = new Property('property-name', Type::fromName('string', false), false, true);
+        $property = new HookedProperty('property-name', Type::fromName('string', false), false, true);
 
         $this->assertTrue($property->hasSetHook());
     }
 
     public function testMayNotHaveSetHook(): void
     {
-        $property = new Property('property-name', Type::fromName('string', false), false, false);
+        $property = new HookedProperty('property-name', Type::fromName('string', false), false, false);
 
         $this->assertFalse($property->hasSetHook());
     }