From: Nikita Popov <nikita.ppv@gmail.com>
Date: Fri, 17 Jul 2020 13:50:06 +0000 (+0200)
Subject: Run arginfo / zpp mismatch tests for methods as well
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c9881fde7ad7536c079a206cd33d4f96d7238214;p=php

Run arginfo / zpp mismatch tests for methods as well

As a side-effect, this also catches issues where classes are not
prepared for use with newInstanceWithoutConstructor.
---

diff --git a/Zend/tests/arginfo_zpp_mismatch.phpt b/Zend/tests/arginfo_zpp_mismatch.phpt
index 9762dd5fce..39a1742743 100644
--- a/Zend/tests/arginfo_zpp_mismatch.phpt
+++ b/Zend/tests/arginfo_zpp_mismatch.phpt
@@ -3,10 +3,27 @@ Test that there is no arginfo/zpp mismatch
 --FILE--
 <?php
 
-foreach (get_defined_functions()["internal"] as $function) {
+function test($function) {
     try {
         @$function(null, null, null, null, null, null, null, null);
-    } catch (ArgumentCountError|Error) {
+    } catch (Throwable) {
+    }
+}
+
+foreach (get_defined_functions()["internal"] as $function) {
+    test($function);
+}
+
+foreach (get_declared_classes() as $class) {
+    try {
+        $rc = new ReflectionClass($class);
+        $obj = $rc->newInstanceWithoutConstructor();
+    } catch (Throwable) {
+        continue;
+    }
+
+    foreach (get_class_methods($class) as $method) {
+        test([$obj, $method]);
     }
 }