Drop support for php5 (#16)

Author: gmponos

.travis.yml

@@ -1,9 +1,6 @@
 language: php
 
 php:
-    - 5.5
-    - 5.6
-    - 7.1
     - 7.2
 
 before_script:

composer.json

@@ -20,13 +20,13 @@
         }
     },
     "require": {
-        "php": ">=5.5",
+        "php": ">=7.2",
         "ext-json": "*",
         "guzzlehttp/guzzle": "6.*",
         "psr/log": "~1.0"
     },
     "require-dev": {
-        "phpunit/phpunit": "4.*|5.*",
+        "phpunit/phpunit": "7.*",
         "webthink/codesniffer": "1.*"
     },
     "scripts": {

src/Handler/ArrayHandler.php

@@ -35,7 +35,7 @@ public function __construct(LogLevelStrategyInterface $logLevelStrategy = null)
      * @param array $options
      * @return void
      */
-    public function log(LoggerInterface $logger, $value, array $options = [])
+    public function log(LoggerInterface $logger, $value, array $options = []): void
     {
         if ($value instanceof ResponseInterface) {
             $context['response']['headers'] = $value->getHeaders();

src/Handler/HandlerInterface.php

@@ -18,5 +18,5 @@ interface HandlerInterface
      * @param array $options
      * @return void
      */
-    public function log(LoggerInterface $logger, $value, array $options = []);
+    public function log(LoggerInterface $logger, $value, array $options = []): void;
 }

src/Handler/LogLevel/LogLevelStrategy.php

@@ -34,7 +34,7 @@ class LogLevelStrategy implements LogLevelStrategyInterface
      * @param array $options
      * @return string LogLevel
      */
-    public function getLevel($value, array $options = [])
+    public function getLevel($value, array $options = []): string
     {
         $this->setOptions($options);
         if ($value instanceof \Exception) {
@@ -60,7 +60,7 @@ public function getLevel($value, array $options = [])
      * @param array $options
      * @return void
      */
-    private function setOptions(array $options)
+    private function setOptions(array $options): void
     {
         if (!isset($options['log'])) {
             return;
@@ -82,7 +82,7 @@ private function setOptions(array $options)
      * @param ResponseInterface $response
      * @return string
      */
-    private function getResponseLevel(ResponseInterface $response)
+    private function getResponseLevel(ResponseInterface $response): string
     {
         $code = $response->getStatusCode();
         if ($code === 0) {

src/Handler/StringHandler.php

@@ -35,7 +35,7 @@ public function __construct(LogLevelStrategyInterface $logLevelStrategy = null)
      * @param array $options
      * @return void
      */
-    public function log(LoggerInterface $logger, $value, array $options = [])
+    public function log(LoggerInterface $logger, $value, array $options = []): void
     {
         $level = $this->logLevelStrategy->getLevel($value, $options);
         if ($value instanceof MessageInterface) {

src/Middleware/LoggerMiddleware.php

@@ -51,8 +51,8 @@ class LoggerMiddleware
     public function __construct(
         LoggerInterface $logger,
         HandlerInterface $handler = null,
-        $onExceptionOnly = false,
-        $logStatistics = false
+        bool $onExceptionOnly = false,
+        bool $logStatistics = false
     ) {
         $this->logger = $logger;
         $this->onExceptionOnly = $onExceptionOnly;
@@ -64,9 +64,9 @@ public function __construct(
      * Called when the middleware is handled by the client.
      *
      * @param callable $handler
-     * @return Closure
+     * @return callable
      */
-    public function __invoke(callable $handler)
+    public function __invoke(callable $handler): callable
     {
         return function (RequestInterface $request, array $options) use ($handler) {
             $this->setOptions($options);
@@ -94,7 +94,7 @@ public function __invoke(callable $handler)
      * @param array $options
      * @return Closure
      */
-    private function handleSuccess(RequestInterface $request, array $options)
+    private function handleSuccess(RequestInterface $request, array $options): callable
     {
         return function (ResponseInterface $response) use ($request, $options) {
             // On exception only is true then it must not log the response since it was successful.
@@ -113,7 +113,7 @@ private function handleSuccess(RequestInterface $request, array $options)
      * @param array $options
      * @return Closure
      */
-    private function handleFailure(RequestInterface $request, array $options)
+    private function handleFailure(RequestInterface $request, array $options): callable
     {
         return function (\Exception $reason) use ($request, $options) {
             if ($this->onExceptionOnly === true) {
@@ -135,7 +135,7 @@ private function handleFailure(RequestInterface $request, array $options)
      * @param array $options
      * @return void
      */
-    private function setOptions(array $options)
+    private function setOptions(array $options): void
     {
         if (!isset($options['log'])) {
             return;

tests/Unit/Handler/StringHandlerTest.php

@@ -8,9 +8,10 @@
 use GuzzleHttp\Psr7\Request;
 use GuzzleHttp\Psr7\Response;
 use GuzzleHttp\TransferStats;
+use PHPUnit\Framework\TestCase;
 use Psr\Log\LogLevel;
 
-final class StringHandlerTest extends \PHPUnit_Framework_TestCase
+final class StringHandlerTest extends TestCase
 {
     /**
      * @var HandlerInterface

tests/Unit/Middleware/LoggerMiddlewareTest.php

@@ -12,9 +12,10 @@
 use GuzzleHttp\HandlerStack;
 use GuzzleHttp\Psr7\Response;
 use GuzzleHttp\RequestOptions;
+use PHPUnit\Framework\TestCase;
 use Psr\Log\LogLevel;
 
-final class LoggerMiddlewareTest extends \PHPUnit_Framework_TestCase
+final class LoggerMiddlewareTest extends TestCase
 {
     /**
      * @var MockHandler