Added logging on HTTP exception (#507)

* Added logging on HTTP exception

* Fixed tests

* Test fix

* Added name to please "composer normalize"

* Dont log in constructor

* Mniir

* PHPStan fix

Author: Nyholm

tests/Unit/Result/CreateTableOutputTest.php

@@ -6,6 +6,7 @@
 use AsyncAws\Core\Test\Http\SimpleMockedResponse;
 use AsyncAws\Core\Test\TestCase;
 use AsyncAws\DynamoDb\Result\CreateTableOutput;
+use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 
 class CreateTableOutputTest extends TestCase
@@ -49,7 +50,7 @@ public function testCreateTableOutput(): void
         }');
 
         $client = new MockHttpClient($response);
-        $result = new CreateTableOutput(new Response($client->request('POST', 'http://localhost'), $client));
+        $result = new CreateTableOutput(new Response($client->request('POST', 'http://localhost'), $client, new NullLogger()));
 
         self::assertEquals('Artist', $result->getTableDescription()->getAttributeDefinitions()[0]->getAttributeName());
         self::assertEquals('SongTitle', $result->getTableDescription()->getAttributeDefinitions()[1]->getAttributeName());

tests/Unit/Result/DeleteItemOutputTest.php

@@ -6,6 +6,7 @@
 use AsyncAws\Core\Test\Http\SimpleMockedResponse;
 use AsyncAws\Core\Test\TestCase;
 use AsyncAws\DynamoDb\Result\DeleteItemOutput;
+use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 
 class DeleteItemOutputTest extends TestCase
@@ -21,7 +22,7 @@ public function testDeleteItemOutput(): void
         }');
 
         $client = new MockHttpClient($response);
-        $result = new DeleteItemOutput(new Response($client->request('POST', 'http://localhost'), $client));
+        $result = new DeleteItemOutput(new Response($client->request('POST', 'http://localhost'), $client, new NullLogger()));
 
         self::assertEquals(1, $result->getConsumedCapacity()->getCapacityUnits());
         self::assertEquals('Music', $result->getConsumedCapacity()->getTableName());

tests/Unit/Result/DeleteTableOutputTest.php

@@ -6,6 +6,7 @@
 use AsyncAws\Core\Test\Http\SimpleMockedResponse;
 use AsyncAws\Core\Test\TestCase;
 use AsyncAws\DynamoDb\Result\DeleteTableOutput;
+use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 
 class DeleteTableOutputTest extends TestCase
@@ -29,7 +30,7 @@ public function testDeleteTableOutput(): void
         }');
 
         $client = new MockHttpClient($response);
-        $result = new DeleteTableOutput(new Response($client->request('POST', 'http://localhost'), $client));
+        $result = new DeleteTableOutput(new Response($client->request('POST', 'http://localhost'), $client, new NullLogger()));
 
         self::assertEquals(0, $result->getTableDescription()->getItemCount());
         self::assertEquals(1, $result->getTableDescription()->getProvisionedThroughput()->getNumberOfDecreasesToday());

tests/Unit/Result/DescribeTableOutputTest.php

@@ -6,6 +6,7 @@
 use AsyncAws\Core\Test\Http\SimpleMockedResponse;
 use AsyncAws\Core\Test\TestCase;
 use AsyncAws\DynamoDb\Result\DescribeTableOutput;
+use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 
 class DescribeTableOutputTest extends TestCase
@@ -49,7 +50,7 @@ public function testDescribeTableOutput(): void
         }');
 
         $client = new MockHttpClient($response);
-        $result = new DescribeTableOutput(new Response($client->request('POST', 'http://localhost'), $client));
+        $result = new DescribeTableOutput(new Response($client->request('POST', 'http://localhost'), $client, new NullLogger()));
 
         self::assertEquals('Artist', $result->getTable()->getAttributeDefinitions()[0]->getAttributeName());
         self::assertEquals('SongTitle', $result->getTable()->getAttributeDefinitions()[1]->getAttributeName());

tests/Unit/Result/GetItemOutputTest.php

@@ -6,6 +6,7 @@
 use AsyncAws\Core\Test\Http\SimpleMockedResponse;
 use AsyncAws\Core\Test\TestCase;
 use AsyncAws\DynamoDb\Result\GetItemOutput;
+use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 
 class GetItemOutputTest extends TestCase
@@ -28,7 +29,7 @@ public function testGetItemOutput(): void
         }');
 
         $client = new MockHttpClient($response);
