Support both client-common ^1.9 and 2.x (#52)

* Support both client-common ^1.9 and 2.x

* Use Traits

* Make sure we test with both 2.0 and 1.9 of client-common

* Drop tests on hhvm

* No need to have special test for client-common:1.9

That is already tested in prefer-lowest

* Run test on php7.1

Author: Nyholm

.travis.yml

@@ -12,6 +12,8 @@ php:
     - 5.6
     - 7.0
     - 7.1
+    - 7.2
+    - 7.3
 
 env:
     global:
@@ -26,16 +28,16 @@ matrix:
     include:
         - php: 5.4
           env: COMPOSER_FLAGS="--prefer-stable --prefer-lowest" COVERAGE=true TEST_COMMAND="composer test-ci"
-        - php: hhvm
-          dist: trusty
+        - php: 7.1
+          env: COMPOSER_FLAGS="--prefer-stable --prefer-lowest"
 
 before_install:
     - if [[ $COVERAGE != true ]]; then phpenv config-rm xdebug.ini || true; fi
 
 install:
     # To be removed when this issue will be resolved: https://github.com/composer/composer/issues/5355
     - if [[ "$COMPOSER_FLAGS" == *"--prefer-lowest"* ]]; then travis_retry composer update --prefer-dist --no-interaction --prefer-stable --quiet; fi
-    - travis_retry composer update ${COMPOSER_FLAGS} --prefer-dist --no-interaction
+    - composer update ${COMPOSER_FLAGS} --prefer-dist --no-interaction
 
 script:
     - $TEST_COMMAND

composer.json

@@ -13,7 +13,7 @@
     "require": {
         "php": "^5.4 || ^7.0",
         "psr/cache": "^1.0",
-        "php-http/client-common": "^1.1",
+        "php-http/client-common": "^1.9 || ^2.0",
         "php-http/message-factory": "^1.0",
         "symfony/options-resolver": "^2.6 || ^3.0 || ^4.0"
     },
@@ -39,5 +39,7 @@
         "branch-alias": {
             "dev-master": "1.6-dev"
         }
-    }
+    },
+    "prefer-stable": true,
+    "minimum-stability": "dev"
 }

src/CachePlugin.php

@@ -23,6 +23,8 @@
  */
 final class CachePlugin implements Plugin
 {
+    use VersionBridgePlugin;
+
     /**
      * @var CacheItemPoolInterface
      */
@@ -124,10 +126,7 @@ public static function serverCache(CacheItemPoolInterface $pool, StreamFactory $
         return new self($pool, $streamFactory, $config);
     }
 
-    /**
-     * {@inheritdoc}
-     */
-    public function handleRequest(RequestInterface $request, callable $next, callable $first)
+    protected function doHandleRequest(RequestInterface $request, callable $next, callable $first)
     {
         $method = strtoupper($request->getMethod());
         // if the request not is cachable, move to $next