fix: ClassMapGenerator get too many classes (#13)

Closes #12

Author: Gashmob

README.md

@@ -31,3 +31,5 @@ $dispatcher->dispatch(new MyEvent());
 ```
 
 That's all!
+
+Please note that if your package is also a Brick, it will scan only `src` and `include` directory for Services.

include/Bricks/LoadBricks.php

@@ -32,16 +32,13 @@
 use Composer\ClassMapGenerator\ClassMapGenerator;
 use Composer\InstalledVersions;
 use ReflectionClass;
-use ReflectionException;
+use Throwable;
 
 /**
  * @internal
  */
 final readonly class LoadBricks implements BricksLoader
 {
-    /**
-     * @throws ReflectionException
-     */
     public function loadInstalledBricks(): array
     {
         $packages = array_unique(InstalledVersions::getInstalledPackagesByType(self::PACKAGE_TYPE));
@@ -64,12 +61,11 @@ public function loadInstalledBricks(): array
 
     /**
      * @return ServiceRepresentation[]
-     * @throws ReflectionException
      */
     private function loadServicesOfPackage(string $package_path): array
     {
         $result = [];
-        $map    = ClassMapGenerator::createMap($package_path);
+        $map    = $this->getClassMapForPackage($package_path);
         foreach ($map as $symbol => $_path) {
             $reflection = new ReflectionClass($symbol);
 
@@ -85,4 +81,22 @@ private function loadServicesOfPackage(string $package_path): array
 
         return $result;
     }
+
+    /**
+     * @return array<class-string, non-empty-string>
+     */
+    private function getClassMapForPackage(string $package_path): array
+    {
+        $generator   = new ClassMapGenerator();
+        $path_to_try = ['src', 'include'];
+        foreach ($path_to_try as $path) {
+            try {
+                $generator->scanPaths($package_path . '/' . $path);
+            } catch (Throwable) { // phpcs:ignore
+                // Ignore it
+            }
+        }
+
+        return $generator->getClassMap()->getMap();
+    }
 }

tests/unit/Bricks/LoadBricksTest.php

@@ -27,8 +27,8 @@
 
 namespace Archict\Core\Bricks;
 
-use Archict\Core\Fixtures\brick1\Service1;
-use Archict\Core\Fixtures\brick1\Service1Configuration;
+use Archict\Core\Fixtures\brick1\src\Service1;
+use Archict\Core\Fixtures\brick1\src\Service1Configuration;
 use Composer\InstalledVersions;
 use PHPUnit\Framework\TestCase;
 

tests/unit/Event/EventManagerTest.php

@@ -27,7 +27,7 @@
 
 namespace Archict\Core\Event;
 
-use Archict\Core\Fixtures\brick1\MyEvent;
+use Archict\Core\Fixtures\brick1\src\MyEvent;
 use Archict\Core\Services\ServiceManager;
 use PHPUnit\Framework\TestCase;
 

tests/unit/Event/LoadEventsTest.php

@@ -28,7 +28,7 @@
 namespace Archict\Core\Event;
 
 use Archict\Brick\Service;
-use Archict\Core\Fixtures\brick1\Service1;
+use Archict\Core\Fixtures\brick1\src\Service1;
 use Archict\Core\Services\ServiceManager;
 use Archict\Core\Services\ServiceRepresentation;
 use PHPUnit\Framework\TestCase;

tests/unit/Fixtures/ServiceWithDependency.php

@@ -27,7 +27,7 @@
 
 namespace Archict\Core\Fixtures;
 
-use Archict\Core\Fixtures\brick1\Service1;
+use Archict\Core\Fixtures\brick1\src\Service1;
 
 final readonly class ServiceWithDependency
 {

tests/unit/Fixtures/brick1/IService1.php …s/unit/Fixtures/brick1/src/IService1.phptests/unit/Fixtures/brick1/IService1.php renamed to tests/unit/Fixtures/brick1/src/IService1.php tests/unit/Fixtures/brick1/IService1.php renamed to tests/unit/Fixtures/brick1/src/IService1.php

@@ -25,7 +25,7 @@
 
 declare(strict_types=1);
 
-namespace Archict\Core\Fixtures\brick1;
+namespace Archict\Core\Fixtures\brick1\src;
 
 interface IService1
 {

tests/unit/Fixtures/brick1/MyEvent.php …sts/unit/Fixtures/brick1/src/MyEvent.phptests/unit/Fixtures/brick1/MyEvent.php renamed to tests/unit/Fixtures/brick1/src/MyEvent.php tests/unit/Fixtures/brick1/MyEvent.php renamed to tests/unit/Fixtures/brick1/src/MyEvent.php

@@ -25,7 +25,7 @@
 
 declare(strict_types=1);
 
-namespace Archict\Core\Fixtures\brick1;
+namespace Archict\Core\Fixtures\brick1\src;
 
 final class MyEvent
 {

tests/unit/Fixtures/brick1/Service1.php …ts/unit/Fixtures/brick1/src/Service1.phptests/unit/Fixtures/brick1/Service1.php renamed to tests/unit/Fixtures/brick1/src/Service1.php tests/unit/Fixtures/brick1/Service1.php renamed to tests/unit/Fixtures/brick1/src/Service1.php

@@ -25,7 +25,7 @@
 
 declare(strict_types=1);
 
-namespace Archict\Core\Fixtures\brick1;
+namespace Archict\Core\Fixtures\brick1\src;
 
 use Archict\Brick\ListeningEvent;
 use Archict\Brick\Service;

…ixtures/brick1/Service1Configuration.php …res/brick1/src/Service1Configuration.phptests/unit/Fixtures/brick1/Service1Configuration.php renamed to tests/unit/Fixtures/brick1/src/Service1Configuration.php tests/unit/Fixtures/brick1/Service1Configuration.php renamed to tests/unit/Fixtures/brick1/src/Service1Configuration.php

@@ -25,7 +25,7 @@
 
 declare(strict_types=1);
 
-namespace Archict\Core\Fixtures\brick1;
+namespace Archict\Core\Fixtures\brick1\src;
 
 final readonly class Service1Configuration
 {