-        $result = new GetItemOutput(new Response($client->request('POST', 'http://localhost'), $client));
+        $result = new GetItemOutput(new Response($client->request('POST', 'http://localhost'), $client, new NullLogger()));
 
         self::assertEquals('Songs About Life', $result->getItem()['AlbumTitle']->getS());
         self::assertEquals('Acme Band', $result->getItem()['Artist']->getS());

tests/Unit/Result/ListTablesOutputTest.php

@@ -6,6 +6,7 @@
 use AsyncAws\Core\Test\Http\SimpleMockedResponse;
 use AsyncAws\Core\Test\TestCase;
 use AsyncAws\DynamoDb\Result\ListTablesOutput;
+use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 
 class ListTablesOutputTest extends TestCase
@@ -24,7 +25,7 @@ public function testListTablesOutput(): void
         }');
 
         $client = new MockHttpClient($response);
-        $result = new ListTablesOutput(new Response($client->request('POST', 'http://localhost'), $client));
+        $result = new ListTablesOutput(new Response($client->request('POST', 'http://localhost'), $client, new NullLogger()));
 
         $tableNames = $result->getTableNames(true);
         foreach ($tableNames as $name) {

tests/Unit/Result/PutItemOutputTest.php

@@ -6,6 +6,7 @@
 use AsyncAws\Core\Test\Http\SimpleMockedResponse;
 use AsyncAws\Core\Test\TestCase;
 use AsyncAws\DynamoDb\Result\PutItemOutput;
+use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 
 class PutItemOutputTest extends TestCase
@@ -22,7 +23,7 @@ public function testPutItemOutput(): void
         }');
 
         $client = new MockHttpClient($response);
-        $result = new PutItemOutput(new Response($client->request('POST', 'http://localhost'), $client));
+        $result = new PutItemOutput(new Response($client->request('POST', 'http://localhost'), $client, new NullLogger()));
 
         self::assertEquals(1, $result->getConsumedCapacity()->getCapacityUnits());
         self::assertEquals('Music', $result->getConsumedCapacity()->getTableName());

tests/Unit/Result/QueryOutputTest.php

@@ -7,6 +7,7 @@
 use AsyncAws\Core\Test\TestCase;
 use AsyncAws\DynamoDb\Result\QueryOutput;
 use AsyncAws\DynamoDb\ValueObject\ConsumedCapacity;
+use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 
 class QueryOutputTest extends TestCase
@@ -31,7 +32,7 @@ public function testQueryOutput(): void
         }');
 
         $client = new MockHttpClient($response);
-        $result = new QueryOutput(new Response($client->request('POST', 'http://localhost'), $client));
+        $result = new QueryOutput(new Response($client->request('POST', 'http://localhost'), $client, new NullLogger()));
 
         $items = $result->getItems(true);
         foreach ($items as $name => $item) {

tests/Unit/Result/ScanOutputTest.php

@@ -6,6 +6,7 @@
 use AsyncAws\Core\Test\Http\SimpleMockedResponse;
 use AsyncAws\Core\Test\TestCase;
 use AsyncAws\DynamoDb\Result\ScanOutput;
+use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 
 class ScanOutputTest extends TestCase
@@ -38,7 +39,7 @@ public function testScanOutput(): void
         }');
 
         $client = new MockHttpClient($response);
-        $result = new ScanOutput(new Response($client->request('POST', 'http://localhost'), $client));
+        $result = new ScanOutput(new Response($client->request('POST', 'http://localhost'), $client, new NullLogger()));
 
         $items = $result->getItems(true);
         foreach ($items as $name => $item) {

tests/Unit/Result/UpdateItemOutputTest.php

@@ -6,6 +6,7 @@
 use AsyncAws\Core\Test\Http\SimpleMockedResponse;
 use AsyncAws\Core\Test\TestCase;
 use AsyncAws\DynamoDb\Result\UpdateItemOutput;
+use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 
 class UpdateItemOutputTest extends TestCase
@@ -31,7 +32,7 @@ public function testUpdateItemOutput(): void
         }');
 
         $client = new MockHttpClient($response);
-        $result = new UpdateItemOutput(new Response($client->request('POST', 'http://localhost'), $client));
+        $result = new UpdateItemOutput(new Response($client->request('POST', 'http://localhost'), $client, new NullLogger()));
 
         $attributes = $result->getAttributes();
         self::assertCount(4, $attributes);