[make:validator] generate final classes

- adds generated file comparison to test suite

Author: jrushlow

src/Maker/MakeValidator.php

@@ -16,9 +16,12 @@
 use Symfony\Bundle\MakerBundle\Generator;
 use Symfony\Bundle\MakerBundle\InputConfiguration;
 use Symfony\Bundle\MakerBundle\Str;
+use Symfony\Bundle\MakerBundle\Util\ClassSource\Model\ClassData;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Validator\Constraint;
+use Symfony\Component\Validator\ConstraintValidator;
 use Symfony\Component\Validator\Validation;
 
 /**
@@ -49,26 +52,35 @@ public function configureCommand(Command $command, InputConfiguration $inputConf
     /** @return void */
     public function generate(InputInterface $input, ConsoleStyle $io, Generator $generator)
     {
-        $validatorClassNameDetails = $generator->createClassNameDetails(
-            $input->getArgument('name'),
-            'Validator\\',
-            'Validator'
+        $validatorClassData = ClassData::create(
+            class: \sprintf('Validator\\%s', $input->getArgument('name')),
+            suffix: 'Validator',
+            extendsClass: ConstraintValidator::class,
+            useStatements: [
+                Constraint::class,
+            ],
         );
 
-        $constraintFullClassName = Str::removeSuffix($validatorClassNameDetails->getFullName(), 'Validator');
+        $constraintDataClass = ClassData::create(
+            class: \sprintf('Validator\\%s', Str::removeSuffix($validatorClassData->getClassName(), 'Validator')),
+            extendsClass: Constraint::class,
+        );
 
         $generator->generateClass(
-            $validatorClassNameDetails->getFullName(),
+            $validatorClassData->getFullClassName(),
             'validator/Validator.tpl.php',
             [
-                'constraint_class_name' => Str::getShortClassName($constraintFullClassName),
+                'class_data' => $validatorClassData,
+                'constraint_class_name' => $constraintDataClass->getClassName(),
             ]
         );
 
         $generator->generateClass(
-            $constraintFullClassName,
+            $constraintDataClass->getFullClassName(),
             'validator/Constraint.tpl.php',
-            []
+            [
+                'class_data' => $constraintDataClass,
+            ]
         );
 
         $generator->writeChanges();

src/Resources/skeleton/validator/Constraint.tpl.php

@@ -1,11 +1,11 @@
 <?= "<?php\n" ?>
 
-namespace <?= $namespace; ?>;
+namespace <?= $class_data->getNamespace(); ?>;
 
-use Symfony\Component\Validator\Constraint;
+<?= $class_data->getUseStatements(); ?>
 
 #[\Attribute(\Attribute::TARGET_PROPERTY | \Attribute::TARGET_METHOD | \Attribute::IS_REPEATABLE)]
-class <?= $class_name ?> extends Constraint
+<?= $class_data->getClassDeclaration(); ?>
 {
     public string $message = 'The string "{{ string }}" contains an illegal character: it can only contain letters or numbers.';
 

src/Resources/skeleton/validator/Validator.tpl.php

@@ -1,11 +1,10 @@
 <?= "<?php\n" ?>
 
-namespace <?= $namespace; ?>;
+namespace <?= $class_data->getNamespace(); ?>;
 
-use Symfony\Component\Validator\Constraint;
-use Symfony\Component\Validator\ConstraintValidator;
+<?= $class_data->getUseStatements(); ?>
 
-class <?= $class_name ?> extends ConstraintValidator
+<?= $class_data->getClassDeclaration(); ?>
 {
     public function validate(mixed $value, Constraint $constraint): void
     {
@@ -18,6 +17,7 @@ public function validate(mixed $value, Constraint $constraint): void
         // TODO: implement the validation here
         $this->context->buildViolation($constraint->message)
             ->setParameter('{{ value }}', $value)
-            ->addViolation();
+            ->addViolation()
+        ;
     }
 }

tests/Maker/MakeValidatorTest.php

@@ -32,6 +32,18 @@ public function getTestDetails(): \Generator
                         'FooBar',
                     ]
                 );
+
+                // Validator
+                $expectedVoterPath = \dirname(__DIR__).'/fixtures/make-validator/expected/FooBarValidator.php';
+                $generatedVoter = $runner->getPath('src/Validator/FooBarValidator.php');
+
+                self::assertSame(file_get_contents($expectedVoterPath), file_get_contents($generatedVoter));
+
+                // Constraint
+                $expectedVoterPath = \dirname(__DIR__).'/fixtures/make-validator/expected/FooBar.php';
+                $generatedVoter = $runner->getPath('src/Validator/FooBar.php');
+
+                self::assertSame(file_get_contents($expectedVoterPath), file_get_contents($generatedVoter));
             }),
         ];
     }

tests/fixtures/make-validator/expected/FooBar.php

@@ -0,0 +1,19 @@
+<?php
+
+namespace App\Validator;
+
+use Symfony\Component\Validator\Constraint;
+
+/**
+ * @Annotation
+ * @Target({"PROPERTY", "METHOD", "ANNOTATION"})
+ */
+#[\Attribute(\Attribute::TARGET_PROPERTY | \Attribute::TARGET_METHOD | \Attribute::IS_REPEATABLE)]
+final class FooBar extends Constraint
+{
+    /*
+     * Any public properties become valid options for the annotation.
+     * Then, use these in your validator class.
+     */
+    public string $message = 'The value "{{ value }}" is not valid.';
+}

tests/fixtures/make-validator/expected/FooBarValidator.php

@@ -0,0 +1,24 @@
+<?php
+
+namespace App\Validator;
+
+use Symfony\Component\Validator\Constraint;
+use Symfony\Component\Validator\ConstraintValidator;
+
+final class FooBarValidator extends ConstraintValidator
+{
+    public function validate(mixed $value, Constraint $constraint): void
+    {
+        /* @var FooBar $constraint */
+
+        if (null === $value || '' === $value) {
+            return;
+        }
+
+        // TODO: implement the validation here
+        $this->context->buildViolation($constraint->message)
+            ->setParameter('{{ value }}', $value)
+            ->addViolation()
+        ;
+    }
+}