Refactor

Author: sebastianbergmann

src/Event/Value/Test/TestMethodBuilder.php

@@ -9,10 +9,7 @@
  */
 namespace PHPUnit\Event\Code;
 
-use const DEBUG_BACKTRACE_IGNORE_ARGS;
-use const DEBUG_BACKTRACE_PROVIDE_OBJECT;
 use function assert;
-use function debug_backtrace;
 use function is_numeric;
 use PHPUnit\Event\TestData\DataFromDataProvider;
 use PHPUnit\Event\TestData\DataFromTestDependency;
@@ -21,6 +18,7 @@
 use PHPUnit\Metadata\Parser\Registry as MetadataRegistry;
 use PHPUnit\Util\Exporter;
 use PHPUnit\Util\Reflection;
+use PHPUnit\Util\Test as TestUtil;
 
 /**
  * @no-named-arguments Parameter names are not covered by the backward compatibility promise for PHPUnit
@@ -53,13 +51,7 @@ public static function fromTestCase(TestCase $testCase): TestMethod
      */
     public static function fromCallStack(): TestMethod
     {
-        foreach (debug_backtrace(DEBUG_BACKTRACE_PROVIDE_OBJECT | DEBUG_BACKTRACE_IGNORE_ARGS) as $frame) {
-            if (isset($frame['object']) && $frame['object'] instanceof TestCase) {
-                return $frame['object']->valueObjectForEvents();
-            }
-        }
-
-        throw new NoTestCaseObjectOnCallStackException;
+        return TestUtil::currentTestCase()->valueObjectForEvents();
     }
 
     private static function dataFor(TestCase $testCase): TestDataCollection

src/Util/Test.php

@@ -9,7 +9,12 @@
  */
 namespace PHPUnit\Util;
 
+use const DEBUG_BACKTRACE_IGNORE_ARGS;
+use const DEBUG_BACKTRACE_PROVIDE_OBJECT;
+use function debug_backtrace;
 use function str_starts_with;
+use PHPUnit\Event\Code\NoTestCaseObjectOnCallStackException;
+use PHPUnit\Framework\TestCase;
 use PHPUnit\Metadata\Parser\Registry;
 use ReflectionMethod;
 
@@ -20,6 +25,20 @@
  */
 final readonly class Test
 {
+    /**
+     * @throws NoTestCaseObjectOnCallStackException
+     */
+    public static function currentTestCase(): TestCase
+    {
+        foreach (debug_backtrace(DEBUG_BACKTRACE_PROVIDE_OBJECT | DEBUG_BACKTRACE_IGNORE_ARGS) as $frame) {
+            if (isset($frame['object']) && $frame['object'] instanceof TestCase) {
+                return $frame['object'];
+            }
+        }
+
+        throw new NoTestCaseObjectOnCallStackException;
+    }
+
     public static function isTestMethod(ReflectionMethod $method): bool
     {
         if (!$method->isPublic()) {

tests/end-to-end/event/test-method-builder-cannot-find-testcase-object.phpt

@@ -10,6 +10,6 @@ require __DIR__ . '/../../bootstrap.php';
 
 \PHPUnit\Event\Code\TestMethodBuilder::fromCallStack();
 --EXPECTF--
-Fatal error: Uncaught PHPUnit\Event\Code\NoTestCaseObjectOnCallStackException: Cannot find TestCase object on call stack in %sTestMethodBuilder.php:%d
+Fatal error: Uncaught PHPUnit\Event\Code\NoTestCaseObjectOnCallStackException: Cannot find TestCase object on call stack in %s:%d
 Stack trace:
 %a