collect autoloading deprecations before running testCase

Author: soyuka

src/Runner/ErrorHandler.php

@@ -59,6 +59,11 @@ final class ErrorHandler
  private ?int $originalErrorReportingLevel = null;
  private readonly Source $source;
 
+ /**
+ * @var list<array{int, string, string, int}>
+ */
+ private array $globalDeprecations = [];
+
  /**
  * @var ?array{functions: list<non-empty-string>, methods: list<array{className: class-string, methodName: non-empty-string}>}
  */
@@ -197,6 +202,18 @@ public function __invoke(int $errorNumber, string $errorString, string $errorFil
  return false;
  }
 
+ public function deprecationHandler(int $errorNumber, string $errorString, string $errorFile, int $errorLine): bool
+ {
+ $this->globalDeprecations[] = [$errorNumber, $errorString, $errorFile, $errorLine];
+
+ return true;
+ }
+
+ public function restoreDeprecationHandler(): void
+ {
+ restore_error_handler();
+ }
+
  public function enable(): void
  {
  if ($this->enabled) {
@@ -213,6 +230,7 @@ public function enable(): void
 
  $this->enabled = true;
  $this->originalErrorReportingLevel = error_reporting();
+ $this->triggerGlobalDeprecations();
 
  error_reporting($this->originalErrorReportingLevel & self::UNHANDLEABLE_LEVELS);
  }
@@ -244,6 +262,14 @@ public function useDeprecationTriggers(array $deprecationTriggers): void
  $this->deprecationTriggers = $deprecationTriggers;
  }
 
+ /**
+ * @param list<array{int, string, string, int}> $deprecations
+ */
+ public function collectGlobalDeprecations(array $deprecations): void
+ {
+ $this->globalDeprecations = $deprecations;
+ }
+
  /**
  * @param non-empty-string $file
  * @param positive-int $line
@@ -422,4 +448,11 @@ private function stackTrace(): string
 
  return $buffer;
  }
+
+ private function triggerGlobalDeprecations(): void
+ {
+ foreach ($this->globalDeprecations ?? [] as $d) {
+ $this->__invoke(...$d);
+ }
+ }
 }

src/TextUI/Application.php

@@ -9,6 +9,7 @@
  */
 namespace PHPUnit\TextUI;
 
+use const E_USER_DEPRECATED;
 use const PHP_EOL;
 use const PHP_VERSION;
 use function assert;
@@ -20,6 +21,7 @@
 use function method_exists;
 use function printf;
 use function realpath;
+use function set_error_handler;
 use function sprintf;
 use function str_contains;
 use function trim;
@@ -178,8 +180,12 @@ public function run(array $argv): int
 
  EventFacade::instance()->seal();
 
+ set_error_handler([ErrorHandler::instance(), 'deprecationHandler'], E_USER_DEPRECATED);
+
  $testSuite = $this->buildTestSuite($configuration);
 
