PSR-18: Network / Request exception inheritance (#158)

#155 Distinct Network and Request exceptions

Author: davidgrayston

CHANGELOG.md

@@ -9,6 +9,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
+### Changed
+
+- `Http\Client\Exception\NetworkException` no longer extends `Http\Client\Exception\RequestException`,
+  in accordance with [PSR-18](https://www.php-fig.org/psr/psr-18/)
 
 ## [2.0.0] - 2018-10-31
 

spec/Exception/NetworkExceptionSpec.php

@@ -17,8 +17,28 @@ function it_is_initializable()
         $this->shouldHaveType('Http\Client\Exception\NetworkException');
     }
 
-    function it_is_a_request_exception()
+    function it_is_a_transfer_exception()
     {
-        $this->shouldHaveType('Http\Client\Exception\RequestException');
+        $this->shouldHaveType('Http\Client\Exception\TransferException');
+    }
+
+    function it_implements_psr_network_exception_interface()
+    {
+        $this->shouldHaveType('Psr\Http\Client\NetworkExceptionInterface');
+    }
+
+    function it_does_not_implement_psr_request_exception_interface()
+    {
+        $this->shouldNotHaveType('Psr\Http\Client\RequestExceptionInterface');
+    }
+
+    function it_is_not_a_request_exception()
+    {
+        $this->shouldNotHaveType('Http\Client\Exception\RequestException');
+    }
+
+    function it_has_a_request(RequestInterface $request)
+    {
+        $this->getRequest()->shouldReturn($request);
     }
 }

spec/Exception/RequestExceptionSpec.php

@@ -2,7 +2,6 @@
 
 namespace spec\Http\Client\Exception;
 
-use Http\Client\Exception\RequestException;
 use Psr\Http\Message\RequestInterface;
 use PhpSpec\ObjectBehavior;
 
@@ -23,6 +22,21 @@ function it_is_a_transfer_exception()
         $this->shouldHaveType('Http\Client\Exception\TransferException');
     }
 
+    function it_implements_psr_request_exception_interface()
+    {
+        $this->shouldHaveType('Psr\Http\Client\RequestExceptionInterface');
+    }
+
+    function it_does_not_implement_psr_network_exception_interface()
+    {
+        $this->shouldNotHaveType('Psr\Http\Client\NetworkExceptionInterface');
+    }
+
+    function it_is_not_a_network_exception()
+    {
+        $this->shouldNotHaveType('Http\Client\Exception\NetworkException');
+    }
+
     function it_has_a_request(RequestInterface $request)
     {
         $this->getRequest()->shouldReturn($request);

src/Exception/NetworkException.php

@@ -2,6 +2,7 @@
 
 namespace Http\Client\Exception;
 
+use Psr\Http\Message\RequestInterface;
 use Psr\Http\Client\NetworkExceptionInterface as PsrNetworkException;
 
 /**
@@ -11,6 +12,19 @@
  *
  * @author Márk Sági-Kazár <[email protected]>
  */
-class NetworkException extends RequestException implements PsrNetworkException
+class NetworkException extends TransferException implements PsrNetworkException
 {
+    use RequestAwareTrait;
+
+    /**
+     * @param string           $message
+     * @param RequestInterface $request
+     * @param \Exception|null  $previous
+     */
+    public function __construct($message, RequestInterface $request, \Exception $previous = null)
+    {
+        $this->setRequest($request);
+
+        parent::__construct($message, 0, $previous);
+    }
 }

src/Exception/RequestAwareTrait.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace Http\Client\Exception;
+
+use Psr\Http\Message\RequestInterface;
+
+trait RequestAwareTrait
+{
+    /**
+     * @var RequestInterface
+     */
+    private $request;
+
+    private function setRequest(RequestInterface $request)
+    {
+        $this->request = $request;
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function getRequest(): RequestInterface
+    {
+        return $this->request;
+    }
+}

src/Exception/RequestException.php

@@ -15,10 +15,7 @@
  */
 class RequestException extends TransferException implements PsrRequestException
 {
-    /**
-     * @var RequestInterface
-     */
-    private $request;
+    use RequestAwareTrait;
 
     /**
      * @param string           $message
@@ -27,13 +24,8 @@ class RequestException extends TransferException implements PsrRequestException
      */
     public function __construct($message, RequestInterface $request, \Exception $previous = null)
     {
-        $this->request = $request;
+        $this->setRequest($request);
 
         parent::__construct($message, 0, $previous);
     }
-
-    public function getRequest(): RequestInterface
-    {
-        return $this->request;
-    }
 }