update deps

.github/workflows/tests.yml

@@ -14,7 +14,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        php-version: ['7.2', '7.3', '7.4', '8.0']
+        php-version: ['7.2', '7.3', '7.4', '8.0', '8.1', '8.2']
     steps:
       - name: Check out code into the workspace
         uses: actions/checkout@v2

README.md

@@ -1,4 +1,4 @@
-![Build Status](https://img.shields.io/github/workflow/status/Neur0toxine/pock/Tests?style=flat-square)
+![Build Status](https://img.shields.io/github/actions/workflow/status/Neur0toxine/pock/tests.yml?branch=master&style=flat-square)
 [![Coverage](https://img.shields.io/codecov/c/gh/Neur0toxine/pock/master.svg?logo=codecov&logoColor=white&style=flat-square)](https://codecov.io/gh/Neur0toxine/pock)
 [![Latest stable](https://img.shields.io/packagist/v/neur0toxine/pock.svg?style=flat-square)](https://packagist.org/packages/neur0toxine/pock)
 [![PHP from Packagist](https://img.shields.io/packagist/php-v/neur0toxine/pock.svg?logo=php&logoColor=white&style=flat-square)](https://packagist.org/packages/neur0toxine/pock)
@@ -97,7 +97,7 @@ if you want to override default behavior.
 ```php
 use Pock\Factory\JsonSerializerFactory;
 use Pock\Factory\XmlSerializerFactory;
-use Pock\Serializer\SymfonySerializerDecorator;
+use Pock\Serializer\SymfonySerializerAdapter;
 use Symfony\Component\Serializer\Encoder\JsonEncoder;
 use Symfony\Component\Serializer\Encoder\XmlEncoder;
 use Symfony\Component\Serializer\Normalizer\ObjectNormalizer;
@@ -105,7 +105,7 @@ use Symfony\Component\Serializer\Serializer;
 
 $encoders = [new XmlEncoder(), new JsonEncoder()];
 $normalizers = [new ObjectNormalizer()];
-$serializer = new SymfonySerializerDecorator(new Serializer($normalizers, $encoders));
+$serializer = new SymfonySerializerAdapter(new Serializer($normalizers, $encoders));
 
 JsonSerializerFactory::setSerializer($serializer);
 XmlSerializerFactory::setSerializer($serializer);
@@ -125,6 +125,7 @@ In order to use unsupported serializer you should create an adapter which implem
 - [x] Regexp matchers for body, query, URI and path.  
 - [x] Form Data body matcher (partial & exact)
 - [x] Multipart form body matcher (just like callback matcher but parses the body as a multipart form data)  
-- [ ] `symfony/http-client` support.  
-- [ ] Real network response for mocked & unmatched requests.  
+- [x] **BREAKING CHANGE:** Rename serializer decorators to serializer adapters.  
+- [x] Real network response for mocked requests.
+- [ ] `symfony/http-client` support.
 - [ ] Document everything (with examples if it’s feasible).

composer.json

@@ -34,22 +34,23 @@
         "php": ">=7.2.0",
         "ext-json": "*",
         "psr/http-client": "^1.0",
-        "psr/http-message": "^1.0",
+        "psr/http-message": "^1.0 || ^2.0",
         "php-http/httplug": "^1.0 || ^2.0",
         "nyholm/psr7": "^1.4",
         "riverline/multipart-parser": "^2.0"
     },
     "require-dev": {
         "squizlabs/php_codesniffer": "^3.6",
-        "phpmd/phpmd": "^2.10",
+        "phpmd/phpmd": "^2.12",
         "dealerdirect/phpcodesniffer-composer-installer": "^0.7.1",
         "phpcompatibility/php-compatibility": "^9.3",
-        "phpstan/phpstan": "^0.12.87",
-        "jms/serializer": "^2 | ^3.12",
+        "phpstan/phpstan": "^1.5",
+        "jms/serializer": "^2 | ^3.17",
         "symfony/phpunit-bridge": "^5.2",
         "symfony/serializer": "^5.2",
         "symfony/property-access": "^5.2",
-        "php-http/multipart-stream-builder": "^1.2"
+        "php-http/multipart-stream-builder": "^1.2",
+        "symfony/http-client": "^5.3"
     },
     "provide": {
         "psr/http-client-implementation": "1.0",
@@ -72,5 +73,11 @@
             "@lint",
             "@phpunit"
         ]
+    },
+    "config": {
+        "allow-plugins": {
+            "dealerdirect/phpcodesniffer-composer-installer": true,
+            "php-http/discovery": true
+        }
     }
 }

phpstan-baseline.neon

@@ -0,0 +1,111 @@
+parameters:
+	ignoreErrors:
+		-
+			message: "#^Method Pock\\\\Client\\:\\:doSendRequest\\(\\) should return Psr\\\\Http\\\\Message\\\\ResponseInterface but returns mixed\\.$#"
+			count: 1
+			path: src/Client.php
+
+		-
+			message: "#^Static property Pock\\\\Comparator\\\\ComparatorLocator\\:\\:\\$comparators \\(array\\<Pock\\\\Comparator\\\\ComparatorInterface\\>\\) does not accept array\\<object\\>\\.$#"
+			count: 1
+			path: src/Comparator/ComparatorLocator.php
+
+		-
+			message: "#^Unsafe access to private property Pock\\\\Comparator\\\\ComparatorLocator\\:\\:\\$comparators through static\\:\\:\\.$#"
+			count: 3
+			path: src/Comparator/ComparatorLocator.php
+
+		-
+			message: "#^Unsafe access to private property Pock\\\\Factory\\\\JsonSerializerFactory\\:\\:\\$mainSerializer through static\\:\\:\\.$#"
+			count: 2
+			path: src/Factory/JsonSerializerFactory.php
+
+		-
+			message: "#^Unsafe access to private property Pock\\\\Factory\\\\XmlSerializerFactory\\:\\:\\$mainSerializer through static\\:\\:\\.$#"
+			count: 2
+			path: src/Factory/XmlSerializerFactory.php
+
+		-
+			message: "#^Parameter \\#4 \\$flags of function preg_match expects TFlags of 0\\|256\\|512\\|768, int given\\.$#"
+			count: 1
+			path: src/Matchers/AbstractRegExpMatcher.php
+
+		-
+			message: "#^Unsafe call to private method Pock\\\\Matchers\\\\ExactHeadersMatcher\\:\\:headerValuesEqual\\(\\) through static\\:\\:\\.$#"
+			count: 2
+			path: src/Matchers/ExactHeadersMatcher.php
+
+		-
+			message: "#^Method Pock\\\\Matchers\\\\JsonBodyMatcher\\:\\:deserialize\\(\\) should return array\\|null but returns mixed\\.$#"
+			count: 1
+			path: src/Matchers/JsonBodyMatcher.php
+
+		-
+			message: "#^Parameter \\#3 \\$depth of function json_decode expects int\\<1, max\\>, int given\\.$#"
+			count: 1
+			path: src/Matchers/JsonBodyMatcher.php
+
+		-
+			message: "#^Method Pock\\\\Matchers\\\\QueryMatcher\\:\\:parseQuery\\(\\) should return array\\<string, mixed\\> but returns array\\<int\\|string, array\\|string\\>\\.$#"
+			count: 1
+			path: src/Matchers/QueryMatcher.php
+
+		-
+			message: "#^Method Pock\\\\Matchers\\\\XmlBodyMatcher\\:\\:sortXmlTags\\(\\) should return string but returns string\\|null\\.$#"
+			count: 1
+			path: src/Matchers/XmlBodyMatcher.php
+
+		-
+			message: "#^Unsafe access to private constant Pock\\\\Matchers\\\\XmlBodyMatcher\\:\\:TAG_SORT_XSLT through static\\:\\:\\.$#"
+			count: 1
+			path: src/Matchers/XmlBodyMatcher.php
+
+		-
+			message: "#^Unsafe access to private property Pock\\\\Matchers\\\\XmlBodyMatcher\\:\\:\\$sorter through static\\:\\:\\.$#"
+			count: 4
+			path: src/Matchers/XmlBodyMatcher.php
+
+		-
+			message: "#^Unsafe call to private method Pock\\\\Matchers\\\\XmlBodyMatcher\\:\\:createDOMDocument\\(\\) through static\\:\\:\\.$#"
+			count: 2
+			path: src/Matchers/XmlBodyMatcher.php
+
+		-
+			message: "#^Unsafe call to private method Pock\\\\Matchers\\\\XmlBodyMatcher\\:\\:getDOMString\\(\\) through static\\:\\:\\.$#"
+			count: 1
+			path: src/Matchers/XmlBodyMatcher.php
+
+		-
+			message: "#^Unsafe call to private method Pock\\\\Matchers\\\\XmlBodyMatcher\\:\\:getSorter\\(\\) through static\\:\\:\\.$#"
+			count: 1
+			path: src/Matchers/XmlBodyMatcher.php
+
+		-
+			message: "#^Unsafe call to private method Pock\\\\Matchers\\\\XmlBodyMatcher\\:\\:hasExtension\\(\\) through static\\:\\:\\.$#"
+			count: 4
+			path: src/Matchers/XmlBodyMatcher.php
+
+		-
+			message: "#^Unsafe call to private method Pock\\\\Matchers\\\\XmlBodyMatcher\\:\\:sortXmlTags\\(\\) through static\\:\\:\\.$#"
+			count: 2
+			path: src/Matchers/XmlBodyMatcher.php
+
+		-
+			message: "#^Parameter \\#1 \\$data of class Pock\\\\Matchers\\\\JsonBodyMatcher constructor expects array, mixed given\\.$#"
+			count: 2
+			path: src/PockBuilder.php
+
+		-
+			message: "#^Parameter \\#3 \\$depth of function json_decode expects int\\<1, max\\>, int given\\.$#"
+			count: 1
+			path: src/PockBuilder.php
+
+		-
+			message: "#^Parameter \\#3 \\$depth of function json_encode expects int\\<1, max\\>, int given\\.$#"
+			count: 1
+			path: src/PockBuilder.php
+
+		-
+			message: "#^Parameter \\#3 \\$depth of function json_encode expects int\\<1, max\\>, int given\\.$#"
+			count: 1
+			path: src/PockResponseBuilder.php

phpstan.neon

@@ -1,3 +1,6 @@
+includes:
+	- phpstan-baseline.neon # TODO: Configure new phpstan
+
 parameters:
     level: max
     paths:

src/Comparator/RecursiveArrayComparator.php

@@ -50,8 +50,12 @@ class RecursiveArrayComparator implements ComparatorInterface
         }
 
         foreach ($first as $key => $value) {
-            if (is_array($value) && !self::recursiveCompareArrays($value, $second[$key])) {
-                return false;
+            if (is_array($value)) {
+                if (!is_array($second[$key]) || !self::recursiveCompareArrays($value, $second[$key])) {
+                    return false;
+                }
+
+                continue;
             }
 
             if ($value !== $second[$key]) {

src/Comparator/RecursiveLtrArrayComparator.php

@@ -47,11 +47,12 @@ class RecursiveLtrArrayComparator extends RecursiveArrayComparator
         }
 
         foreach ($needle as $key => $value) {
-            if (
-                is_array($value) &&
-                (!is_array($haystack[$key]) || !self::recursiveCompareArrays($value, $haystack[$key]))
-            ) {
-                return false;
+            if (is_array($value)) {
+                if (!is_array($haystack[$key]) || !self::recursiveCompareArrays($value, $haystack[$key])) {
+                    return false;
+                }
+
+                continue;
             }
 
             if ($value !== $haystack[$key]) {

src/Creator/AbstractJmsSerializerCreator.php

@@ -10,7 +10,7 @@
 namespace Pock\Creator;
 
 use Throwable;
-use Pock\Serializer\JmsSerializerDecorator;
+use Pock\Serializer\JmsSerializerAdapter;
 use Pock\Serializer\SerializerInterface;
 
 /**
@@ -30,13 +30,13 @@ abstract class AbstractJmsSerializerCreator implements SerializerCreatorInterfac
     {
         if (
             class_exists(self::BUILDER_CLASS) &&
-            method_exists(self::BUILDER_CLASS, 'create')
+            method_exists(self::BUILDER_CLASS, 'create') // @phpstan-ignore-line
         ) {
             try {
-                $builder = call_user_func([self::BUILDER_CLASS, 'create']);
+                $builder = call_user_func([self::BUILDER_CLASS, 'create']); // @phpstan-ignore-line
 
-                if (null !== $builder && method_exists($builder, 'build')) {
-                    return new JmsSerializerDecorator($builder->build(), static::getFormat()); // @phpstan-ignore-line
+                if (null !== $builder && method_exists($builder, 'build')) { // @phpstan-ignore-line
+                    return new JmsSerializerAdapter($builder->build(), static::getFormat()); // @phpstan-ignore-line
                 }
             } catch (Throwable $throwable) {
                 return null;

src/Creator/AbstractSymfonySerializerCreator.php

@@ -10,7 +10,7 @@
 namespace Pock\Creator;
 
 use Pock\Serializer\SerializerInterface;
-use Pock\Serializer\SymfonySerializerDecorator;
+use Pock\Serializer\SymfonySerializerAdapter;
 
 /**
  * Class AbstractSymfonySerializerCreator
@@ -33,8 +33,8 @@ abstract class AbstractSymfonySerializerCreator implements SerializerCreatorInte
             $normalizer = self::OBJECT_NORMALIZER_CLASS;
             $encoder = static::getEncoderClass();
 
-            return new SymfonySerializerDecorator(
-                new $serializer([new $normalizer()], [new $encoder()]),
+            return new SymfonySerializerAdapter(
+                new $serializer([new $normalizer()], [new $encoder()]), // @phpstan-ignore-line
                 static::getFormat()
             );
         }

src/Matchers/BodyMatcher.php

@@ -79,5 +79,7 @@ class BodyMatcher implements RequestMatcherInterface
         if (is_resource($contents)) {
             return static::readAllResource($contents);
         }
+
+        return '';
     }
 }

src/Matchers/FormDataMatcher.php

@@ -10,7 +10,7 @@
 namespace Pock\Matchers;
 
 use Pock\Comparator\ComparatorLocator;
-use Pock\Comparator\RecursiveLtrArrayComparator;
+use Pock\Comparator\RecursiveArrayComparator;
 use Pock\Traits\SeekableStreamDataExtractor;
 use Psr\Http\Message\RequestInterface;
 
@@ -35,6 +35,6 @@ class FormDataMatcher extends QueryMatcher
             return false;
         }
 
-        return ComparatorLocator::get(RecursiveLtrArrayComparator::class)->compare($this->query, $query);
+        return ComparatorLocator::get(RecursiveArrayComparator::class)->compare($this->query, $query);
     }
 }

src/Matchers/PortMatcher.php

@@ -0,0 +1,56 @@
+<?php
+
+/**
+ * PHP version 7.3
+ *
+ * @category PortMatcher
+ * @package  Pock\Matchers
+ */
+
+namespace Pock\Matchers;
+
+use Pock\Enum\RequestScheme;
+use Psr\Http\Message\RequestInterface;
+
+/**
+ * Class PortMatcher
+ *
+ * @category PortMatcher
+ * @package  Pock\Matchers
+ */
+class PortMatcher implements RequestMatcherInterface
+{
+    /** @var int */
+    protected $port;
+
+    /**
+     * PortMatcher constructor.
+     *
+     * @param int $port
+     */
+    public function __construct(int $port)
+    {
+        $this->port = $port;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function matches(RequestInterface $request): bool
+    {
+        $port = $request->getUri()->getPort();
+
+        if (null === $port) {
+            switch ($request->getUri()->getScheme()) {
+                case RequestScheme::HTTP:
+                    return 80 === $this->port;
+                case RequestScheme::HTTPS:
+                    return 443 === $this->port;
+                default:
+                    return false;
+            }
+        }
+
+        return $port === $this->port;
+    }
+}

src/PockBuilder.php

@@ -33,6 +33,7 @@ use Pock\Matchers\MethodMatcher;
 use Pock\Matchers\MultipartFormDataMatcher;
 use Pock\Matchers\MultipleMatcher;
 use Pock\Matchers\PathMatcher;
+use Pock\Matchers\PortMatcher;
 use Pock\Matchers\QueryMatcher;
 use Pock\Matchers\RegExpBodyMatcher;
 use Pock\Matchers\RegExpPathMatcher;
@@ -46,7 +47,11 @@ use Pock\Traits\JsonDecoderTrait;
 use Pock\Traits\JsonSerializerAwareTrait;
 use Pock\Traits\XmlSerializerAwareTrait;
 use Psr\Http\Client\ClientInterface;
+use Psr\Http\Message\RequestInterface;
+use Psr\Http\Message\ResponseInterface;
 use RuntimeException;
+use Symfony\Component\HttpClient\Psr18Client;
+use Symfony\Contracts\HttpClient\HttpClientInterface;
 use Throwable;
 
 /**
@@ -58,6 +63,7 @@ use Throwable;
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  * @SuppressWarnings(PHPMD.TooManyPublicMethods)
  * @SuppressWarnings(PHPMD.TooManyMethods)
+ * @SuppressWarnings(PHPMD.ExcessivePublicCount)
  * @SuppressWarnings(PHPMD.ExcessiveClassComplexity)
  */
 class PockBuilder
@@ -134,6 +140,18 @@ class PockBuilder
         return $this->addMatcher(new HostMatcher($host));
     }
 
+    /**
+     * Matches request by the port.
+     *
+     * @param int $port
+     *
+     * @return self
+     */
+    public function matchPort(int $port): self
+    {
+        return $this->addMatcher(new PortMatcher($port));
+    }
+
     /**
      * Matches request by origin.
      *
@@ -158,6 +176,10 @@ class PockBuilder
             $this->matchHost($parsed['host']);
         }
 
+        if (array_key_exists('port', $parsed) && is_int($parsed['port']) && $parsed['port'] > 0) {
+            $this->matchPort($parsed['port']);
+        }
+
         return $this;
     }
 
@@ -645,6 +667,22 @@ class PockBuilder
         $this->replyWithFactory(new CallbackReplyFactory($callback));
     }
 
+    /**
+     * Reply to the request using provided client. Can be used to send real network request.
+     *
+     * @param \Psr\Http\Client\ClientInterface $client
+     * @SuppressWarnings(unused)
+     */
+    public function replyWithClient(ClientInterface $client): void
+    {
+        $this->replyWithCallback(function (
+            RequestInterface $request,
+            PockResponseBuilder $responseBuilder
+        ) use ($client): ResponseInterface {
+            return $client->sendRequest($request);
+        });
+    }
+
     /**
      * Resets the builder.
      *

src/Serializer/CallbackSerializerAdapter.php

@@ -3,7 +3,7 @@
 /**
  * PHP 7.1
  *
- * @category CallbackSerializerDecorator
+ * @category CallbackSerializerAdapter
  * @package  Pock\Serializer
  */
 
@@ -12,18 +12,18 @@ namespace Pock\Serializer;
 use RuntimeException;
 
 /**
- * Class CallbackSerializerDecorator
+ * Class CallbackSerializerAdapter
  *
- * @category CallbackSerializerDecorator
+ * @category CallbackSerializerAdapter
  * @package  Pock\Serializer
  */
-class CallbackSerializerDecorator implements SerializerInterface
+class CallbackSerializerAdapter implements SerializerInterface
 {
     /** @var callable */
     private $callback;
 
     /**
-     * CallbackSerializerDecorator constructor.
+     * CallbackSerializerAdapter constructor.
      *
      * @param callable $callback
      */

src/Serializer/JmsSerializerAdapter.php

@@ -3,19 +3,19 @@
 /**
  * PHP 7.2
  *
- * @category JmsSerializerDecorator
+ * @category JmsSerializerAdapter
  * @package  Pock\Serializer
  */
 
 namespace Pock\Serializer;
 
 /**
- * Class JmsSerializerDecorator
+ * Class JmsSerializerAdapter
  *
- * @category JmsSerializerDecorator
+ * @category JmsSerializerAdapter
  * @package  Pock\Serializer
  */
-class JmsSerializerDecorator implements SerializerInterface
+class JmsSerializerAdapter implements SerializerInterface
 {
     /** @var object */
     private $serializer;
@@ -24,7 +24,7 @@ class JmsSerializerDecorator implements SerializerInterface
     private $format;
 
     /**
-     * JmsSerializerDecorator constructor.
+     * JmsSerializerAdapter constructor.
      *
      * @param object $serializer
      */

src/Serializer/SymfonySerializerAdapter.php

@@ -0,0 +1,20 @@
+<?php
+
+/**
+ * PHP 7.1
+ *
+ * @category SymfonySerializerAdapter
+ * @package  Pock\Serializer
+ */
+
+namespace Pock\Serializer;
+
+/**
+ * Class SymfonySerializerAdapter
+ *
+ * @category SymfonySerializerAdapter
+ * @package  Pock\Serializer
+ */
+class SymfonySerializerAdapter extends JmsSerializerAdapter
+{
+}

src/Serializer/SymfonySerializerDecorator.php

@@ -1,20 +0,0 @@
-<?php
-
-/**
- * PHP 7.1
- *
- * @category SymfonySerializerDecorator
- * @package  Pock\Serializer
- */
-
-namespace Pock\Serializer;
-
-/**
- * Class SymfonySerializerDecorator
- *
- * @category SymfonySerializerDecorator
- * @package  Pock\Serializer
- */
-class SymfonySerializerDecorator extends JmsSerializerDecorator
-{
-}

tests/src/Comparator/RecursiveArrayComparatorTest.php

@@ -0,0 +1,75 @@
+<?php
+
+/**
+ * PHP version 7.3
+ *
+ * @category RecursiveArrayComparatorTest
+ * @package  Pock\Tests\Comparator
+ */
+
+namespace Pock\Tests\Comparator;
+
+use PHPUnit\Framework\TestCase;
+use Pock\Comparator\ComparatorLocator;
+use Pock\Comparator\RecursiveArrayComparator;
+
+/**
+ * Class RecursiveArrayComparatorTest
+ *
+ * @category RecursiveArrayComparatorTest
+ * @package  Pock\Tests\Comparator
+ */
+class RecursiveArrayComparatorTest extends TestCase
+{
+    public function testMatches(): void
+    {
+        $needle = [
+            'filter' => [
+                'createdAtFrom' => '2020-01-01 00:00:00',
+                'createdAtTo' => '2021-08-01 00:00:00',
+            ],
+            'test' => ''
+        ];
+        $haystack = [
+            'filter' => [
+                'createdAtFrom' => '2020-01-01 00:00:00',
+                'createdAtTo' => '2021-08-01 00:00:00',
+            ],
+            'test' => ''
+        ];
+
+        self::assertTrue(ComparatorLocator::get(RecursiveArrayComparator::class)->compare($needle, $haystack));
+    }
+
+    public function testNotMatches(): void
+    {
+        $needle = [
+            'filter' => [
+                'createdAtFrom' => '2020-01-01 00:00:00',
+                'createdAtTo' => '2021-08-01 00:00:00',
+            ],
+            'test2' => [1]
+        ];
+        $haystack = [
+            'filter' => [
+                'createdAtFrom' => '2020-01-01 00:00:00',
+                'createdAtTo' => '2021-08-01 00:00:00',
+            ],
+            'test2' => 1
+        ];
+
+        self::assertFalse(ComparatorLocator::get(RecursiveArrayComparator::class)->compare($needle, $haystack));
+    }
+
+    public function testNotMatchesKeyPositions(): void
+    {
+        $needle = [
+            'source' => json_decode('{"medium":"tiktok","source":"Test Ad","campaign":"Test Campaign"}', true)
+        ];
+        $haystack = [
+            'source' => json_decode('{"source":"Test Ad","medium":"tiktok","campaign":"Test Campaign"}', true)
+        ];
+
+        self::assertTrue(ComparatorLocator::get(RecursiveArrayComparator::class)->compare($needle, $haystack));
+    }
+}

tests/src/Comparator/RecursiveArrayLtrComparatorTest.php

@@ -39,4 +39,37 @@ class RecursiveArrayLtrComparatorTest extends TestCase
 
         self::assertTrue(ComparatorLocator::get(RecursiveLtrArrayComparator::class)->compare($needle, $haystack));
     }
+
+    public function testNotMatches(): void
+    {
+        $needle = [
+            'filter' => [
+                'createdAtFrom' => '2020-01-01 00:00:00',
+                'createdAtTo' => '2021-08-01 00:00:00',
+            ],
+            'test2' => [1]
+        ];
+        $haystack = [
+            'filter' => [
+                'createdAtFrom' => '2020-01-01 00:00:00',
+                'createdAtTo' => '2021-08-01 00:00:00',
+            ],
+            'test2' => 1,
+            'test' => ''
+        ];
+
+        self::assertFalse(ComparatorLocator::get(RecursiveLtrArrayComparator::class)->compare($needle, $haystack));
+    }
+
+    public function testNotMatchesKeyPositions(): void
+    {
+        $needle = [
+            'source' => json_decode('{"medium":"tiktok","source":"Test Ad","campaign":"Test Campaign"}', true)
+        ];
+        $haystack = [
+            'source' => json_decode('{"source":"Test Ad","medium":"tiktok","campaign":"Test Campaign"}', true)
+        ];
+
+        self::assertTrue(ComparatorLocator::get(RecursiveLtrArrayComparator::class)->compare($needle, $haystack));
+    }
 }

tests/src/Decorator/CallbackSerializerDecoratorTest.php

@@ -1,29 +0,0 @@
-<?php
-
-/**
- * PHP 7.1
- *
- * @category CallbackSerializerDecoratorTest
- * @package  Pock\Tests\Decorator
- */
-
-namespace Pock\Tests\Decorator;
-
-use PHPUnit\Framework\TestCase;
-use Pock\Serializer\CallbackSerializerDecorator;
-
-/**
- * Class CallbackSerializerDecoratorTest
- *
- * @category CallbackSerializerDecoratorTest
- * @package  Pock\Tests\Decorator
- */
-class CallbackSerializerDecoratorTest extends TestCase
-{
-    public function testSerialize(): void
-    {
-        self::assertEquals('{}', (new CallbackSerializerDecorator(function ($data) {
-            return $data;
-        }))->serialize('{}'));
-    }
-}

tests/src/Factory/AbstractSerializerFactoryTest.php

@@ -12,7 +12,7 @@ namespace Pock\Tests\Factory;
 use PHPUnit\Framework\TestCase;
 use Pock\Factory\JsonSerializerFactory;
 use Pock\Factory\XmlSerializerFactory;
-use Pock\Serializer\CallbackSerializerDecorator;
+use Pock\Serializer\CallbackSerializerAdapter;
 use Pock\Serializer\SerializerInterface;
 use Pock\TestUtils\EmptyJsonSerializerDecorator;
 use Pock\TestUtils\EmptyXmlSerializerDecorator;

tests/src/Matchers/FormDataMatcherTest.php

@@ -44,11 +44,19 @@ class FormDataMatcherTest extends PockTestCase
         self::assertFalse($matcher->matches($request));
     }
 
-    public function testMatches(): void
+    public function testNoMatchesExtraValues(): void
     {
         $matcher = new FormDataMatcher(['field2' => 'value2']);
         $request = self::getRequestWithBody('field1=value1&field2=value2');
 
+        self::assertFalse($matcher->matches($request));
+    }
+
+    public function testMatches(): void
+    {
+        $matcher = new FormDataMatcher(['field1' => 'value1', 'field2' => 'value2']);
+        $request = self::getRequestWithBody('field1=value1&field2=value2');
+
         self::assertTrue($matcher->matches($request));
     }
 }

tests/src/Matchers/PortMatcherTest.php

@@ -0,0 +1,60 @@
+<?php
+
+/**
+ * PHP version 7.3
+ *
+ * @category PortMatcherTest
+ * @package  Pock\Tests\Matchers
+ */
+
+namespace Pock\Tests\Matchers;
+
+use Nyholm\Psr7\Uri;
+use Pock\Enum\RequestScheme;
+use Pock\Matchers\PortMatcher;
+use Pock\TestUtils\PockTestCase;
+
+/**
+ * Class PortMatcherTest
+ *
+ * @category PortMatcherTest
+ * @package  Pock\Tests\Matchers
+ */
+class PortMatcherTest extends PockTestCase
+{
+    public function testNotMatches(): void
+    {
+        self::assertFalse((new PortMatcher(80))->matches(static::getTestRequest()));
+    }
+
+    public function testMatches(): void
+    {
+        self::assertTrue((new PortMatcher(443))->matches(static::getTestRequest()));
+    }
+
+    public function testMatchesWithoutProto(): void
+    {
+        self::assertTrue((new PortMatcher(80))->matches(static::getTestRequest()->withUri(new class extends Uri {
+            public function getScheme(): string
+            {
+                return RequestScheme::HTTP;
+            }
+
+            public function getPort(): ?int
+            {
+                return null;
+            }
+        })));
+        self::assertTrue((new PortMatcher(443))->matches(static::getTestRequest()->withUri(new class extends Uri {
+            public function getScheme(): string
+            {
+                return RequestScheme::HTTPS;
+            }
+
+            public function getPort(): ?int
+            {
+                return null;
+            }
+        })));
+    }
+}

tests/src/PockBuilderTest.php

@@ -174,7 +174,7 @@ class PockBuilderTest extends PockTestCase
 
     public function testMatchOrigin(): void
     {
-        $origin = RequestScheme::HTTPS . '://' . self::TEST_HOST;
+        $origin = RequestScheme::HTTPS . '://' . self::TEST_HOST . ':443';
         $builder = new PockBuilder();
 
         $builder->matchMethod(RequestMethod::GET)
@@ -681,7 +681,7 @@ EOF;
         $builder = new PockBuilder();
         $builder->matchMethod(RequestMethod::GET)
             ->matchUri(self::TEST_URI)
-            ->matchFormData(['field2' => 'value2'])
+            ->matchFormData(['field1' => 'value1', 'field2' => 'value2'])
             ->reply(200);
 
         $response = $builder->getClient()->sendRequest(self::getRequestWithBody('field1=value1&field2=value2'));
@@ -1035,10 +1035,29 @@ EOF;
                 throw new RuntimeException('Exception from the callback');
             });
 
-         $builder->getClient()->sendRequest(self::getPsr17Factory()->createRequest(
-             RequestMethod::GET,
-             self::TEST_URI
-         ));
+        $builder->getClient()->sendRequest(self::getPsr17Factory()->createRequest(
+            RequestMethod::GET,
+            self::TEST_URI
+        ));
+    }
+
+    public function testReplyWithClient(): void
+    {
+        $inlined = new PockBuilder();
+        $inlined->reply(429);
+
+        $builder = new PockBuilder();
+        $builder->matchMethod(RequestMethod::GET)
+            ->matchUri(self::TEST_URI)
+            ->always()
+            ->replyWithClient($inlined->getClient());
+
+        $response = $builder->getClient()->sendRequest(self::getPsr17Factory()->createRequest(
+            RequestMethod::GET,
+            self::TEST_URI
+        ));
+
+        self::assertEquals(429, $response->getStatusCode());
     }
 
     public function matchXmlNoXslProvider(): array

tests/src/Serializer/CallbackSerializerAdapterTest.php

@@ -0,0 +1,29 @@
+<?php
+
+/**
+ * PHP 7.1
+ *
+ * @category CallbackSerializerAdapterTest
+ * @package  Pock\Tests\Serializer
+ */
+
+namespace Pock\Tests\Serializer;
+
+use PHPUnit\Framework\TestCase;
+use Pock\Serializer\CallbackSerializerAdapter;
+
+/**
+ * Class CallbackSerializerAdapterTest
+ *
+ * @category CallbackSerializerAdapterTest
+ * @package  Pock\Tests\Serializer
+ */
+class CallbackSerializerAdapterTest extends TestCase
+{
+    public function testSerialize(): void
+    {
+        self::assertEquals('{}', (new CallbackSerializerAdapter(function ($data) {
+            return $data;
+        }))->serialize('{}'));
+    }
+}