Merge pull request #2 from ddrv-fork/master

Added github actions CI configuration

Author: ddrv

.github/workflows/ci.yml

@@ -0,0 +1,33 @@
+name: CI
+
+on: [push, pull_request]
+
+jobs:
+  tests:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        php: [7.0, 7.1, 7.2, 7.3, 7.4, 8.0]
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          coverage: none
+
+      - name: Validate composer.json and composer.lock
+        run: composer validate
+
+      - name: Install dependencies
+        run: composer install --prefer-dist --no-progress --no-interaction --no-suggest
+
+      - name: Check code style
+        run: vendor/bin/phpcs
+
+      - name: Run test suite
+        run: vendor/bin/phpunit

composer.json

@@ -19,8 +19,9 @@
     "psr/http-server-handler": "^1.0"
   },
   "require-dev": {
-    "nyholm/psr7": "^1.3",
-    "phpunit/phpunit": "^6.5",
+    "psr/http-factory": "^1.0",
+    "guzzlehttp/psr7": "^1.7",
+    "phpunit/phpunit": ">=6.5",
     "squizlabs/php_codesniffer": "^3.5"
   },
   "provide": {

phpunit.xml.dist

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit bootstrap="vendor/autoload.php" colors="true">
+<phpunit bootstrap="vendor/autoload.php" colors="true" cacheResult="false">
     <testsuites>
         <testsuite name="all">
             <directory>tests</directory>

stuff/Factory/HttpFactory.php

@@ -0,0 +1,46 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Webclient\Stuff\Fake\Factory;
+
+use GuzzleHttp\Psr7\Request;
+use GuzzleHttp\Psr7\Response;
+use GuzzleHttp\Psr7\ServerRequest;
+use Psr\Http\Message\RequestFactoryInterface;
+use Psr\Http\Message\RequestInterface;
+use Psr\Http\Message\ResponseFactoryInterface;
+use Psr\Http\Message\ResponseInterface;
+use Psr\Http\Message\ServerRequestFactoryInterface;
+use Psr\Http\Message\ServerRequestInterface;
+use Psr\Http\Message\UriInterface;
+
+class HttpFactory implements RequestFactoryInterface, ServerRequestFactoryInterface, ResponseFactoryInterface
+{
+
+    /**
+     * @param string $method
+     * @param UriInterface|string $uri
+     * @return RequestInterface
+     */
+    public function createRequest(string $method, $uri): RequestInterface
+    {
+        return new Request($method, $uri);
+    }
+
+    /**
+     * @param string $method
+     * @param UriInterface|string $uri
+     * @param array $serverParams
+     * @return ServerRequestInterface
+     */
+    public function createServerRequest(string $method, $uri, array $serverParams = []): ServerRequestInterface
+    {
+        return new ServerRequest($method, $uri, [], null, '1.1', $serverParams);
+    }
+
+    public function createResponse(int $code = 200, string $reasonPhrase = ''): ResponseInterface
+    {
+        return new Response($code, [], null, '1.1', $reasonPhrase);
+    }
+}

tests/ClientTest.php

@@ -5,9 +5,9 @@
 namespace Webclient\Tests\Fake;
 
 use Webclient\Fake\Client;
+use Webclient\Stuff\Fake\Factory\HttpFactory;
 use Webclient\Stuff\Fake\Handler\ErrorHandler;
 use Webclient\Stuff\Fake\Handler\UniversalHandler;
-use Nyholm\Psr7\Factory\Psr17Factory;
 use PHPUnit\Framework\TestCase;
 use Psr\Http\Client\ClientExceptionInterface;
 use Psr\Http\Client\NetworkExceptionInterface;
@@ -16,21 +16,16 @@ class ClientTest extends TestCase
 {
 
     /**
-     * @var Psr17Factory
+     * @var HttpFactory
      */
     private $factory;
 
-    public function setUp()
-    {
-        parent::setUp();
-        $this->factory = new Psr17Factory();
-    }
-
     /**
      * @throws ClientExceptionInterface
      */
     public function testSuccessWithRequest()
     {
+        $this->init();
         $request = $this->factory->createRequest('GET', 'http://phpunit.de/?return=302&redirect=https://phpunit.de');
         $client = new Client(new UniversalHandler($this->factory));
         $response = $client->sendRequest($request);
@@ -44,6 +39,7 @@ public function testSuccessWithRequest()
      */
     public function testSuccessWithServerRequest()
     {
+        $this->init();
         $request = $this->factory->createServerRequest(
             'GET',
             'https://phpunit.de',
@@ -64,6 +60,7 @@ public function testSuccessWithServerRequest()
      */
     public function testSuccessWithPreparedServerRequest()
     {
+        $this->init();
         $request = $this->factory->createServerRequest(
             'GET',
             'https://phpunit.de',
@@ -83,9 +80,15 @@ public function testSuccessWithPreparedServerRequest()
      */
     public function testFailWithNetworkError()
     {
+        $this->init();
         $request = $this->factory->createRequest('GET', '/');
         $client = new Client(new ErrorHandler());
         $this->expectException(NetworkExceptionInterface::class);
         $client->sendRequest($request);
     }
+
+    private function init()
+    {
+        $this->factory = new HttpFactory();
+    }
 }