More callable param tests (#295)

More tests related to bug #288

Follow-up to #294

src/RuleErrors/DisallowedFunctionRuleErrors.php

@@ -65,7 +65,7 @@ public function get(FuncCall $node, Scope $scope, array $disallowedCalls): array
 	 */
 	public function getByString(string $name, Scope $scope, array $disallowedCalls): array
 	{
-		return $this->getErrors(new Name($name), $scope, null, null, $disallowedCalls);
+		return $name === '' ? [] : $this->getErrors(new Name($name), $scope, null, null, $disallowedCalls);
 	}
 
 

tests/Bugs/Bug288DocblockTemplateCallableParamTest.php

@@ -47,7 +47,12 @@ protected function getRule(): Rule
 
 	public function testRule(): void
 	{
-		$this->analyse([__DIR__ . '/../src/bugs/Bug288DocblockTemplateCallableParam.php'], []);
+		$this->analyse([__DIR__ . '/../src/bugs/Bug288DocblockTemplateCallableParam.php'], [
+			[
+				'Calling disallowedFunction() is forbidden.',
+				15,
+			],
+		]);
 	}
 
 

tests/src/bugs/Bug288DocblockTemplateCallableParam.php

@@ -12,6 +12,8 @@ public function testItDoesNothing(): void
 	{
 		// the $expected param here is a string param, not a callable param, no error should be reported here
 		$this->assertSame('disallowedFunction', 'whatever');
+		$this->constantStringIsCallable('disallowedFunction');
+		$this->callableString('');
 	}
 
 	/**
@@ -25,4 +27,20 @@ public function assertSame(mixed $expected, mixed $actual, string $message = '')
 	{
 	}
 
+
+	/**
+	 * @param 'disallowedFunction' $callable
+	 */
+	public function constantStringIsCallable(string $callable)
+	{
+	}
+
+
+	/**
+	 * @param callable-string $callable
+	 */
+	public function callableString(string $callable)
+	{
+	}
+
 }