fix(symfony): ECMA-262 pattern with RegExp validator

soyuka · Oct 17, 2024 · a0409fc · a0409fc
1 parent 4171d5f
commit a0409fc
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/...idator/Metadata/Resource/Factory/ParameterValidationResourceMetadataCollectionFactory.php b/...idator/Metadata/Resource/Factory/ParameterValidationResourceMetadataCollectionFactory.php
@@ -130,7 +130,7 @@ private function addSchemaValidation(Parameter $parameter, ?array $schema = null
         }
 
         if (isset($schema['pattern'])) {
-            $assertions[] = new Regex($schema['pattern']);
+            $assertions[] = new Regex('#'.$schema['pattern'].'#');
         }
 
         if (isset($schema['maxLength']) || isset($schema['minLength'])) {

diff --git a/tests/Fixtures/TestBundle/ApiResource/WithParameter.php b/tests/Fixtures/TestBundle/ApiResource/WithParameter.php
@@ -62,7 +62,7 @@
         'length' => new QueryParameter(schema: ['maxLength' => 1, 'minLength' => 3]),
         'array' => new QueryParameter(schema: ['minItems' => 2, 'maxItems' => 3]),
         'multipleOf' => new QueryParameter(schema: ['multipleOf' => 2]),
-        'pattern' => new QueryParameter(schema: ['pattern' => '/\d/']),
+        'pattern' => new QueryParameter(schema: ['pattern' => '\d']),
     ],
     provider: [self::class, 'collectionProvider']
 )]

diff --git a/tests/Functional/Parameters/ValidationTest.php b/tests/Functional/Parameters/ValidationTest.php
@@ -113,7 +113,7 @@ public static function provideQueryStrings(): array
 
     public function testBlank(): void
     {
-        $response = self::createClient()->request('GET', 'validate_parameters?blank=f');
+        self::createClient()->request('GET', 'validate_parameters?blank=f');
         $this->assertResponseIsSuccessful();
     }
 
@@ -146,4 +146,10 @@ public function testValidatePropertyPlaceholder(): void
             ],
         ], $response->toArray(false));
     }
+
+    public function testValidatePattern(): void
+    {
+        self::createClient()->request('GET', 'validate_parameters?pattern=2');
+        $this->assertResponseIsSuccessful();
+    }
 }