Changes the test to leverage failing on warnings.

Signed-off-by: Alexandre Lemaire <saeven@saeven.net>

Author: Saeven

phpunit.xml.dist

@@ -4,6 +4,7 @@
          bootstrap="./vendor/autoload.php"
          beStrictAboutOutputDuringTests="true"
          beStrictAboutTestsThatDoNotTestAnything="true"
+         failOnWarning="true"
          colors="true">
     <testsuites>
         <testsuite name="laminas-validator Test Suite">

test/EmailAddressTest.php

@@ -18,14 +18,12 @@
 use function implode;
 use function next;
 use function preg_replace;
-use function restore_error_handler;
 use function set_error_handler;
 use function sprintf;
 use function str_repeat;
 use function strstr;
 
 use const E_USER_NOTICE;
-use const E_WARNING;
 
 /**
  * @group      Laminas_Validator
@@ -934,15 +932,6 @@ public function testWillNotCheckEmptyDeepMxChecks(): void
             'useDeepMxCheck' => true,
         ]);
 
-        $called = false;
-        /** @psalm-suppress InvalidArgument **/
-        set_error_handler(static function () use (&$called) {
-            $called = true;
-        }, E_WARNING);
-
-        $validator->isValid('jon@example.com');
-
-        restore_error_handler();
-        $this->assertFalse($called);
+        $this->assertFalse($validator->isValid('jon@example.com'));
     }
 }