MNT Fix tests

Author: GuySartorelli

tests/php/ValidatorTests/AjaxCompositeValidatorTest.php

@@ -88,13 +88,13 @@ public function testGetOrAddValidatorByType(): void
         // Confirm validator starts empty.
         $this->assertCount(0, $compositeValidator->getValidators());
         // One validator should be created and added.
-        $simpleFieldsValidator1 = $compositeValidator->getOrAddValidatorByType(SimpleFieldsValidator::class); // @TODO replace with another
+        $validator1 = $compositeValidator->getOrAddValidatorByType(WarningFieldsValidator::class);
         $this->assertCount(1, $compositeValidator->getValidators());
         // The validator previously created should be fetched, rather than instantiating a new one.
-        $simpleFieldsValidator2 = $compositeValidator->getOrAddValidatorByType(SimpleFieldsValidator::class); // @TODO replace with another
+        $validator2 = $compositeValidator->getOrAddValidatorByType(WarningFieldsValidator::class);
         $this->assertCount(1, $compositeValidator->getValidators());
         // Confirm both simple fields validators are the exact same instance.
-        $this->assertTrue($simpleFieldsValidator1 === $simpleFieldsValidator2);
+        $this->assertTrue($validator1 === $validator2);
     }
 
     /**

tests/php/ValidatorTests/ConstraintsValidatorTest.php

@@ -2,14 +2,15 @@
 
 namespace Signify\ComposableValidators\Tests;
 
+use PHPUnit\Framework\Attributes\DataProvider;
 use Signify\ComposableValidators\Validators\ConstraintsValidator;
 use SilverStripe\Dev\SapphireTest;
 use Symfony\Component\Validator\Constraints\Ip;
 use Symfony\Component\Validator\Constraints\NotBlank;
 
 class ConstraintsValidatorTest extends SapphireTest
 {
-    public function provideValidation(): array
+    public static function provideValidation(): array
     {
         return [
             [
@@ -25,9 +26,7 @@ public function provideValidation(): array
         ];
     }
 
-    /**
-     * @dataProvider provideValidation
-     */
+    #[DataProvider('provideValidation')]
     public function testValidation(array $fields, array $constraints, bool $isValid): void
     {
         $form = TestFormGenerator::getForm($fields, new ConstraintsValidator($constraints));