Merge pull request #348 from localheinz/feature/tag

Enhancement: Tag clients

Author: dbu

CHANGELOG.md

@@ -2,6 +2,12 @@
 
 The change log describes what is "Added", "Removed", "Changed" or "Fixed" between each release.
 
+## Unreleased
+
+### Added
+
+- Configured clients are now tagged with `'httplug.client'`
+
 ## 1.16.0 - 2019-06-05
 
 ### Changed

src/DependencyInjection/HttplugExtension.php

@@ -37,6 +37,8 @@
  */
 class HttplugExtension extends Extension
 {
+    public const HTTPLUG_CLIENT_TAG = 'httplug.client';
+
     /**
      * Used to check is the VCR plugin is installed.
      *
@@ -419,6 +421,7 @@ function ($id) {
             ->addArgument([
                 'client_name' => $clientName,
             ])
+            ->addTag(self::HTTPLUG_CLIENT_TAG)
         ;
 
         if (is_bool($arguments['public'])) {

tests/Unit/DependencyInjection/HttplugExtensionTest.php

@@ -31,6 +31,11 @@ protected function getContainerExtensions()
         ];
     }
 
+    public function testConstants(): void
+    {
+        self::assertSame('httplug.client', HttplugExtension::HTTPLUG_CLIENT_TAG);
+    }
+
     public function testConfigLoadDefault(): void
     {
         $this->load();
@@ -452,6 +457,25 @@ public function testBatchClientCanBePublic(): void
         }
     }
 
+    public function testClientIsTaggedWithHttplugClientTag(): void
+    {
+        $this->load([
+            'clients' => [
+                'acme' => null,
+            ],
+        ]);
+
+        $serviceId = 'httplug.client.acme';
+
+        $this->assertContainerBuilderHasService($serviceId);
+
+        $this->assertTrue($this->container->getDefinition($serviceId)->hasTag(HttplugExtension::HTTPLUG_CLIENT_TAG), sprintf(
+            'Failed asserting that client with service identifier "%s" has been tagged with "%s".',
+            $serviceId,
+            HttplugExtension::HTTPLUG_CLIENT_TAG
+        ));
+    }
+
     /**
      * @dataProvider provideVcrPluginConfig
      * @group vcr-plugin