Merge branch 'remove-provider'

Author: codedmonkey

src/Doctrine/EventListener/PackageListener.php

@@ -4,19 +4,29 @@
 
 use CodedMonkey\Dirigent\Doctrine\Entity\Package;
 use CodedMonkey\Dirigent\Doctrine\Repository\PackageRepository;
+use CodedMonkey\Dirigent\Message\RemovePackageProvider;
 use Doctrine\Bundle\DoctrineBundle\Attribute\AsEntityListener;
 use Doctrine\ORM\Event\PreRemoveEventArgs;
 use Doctrine\ORM\Events;
+use Symfony\Component\Messenger\MessageBusInterface;
 
 #[AsEntityListener(Events::preRemove, entity: Package::class)]
-class PackageListener
+readonly class PackageListener
 {
+    public function __construct(
+        private MessageBusInterface $messenger,
+    ) {
+    }
+
     public function preRemove(Package $package, PreRemoveEventArgs $event): void
     {
         /** @var PackageRepository $repository */
         $repository = $event->getObjectManager()->getRepository(Package::class);
 
         // Delete existing package links
         $repository->deletePackageLinks($package);
+
+        // Remove package provider
+        $this->messenger->dispatch(new RemovePackageProvider($package->getId()));
     }
 }

src/Message/RemovePackageProvider.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace CodedMonkey\Dirigent\Message;
+
+use Symfony\Component\Messenger\Attribute\AsMessage;
+
+#[AsMessage]
+readonly class RemovePackageProvider
+{
+    public function __construct(
+        public int $packageId,
+    ) {
+    }
+}

src/Message/RemovePackageProviderHandler.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace CodedMonkey\Dirigent\Message;
+
+use CodedMonkey\Dirigent\Doctrine\Repository\PackageRepository;
+use CodedMonkey\Dirigent\Package\PackageProviderManager;
+use Symfony\Component\Messenger\Attribute\AsMessageHandler;
+
+#[AsMessageHandler]
+readonly class RemovePackageProviderHandler
+{
+    use PackageHandlerTrait;
+
+    public function __construct(
+        private PackageRepository $packageRepository,
+        private PackageProviderManager $providerManager,
+    ) {
+    }
+
+    public function __invoke(RemovePackageProvider $message): void
+    {
+        $package = $this->getPackage($this->packageRepository, $message->packageId);
+
+        $this->providerManager->remove($package);
+    }
+}

src/Package/PackageProviderManager.php

@@ -44,6 +44,14 @@ public function dump(Package $package): void
         $package->setDumpedAt(new \DateTimeImmutable());
     }
 
