Fix TestCase::expectErrorLog() with open_basedir php.ini

Author: mvorisek

src/Framework/TestCase.php

@@ -32,6 +32,7 @@
 use function is_int;
 use function is_object;
 use function is_string;
+use function is_writable;
 use function libxml_clear_errors;
 use function method_exists;
 use function ob_end_clean;
@@ -1252,27 +1253,40 @@ private function runTest(): mixed
     {
         $testArguments = array_merge($this->data, array_values($this->dependencyInput));
 
-        $capture          = tmpfile();
-        $errorLogPrevious = ini_set('error_log', stream_get_meta_data($capture)['uri']);
+        $capture = tmpfile();
+
+        if ($capture !== false) {
+            $capturePath = stream_get_meta_data($capture)['uri'];
+
+            if (@is_writable($capturePath)) {
+                $errorLogPrevious = ini_set('error_log', $capturePath);
+            } else {
+                $capture = false;
+            }
+        }
 
         try {
             /** @phpstan-ignore method.dynamicName */
             $testResult = $this->{$this->methodName}(...$testArguments);
 
-            $errorLogOutput = stream_get_contents($capture);
+            if ($capture !== false) {
+                $errorLogOutput = stream_get_contents($capture);
 
-            if ($this->expectErrorLog) {
-                $this->assertNotEmpty($errorLogOutput, 'Test did not call error_log().');
-            } else {
-                // strip date from logged error, see https://github.com/php/php-src/blob/c696087e323263e941774ebbf902ac249774ec9f/main/main.c#L905
-                print preg_replace('/\[.+\] /', '', $errorLogOutput);
+                if ($this->expectErrorLog) {
+                    $this->assertNotEmpty($errorLogOutput, 'Test did not call error_log().');
+                } else {
+                    // strip date from logged error, see https://github.com/php/php-src/blob/c696087e323263e941774ebbf902ac249774ec9f/main/main.c#L905
+                    print preg_replace('/\[.+\] /', '', $errorLogOutput);
+                }
             }
         } catch (Throwable $exception) {
-            if (!$this->expectErrorLog) {
-                $errorLogOutput = stream_get_contents($capture);
+            if ($capture !== false) {
+                if (!$this->expectErrorLog) {
+                    $errorLogOutput = stream_get_contents($capture);
 
-                // strip date from logged error, see https://github.com/php/php-src/blob/c696087e323263e941774ebbf902ac249774ec9f/main/main.c#L905
-                print preg_replace('/\[.+\] /', '', $errorLogOutput);
+                    // strip date from logged error, see https://github.com/php/php-src/blob/c696087e323263e941774ebbf902ac249774ec9f/main/main.c#L905
+                    print preg_replace('/\[.+\] /', '', $errorLogOutput);
+                }
             }
 
             if (!$this->shouldExceptionExpectationsBeVerified($exception)) {
@@ -1285,9 +1299,10 @@ private function runTest(): mixed
         } finally {
             if ($capture !== false) {
                 fclose($capture);
-            }
 
-            ini_set('error_log', $errorLogPrevious);
+                /** @phpstan-ignore variable.undefined (https://github.com/phpstan/phpstan/issues/12992) */
+                ini_set('error_log', $errorLogPrevious);
+            }
         }
 
         $this->expectedExceptionWasNotRaised();

tests/end-to-end/generic/expect-error-log-fail-with-open_basedir.phpt

@@ -0,0 +1,36 @@
+--TEST--
+https://github.com/sebastianbergmann/phpunit/issues/6197
+--FILE--
+<?php declare(strict_types=1);
+$_SERVER['argv'][] = '--do-not-cache-result';
+$_SERVER['argv'][] = '--no-configuration';
+$_SERVER['argv'][] = '--testdox';
+$_SERVER['argv'][] = __DIR__ . '/_files/ExpectErrorLogFailTest.php';
+
+/*
+ * Expected result should math expect-error-log-fail-with-open_basedir.phpt test,
+ * but at least one of these feature requests needs to be implemented in php-src:
+ * - https://github.com/php/php-src/issues/17817
+ * - https://github.com/php/php-src/issues/18530
+ *
+ * Until then, ignore TestCase::expectErrorLog() if open_basedir php.ini is in effect.
+ */
+
+ini_set('open_basedir', (ini_get('open_basedir') ? ini_get('open_basedir') . PATH_SEPARATOR : '') . dirname(__DIR__, 3));
+
+require_once __DIR__ . '/../../bootstrap.php';
+
+(new PHPUnit\TextUI\Application)->run($_SERVER['argv']);
+--EXPECTF--
+PHPUnit %s by Sebastian Bergmann and contributors.
+
+Runtime: %s
+
+.                                                                   1 / 1 (100%)
+
+Time: %s, Memory: %s
+
+Expect Error Log Fail (PHPUnit\TestFixture\ExpectNoErrorLog\ExpectErrorLogFail)
+ ✔ One
+
+OK (1 test, 1 assertion)

tests/end-to-end/generic/expect-error-log-with-open_basedir.phpt

@@ -0,0 +1,28 @@
+--TEST--
+https://github.com/sebastianbergmann/phpunit/issues/6197
+--FILE--
+<?php declare(strict_types=1);
+$_SERVER['argv'][] = '--do-not-cache-result';
+$_SERVER['argv'][] = '--no-configuration';
+$_SERVER['argv'][] = '--testdox';
+$_SERVER['argv'][] = __DIR__ . '/_files/ExpectErrorLogTest.php';
+
+ini_set('open_basedir', (ini_get('open_basedir') ? ini_get('open_basedir') . PATH_SEPARATOR : '') . dirname(__DIR__, 3));
+
+require_once __DIR__ . '/../../bootstrap.php';
+
+(new PHPUnit\TextUI\Application)->run($_SERVER['argv']);
+--EXPECTF--
+PHPUnit %s by Sebastian Bergmann and contributors.
+
+Runtime: %s
+
+logged a side effect
+.                                                                   1 / 1 (100%)
+
+Time: %s, Memory: %s
+
+Expect Error Log (PHPUnit\TestFixture\ExpectErrorLog\ExpectErrorLog)
+ ✔ One
+
+OK (1 test, 1 assertion)