AbstractPersistProcessorTest: replace expectWarning mit expectException

From sebastianbergmann/phpunit#5062 Issue: ecamp#3337
BacLuc · May 6, 2023 · cdefa9d · cdefa9d
1 parent 6664a62
commit cdefa9d
Showing 1 changed file with 18 additions and 1 deletion.
diff --git a/api/tests/State/Util/AbstractPersistProcessorTest.php b/api/tests/State/Util/AbstractPersistProcessorTest.php
@@ -42,6 +42,20 @@ protected function setUp(): void {
         );
 
         $this->processor->method('onBefore')->willReturnArgument(0);
+
+        set_error_handler(
+            /**
+             * @throws WarningException
+             */
+            static function (int $errno, string $errstr): never {
+                throw new WarningException();
+            },
+            E_WARNING
+        );
+    }
+
+    protected function tearDown(): void {
+        restore_error_handler();
     }
 
     public function testThrowsIfOnePropertyChangeListenerIsOfWrongType() {
@@ -96,7 +110,7 @@ public function testThrowErrorIfExtractPropertyFails() {
         );
         $this->closure->expects(self::never())->method('call');
 
-        $this->expectWarning();
+        $this->expectException(WarningException::class);
         $this->processor->process($toPersist, new Patch(), [], $context);
     }
 
@@ -156,3 +170,6 @@ class MyEntity extends BaseEntity {
 
 class MyEmptyEntity extends BaseEntity {
 }
+
+class WarningException extends \Exception {
+}