Add Support for PHP 8.x

Author: chadicus

composer.json

@@ -14,12 +14,12 @@
         "sort-packages": true
     },
     "require": {
-        "php": "^7.0",
+        "php": "^7.0||^8.0",
         "guzzlehttp/guzzle": "^6.3",
-        "traderinteractive/util-arrays": "^3.1"
+        "traderinteractive/util-arrays": "^3.1||^4.0"
     },
     "require-dev": {
-        "phpunit/phpunit": "^6.0",
+        "phpunit/phpunit": "^6.0 || ^7.0 || ^8.0 || ^9.0",
         "squizlabs/php_codesniffer": "^3.2"
     },
     "autoload": {

phpunit.xml.dist

@@ -1,4 +1,4 @@
-<phpunit colors="true" forceCoversAnnotation="true" strict="true">
+<phpunit colors="true" forceCoversAnnotation="true">
     <testsuite name="Unit Tests">
         <directory>./tests</directory>
     </testsuite>

tests/Databases/AbstractNetAcuityDatabaseTest.php

@@ -151,12 +151,12 @@ public function getGeoWithExtraFieldEdge()
      *
      * @test
      * @covers ::fetch
-     * @expectedException Exception
-     * @expectedExceptionMessage NetAcuity API rejected the request, Reason: Invalid IP (1)
-     * @expectedExceptionCode 400
      */
     public function getGeoNonStringIp()
     {
+        $this->expectExceptionMessage('NetAcuity API rejected the request, Reason: Invalid IP (1)');
+        $this->expectExceptionCode(400);
+
         $mockException = $this->getMockClientException(400, 'Invalid IP (1)');
         $mockClient = $this->getMockGuzzleClient();
         $mockClient->method('send')->will($this->throwException($mockException));
@@ -167,15 +167,13 @@ public function getGeoNonStringIp()
 
     /**
      * @test
-     *
      * @covers ::fetch
-     *
-     * @expectedException Exception
-     * @expectedExceptionMessage NetAcuity API rejected the provided api user token.
-     * @expectedExceptionCode 403
      */
     public function netAcuityUserTokenInvalid()
     {
+        $this->expectExceptionMessage('NetAcuity API rejected the provided api user token.');
+        $this->expectExceptionCode(403);
+
         $mockException = $this->getMockClientException(403, 'Invalid IP (1)');
         $mockClient = $this->getMockGuzzleClient();
         $mockClient->method('send')->will($this->throwException($mockException));

tests/NetAcuityTest.php

@@ -2,13 +2,14 @@
 
 namespace TraderInteractive\NetAcuity\Tests;
 
+use PHPUnit\Framework\TestCase;
 use TraderInteractive\NetAcuity\NetAcuity;
 
 /**
  * @coversDefaultClass \TraderInteractive\NetAcuity\NetAcuity
  * @covers ::__construct
  */
-final class NetAcuityTest extends NetAcuityTestSuite
+final class NetAcuityTest extends TestCase
 {
     /**
      * @test

tests/NetAcuityTestSuite.php

@@ -4,53 +4,28 @@
 
 use GuzzleHttp\ClientInterface;
 use GuzzleHttp\Exception\ClientException;
+use GuzzleHttp\Psr7\Request;
+use GuzzleHttp\Psr7\Response;
+use GuzzleHttp\Psr7\Utils;
 use PHPUnit\Framework\TestCase;
 use Psr\Http\Message\ResponseInterface;
 
 abstract class NetAcuityTestSuite extends TestCase
 {
     protected function getMockGuzzleClient(): ClientInterface
     {
-        return $this->getMockBuilder(
-            '\GuzzleHttp\Client'
-        )->disableOriginalConstructor()->setMethods(['send'])->getMock();
+        return $this->getMockBuilder(ClientInterface::class)->getMock();
     }
 
     protected function getMockClientException(int $code, string $errorMessage): ClientException
     {
-        $mockStream = $this->getMockBuilder(
-            '\GuzzleHttp\Psr7\Stream'
-        )->disableOriginalConstructor()->setMethods(['getContents'])->getMock();
-        $mockStream->method(
-            'getContents'
-        )->willReturn(json_encode(['error' => ['message' => $errorMessage]]));
-
-        $mockResponse = $this->getMockBuilder(
-            '\GuzzleHttp\Psr7\Response'
-        )->disableOriginalConstructor()->setMethods(['getStatusCode', 'getBody'])->getMock();
-        $mockResponse->method('getStatusCode')->willReturn($code);
-        $mockResponse->method('getBody')->willReturn($mockStream);
-
-        $mockException = $this->getMockBuilder(
-            '\GuzzleHttp\Exception\ClientException'
-        )->disableOriginalConstructor()->setMethods(['getResponse'])->getMock();
-        $mockException->method('getResponse')->willReturn($mockResponse);
-
-        return $mockException;
+        $stream = Utils::streamFor(json_encode(['error' => ['message' => $errorMessage]]));
+        $response = new Response($code, [], $stream);
+        return new ClientException($errorMessage, new Request('GET', 'localhost'), $response);
     }
 
     protected function getMockResponse(array $response): ResponseInterface
     {
-        $mockStream = $this->getMockBuilder(
-            '\GuzzleHttp\Psr7\Stream'
-        )->disableOriginalConstructor()->setMethods(['getContents'])->getMock();
-        $mockStream->method('getContents')->willReturn(json_encode(['response' => $response], true));
-
-        $mockResponse = $this->getMockBuilder(
-            '\GuzzleHttp\Psr7\Response'
-        )->disableOriginalConstructor()->setMethods(['getBody'])->getMock();
-        $mockResponse->method('getBody')->willReturn($mockStream);
-
-        return $mockResponse;
+        return new Response(200, [], Utils::streamFor(json_encode(['response' => $response])));
     }
 }