Fix #7719: ReflectionMethod with invokeArgs: Dead catch - exception is never thrown in the try block.

conf/config.neon

@@ -125,6 +125,7 @@ parameters:
 		- ../stubs/Redis.stub
 		- ../stubs/ReflectionAttribute.stub
 		- ../stubs/ReflectionClassConstant.stub
+		- ../stubs/ReflectionFunction.stub
 		- ../stubs/ReflectionFunctionAbstract.stub
 		- ../stubs/ReflectionMethod.stub
 		- ../stubs/ReflectionParameter.stub

stubs/ReflectionFunction.stub

@@ -0,0 +1,16 @@
+<?php
+
+class ReflectionFunction
+{
+
+	/**
+	 * @throws \Throwable
+	 */
+	public function invoke(mixed ...$args): mixed {}
+
+	/**
+	 * @throws \Throwable
+	 */
+	public function invokeArgs(array $args): mixed {}
+
+}

stubs/ReflectionMethod.stub

@@ -13,4 +13,14 @@ class ReflectionMethod
 	 */
 	public $name;
 
+	/**
+	 * @throws \Throwable
+	 */
+	public function invoke(?object $object, mixed ...$args): mixed {}
+
+	/**
+	 * @throws \Throwable
+	 */
+	public function invokeArgs(?object $object, array $args): mixed {}
+
 }

tests/PHPStan/Rules/Exceptions/CatchWithUnthrownExceptionRuleTest.php

@@ -604,6 +604,16 @@ public function testBug9568(): void
 		$this->analyse([__DIR__ . '/data/bug-9568.php'], []);
 	}
 
+	public function testBug7719(): void
+	{
+		$this->analyse([__DIR__ . '/data/bug-7719.php'], []);
+	}
+
+	public function testBug9267(): void
+	{
+		$this->analyse([__DIR__ . '/data/bug-9267.php'], []);
+	}
+
 	#[RequiresPhp('>= 8.4')]
 	public function testPropertyHooks(): void
 	{

tests/PHPStan/Rules/Exceptions/data/bug-7719.php

@@ -0,0 +1,56 @@
+<?php declare(strict_types = 1);
+
+namespace Bug7719;
+
+class Endpoint {
+	public function run(int $id): int {
+		return $id;
+	}
+}
+
+class HelloWorld
+{
+	public function sayHello(Endpoint $endpoint, string $methodName): void
+	{
+		try {
+			$methodResponse = (new \ReflectionMethod($endpoint, $methodName))->invokeArgs($endpoint, ['id' => 2]);
+		} catch (\RuntimeException $e) {
+			echo $e->getMessage();
+			die;
+		}
+		var_dump($methodResponse);
+	}
+
+	public function sayHelloWithInvoke(Endpoint $endpoint, string $methodName): void
+	{
+		try {
+			$methodResponse = (new \ReflectionMethod($endpoint, $methodName))->invoke($endpoint, 2);
+		} catch (\RuntimeException $e) {
+			echo $e->getMessage();
+			die;
+		}
+		var_dump($methodResponse);
+	}
+
+	public function sayHelloWithFunction(string $functionName): void
+	{
+		try {
+			$result = (new \ReflectionFunction($functionName))->invokeArgs([1, 2]);
+		} catch (\RuntimeException $e) {
+			echo $e->getMessage();
+			die;
+		}
+		var_dump($result);
+	}
+
+	public function sayHelloWithFunctionInvoke(string $functionName): void
+	{
+		try {
+			$result = (new \ReflectionFunction($functionName))->invoke(1, 2);
+		} catch (\RuntimeException $e) {
+			echo $e->getMessage();
+			die;
+		}
+		var_dump($result);
+	}
+}

tests/PHPStan/Rules/Exceptions/data/bug-9267.php

@@ -0,0 +1,30 @@
+<?php declare(strict_types = 1);
+
+namespace Bug9267;
+
+class FooException extends \Exception {}
+
+class C {
+	/** @return never */
+	public function test(): never {
+		throw new FooException("");
+	}
+}
+
+function bar(\ReflectionMethod $r): void {
+	try {
+		$r->invokeArgs(new C, array());
+	}
+	catch (FooException $e) {
+		print "CAUGHT FOO!\n";
+	}
+}
+
+function baz(\ReflectionMethod $r): void {
+	try {
+		$r->invoke(new C);
+	}
+	catch (FooException $e) {
+		print "CAUGHT FOO!\n";
+	}
+}