Merge pull request #338 from WyriHaximus/only-allow-existing-objects-as-events

Only allow existing objects as events

Author: WyriHaximus

src/Composer/Collector.php

@@ -14,6 +14,7 @@
 use WyriHaximus\Composer\GenerativePluginTooling\ItemCollector;
 
 use function array_map;
+use function class_exists;
 use function in_array;
 
 final class Collector implements ItemCollector
@@ -56,6 +57,10 @@ public function collect(ReflectionClass $class): iterable
             }
 
             foreach ($eventTypes as $eventType) {
+                if (! class_exists((string) $eventType)) {
+                    continue;
+                }
+
                 yield new Item(
                     (string) $eventType,
                     $class->getName(),

src/Dummy/Listener.php

@@ -24,6 +24,11 @@ public function handle(Event $event): void
         ($this->handler)($event);
     }
 
+    public function thisShouldNotBeDetected(string $event): void
+    {
+        ($this->handler)($event);
+    }
+
     public function handleBoth(Event|stdClass $event): void
     {
         ($this->handler)($event);

tests/Composer/InstallerTest.php

@@ -155,6 +155,9 @@ public function write($messages, bool $newline = true, int $verbosity = self::NO
         self::assertStringContainsStringIgnoringCase('\'\WyriHaximus\Broadcast\Dummy\Listener::handleBothStaticly\'', $fileContents);
         self::assertStringContainsStringIgnoringCase('fn (\WyriHaximus\Broadcast\Dummy\Event $event) => await(async(fn (\WyriHaximus\Broadcast\Dummy\Event $event) => $this->container()->get(\WyriHaximus\Broadcast\Dummy\AsyncListener::class)->handle' . "\n" . ' ($event))($event))', $fileContents);
         self::assertStringContainsStringIgnoringCase('static fn (\WyriHaximus\Broadcast\Dummy\Event $event) => await(async(static fn (\WyriHaximus\Broadcast\Dummy\Event $event) => \WyriHaximus\Broadcast\Dummy\AsyncListener::handleStatic ($event))($event))', $fileContents);
+
+        self::assertStringNotContainsStringIgnoringCase('\string::class => [', $fileContents);
+        self::assertStringNotContainsStringIgnoringCase('thisShouldNotBeDetected', $fileContents);
     }
 
     private function recurseCopy(string $src, string $dst): void