+    public function remove(Package $package): void
+    {
+        $packageName = $package->getName();
+
+        $this->filesystem->remove($this->path($packageName));
+        $this->filesystem->remove($this->path("$packageName~dev"));
+    }
+
     public function exists(string $packageName): bool
     {
         return $this->filesystem->exists($this->path($packageName));

tests/FunctionalTests/Controller/ApiControllerPublicTest.php

@@ -6,6 +6,7 @@
 use CodedMonkey\Dirigent\Doctrine\Entity\Registry;
 use CodedMonkey\Dirigent\Doctrine\Entity\RegistryPackageMirroring;
 use CodedMonkey\Dirigent\Tests\FunctionalTests\PublicKernel;
+use CodedMonkey\Dirigent\Tests\Helper\EntityManagerTestTrait;
 use CodedMonkey\Dirigent\Tests\Helper\KernelTestCaseTrait;
 use CodedMonkey\Dirigent\Tests\Helper\MockEntityFactoryTrait;
 use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
@@ -14,6 +15,7 @@
 
 class ApiControllerPublicTest extends KernelTestCase
 {
+    use EntityManagerTestTrait;
     use KernelTestCaseTrait;
     use MockEntityFactoryTrait;
 

tests/FunctionalTests/Controller/Dashboard/DashboardAccountControllerTest.php

@@ -4,6 +4,7 @@
 
 use CodedMonkey\Dirigent\Controller\Dashboard\DashboardAccountController;
 use CodedMonkey\Dirigent\Doctrine\Entity\User;
+use CodedMonkey\Dirigent\Tests\Helper\EntityManagerTestTrait;
 use CodedMonkey\Dirigent\Tests\Helper\MockEntityFactoryTrait;
 use CodedMonkey\Dirigent\Tests\Helper\WebTestCaseTrait;
 use Doctrine\ORM\EntityManagerInterface;
@@ -15,6 +16,7 @@
 #[CoversClass(DashboardAccountController::class)]
 class DashboardAccountControllerTest extends WebTestCase
 {
+    use EntityManagerTestTrait;
     use MockEntityFactoryTrait;
     use WebTestCaseTrait;
 

tests/FunctionalTests/Controller/Dashboard/DashboardCredentialsControllerTest.php

@@ -5,6 +5,7 @@
 use CodedMonkey\Dirigent\Controller\Dashboard\DashboardCredentialsController;
 use CodedMonkey\Dirigent\Doctrine\Entity\Credentials;
 use CodedMonkey\Dirigent\Doctrine\Entity\CredentialsType;
+use CodedMonkey\Dirigent\Tests\Helper\EntityManagerTestTrait;
 use CodedMonkey\Dirigent\Tests\Helper\MockEntityFactoryTrait;
 use CodedMonkey\Dirigent\Tests\Helper\WebTestCaseTrait;
 use PHPUnit\Framework\Attributes\CoversClass;
@@ -14,6 +15,7 @@
 #[CoversClass(DashboardCredentialsController::class)]
 class DashboardCredentialsControllerTest extends WebTestCase
 {
+    use EntityManagerTestTrait;
     use MockEntityFactoryTrait;
     use WebTestCaseTrait;
 

tests/FunctionalTests/Controller/Dashboard/DashboardPackagesControllerTest.php

@@ -5,13 +5,15 @@
 use CodedMonkey\Dirigent\Doctrine\Entity\Package;
 use CodedMonkey\Dirigent\Doctrine\Repository\PackageRepository;
 use CodedMonkey\Dirigent\Doctrine\Repository\RegistryRepository;
+use CodedMonkey\Dirigent\Tests\Helper\EntityManagerTestTrait;
 use CodedMonkey\Dirigent\Tests\Helper\MockEntityFactoryTrait;
 use CodedMonkey\Dirigent\Tests\Helper\WebTestCaseTrait;
 use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
 use Symfony\Component\HttpFoundation\Response;
 
 class DashboardPackagesControllerTest extends WebTestCase
 {
+    use EntityManagerTestTrait;
     use MockEntityFactoryTrait;
     use WebTestCaseTrait;
 

tests/FunctionalTests/Controller/Dashboard/DashboardPackagesInfoControllerTest.php

@@ -3,13 +3,15 @@
 namespace CodedMonkey\Dirigent\Tests\FunctionalTests\Controller\Dashboard;
 
 use CodedMonkey\Dirigent\Doctrine\Repository\PackageRepository;
+use CodedMonkey\Dirigent\Tests\Helper\EntityManagerTestTrait;
 use CodedMonkey\Dirigent\Tests\Helper\MockEntityFactoryTrait;
 use CodedMonkey\Dirigent\Tests\Helper\WebTestCaseTrait;
 use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
 use Symfony\Component\HttpFoundation\Response;
 
 class DashboardPackagesInfoControllerTest extends WebTestCase
 {
+    use EntityManagerTestTrait;
     use MockEntityFactoryTrait;
     use WebTestCaseTrait;
 

tests/Helper/EntityManagerTestTrait.php

@@ -0,0 +1,47 @@
+<?php
+
+namespace CodedMonkey\Dirigent\Tests\Helper;
+
+use Doctrine\ORM\EntityManagerInterface;
+
+trait EntityManagerTestTrait
+{
+    /**
+     * Find a single entity by its ID or an array of criteria.
+     *
+     * @template T of object
+     *
+     * @param class-string<T> $className
+     *
+     * @return T|null
+     */
+    protected function findEntity(string $className, array|int $criteria): ?object
+    {
+        if (is_array($criteria)) {
+            return $this->getService(EntityManagerInterface::class)->getRepository($className)->findOneBy($criteria);
+        }
+
+        return $this->getService(EntityManagerInterface::class)->find($className, $criteria);
+    }
+
+    /**
+     * Persist and flush all given entities.
+     *
+     * @param object ...$entities
+     */
+    protected function persistEntities(...$entities): void
+    {
+        $entityManager = $this->getService(EntityManagerInterface::class);
+
+        foreach ($entities as $entity) {
+            $entityManager->persist($entity);
+        }
+
+        $entityManager->flush();
+    }
+
+    protected function clearEntities(): void
+    {
+        $this->getService(EntityManagerInterface::class)->clear();
+    }
+}