improve tests

Author: DavidBadura

…s/Unit/TestCaseAllPublicCompilerPass.php tests/Unit/AllPublicCompilerPass.phptests/Unit/TestCaseAllPublicCompilerPass.php renamed to tests/Unit/AllPublicCompilerPass.php tests/Unit/TestCaseAllPublicCompilerPass.php renamed to tests/Unit/AllPublicCompilerPass.php

@@ -7,7 +7,7 @@
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 
-final class TestCaseAllPublicCompilerPass implements CompilerPassInterface
+final class AllPublicCompilerPass implements CompilerPassInterface
 {
     private const SERVICE_PREFIX = 'event_sourcing.';
     private const NAMESPACE_PREFIX = 'Patchlevel\\';
@@ -21,7 +21,7 @@ public function process(ContainerBuilder $container): void
         }
 
         foreach ($container->getAliases() as $id => $alias) {
-            if ($this->isOwnService($id)) {
+            if ($this->isOwnService($id) || $this->isOwnService((string)$alias)) {
                 $alias->setPublic(true);
             }
         }

tests/Unit/NoLazyCompilerPass.php

@@ -0,0 +1,16 @@
+<?php
+
+namespace Patchlevel\EventSourcingBundle\Tests\Unit;
+
+use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+
+final class NoLazyCompilerPass implements CompilerPassInterface
+{
+    public function process(ContainerBuilder $container): void
+    {
+        foreach ($container->getDefinitions() as $definition) {
+            $definition->setLazy(false);
+        }
+    }
+}

tests/Unit/PatchlevelEventSourcingBundleTest.php

@@ -4,7 +4,6 @@
 
 use ArrayObject;
 use Doctrine\DBAL\Connection;
-use Doctrine\DBAL\Platforms\PostgreSQLPlatform;
 use Doctrine\Migrations\Tools\Console\Command\CurrentCommand;
 use Doctrine\Migrations\Tools\Console\Command\DiffCommand;
 use Doctrine\Migrations\Tools\Console\Command\ExecuteCommand;
@@ -378,6 +377,7 @@ public function testMigrateStore(): void
         $container = new ContainerBuilder();
 
         $container->register('my_translator', ExcludeEventWithHeaderTranslator::class)
+            ->setPublic(true)
             ->setArguments([ArchivedHeader::class]);
 
         $this->compileContainer(
@@ -1629,10 +1629,6 @@ private function compileContainer(ContainerBuilder $container, array $config): v
         $container->setParameter('kernel.project_dir', __DIR__);
 
         $connection = $this->createMock(Connection::class);
-        $connection
-            ->expects($this->never())
-            ->method('getDatabasePlatform')
-            ->willReturn(new PostgreSQLPlatform());
 
         $container->set('doctrine.dbal.eventstore_connection', $connection);
         $container->set('event.bus', $this->createMock(MessageBusInterface::class));
@@ -1648,10 +1644,19 @@ private function compileContainer(ContainerBuilder $container, array $config): v
 
         $compilerPassConfig = $container->getCompilerPassConfig();
         $compilerPassConfig->setRemovingPasses([]);
-        $compilerPassConfig->addPass(new TestCaseAllPublicCompilerPass());
+        $compilerPassConfig->addPass(new AllPublicCompilerPass());
+        $compilerPassConfig->addPass(new NoLazyCompilerPass());
 
         $container->compile();
 
+        foreach ($container->getServiceIds() as $id) {
+            if (str_ends_with($id, '.inner')) {
+                continue;
+            }
+
+            $container->get($id);
+        }
+
         (new XmlDumper($container))->dump();
     }
 }