Refactor

Author: staabm

src/Runner/Phpt/TestCase.php

@@ -350,23 +350,7 @@ private function shouldTestBeSkipped(array $sections, array $settings): bool
             $output = $this->runCodeInLocalSandbox($skipIfCode);
         }
 
-        if (str_contains($output, 'Parse error:')) {
-            EventFacade::emitter()->testRunnerTriggeredPhpunitWarning(
-                sprintf(
-                    'SKIPIF section triggered a parse error: %s',
-                    $output,
-                ),
-            );
-        }
-
-        if (str_contains($output, 'Fatal error:')) {
-            EventFacade::emitter()->testRunnerTriggeredPhpunitWarning(
-                sprintf(
-                    'SKIPIF section triggered a fatal error: %s',
-                    $output,
-                ),
-            );
-        }
+        $this->triggerWarningOnPhpErrors($output);
 
         if (strncasecmp('skip', ltrim($output), 4) === 0) {
             $message = '';
@@ -698,4 +682,25 @@ private function settings(bool $collectCoverage): array
 
         return $settings;
     }
+
+    private function triggerWarningOnPhpErrors(string $output): void
+    {
+        if (str_contains($output, 'Parse error:')) {
+            EventFacade::emitter()->testRunnerTriggeredPhpunitWarning(
+                sprintf(
+                    'SKIPIF section triggered a parse error: %s',
+                    $output,
+                ),
+            );
+        }
+
+        if (str_contains($output, 'Fatal error:')) {
+            EventFacade::emitter()->testRunnerTriggeredPhpunitWarning(
+                sprintf(
+                    'SKIPIF section triggered a fatal error: %s',
+                    $output,
+                ),
+            );
+        }
+    }
 }