Minor updates

Author: Nyholm

ClientFactory/ClientFactoryInterface.php

@@ -6,7 +6,7 @@
 use Http\HttplugBundle\Exception\ClientWasNotCreated;
 
 /**
- * Interface ClientFactoryInterface.
+ * @author Tobias Nyholm <tobias.nyholm@gmail.com>
  */
 interface ClientFactoryInterface
 {

ClientFactory/DummyFactory.php

@@ -2,7 +2,6 @@
 
 namespace Http\HttplugBundle\ClientFactory;
 
-use Http\Client\HttpClient;
 use Http\HttplugBundle\Exception\ClientWasNotCreated;
 
 /**

ClientFactory/GenericClientFactory.php

@@ -25,7 +25,7 @@ class GenericClientFactory
     public function getClient($name, array $config = [])
     {
         if (!isset($this->clientFactories[$name])) {
-            throw new \InvalidArgumentException(sprintf('No configurator named %s was found.', $name));
+            throw new \InvalidArgumentException(sprintf('No factory named %s was found. Configured factory names are: %s', $name, implode(', ', array_keys($this->clientFactories))));
         }
 
         return $this->clientFactories[$name]->createClient($config);

DependencyInjection/Configuration.php

@@ -78,14 +78,16 @@ public function getConfigTreeBuilder()
 
     protected function configureClients(ArrayNodeDefinition $root)
     {
-        $validAdapters = ['guzzle5', 'guzzle6'];
-
         $root->children()
             ->arrayNode('clients')
                 ->useAttributeAsKey('name')
                 ->prototype('array')
                 ->children()
-                    ->enumNode('adapter')->values($validAdapters)->isRequired()->cannotBeEmpty()->end()
+                    ->scalarNode('adapter')
+                        ->isRequired()
+                        ->cannotBeEmpty()
+                        ->info('The name of the adapter to use. There must be a factory corresponding to this name. Example values: guzzle5, guzzle6')
+                    ->end()
                     ->variableNode('config')->end()
                 ->end()
             ->end();