Refactor - Add helper classes for testing internal class defaults and final methods in StopwatchDecorator

Author: tarlepp

tests/Integration/Decorator/FinalTestService.php

@@ -0,0 +1,21 @@
+<?php
+declare(strict_types = 1);
+/**
+ * /tests/Integration/Decorator/FinalTestService.php
+ *
+ * @author TLe, Tarmo Leppänen <tarmo.leppanen@pinja.com>
+ */
+
+namespace App\Tests\Integration\Decorator;
+
+/**
+ * Helper final class for testing
+ */
+final class FinalTestService
+{
+    public function testMethod(): string
+    {
+        return 'final-test';
+    }
+}
+

tests/Integration/Decorator/ServiceWithInternalDefaults.php

@@ -0,0 +1,35 @@
+<?php
+declare(strict_types = 1);
+/**
+ * /tests/Integration/Decorator/ServiceWithInternalDefaults.php
+ *
+ * @author TLe, Tarmo Leppänen <tarmo.leppanen@pinja.com>
+ */
+
+namespace App\Tests\Integration\Decorator;
+
+use SplFileInfo;
+
+/**
+ * Helper class for testing internal class default values
+ * Extends an internal class to potentially trigger reflection edge cases
+ */
+class ServiceWithInternalDefaults extends SplFileInfo
+{
+    public function __construct()
+    {
+        parent::__construct(__FILE__);
+    }
+
+    /**
+     * Method with parameter that has complex default values
+     * This is for testing the catch block in getDefaultValueString when the decorator
+     * tries to get the default value via reflection for certain edge cases
+     *
+     * @param array<string, mixed> $options
+     */
+    public function methodWithInternalDefault(array $options = []): string
+    {
+        return 'test';
+    }
+}

tests/Integration/Decorator/StopwatchDecoratorTest.php

@@ -203,39 +203,3 @@ public static function dataProviderTestThatDecorateMethodReturnsExpected(): Gene
         yield [stdClass::class, new stdClass()];
     }
 }
-
-/**
- * Helper final class for testing
- */
-final class FinalTestService
-{
-    public function testMethod(): string
-    {
-        return 'final-test';
-    }
-}
-
-/**
- * Helper class for testing internal class default values
- * Extends an internal class to potentially trigger reflection edge cases
- */
-class ServiceWithInternalDefaults extends \SplFileInfo
-{
-    public function __construct()
-    {
-        parent::__construct(__FILE__);
-    }
-
-    /**
-     * Method with parameter that has complex default values
-     * This is for testing the catch block in getDefaultValueString when the decorator
-     * tries to get the default value via reflection for certain edge cases
-     *
-     * @param array<string, mixed> $options
-     */
-    public function methodWithInternalDefault(array $options = []): string
-    {
-        return 'test';
-    }
-}
-