add test for createWithConfig

Author: dbu

.github/workflows/static.yml

@@ -1,5 +1,11 @@
-on: [push, pull_request]
 name: Static analysis
+on:
+  push:
+    branches:
+      - '[0-9]+.x'
+      - '[0-9]+.[0-9]+'
+      - '[0-9]+.[0-9]+.x'
+  pull_request:
 
 jobs:
   phpstan:

src/Promise.php

@@ -53,13 +53,13 @@ public function __construct(PromiseInterface $promise, RequestInterface $request
             $this->state = self::FULFILLED;
 
             return $response;
-        }, function ($reason) use ($request) {
+        }, function ($reason) {
             $this->state = self::REJECTED;
 
             if ($reason instanceof HttplugException) {
                 $this->exception = $reason;
             } elseif ($reason instanceof GuzzleExceptions\GuzzleException) {
-                $this->exception = $this->handleException($reason, $request);
+                $this->exception = $this->handleException($reason);
             } elseif ($reason instanceof \Throwable) {
                 $this->exception = new HttplugException\TransferException('Invalid exception returned from Guzzle7', 0, $reason);
             } else {
@@ -85,7 +85,7 @@ public function wait($unwrap = true)
         $this->promise->wait(false);
 
         if ($unwrap) {
-            if (self::REJECTED == $this->getState()) {
+            if (self::REJECTED === $this->getState()) {
                 throw $this->exception;
             }
 
@@ -98,7 +98,7 @@ public function wait($unwrap = true)
      *
      * @return HttplugException
      */
-    private function handleException(GuzzleExceptions\GuzzleException $exception, RequestInterface $request)
+    private function handleException(GuzzleExceptions\GuzzleException $exception)
     {
         if ($exception instanceof GuzzleExceptions\ConnectException) {
             return new HttplugException\NetworkException($exception->getMessage(), $exception->getRequest(), $exception);

tests/DefaultHttpAdapterWithConfigTest.php

@@ -0,0 +1,24 @@
+<?php
+
+declare(strict_types=1);
+
+use Http\Adapter\Guzzle7\Client;
+use Http\Client\Tests\HttpClientTest;
+use Psr\Http\Client\ClientInterface;
+
+/**
+ * @author David Buchmann <[email protected]>
+ */
+class DefaultHttpAdapterWithConfigTest extends HttpClientTest
+{
+    protected function createHttpAdapter(): ClientInterface
+    {
+        $this->defaultHeaders['X-Test'] = 'configuration-value';
+
+        return Client::createWithConfig([
+            'headers' => [
+                'X-Test' => 'configuration-value',
+            ],
+        ]);
+    }
+}