We do not actually require Psr\Log

adamwinn · May 5, 2015 · 5ec80d5 · 5ec80d5
1 parent 89a9b30
commit 5ec80d5
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 17 deletions.
diff --git a/composer.json b/composer.json
@@ -30,8 +30,7 @@
         "ext-json": "*",
         "ext-dom": "*",
         "ext-simplexml": "*",
-        "phpunit/phpunit": "^4.0",
-        "psr/log": "^1.0"
+        "phpunit/phpunit": "^4.0"
     },
     "suggest": {
         "ext-openssl": "Allows working with CloudFront private distributions and verifying received SNS messages",

diff --git a/tests/ClientResolverTest.php b/tests/ClientResolverTest.php
@@ -276,21 +276,6 @@ public function testCanUseCustomEndpointProviderWithExtraData()
         $this->assertEquals('v2', $conf['config']['signature_version']);
     }
 
-    public function testAddsLogger()
-    {
-        $r = new ClientResolver(ClientResolver::getDefaultArguments());
-        $logger = $this->getMockBuilder('Psr\Log\LoggerInterface')
-            ->getMockForAbstractClass();
-        $conf = $r->resolve([
-            'service'      => 'sqs',
-            'region'       => 'x',
-            'retries'      => 2,
-            'retry_logger' => $logger,
-            'endpoint'     => 'http://us-east-1.foo.amazonaws.com',
-            'version'      => 'latest'
-        ], new HandlerList());
-    }
-
     public function testAddsLoggerWithDebugSettings()
     {
         $r = new ClientResolver(ClientResolver::getDefaultArguments());