Merge pull request #15 from elcuro/remove-inflector-check

Removed method inflector service check
thephpleague · Apr 21, 2016 · 14b38eb · 14b38eb
2 parents db454e4 + 9cd6fda
commit 14b38eb
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 17 deletions.
diff --git a/DependencyInjection/TacticianExtension.php b/DependencyInjection/TacticianExtension.php
@@ -59,20 +59,13 @@ function ($middlewareServiceId) {
      *
      * @param array $mergedConfig
      * @param ContainerBuilder $container
-     * @throws \Exception
      */
     private function injectMethodNameInflector(array $mergedConfig, ContainerBuilder $container)
     {
         if (! $container->has('tactician.middleware.command_handler')) {
             return;
         }
 
-        if (! $container->has($mergedConfig['method_inflector'])) {
-            throw new \Exception(
-                'Unable to find requested method_inflector service definition: ' . $mergedConfig['method_inflector']
-            );
-        }
-
         $inflectorReference = new Reference($mergedConfig['method_inflector']);
 
         $handlerLocator = $container->findDefinition('tactician.middleware.command_handler');

diff --git a/Tests/DependencyInjection/TacticianExtensionTest.php b/Tests/DependencyInjection/TacticianExtensionTest.php
@@ -121,14 +121,4 @@ public function testMethodNameInflectorNonDefault()
             new Reference('tactician.handler.method_name_inflector.handle_class_name_without_suffix')
         );
     }
-
-    /**
-     * @expectedException \Exception
-     */
-    public function testMethodNameInflectorThrowsErrorIfNonExistingService()
-    {
-        $this->load([
-            'method_inflector' => 'i.do.not.exist'
-        ]);
-    }
 }