mirror of
https://github.com/danog/psalm.git
synced 2024-12-12 09:19:40 +01:00
0bd4dbcbc4
Fixes #1650
530 lines
18 KiB
PHP
530 lines
18 KiB
PHP
<?php
|
|
namespace Psalm\Tests;
|
|
|
|
use Psalm\Config;
|
|
use Psalm\Context;
|
|
|
|
class MagicMethodAnnotationTest extends TestCase
|
|
{
|
|
use Traits\InvalidCodeAnalysisTestTrait;
|
|
use Traits\ValidCodeAnalysisTestTrait;
|
|
|
|
/**
|
|
* @return void
|
|
*/
|
|
public function testPhpDocMethodWhenUndefined()
|
|
{
|
|
Config::getInstance()->use_phpdoc_method_without_magic_or_parent = true;
|
|
|
|
$this->addFile(
|
|
'somefile.php',
|
|
'<?php
|
|
/**
|
|
* @method string getString()
|
|
* @method void setInteger(int $integer)
|
|
* @method setString(int $integer)
|
|
* @method getBool(string $foo) : bool
|
|
* @method (string|int)[] getArray() : array
|
|
* @method (callable() : string) getCallable() : callable
|
|
*/
|
|
class Child {}
|
|
|
|
$child = new Child();
|
|
|
|
$a = $child->getString();
|
|
$child->setInteger(4);
|
|
/** @psalm-suppress MixedAssignment */
|
|
$b = $child->setString(5);
|
|
$c = $child->getBool("hello");
|
|
$d = $child->getArray();
|
|
$e = $child->getCallable();'
|
|
);
|
|
|
|
$this->analyzeFile('somefile.php', new Context());
|
|
}
|
|
|
|
/**
|
|
*
|
|
* @return void
|
|
*/
|
|
public function testAnnotationWithoutCallConfig()
|
|
{
|
|
$this->expectExceptionMessage('UndefinedMethod');
|
|
$this->expectException(\Psalm\Exception\CodeException::class);
|
|
Config::getInstance()->use_phpdoc_method_without_magic_or_parent = false;
|
|
|
|
$this->addFile(
|
|
'somefile.php',
|
|
'<?php
|
|
/**
|
|
* @method string getString()
|
|
*/
|
|
class Child {}
|
|
|
|
$child = new Child();
|
|
|
|
$child->getString();'
|
|
);
|
|
|
|
$context = new Context();
|
|
|
|
$this->analyzeFile('somefile.php', $context);
|
|
}
|
|
|
|
/**
|
|
* @return void
|
|
*/
|
|
public function testOverrideParentClassRetunType()
|
|
{
|
|
Config::getInstance()->use_phpdoc_method_without_magic_or_parent = true;
|
|
|
|
$this->addFile(
|
|
'somefile.php',
|
|
'<?php
|
|
class ParentClass {
|
|
public static function getMe() : self {
|
|
return new self();
|
|
}
|
|
}
|
|
|
|
/**
|
|
* @method getMe() : Child
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = Child::getMe();'
|
|
);
|
|
|
|
$context = new Context();
|
|
|
|
$this->analyzeFile('somefile.php', $context);
|
|
|
|
$this->assertSame('Child', (string) $context->vars_in_scope['$child']);
|
|
}
|
|
|
|
/**
|
|
* @return iterable<string,array{string,assertions?:array<string,string>,error_levels?:string[]}>
|
|
*/
|
|
public function providerValidCodeParse()
|
|
{
|
|
return [
|
|
'validSimpleAnnotations' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method string getString() dsa sada
|
|
* @method void setInteger(int $integer) dsa sada
|
|
* @method setString(int $integer) dsa sada
|
|
* @method setMixed(mixed $foo) dsa sada
|
|
* @method setImplicitMixed($foo) dsa sada
|
|
* @method setAnotherImplicitMixed( $foo, $bar,$baz) dsa sada
|
|
* @method setYetAnotherImplicitMixed( $foo ,$bar, $baz ) dsa sada
|
|
* @method getBool(string $foo) : bool dsa sada
|
|
* @method (string|int)[] getArray() : array with some text dsa sada
|
|
* @method (callable() : string) getCallable() : callable dsa sada
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$a = $child->getString();
|
|
$child->setInteger(4);
|
|
/** @psalm-suppress MixedAssignment */
|
|
$b = $child->setString(5);
|
|
$c = $child->getBool("hello");
|
|
$d = $child->getArray();
|
|
$child->setArray(["boo"]);
|
|
$e = $child->getCallable();
|
|
$child->setMixed("hello");
|
|
$child->setMixed(4);
|
|
$child->setImplicitMixed("hello");
|
|
$child->setImplicitMixed(4);',
|
|
'assertions' => [
|
|
'$a' => 'string',
|
|
'$b' => 'mixed',
|
|
'$c' => 'bool',
|
|
'$d' => 'array<array-key, string|int>',
|
|
'$e' => 'callable():string',
|
|
],
|
|
],
|
|
'validAnnotationWithDefault' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method void setArray(array $arr = array(), int $foo = 5) with some more text
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$child->setArray(["boo"]);
|
|
$child->setArray(["boo"], 8);',
|
|
],
|
|
'validStaticAnnotationWithDefault' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public static function __callStatic(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method static string getString(int $foo) with some more text
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$a = $child::getString(5);',
|
|
'assertions' => [
|
|
'$a' => 'string',
|
|
],
|
|
],
|
|
'validAnnotationWithVariadic' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method void setInts(int ...$foo) with some more text
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$child->setInts(1, 2, 3, 4);',
|
|
],
|
|
'validUnionAnnotations' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method setBool(string $foo, string|bool $bar) : bool dsa sada
|
|
* @method void setAnotherArray(int[]|string[] $arr = [], int $foo = 5) with some more text
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$b = $child->setBool("hello", true);
|
|
$c = $child->setBool("hello", "true");
|
|
$child->setAnotherArray(["boo"]);',
|
|
'assertions' => [
|
|
'$b' => 'bool',
|
|
'$c' => 'bool',
|
|
],
|
|
],
|
|
'namespacedValidAnnotations' => [
|
|
'<?php
|
|
namespace Foo;
|
|
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method setBool(string $foo, string|bool $bar) : bool
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$c = $child->setBool("hello", true);
|
|
$c = $child->setBool("hello", "true");',
|
|
],
|
|
'globalMethod' => [
|
|
'<?php
|
|
/** @method void global() */
|
|
class A {
|
|
public function __call(string $s) {}
|
|
}',
|
|
],
|
|
'magicMethodInternalCall' => [
|
|
'<?php
|
|
/**
|
|
* @method I[] work()
|
|
*/
|
|
class I {
|
|
function __call(string $method, array $args) { return [new I, new I]; }
|
|
|
|
function zugzug(): void {
|
|
echo count($this->work());
|
|
}
|
|
}',
|
|
],
|
|
'magicMethodOverridesParentWithMoreSpecificType' => [
|
|
'<?php
|
|
class C {}
|
|
class D extends C {}
|
|
|
|
class A {
|
|
public function foo(string $s) : C {
|
|
return new C;
|
|
}
|
|
}
|
|
|
|
/** @method D foo(string $s) */
|
|
class B extends A {}',
|
|
],
|
|
'complicatedMagicMethodInheritance' => [
|
|
'<?php
|
|
class BaseActiveRecord {
|
|
/**
|
|
* @param string $class
|
|
* @param array $link
|
|
* @return ActiveQueryInterface
|
|
*/
|
|
public function hasMany($class, $link)
|
|
{
|
|
return new ActiveQuery();
|
|
}
|
|
}
|
|
|
|
/**
|
|
* @method ActiveQuery hasMany($class, array $link)
|
|
*/
|
|
class ActiveRecord extends BaseActiveRecord {}
|
|
|
|
interface ActiveQueryInterface {}
|
|
|
|
class ActiveQuery implements ActiveQueryInterface {
|
|
/**
|
|
* @param string $tableName
|
|
* @param array $link
|
|
* @param callable $callable
|
|
* @return $this
|
|
*/
|
|
public function viaTable($tableName, $link, callable $callable = null)
|
|
{
|
|
return $this;
|
|
}
|
|
}
|
|
|
|
class Boom extends ActiveRecord {
|
|
/**
|
|
* @return ActiveQuery
|
|
*/
|
|
public function getUsers()
|
|
{
|
|
$query = $this->hasMany("User", ["id" => "user_id"])
|
|
->viaTable("account_to_user", ["account_id" => "id"]);
|
|
|
|
return $query;
|
|
}
|
|
}',
|
|
],
|
|
'magicMethodReturnSelf' => [
|
|
'<?php
|
|
/**
|
|
* @method static self getSelf()
|
|
* @method $this getThis()
|
|
*/
|
|
class C {
|
|
public static function __callStatic(string $c, array $args) {}
|
|
public function __call(string $c, array $args) {}
|
|
}
|
|
|
|
$a = C::getSelf();
|
|
$b = (new C)->getThis();',
|
|
[
|
|
'$a' => 'C',
|
|
'$b' => 'C',
|
|
],
|
|
],
|
|
'allowMagicMethodStatic' => [
|
|
'<?php
|
|
/** @method static getStatic() */
|
|
class C {
|
|
public function __call(string $c, array $args) {}
|
|
}
|
|
|
|
class D extends C {}
|
|
|
|
$c = (new C)->getStatic();
|
|
$d = (new D)->getStatic();',
|
|
[
|
|
'$c' => 'C',
|
|
'$d' => 'D',
|
|
],
|
|
],
|
|
'validSimplePsalmAnnotations' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @psalm-method string getString() dsa sada
|
|
* @psalm-method void setInteger(int $integer) dsa sada
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$a = $child->getString();
|
|
$child->setInteger(4);',
|
|
'assertions' => [
|
|
'$a' => 'string',
|
|
],
|
|
],
|
|
'overrideMethodAnnotations' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method int getString() dsa sada
|
|
* @method void setInteger(string $integer) dsa sada
|
|
* @psalm-method string getString() dsa sada
|
|
* @psalm-method void setInteger(int $integer) dsa sada
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$a = $child->getString();
|
|
$child->setInteger(4);',
|
|
'assertions' => [
|
|
'$a' => 'string',
|
|
],
|
|
],
|
|
];
|
|
}
|
|
|
|
/**
|
|
* @return iterable<string,array{string,error_message:string,2?:string[],3?:bool,4?:string}>
|
|
*/
|
|
public function providerInvalidCodeParse()
|
|
{
|
|
return [
|
|
'annotationWithBadDocblock' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method string getString(\)
|
|
*/
|
|
class Child extends ParentClass {}',
|
|
'error_message' => 'InvalidDocblock',
|
|
],
|
|
'annotationWithByRefParam' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method string getString(&$a)
|
|
*/
|
|
class Child extends ParentClass {}',
|
|
'error_message' => 'InvalidDocblock',
|
|
],
|
|
'annotationWithSealed' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method string getString()
|
|
* @psalm-seal-methods
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
$child->getString();
|
|
$child->foo();',
|
|
'error_message' => 'UndefinedMethod - src' . DIRECTORY_SEPARATOR . 'somefile.php:14:29 - Method Child::foo does not exist',
|
|
],
|
|
'annotationInvalidArg' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method setString(int $integer)
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$child->setString("five");',
|
|
'error_message' => 'InvalidScalarArgument',
|
|
],
|
|
'unionAnnotationInvalidArg' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method setBool(string $foo, string|bool $bar) : bool dsa sada
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$b = $child->setBool("hello", 5);',
|
|
'error_message' => 'InvalidScalarArgument',
|
|
],
|
|
'validAnnotationWithInvalidVariadicCall' => [
|
|
'<?php
|
|
class ParentClass {
|
|
public function __call(string $name, array $args) {}
|
|
}
|
|
|
|
/**
|
|
* @method void setInts(int ...$foo) with some more text
|
|
*/
|
|
class Child extends ParentClass {}
|
|
|
|
$child = new Child();
|
|
|
|
$child->setInts([1, 2, 3]);',
|
|
'error_message' => 'InvalidArgument',
|
|
],
|
|
'magicMethodOverridesParentWithDifferentReturnType' => [
|
|
'<?php
|
|
class C {}
|
|
class D {}
|
|
|
|
class A {
|
|
public function foo(string $s) : C {
|
|
return new C;
|
|
}
|
|
}
|
|
|
|
/** @method D foo(string $s) */
|
|
class B extends A {}',
|
|
'error_message' => 'ImplementedReturnTypeMismatch - src/somefile.php:11:33',
|
|
],
|
|
'magicMethodOverridesParentWithDifferentParamType' => [
|
|
'<?php
|
|
class C {}
|
|
class D extends C {}
|
|
|
|
class A {
|
|
public function foo(string $s) : C {
|
|
return new C;
|
|
}
|
|
}
|
|
|
|
/** @method D foo(int $s) */
|
|
class B extends A {}',
|
|
'error_message' => 'ImplementedParamTypeMismatch - src/somefile.php:11:21',
|
|
],
|
|
];
|
|
}
|
|
}
|