Add more tests

src/Client.php

@@ -637,7 +637,7 @@ protected function _buildTransport(array $config): Transport
         // API key
         if (!empty($config['api_key'])) {
             if (!empty($config['username'])) {
-                throw new InvalidException('You cannot use APIKey and Basic Authentication together');
+                throw new InvalidException('You cannot use APIKey and Basic Authentication together.');
             }
 
             $transport->setHeader('Authorization', \sprintf('ApiKey %s', $config['api_key']));

tests/ClientTest.php

@@ -6,6 +6,8 @@
 use Elastica\ClientConfiguration;
 use Elastica\Exception\InvalidException;
 use Elastica\Test\Base as BaseTest;
+use Psr\Log\LoggerInterface;
+use Psr\Log\NullLogger;
 
 /**
  * @group unit
@@ -154,4 +156,73 @@ public function testClientConnectionWithCloudId(): void
 
         $this->assertEquals('4de46ced8d8d459696e544fe5f32b999.eu-central-1.aws.cloud.es.io', $node->getUri()->getHost());
     }
+
+    public function testItSetLogger(): void
+    {
+        $client = new Client();
+
+        self::assertInstanceOf(NullLogger::class, $client->getLogger());
+
+        $client->setLogger(
+            new class() implements LoggerInterface {
+                public function emergency($message, array $context = [])
+                {
+                }
+
+                public function alert($message, array $context = [])
+                {
+                }
+
+                public function critical($message, array $context = [])
+                {
+                }
+
+                public function error($message, array $context = [])
+                {
+                }
+
+                public function warning($message, array $context = [])
+                {
+                }
+
+                public function notice($message, array $context = [])
+                {
+                }
+
+                public function info($message, array $context = [])
+                {
+                }
+
+                public function debug($message, array $context = [])
+                {
+                }
+
+                public function log($level, $message, array $context = [])
+                {
+                }
+            }
+        );
+
+        self::assertNotInstanceOf(NullLogger::class, $client->getLogger());
+    }
+
+    public function testItThrowsAnExceptionWhenApiKeyAndUserNameInConfigAtTheSameTime(): void
+    {
+        $this->expectException(InvalidException::class);
+        $this->expectExceptionMessage('You cannot use APIKey and Basic Authentication together.');
+
+        new Client([
+            'username' => 'user',
+            'api_key' => 'key',
+        ]);
+    }
+
+    public function testItSetsAuthorizationHeaderIfApiKeyPassed(): void
+    {
+        $apiKey = 'key';
+
+        $client = new Client(['api_key' => $apiKey]);
+
+        self::assertSame(['Authorization' => \sprintf('ApiKey %s', $apiKey)], $client->getTransport()->getHeaders());
+    }
 }