Make it possible to create public httplug clients

Author: ruudk

CHANGELOG.md

@@ -2,6 +2,12 @@
 
 The change log describes what is "Added", "Removed", "Changed" or "Fixed" between each release.
 
+## 1.12.0 - 2018-10-24
+
+### Added
+
+- Add configuration option to create public clients
+
 ## 1.11.0 - 2018-07-07
 
 ### Added

DependencyInjection/Configuration.php

@@ -195,6 +195,10 @@ private function configureClients(ArrayNodeDefinition $root)
                         ->defaultNull()
                         ->info('The service id of the client to use.')
                     ->end()
+                    ->booleanNode('public')
+                        ->defaultFalse()
+                        ->info('Set true if you really cannot use dependency injection and need to make the client service public.')
+                    ->end()
                     ->booleanNode('flexible_client')
                         ->defaultFalse()
                         ->info('Set to true to get the client wrapped in a FlexibleHttpClient which emulates async or sync behavior.')

DependencyInjection/HttplugExtension.php

@@ -316,15 +316,15 @@ private function configureClient(ContainerBuilder $container, $clientName, array
             $container
                 ->register($serviceId.'.client', HttpClient::class)
                 ->setFactory([new Reference($arguments['factory']), 'createClient'])
-                ->addArgument($arguments['config'])
-                ->setPublic(false);
+                ->addArgument($arguments['config']);
         } else {
             $container
                 ->setAlias($serviceId.'.client', new Alias($arguments['service'], false));
         }
 
         $container
             ->register($serviceId, PluginClient::class)
+            ->setPublic($arguments['public'])
             ->setFactory([new Reference(PluginClientFactory::class), 'createClient'])
             ->addArgument(new Reference($serviceId.'.client'))
             ->addArgument(

Tests/Unit/DependencyInjection/ConfigurationTest.php

@@ -119,6 +119,7 @@ public function testSupportsAllConfigFormats()
                     'factory' => 'httplug.factory.guzzle6',
                     'http_methods_client' => true,
                     'service' => null,
+                    'public' => false,
                     'flexible_client' => false,
                     'batch_client' => false,
                     'plugins' => [

Tests/Unit/DependencyInjection/HttplugExtensionTest.php

@@ -267,4 +267,30 @@ private function verifyProfilingDisabled()
             );
         }
     }
+
+    public function testClientShouldBePrivatByDefault()
+    {
+        $this->load([
+            'clients' => [
+                'acme' => [],
+            ],
+        ]);
+
+        $this->assertContainerBuilderHasService('httplug.client.acme');
+        $this->assertFalse($this->container->getDefinition('httplug.client.acme')->isPublic());
+    }
+
+    public function testClientCanBePublic()
+    {
+        $this->load([
+            'clients' => [
+                'acme' => [
+                    'public' => true,
+                ],
+            ],
+        ]);
+
+        $this->assertContainerBuilderHasService('httplug.client.acme');
+        $this->assertTrue($this->container->getDefinition('httplug.client.acme')->isPublic());
+    }
 }