Merge pull request #413 from bearsunday/spike

Fix primary_router binding

src/Provide/Router/RouterCollectionProvider.php

@@ -5,16 +5,14 @@
 namespace BEAR\Package\Provide\Router;
 
 use BEAR\Sunday\Extension\Router\RouterInterface;
-use Ray\Di\Di\Inject;
 use Ray\Di\Di\Named;
 use Ray\Di\ProviderInterface;
 
 /** @implements ProviderInterface<RouterCollection> */
 class RouterCollectionProvider implements ProviderInterface
 {
-    #[Inject]
     public function __construct(
-        #[Named('router')] private RouterInterface $primaryRouter,
+        #[Named('primary_router')] private RouterInterface $primaryRouter,
         private WebRouterInterface $webRouter,
     ) {
     }

tests/Fake/fake-app/src/Module/AppModule.php

@@ -33,6 +33,7 @@ protected function configure()
             [NullInterceptor::class]
         );
         $this->bind(FakeFoo::class);
+        $this->bind(FakeDep::class);
         $this->bind(Auth::class)->toProvider(AuthProvider::class);
     }
 }

tests/Provide/Router/RouterCollectionProviderTest.php

@@ -0,0 +1,33 @@
+<?php
+
+declare(strict_types=1);
+
+namespace BEAR\Package\Provide\Router;
+
+use BEAR\Package\FakeWebRouter;
+use BEAR\Package\PackageModule;
+use BEAR\Sunday\Extension\Router\RouterInterface;
+use PHPUnit\Framework\TestCase;
+use Ray\Di\AbstractModule;
+use Ray\Di\Injector;
+
+class RouterCollectionProviderTest extends TestCase
+{
+    public function testPrimaryRouter(): void
+    {
+        $module = new class extends AbstractModule
+        {
+            protected function configure(): void
+            {
+                $this->install(new PackageModule());
+                $this->bind(RouterInterface::class)->annotatedWith('primary_router')->toInstance(new FakeWebRouter('page://self', new HttpMethodParams()));
+                $this->bind(WebRouterInterface::class)->to(WebRouter::class);
+                $this->bind(RouterInterface::class)->toProvider(RouterCollectionProvider::class);
+            }
+        };
+        $i = new Injector();
+        $injector = new Injector($module);
+        $router = $injector->getInstance(RouterInterface::class);
+        $this->assertInstanceOf(RouterInterface::class, $router);
+    }
+}