do not base services on PHPUnit mocks

xabbuh · xabbuh · commit 6b0ec7c93a69 · 2024-09-29T10:34:43.000+02:00
Using the service container to build services bypasses PHPUnit's mock system
which means that there is no guarantuee that objects are initialized in a way
expected by PHPUnit. Thus mocks may or may not work as expected.
diff --git a/src/Symfony/Bundle/WebProfilerBundle/Tests/DependencyInjection/WebProfilerExtensionTest.php b/src/Symfony/Bundle/WebProfilerBundle/Tests/DependencyInjection/WebProfilerExtensionTest.php
@@ -15,15 +15,16 @@
 use Symfony\Bundle\WebProfilerBundle\Tests\TestCase;
 use Symfony\Component\DependencyInjection\Container;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\Definition;
 use Symfony\Component\DependencyInjection\Reference;
 use Symfony\Component\ErrorHandler\ErrorRenderer\HtmlErrorRenderer;
 use Symfony\Component\EventDispatcher\DependencyInjection\RegisterListenersPass;
 use Symfony\Component\EventDispatcher\EventDispatcher;
 use Symfony\Component\HttpKernel\DataCollector\DumpDataCollector;
 use Symfony\Component\HttpKernel\KernelInterface;
+use Symfony\Component\HttpKernel\Profiler\FileProfilerStorage;
 use Symfony\Component\HttpKernel\Profiler\Profiler;
-use Symfony\Component\HttpKernel\Profiler\ProfilerStorageInterface;
+use Symfony\Component\Routing\RequestContext;
+use Symfony\Component\Routing\RouteCollection;
 use Symfony\Component\Routing\RouterInterface;
 
 class WebProfilerExtensionTest extends TestCase
@@ -58,15 +59,11 @@ protected function setUp(): void
 
         $this->kernel = $this->createMock(KernelInterface::class);
 
-        $profiler = $this->createMock(Profiler::class);
-        $profilerStorage = $this->createMock(ProfilerStorageInterface::class);
-        $router = $this->createMock(RouterInterface::class);
-
         $this->container = new ContainerBuilder();
         $this->container->register('data_collector.dump', DumpDataCollector::class)->setPublic(true);
         $this->container->register('error_handler.error_renderer.html', HtmlErrorRenderer::class)->setPublic(true);
         $this->container->register('event_dispatcher', EventDispatcher::class)->setPublic(true);
-        $this->container->register('router', \get_class($router))->setPublic(true);
+        $this->container->register('router', Router::class)->setPublic(true);
         $this->container->register('twig', 'Twig\Environment')->setPublic(true);
         $this->container->register('twig_loader', 'Twig\Loader\ArrayLoader')->addArgument([])->setPublic(true);
         $this->container->register('twig', 'Twig\Environment')->addArgument(new Reference('twig_loader'))->setPublic(true);
@@ -78,9 +75,10 @@ protected function setUp(): void
         $this->container->setParameter('kernel.charset', 'UTF-8');
         $this->container->setParameter('debug.file_link_format', null);
         $this->container->setParameter('profiler.class', ['Symfony\\Component\\HttpKernel\\Profiler\\Profiler']);
-        $this->container->register('profiler', \get_class($profiler))
+        $this->container->register('profiler', Profiler::class)
             ->setPublic(true)
-            ->addArgument(new Definition(\get_class($profilerStorage)));
+            ->addArgument(new Reference('profiler.storage'));
+        $this->container->register('profiler.storage', FileProfilerStorage::class)->addArgument('file:/'.sys_get_temp_dir());
         $this->container->setParameter('data_collector.templates', []);
         $this->container->set('kernel', $this->kernel);
         $this->container->addCompilerPass(new RegisterListenersPass());
@@ -212,3 +210,27 @@ private function getCompiledContainer()
         return $this->container;
     }
 }
+
+class Router implements RouterInterface
+{
+    public function setContext(RequestContext $context): void
+    {
+    }
+
+    public function getContext(): RequestContext
+    {
+    }
+
+    public function getRouteCollection(): RouteCollection
+    {
+        return new RouteCollection();
+    }
+
+    public function generate(string $name, array $parameters = [], int $referenceType = self::ABSOLUTE_PATH): string
+    {
+    }
+
+    public function match(string $pathinfo): array
+    {
+    }
+}
