Feature: Bind a service with tags (#7)

Signed-off-by: Nathanael Esayeas <nathanael.esayeas@protonmail.com>
ghostwriter · May 26, 2022 · 008e594 · 008e594
2 parents 30ae497 + ab5b99e
commit 008e594
Show file tree

Hide file tree

Showing 4 changed files with 42 additions and 26 deletions.
diff --git a/composer.lock b/composer.lock
diff --git a/src/Container.php b/src/Container.php
@@ -113,7 +113,7 @@ public function alias(string $alias, string $id): void
         $this->services[self::ALIASES][$alias] = $id;
     }
 
-    public function bind(string $abstract, ?string $concrete = null): void
+    public function bind(string $abstract, ?string $concrete = null, iterable $tags = []): void
     {
         if ('' === trim($abstract)) {
             throw InvalidArgumentException::emptyServiceId();
@@ -129,6 +129,12 @@ public function bind(string $abstract, ?string $concrete = null): void
 
         $this->services[self::FACTORIES][$abstract] =
             static fn (ContainerInterface $container): object => $container->build($concrete ?? $abstract);
+
+        if ([] === $tags) {
+            return;
+        }
+
+        $this->tag($abstract, $tags);
     }
 
     public function build(string $class, array $arguments = []): object

diff --git a/src/Contract/ContainerInterface.php b/src/Contract/ContainerInterface.php
@@ -155,10 +155,12 @@ public function alias(string $alias, string $id): void;
     /**
      * Bind abstract classes or interfaces to concrete implementations.
      *
+     * @param iterable<string> $tags
+     *
      * @throws InvalidArgumentException if $abstract is empty
      * @throws LogicException           if $abstract is already registered
      */
-    public function bind(string $abstract, ?string $concrete = null): void;
+    public function bind(string $abstract, ?string $concrete = null, iterable $tags = []): void;
 
     /**
      * Create an object using the given Container to resolve dependencies.

diff --git a/tests/Unit/ContainerTest.php b/tests/Unit/ContainerTest.php
@@ -513,19 +513,22 @@ public function testContainerAlias(): void
      * @covers \Ghostwriter\Container\Container::getInstance
      * @covers \Ghostwriter\Container\Container::has
      * @covers \Ghostwriter\Container\Container::resolve
+     * @covers \Ghostwriter\Container\Container::tag
+     * @covers \Ghostwriter\Container\Container::tagged
      *
      * @throws Throwable
      */
     public function testContainerBind(): void
     {
-        $this->container->bind(DummyInterface::class, Foo::class);
+        $this->container->bind(DummyInterface::class, Foo::class, ['taggable']);
         $this->container->bind(Baz::class);
 
         self::assertTrue($this->container->has(DummyInterface::class));
         self::assertTrue($this->container->has(Baz::class));
 
         self::assertInstanceOf(Foo::class, $this->container->get(DummyInterface::class));
         self::assertInstanceOf(Baz::class, $this->container->get(Baz::class));
+        self::assertCount(1, $this->container->tagged('taggable'));
     }
 
     /**