Add name to service in handler stack

gregurco · Nov 3, 2017 · 318fb2e · 318fb2e
1 parent 3bdcc17
commit 318fb2e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/GuzzleBundleCachePlugin.php b/src/GuzzleBundleCachePlugin.php
@@ -44,7 +44,7 @@ public function loadForClient(array $config, ContainerBuilder $container, string
 
  $cacheMiddlewareExpression = new Expression(sprintf('service("%s")', $cacheMiddlewareDefinitionName));
 
- $handler->addMethodCall('push', [$cacheMiddlewareExpression]);
+ $handler->addMethodCall('push', [$cacheMiddlewareExpression, 'cache']);
  }
  }
 

diff --git a/tests/GuzzleBundleCachePluginTest.php b/tests/GuzzleBundleCachePluginTest.php
@@ -76,6 +76,8 @@ public function testLoadForClientWithNoStrategy()
 
  $this->assertTrue($container->hasDefinition('guzzle_bundle_cache_plugin.middleware.api_payment'));
  $this->assertCount(1, $handler->getMethodCalls());
+ $this->assertTrue(isset($handler->getMethodCalls()[0][1][1]), 'No name provided for middleware');
+ $this->assertEquals('cache', $handler->getMethodCalls()[0][1][1]);
 
  $clientMiddlewareDefinition = $container->getDefinition('guzzle_bundle_cache_plugin.middleware.api_payment');
  $this->assertCount(0, $clientMiddlewareDefinition->getArguments());