Extract method

Author: sebastianbergmann

src/Framework/TestCase.php

@@ -219,6 +219,7 @@ abstract class TestCase extends Assert implements Reorderable, SelfDescribing, T
      * @var list<non-empty-string>
      */
     private array $expectedUserDeprecationMessageRegularExpression = [];
+    private false|string $previousErrorLogTarget                   = false;
 
     /**
      * @param non-empty-string $name
@@ -1298,21 +1299,7 @@ private function runTest(): mixed
     {
         $testArguments = array_merge($this->data, array_values($this->dependencyInput));
 
-        $capture = tmpfile();
-
-        if (ini_get('display_errors') === '0') {
-            ShutdownHandler::setMessage('Fatal error: Premature end of PHP process. Use display_errors=On to see the error message.');
-        }
-
-        if ($capture !== false) {
-            $capturePath = stream_get_meta_data($capture)['uri'];
-
-            if (@is_writable($capturePath)) {
-                $errorLogPrevious = ini_set('error_log', $capturePath);
-            } else {
-                $capture = false;
-            }
-        }
+        $capture = $this->startErrorLogCapture();
 
         try {
             /** @phpstan-ignore method.dynamicName */
@@ -1352,10 +1339,12 @@ private function runTest(): mixed
         } finally {
             if ($capture !== false) {
                 ShutdownHandler::resetMessage();
+
                 fclose($capture);
 
-                /** @phpstan-ignore variable.undefined (https://github.com/phpstan/phpstan/issues/12992) */
-                ini_set('error_log', $errorLogPrevious);
+                if ($this->previousErrorLogTarget !== false) {
+                    ini_set('error_log', $this->previousErrorLogTarget);
+                }
             }
         }
 
@@ -2347,6 +2336,30 @@ private function handleExceptionFromInvokedCountMockObjectRule(Throwable $t): vo
         }
     }
 
+    /**
+     * @return false|resource
+     */
+    private function startErrorLogCapture(): mixed
+    {
+        if (ini_get('display_errors') === '0') {
+            ShutdownHandler::setMessage('Fatal error: Premature end of PHP process. Use display_errors=On to see the error message.');
+        }
+
+        $capture = tmpfile();
+
+        if ($capture !== false) {
+            $capturePath = stream_get_meta_data($capture)['uri'];
+
+            if (@is_writable($capturePath)) {
+                $this->previousErrorLogTarget = ini_set('error_log', $capturePath);
+            } else {
+                $capture = false;
+            }
+        }
+
+        return $capture;
+    }
+
     /**
      * Creates a test stub for the specified interface or class.
      *