+ ErrorHandler::instance()->restoreDeprecationHandler();
+
  $this->executeCommandsThatRequireTheTestSuite($configuration, $cliConfiguration, $testSuite);
 
  if ($testSuite->isEmpty() && !$configuration->hasCliArguments() && $configuration->testSuite()->isEmpty()) {

tests/end-to-end/deprecation-trigger/_files/deprecation-trigger-class/phpunit.xml

@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+ xsi:noNamespaceSchemaLocation="../../../../../phpunit.xsd"
+ bootstrap="vendor/autoload.php"
+>
+ <testsuites>
+ <testsuite name="default">
+ <directory>tests</directory>
+ </testsuite>
+ </testsuites>
+
+ <source>
+ <include>
+ <directory>src</directory>
+ </include>
+ </source>
+</phpunit>

tests/end-to-end/deprecation-trigger/_files/deprecation-trigger-class/src/FirstPartyClass.php

@@ -0,0 +1,18 @@
+<?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\SelfDirectIndirect;
+
+final class FirstPartyClass
+{
+ public function method(): true
+ {
+ return ThirdPartyClass::A;
+ }
+}

tests/end-to-end/deprecation-trigger/_files/deprecation-trigger-class/tests/FirstPartyClassTest.php

@@ -0,0 +1,20 @@
+<?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\SelfDirectIndirect;
+
+use PHPUnit\Framework\TestCase;
+
+final class FirstPartyClassTest extends TestCase
+{
+ public function testOne(): void
+ {
+ $this->assertTrue((new FirstPartyClass)->method());
+ }
+}

tests/end-to-end/deprecation-trigger/_files/deprecation-trigger-class/vendor/ThirdPartyClass.php

@@ -0,0 +1,9 @@
+<?php declare(strict_types=1);
+namespace PHPUnit\TestFixture\SelfDirectIndirect;
+
+@trigger_error('This class is deprecated', E_USER_DEPRECATED);
+
+final class ThirdPartyClass
+{
+ const A = true;
+}

tests/end-to-end/deprecation-trigger/_files/deprecation-trigger-class/vendor/autoload.php

@@ -0,0 +1,12 @@
+<?php declare(strict_types=1);
+
+spl_autoload_register(function ($class) {
+ $parts = explode('\\', $class);
+ $file = end($parts) . '.php';
+
+ match ($file) {
+ 'FirstPartyClass.php' => require __DIR__ . '/../src/' . $file,
+ 'ThirdPartyClass.php' => require __DIR__ . '/' . $file,
+ default => throw new LogicException
+ };
+});

tests/end-to-end/deprecation-trigger/deprecation-trigger-class.phpt

@@ -0,0 +1,36 @@
+--TEST--
+The right events are emitted in the right order for a test that loads a class that triggers E_USER_DEPRECATED
+--FILE--
+<?php declare(strict_types=1);
+$_SERVER['argv'][] = '--do-not-cache-result';
+$_SERVER['argv'][] = '--debug';
+$_SERVER['argv'][] = '--configuration';
+$_SERVER['argv'][] = __DIR__ . '/_files/deprecation-trigger-class';
+
+require __DIR__ . '/../../bootstrap.php';
+
+(new PHPUnit\TextUI\Application)->run($_SERVER['argv']);
+--EXPECTF--
+PHPUnit Started (PHPUnit %s using %s)
+Test Runner Configured
+Bootstrap Finished (%sautoload.php)
+Event Facade Sealed
+Test Suite Loaded (1 test)
+Test Runner Started
+Test Suite Sorted
+Test Runner Execution Started (1 test)
+Test Suite Started (%sphpunit.xml, 1 test)
+Test Suite Started (default, 1 test)
+Test Suite Started (PHPUnit\TestFixture\SelfDirectIndirect\FirstPartyClassTest, 1 test)
+Test Preparation Started (PHPUnit\TestFixture\SelfDirectIndirect\FirstPartyClassTest::testOne)
+Test Prepared (PHPUnit\TestFixture\SelfDirectIndirect\FirstPartyClassTest::testOne)
+Test Triggered Deprecation (PHPUnit\TestFixture\SelfDirectIndirect\FirstPartyClassTest::testOne, issue triggered by third-party code, suppressed using operator) in %s:%d
+This class is deprecated
+Test Passed (PHPUnit\TestFixture\SelfDirectIndirect\FirstPartyClassTest::testOne)
+Test Finished (PHPUnit\TestFixture\SelfDirectIndirect\FirstPartyClassTest::testOne)
+Test Suite Finished (PHPUnit\TestFixture\SelfDirectIndirect\FirstPartyClassTest, 1 test)
+Test Suite Finished (default, 1 test)
+Test Suite Finished (%sphpunit.xml, 1 test)
+Test Runner Execution Finished
+Test Runner Finished
+PHPUnit Finished (Shell Exit Code: 0)