From 58bea54f50e9a3e554bf3fdf2eaeed2a1836498a Mon Sep 17 00:00:00 2001
From: "tien.xuan.vo" <tien.xuan.vo@gmail.com>
Date: Mon, 2 Dec 2024 13:02:10 +0700
Subject: [PATCH] chore: Don't validate example value with regex

---
 .../Exception/InvalidRegexException.php       |  7 ------
 .../Consumer/Matcher/Matchers/Regex.php       | 22 -------------------
 .../Matcher/Matchers/EachValueTest.php        |  2 +-
 .../Consumer/Matcher/Matchers/RegexTest.php   | 12 ----------
 4 files changed, 1 insertion(+), 42 deletions(-)
 delete mode 100644 src/PhpPact/Consumer/Matcher/Exception/InvalidRegexException.php

diff --git a/src/PhpPact/Consumer/Matcher/Exception/InvalidRegexException.php b/src/PhpPact/Consumer/Matcher/Exception/InvalidRegexException.php
deleted file mode 100644
index 97b72f6d..00000000
--- a/src/PhpPact/Consumer/Matcher/Exception/InvalidRegexException.php
+++ /dev/null
@@ -1,7 +0,0 @@
-<?php
-
-namespace PhpPact\Consumer\Matcher\Exception;
-
-class InvalidRegexException extends MatcherException
-{
-}
diff --git a/src/PhpPact/Consumer/Matcher/Matchers/Regex.php b/src/PhpPact/Consumer/Matcher/Matchers/Regex.php
index 91d49e46..7aaa6340 100644
--- a/src/PhpPact/Consumer/Matcher/Matchers/Regex.php
+++ b/src/PhpPact/Consumer/Matcher/Matchers/Regex.php
@@ -3,7 +3,6 @@
 namespace PhpPact\Consumer\Matcher\Matchers;
 
 use PhpPact\Consumer\Matcher\Exception\InvalidValueException;
-use PhpPact\Consumer\Matcher\Exception\InvalidRegexException;
 use PhpPact\Consumer\Matcher\Generators\Regex as RegexGenerator;
 use PhpPact\Consumer\Matcher\Model\Attributes;
 use PhpPact\Consumer\Matcher\Model\Expression;
@@ -11,9 +10,6 @@
 use PhpPact\Consumer\Matcher\Model\Matcher\JsonFormattableInterface;
 use PhpPact\Consumer\Matcher\Trait\JsonFormattableTrait;
 
-use function preg_last_error;
-use function preg_match;
-
 class Regex extends GeneratorAwareMatcher implements JsonFormattableInterface, ExpressionFormattableInterface
 {
     use JsonFormattableTrait;
@@ -27,28 +23,10 @@ public function __construct(
     ) {
         if ($values === null) {
             $this->setGenerator(new RegexGenerator($this->regex));
-        } else {
-            $this->validateRegex();
         }
         parent::__construct();
     }
 
-    /**
-     * @todo Use json_validate()
-     */
-    private function validateRegex(): void
-    {
-        foreach ((array) $this->values as $value) {
-            $result = preg_match("/$this->regex/", $value);
-
-            if ($result !== 1) {
-                $errorCode = preg_last_error();
-
-                throw new InvalidRegexException("The value '{$value}' doesn't match pattern '{$this->regex}'. Failed with error code {$errorCode}.");
-            }
-        }
-    }
-
     public function formatJson(): Attributes
     {
         return $this->mergeJson(new Attributes([
diff --git a/tests/PhpPact/Consumer/Matcher/Matchers/EachValueTest.php b/tests/PhpPact/Consumer/Matcher/Matchers/EachValueTest.php
index 98a16879..72eb7646 100644
--- a/tests/PhpPact/Consumer/Matcher/Matchers/EachValueTest.php
+++ b/tests/PhpPact/Consumer/Matcher/Matchers/EachValueTest.php
@@ -55,7 +55,7 @@ public function testFormatJson(): void
                         "value": null
                     }
                 ]
-                }
+            }
             JSON,
             $jsonEncoded
         );
diff --git a/tests/PhpPact/Consumer/Matcher/Matchers/RegexTest.php b/tests/PhpPact/Consumer/Matcher/Matchers/RegexTest.php
index 9306fd84..3a82f0f5 100644
--- a/tests/PhpPact/Consumer/Matcher/Matchers/RegexTest.php
+++ b/tests/PhpPact/Consumer/Matcher/Matchers/RegexTest.php
@@ -2,7 +2,6 @@
 
 namespace PhpPactTest\Consumer\Matcher\Matchers;
 
-use PhpPact\Consumer\Matcher\Exception\InvalidRegexException;
 use PhpPact\Consumer\Matcher\Exception\InvalidValueException;
 use PhpPact\Consumer\Matcher\Formatters\Expression\ExpressionFormatter;
 use PhpPact\Consumer\Matcher\Matchers\Regex;
@@ -14,17 +13,6 @@ class RegexTest extends TestCase
 {
     private string $regex = '\d+';
 
-    #[TestWith(['number', true])]
-    #[TestWith(['integer', false])]
-    public function testInvalidRegex(string $value, bool $isArray): void
-    {
-        $values = $isArray ? [$value] : $value;
-        $this->expectException(InvalidRegexException::class);
-        $value = is_array($values) ? $values[0] : $values;
-        $this->expectExceptionMessage("The value '{$value}' doesn't match pattern '{$this->regex}'. Failed with error code 0.");
-        new Regex($this->regex, $values);
-    }
-
     /**
      * @param string|string[]|null $values
      */