From d8c5cc08904851450211c7938ff0b907ef90d9aa Mon Sep 17 00:00:00 2001 From: Sebastian Mendel Date: Wed, 19 Aug 2026 21:39:59 +0200 Subject: [PATCH] chore: move TestExtension out of the shipped source TestExtension is test infrastructure: it forces Parser, Compiler and the output node renderer public so tests can pull them from the container, pins the clock to a fixed date with MockClock, and registers Monolog's TestHandler so functional tests can assert on log records. Its only user is tests/ApplicationTestCase.php. It reached packages/guides by accident rather than by decision. It was created in packages/guides-symfony alongside ContainerFactory (1d10cbd8), and that package was dissolved the next day, so the rename carried it into packages/guides (54daa48f). Later commits added the Monolog handler (ba041eb8) and the mocked clock (541fbf4d) without anyone revisiting where it lives. The fully qualified name does not change: packages/guides/tests/unit maps to the same phpDocumentor\Guides namespace, and the root autoload-dev already covers that path, so no import or consumer is touched. Because every package marks /tests export-ignore, the class now stays out of the published phpdocumentor/guides, and with it Monolog and MockClock as production symbols. Refs #1355 Signed-off-by: Sebastian Mendel Assisted-by: claude-code:claude-fable-5 Agent-Session: https://claude.ai/code/session_0114KJz3vqq2WWfx4FUdmcss Agent-Host: 0493f0 --- .../{src => tests/unit}/DependencyInjection/TestExtension.php | 0 1 file changed, 0 insertions(+), 0 deletions(-) rename packages/guides/{src => tests/unit}/DependencyInjection/TestExtension.php (100%) diff --git a/packages/guides/src/DependencyInjection/TestExtension.php b/packages/guides/tests/unit/DependencyInjection/TestExtension.php similarity index 100% rename from packages/guides/src/DependencyInjection/TestExtension.php rename to packages/guides/tests/unit/DependencyInjection/TestExtension.php