feat(discovery): add ability to hide classes from discovery (#512)

Author: innocenzi

src/Tempest/Core/src/DoNotDiscover.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tempest\Core;
+
+use Attribute;
+
+#[Attribute(Attribute::TARGET_CLASS)]
+final class DoNotDiscover
+{
+}

src/Tempest/Core/src/Kernel/LoadDiscoveryClasses.php

@@ -12,6 +12,7 @@
 use Tempest\Container\Container;
 use Tempest\Core\DiscoversPath;
 use Tempest\Core\Discovery;
+use Tempest\Core\DoNotDiscover;
 use Tempest\Core\Kernel;
 use Tempest\Reflection\ClassReflector;
 use Throwable;
@@ -94,6 +95,10 @@ public function __invoke(): void
                         }
                     }
 
+                    if ($input instanceof ClassReflector && $input->hasAttribute(DoNotDiscover::class)) {
+                        continue;
+                    }
+
                     if ($input instanceof ClassReflector) {
                         $discovery->discover($input);
                     } elseif ($discovery instanceof DiscoversPath) {

tests/Fixtures/Discovery/HiddenMigration.php

@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Tempest\Fixtures\Discovery;
+
+use Tempest\Core\DoNotDiscover;
+use Tempest\Database\Migration;
+use Tempest\Database\QueryStatement;
+
+#[DoNotDiscover]
+final class HiddenMigration implements Migration
+{
+    public function getName(): string
+    {
+        return 'hidden-migration';
+    }
+
+    public function up(): ?QueryStatement
+    {
+        return null;
+    }
+
+    public function down(): ?QueryStatement
+    {
+        return null;
+    }
+}

tests/Integration/Core/LoadDiscoveryClassesTest.php

@@ -0,0 +1,36 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Tempest\Integration\Core;
+
+use function PHPUnit\Framework\assertNotContains;
+use Tempest\Core\Kernel\LoadDiscoveryClasses;
+use Tempest\Database\DatabaseConfig;
+use Tempest\Database\MigrationDiscovery;
+use function Tempest\get;
+use Tests\Tempest\Fixtures\Discovery\HiddenMigration;
+use Tests\Tempest\Integration\FrameworkIntegrationTestCase;
+
+/**
+ * @internal
+ */
+final class LoadDiscoveryClassesTest extends FrameworkIntegrationTestCase
+{
+    public function test_hidden_from_discovery(): void
+    {
+        $this->kernel->discoveryClasses = [
+            MigrationDiscovery::class,
+        ];
+
+        $this->kernel->discoveryLocations = [
+            realpath(__DIR__.'../../Fixtures/Discovery'),
+        ];
+
+        (new LoadDiscoveryClasses($this->kernel, $this->container));
+
+        $migrations = get(DatabaseConfig::class)->getMigrations();
+
+        assertNotContains(HiddenMigration::class, $migrations);
+    }
+}