Merge branch '10.5' into 11.1

Author: sebastianbergmann

ChangeLog-11.1.md

@@ -6,6 +6,7 @@ All notable changes of the PHPUnit 11.1 release series are documented in this fi
 
 ### Fixed
 
+* [#5771](https://github.com/sebastianbergmann/phpunit/issues/5771): JUnit XML logger may crash when test that is run in separate process exits unexpectedly
 * [#5819](https://github.com/sebastianbergmann/phpunit/issues/5819): Duplicate keys from different data providers are not handled properly
 
 ## [11.1.2] - 2024-04-14

src/Logging/JUnit/JunitXmlLogger.php

@@ -202,7 +202,7 @@ public function testPrepared(): void
      */
     public function testFinished(Finished $event): void
     {
-        if ($this->preparationFailed) {
+        if (!$this->prepared || $this->preparationFailed) {
             return;
         }
 

tests/end-to-end/regression/5771.phpt

@@ -0,0 +1,23 @@
+--TEST--
+https://github.com/sebastianbergmann/phpunit/issues/5771
+--FILE--
+<?php declare(strict_types=1);
+$_SERVER['argv'][] = '--no-configuration';
+$_SERVER['argv'][] = '--no-output';
+$_SERVER['argv'][] = __DIR__ . '/5771/Issue5771Test.php';
+$_SERVER['argv'][] = '--log-junit';
+$_SERVER['argv'][] = 'php://stdout';
+
+require_once __DIR__ . '/../../bootstrap.php';
+
+(new PHPUnit\TextUI\Application)->run($_SERVER['argv']);
+--EXPECTF--
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuites>
+  <testsuite name="PHPUnit\TestFixture\Issue5771\Issue5771Test" file="%sIssue5771Test.php" tests="1" assertions="0" errors="1" failures="0" skipped="0" time="%s">
+    <testcase name="testOne" file="%sIssue5771Test.php" line="18" class="PHPUnit\TestFixture\Issue5771\Issue5771Test" classname="PHPUnit.TestFixture.Issue5771.Issue5771Test" assertions="0" time="%s">
+      <error type="PHPUnit\Framework\AssertionFailedError">PHPUnit\TestFixture\Issue5771\Issue5771Test::testOne
+Test was run in child process and ended unexpectedly</error>
+    </testcase>
+  </testsuite>
+</testsuites>

tests/end-to-end/regression/5771/Issue5771Test.php

@@ -0,0 +1,22 @@
+<?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\TestFixture\Issue5771;
+
+use PHPUnit\Framework\Attributes\RunInSeparateProcess;
+use PHPUnit\Framework\TestCase;
+
+final class Issue5771Test extends TestCase
+{
+    #[RunInSeparateProcess]
+    public function testOne(): void
+    {
+        exit;
+    }
+}