Fix Exception

Author: barryvdh

src/Common/Http/Exception.php

@@ -26,6 +26,6 @@ public function __construct(string $message, RequestInterface $request, Throwabl
      */
     public function getRequest(): RequestInterface
     {
-        return $this->getRequest();
+        return $this->request;
     }
 }

tests/Omnipay/Common/Http/ClientTest.php

@@ -8,6 +8,7 @@
 use GuzzleHttp\Psr7\Request;
 use Http\Client\HttpClient;
 use Http\Message\RequestFactory;
+use Omnipay\Common\Http\Exception\RequestException;
 use Omnipay\Tests\TestCase;
 
 class ClientTest extends TestCase
@@ -99,4 +100,41 @@ public function testSendNetworkException()
 
         $client->request('GET', '/path');
     }
+
+    public function testSendExceptionGetRequest()
+    {
+        $mockClient = m::mock(HttpClient::class);
+        $mockFactory = m::mock(RequestFactory::class);
+        $client = new Client($mockClient, $mockFactory);
+
+        $request = new Request('GET', '/path');
+        $response = new Response();
+
+        $mockFactory->shouldReceive('createRequest')->withArgs([
+            'GET',
+            '/path',
+            [],
+            null,
+            '1.1',
+        ])->andReturn($request);
+
+        $exception = new \Exception('Something went wrong');
+
+        $mockClient->shouldReceive('sendRequest')
+            ->with($request)
+            ->andThrow($exception);
+
+        $this->expectException(\Omnipay\Common\Http\Exception\RequestException::class);
+        $this->expectExceptionMessage('Something went wrong');
+
+
+        try {
+            $client->request('GET', '/path');
+        } catch (RequestException $e) {
+            $this->assertSame($request, $e->getRequest());
+            $this->assertSame($exception, $e->getPrevious());
+
+            throw $e;
+        }
+    }
 }

tests/Omnipay/Common/Http/ExceptionTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Omnipay\Common\Http;
+
+use GuzzleHttp\Psr7\Request;
+use Omnipay\Common\Http\Exception\NetworkException;
+use Omnipay\Tests\TestCase;
+
+class ExceptionTest extends TestCase
+{
+    public function testConstruct()
+    {
+        $request = new Request('GET', '/path');
+
+        $previous = new \Exception('Whoops');
+        $exception = new NetworkException('Something went wrong', $request, $previous);
+
+        $this->assertSame($request, $exception->getRequest());
+        $this->assertSame('Something went wrong', $exception->getMessage());
+        $this->assertSame($previous, $exception->getPrevious());
+    }
+}