1
0
mirror of https://github.com/danog/psalm.git synced 2024-11-30 04:39:00 +01:00
psalm/tests/PropertyTypeTest.php

3826 lines
126 KiB
PHP
Raw Normal View History

2016-07-12 06:53:36 +02:00
<?php
2016-07-26 00:37:44 +02:00
namespace Psalm\Tests;
2016-07-12 06:53:36 +02:00
2021-12-03 21:40:18 +01:00
use DateTime;
2016-11-02 07:29:00 +01:00
use Psalm\Config;
use Psalm\Context;
2021-12-03 20:29:06 +01:00
use Psalm\Exception\CodeException;
2021-12-04 21:55:53 +01:00
use Psalm\Tests\Traits\InvalidCodeAnalysisTestTrait;
use Psalm\Tests\Traits\ValidCodeAnalysisTestTrait;
2016-07-12 06:53:36 +02:00
2021-06-08 04:55:21 +02:00
use const DIRECTORY_SEPARATOR;
class PropertyTypeTest extends TestCase
2016-07-12 06:53:36 +02:00
{
2021-12-04 21:55:53 +01:00
use InvalidCodeAnalysisTestTrait;
use ValidCodeAnalysisTestTrait;
public function testForgetPropertyAssignments(): void
{
2019-05-17 00:36:36 +02:00
$this->expectExceptionMessage('NullableReturnStatement');
2021-12-03 20:29:06 +01:00
$this->expectException(CodeException::class);
Config::getInstance()->remember_property_assignments_after_call = false;
Refactor scanning and analysis, introducing multithreading (#191) * Add failing test * Add visitor to soup up classlike references * Move a whole bunch of code into the visitor * Move some methods back, move onto analysis stage * Use the getAliases method everywhere * Fix refs * Fix more refs * Fix some tests * Fix more tests * Fix include tests * Shift config class finding to project checker and fix bugs * Fix a few more tests * transition test to new syntax * Remove var_dump * Delete a bunch of code and fix mutation test * Remove unnecessary visitation * Transition to better mocked out file provider, breaking some cached statement loading * Use different scheme for naming anonymous classes * Fix anonymous class issues * Refactor file/statement loading * Add specific property types * Fix mapped property assignment * Improve how we deal with traits * Fix trait checking * Pass Psalm checks * Add multi-process support * Delay console output until the end * Remove PHP 7 syntax * Update file storage with classes * Fix scanning individual files and add reflection return types * Always turn XDebug off * Add quicker method of getting method mutations * Queue return types for crawling * Interpret all strings as possible classes once we see a `get_class` call * Check invalid return types again * Fix template namespacing issues * Default to class-insensitive file names for includes * Don’t overwrite existing issues data * Add var docblocks for scanning * Add null check * Fix loading of external classes in templates * Only try to populate class when we haven’t yet seen it’s not a class * Fix trait property accessibility * Only ever improve docblock param type * Make param replacement more robust * Fix static const missing inferred type * Fix a few more tests * Register constant definitions * Fix trait aliasing * Skip constant type tests for now * Fix linting issues * Make sure caching is off for tests * Remove unnecessary return * Use emulative parser if on PHP 5.6 * Cache parser for faster first-time parse * Fix constant resolution when scanning classes * Remove test that’s beyond a practical scope * Add back --diff support * Add --help for --threads * Remove unused vars
2017-07-25 22:11:02 +02:00
$this->addFile(
'somefile.php',
'<?php
class XCollector {
/** @var X[] */
private static array $xs = [];
public static function modify() : void {
foreach (self::$xs as $x) {
$x->x = null;
}
}
}
Refactor scanning and analysis, introducing multithreading (#191) * Add failing test * Add visitor to soup up classlike references * Move a whole bunch of code into the visitor * Move some methods back, move onto analysis stage * Use the getAliases method everywhere * Fix refs * Fix more refs * Fix some tests * Fix more tests * Fix include tests * Shift config class finding to project checker and fix bugs * Fix a few more tests * transition test to new syntax * Remove var_dump * Delete a bunch of code and fix mutation test * Remove unnecessary visitation * Transition to better mocked out file provider, breaking some cached statement loading * Use different scheme for naming anonymous classes * Fix anonymous class issues * Refactor file/statement loading * Add specific property types * Fix mapped property assignment * Improve how we deal with traits * Fix trait checking * Pass Psalm checks * Add multi-process support * Delay console output until the end * Remove PHP 7 syntax * Update file storage with classes * Fix scanning individual files and add reflection return types * Always turn XDebug off * Add quicker method of getting method mutations * Queue return types for crawling * Interpret all strings as possible classes once we see a `get_class` call * Check invalid return types again * Fix template namespacing issues * Default to class-insensitive file names for includes * Don’t overwrite existing issues data * Add var docblocks for scanning * Add null check * Fix loading of external classes in templates * Only try to populate class when we haven’t yet seen it’s not a class * Fix trait property accessibility * Only ever improve docblock param type * Make param replacement more robust * Fix static const missing inferred type * Fix a few more tests * Register constant definitions * Fix trait aliasing * Skip constant type tests for now * Fix linting issues * Make sure caching is off for tests * Remove unnecessary return * Use emulative parser if on PHP 5.6 * Cache parser for faster first-time parse * Fix constant resolution when scanning classes * Remove test that’s beyond a practical scope * Add back --diff support * Add --help for --threads * Remove unused vars
2017-07-25 22:11:02 +02:00
class X {
public ?int $x = null;
Refactor scanning and analysis, introducing multithreading (#191) * Add failing test * Add visitor to soup up classlike references * Move a whole bunch of code into the visitor * Move some methods back, move onto analysis stage * Use the getAliases method everywhere * Fix refs * Fix more refs * Fix some tests * Fix more tests * Fix include tests * Shift config class finding to project checker and fix bugs * Fix a few more tests * transition test to new syntax * Remove var_dump * Delete a bunch of code and fix mutation test * Remove unnecessary visitation * Transition to better mocked out file provider, breaking some cached statement loading * Use different scheme for naming anonymous classes * Fix anonymous class issues * Refactor file/statement loading * Add specific property types * Fix mapped property assignment * Improve how we deal with traits * Fix trait checking * Pass Psalm checks * Add multi-process support * Delay console output until the end * Remove PHP 7 syntax * Update file storage with classes * Fix scanning individual files and add reflection return types * Always turn XDebug off * Add quicker method of getting method mutations * Queue return types for crawling * Interpret all strings as possible classes once we see a `get_class` call * Check invalid return types again * Fix template namespacing issues * Default to class-insensitive file names for includes * Don’t overwrite existing issues data * Add var docblocks for scanning * Add null check * Fix loading of external classes in templates * Only try to populate class when we haven’t yet seen it’s not a class * Fix trait property accessibility * Only ever improve docblock param type * Make param replacement more robust * Fix static const missing inferred type * Fix a few more tests * Register constant definitions * Fix trait aliasing * Skip constant type tests for now * Fix linting issues * Make sure caching is off for tests * Remove unnecessary return * Use emulative parser if on PHP 5.6 * Cache parser for faster first-time parse * Fix constant resolution when scanning classes * Remove test that’s beyond a practical scope * Add back --diff support * Add --help for --threads * Remove unused vars
2017-07-25 22:11:02 +02:00
public function getX(): int {
$this->x = 5;
XCollector::modify();
return $this->x;
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testForgetPropertyAssignmentsPassesNormally(): void
{
$this->addFile(
'somefile.php',
'<?php
class XCollector {
/** @var X[] */
private static array $xs = [];
public static function modify() : void {
foreach (self::$xs as $x) {
$x->x = null;
}
}
}
class X {
public ?int $x = null;
public function getX(): int {
$this->x = 5;
XCollector::modify();
Refactor scanning and analysis, introducing multithreading (#191) * Add failing test * Add visitor to soup up classlike references * Move a whole bunch of code into the visitor * Move some methods back, move onto analysis stage * Use the getAliases method everywhere * Fix refs * Fix more refs * Fix some tests * Fix more tests * Fix include tests * Shift config class finding to project checker and fix bugs * Fix a few more tests * transition test to new syntax * Remove var_dump * Delete a bunch of code and fix mutation test * Remove unnecessary visitation * Transition to better mocked out file provider, breaking some cached statement loading * Use different scheme for naming anonymous classes * Fix anonymous class issues * Refactor file/statement loading * Add specific property types * Fix mapped property assignment * Improve how we deal with traits * Fix trait checking * Pass Psalm checks * Add multi-process support * Delay console output until the end * Remove PHP 7 syntax * Update file storage with classes * Fix scanning individual files and add reflection return types * Always turn XDebug off * Add quicker method of getting method mutations * Queue return types for crawling * Interpret all strings as possible classes once we see a `get_class` call * Check invalid return types again * Fix template namespacing issues * Default to class-insensitive file names for includes * Don’t overwrite existing issues data * Add var docblocks for scanning * Add null check * Fix loading of external classes in templates * Only try to populate class when we haven’t yet seen it’s not a class * Fix trait property accessibility * Only ever improve docblock param type * Make param replacement more robust * Fix static const missing inferred type * Fix a few more tests * Register constant definitions * Fix trait aliasing * Skip constant type tests for now * Fix linting issues * Make sure caching is off for tests * Remove unnecessary return * Use emulative parser if on PHP 5.6 * Cache parser for faster first-time parse * Fix constant resolution when scanning classes * Remove test that’s beyond a practical scope * Add back --diff support * Add --help for --threads * Remove unused vars
2017-07-25 22:11:02 +02:00
return $this->x;
}
2022-12-18 17:15:15 +01:00
}',
Refactor scanning and analysis, introducing multithreading (#191) * Add failing test * Add visitor to soup up classlike references * Move a whole bunch of code into the visitor * Move some methods back, move onto analysis stage * Use the getAliases method everywhere * Fix refs * Fix more refs * Fix some tests * Fix more tests * Fix include tests * Shift config class finding to project checker and fix bugs * Fix a few more tests * transition test to new syntax * Remove var_dump * Delete a bunch of code and fix mutation test * Remove unnecessary visitation * Transition to better mocked out file provider, breaking some cached statement loading * Use different scheme for naming anonymous classes * Fix anonymous class issues * Refactor file/statement loading * Add specific property types * Fix mapped property assignment * Improve how we deal with traits * Fix trait checking * Pass Psalm checks * Add multi-process support * Delay console output until the end * Remove PHP 7 syntax * Update file storage with classes * Fix scanning individual files and add reflection return types * Always turn XDebug off * Add quicker method of getting method mutations * Queue return types for crawling * Interpret all strings as possible classes once we see a `get_class` call * Check invalid return types again * Fix template namespacing issues * Default to class-insensitive file names for includes * Don’t overwrite existing issues data * Add var docblocks for scanning * Add null check * Fix loading of external classes in templates * Only try to populate class when we haven’t yet seen it’s not a class * Fix trait property accessibility * Only ever improve docblock param type * Make param replacement more robust * Fix static const missing inferred type * Fix a few more tests * Register constant definitions * Fix trait aliasing * Skip constant type tests for now * Fix linting issues * Make sure caching is off for tests * Remove unnecessary return * Use emulative parser if on PHP 5.6 * Cache parser for faster first-time parse * Fix constant resolution when scanning classes * Remove test that’s beyond a practical scope * Add back --diff support * Add --help for --threads * Remove unused vars
2017-07-25 22:11:02 +02:00
);
$this->analyzeFile('somefile.php', new Context());
}
public function testForgetPropertyAssignmentsInBranch(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->addFile(
'somefile.php',
'<?php
class XCollector {
/** @var X[] */
private static array $xs = [];
public static function modify() : void {
foreach (self::$xs as $x) {
$x->x = null;
}
}
}
class X {
public ?int $x = null;
}
function testX(X $x): void {
$x->x = 5;
if (rand(0, 1)) {
XCollector::modify();
}
if ($x->x === null) {}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testFooBar(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->addFile(
'somefile.php',
'<?php
class A {
private ?int $bar = null;
public function baz(): void
{
$this->bar = null;
foreach (range(1, 5) as $part) {
if ($part === 3) {
$this->foo();
}
}
if ($this->bar === null) {}
}
private function foo() : void {
$this->bar = 5;
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testForgetFinalMethodCalls(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->addFile(
'somefile.php',
'<?php
class XCollector {
/** @var X[] */
private static array $xs = [];
public static function modify() : void {
foreach (self::$xs as $x) {
$x->x = null;
}
}
}
class X {
public ?int $x = null;
public function __construct(?int $x) {
$this->x = $x;
}
public final function getX() : ?int {
return $this->x;
}
}
function testX(X $x): void {
if ($x->getX()) {
XCollector::modify();
if ($x->getX() === null) {}
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testRememberImmutableMethodCalls(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->expectExceptionMessage('TypeDoesNotContainNull - somefile.php:22:29');
2021-12-03 20:29:06 +01:00
$this->expectException(CodeException::class);
$this->addFile(
'somefile.php',
'<?php
class XCollector {
public static function modify() : void {}
}
/** @psalm-immutable */
class X {
public ?int $x = null;
public function __construct(?int $x) {
$this->x = $x;
}
public function getX() : ?int {
return $this->x;
}
}
function testX(X $x): void {
if ($x->getX()) {
XCollector::modify();
if ($x->getX() === null) {}
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testRememberImmutableProperties(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->expectExceptionMessage('TypeDoesNotContainNull - somefile.php:18:29');
2021-12-03 20:29:06 +01:00
$this->expectException(CodeException::class);
$this->addFile(
'somefile.php',
'<?php
class XCollector {
public static function modify() : void {}
}
/** @psalm-immutable */
class X {
public ?int $x = null;
public function __construct(?int $x) {
$this->x = $x;
}
}
function testX(X $x): void {
if ($x->x) {
XCollector::modify();
if ($x->x === null) {}
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
2020-12-10 19:51:49 +01:00
public function testNoCrashInTryCatch(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->addFile(
'somefile.php',
'<?php
function maybeMutates() : void {}
class X {
public int $f = 0;
public function validate(): void {
try {
} finally {
$this->f = 1;
maybeMutates();
}
}
2022-12-18 17:15:15 +01:00
}',
2020-12-10 19:51:49 +01:00
);
$this->analyzeFile('somefile.php', new Context());
}
public function testAssertionInsideWhileOne(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->addFile(
'somefile.php',
'<?php
class Foo {
public array $a = [];
public array $b = [];
public array $c = [];
public function one(): bool {
$has_changes = false;
while ($this->a) {
$has_changes = true;
$this->alter();
}
return $has_changes;
}
public function alter() : void {
if (rand(0, 1)) {
array_pop($this->a);
}
if (rand(0, 1)) {
array_pop($this->b);
}
if (rand(0, 1)) {
array_pop($this->c);
}
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testAssertionInsideWhileTwo(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->addFile(
'somefile.php',
'<?php
class Foo {
public array $a = [];
public array $b = [];
public function two(): bool {
$has_changes = false;
while ($this->a || $this->b) {
$has_changes = true;
$this->alter();
}
return $has_changes;
}
Add support for strict arrays, fix type alias intersection, fix array_is_list assertion on non-lists (#8395) * Immutable CodeLocation * Remove excess clones * Remove external clones * Remove leftover clones * Fix final clone issue * Immutable storages * Refactoring * Fixes * Fixes * Fix * Fix * Fixes * Simplify * Fixes * Fix * Fixes * Update * Fix * Cache global types * Fix * Update * Update * Fixes * Fixes * Refactor * Fixes * Fix * Fix * More caching * Fix * Fix * Update * Update * Fix * Fixes * Update * Refactor * Update * Fixes * Break one more test * Fix * FIx * Fix * Fix * Fix * Fix * Improve performance and readability * Equivalent logic * Fixes * Revert * Revert "Revert" This reverts commit f9175100c8452c80559234200663fd4c4f4dd889. * Fix * Fix reference bug * Make default TypeVisitor immutable * Bugfix * Remove clones * Partial refactoring * Refactoring * Fixes * Fix * Fixes * Fixes * cs-fix * Fix final bugs * Add test * Misc fixes * Update * Fixes * Experiment with removing different property * revert "Experiment with removing different property" This reverts commit ac1156e077fc4ea633530d51096d27b6e88bfdf9. * Uniform naming * Uniform naming * Hack hotfix * Clean up $_FILES ref #8621 * Undo hack, try fixing properly * Helper method * Remove redundant call * Partially fix bugs * Cleanup * Change defaults * Fix bug * Fix (?, hope this doesn't break anything else) * cs-fix * Review fixes * Bugfix * Bugfix * Improve logic * Add support for list{} and callable-list{} types, properly implement array_is_list assertions (fixes #8389) * Default to sealed arrays * Fix array_merge bug * Fixes * Fix * Sealed type checks * Properly infer properties-of and get_object_vars on final classes * Fix array_map zipping * Fix tests * Fixes * Fixes * Fix more stuff * Recursively resolve type aliases * Fix typo * Fixes * Fix array_is_list assertion on keyed array * Add BC docs * Fixes * fix * Update * Update * Update * Update * Seal arrays with count assertions * Fix #8528 * Fix * Update * Improve sealed array foreach logic * get_object_vars on template properties * Fix sealed array assertion reconciler logic * Improved reconciler * Add tests * Single source of truth for test types * Fix tests * Fixup tests * Fixup tests * Fixup tests * Update * Fix tests * Fix tests * Final fixes * Fixes * Use list syntax only when needed * Fix tests * Cs-fix * Update docs * Update docs * Update docs * Update docs * Update docs * Document missing types * Update docs * Improve class-string-map docs * Update * Update * I love working on psalm :) * Keep arrays unsealed by default * Fixup tests * Fix syntax mistake * cs-fix * Fix typo * Re-import missing types * Keep strict types only in return types * argc/argv fixes * argc/argv fixes * Fix test * Comment-out valinor code, pinging @romm pls merge https://github.com/CuyZ/Valinor/pull/246 so we can add valinor to the psalm docs :)
2022-11-05 22:34:42 +01:00
public function alter() : void {
if (rand(0, 1)) {
array_pop($this->a);
}
if (rand(0, 1)) {
array_pop($this->b);
}
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testAssertionInsideWhileThree(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->addFile(
'somefile.php',
'<?php
class Foo {
public array $a = [];
public array $b = [];
public array $c = [];
public function three(): bool {
$has_changes = false;
while ($this->a || $this->b || $this->c) {
$has_changes = true;
$this->alter();
}
return $has_changes;
}
public function alter() : void {
if (rand(0, 1)) {
array_pop($this->a);
}
if (rand(0, 1)) {
array_pop($this->b);
}
if (rand(0, 1)) {
array_pop($this->c);
}
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testAssertionInsideWhileFour(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->addFile(
'somefile.php',
'<?php
class Foo {
public array $a = [];
public array $b = [];
public array $c = [];
public function four(): bool {
$has_changes = false;
while (($this->a && $this->b) || $this->c) {
$has_changes = true;
$this->alter();
}
return $has_changes;
}
public function alter() : void {
if (rand(0, 1)) {
array_pop($this->a);
}
if (rand(0, 1)) {
array_pop($this->b);
}
if (rand(0, 1)) {
array_pop($this->c);
}
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testAssertionInsideWhileFive(): void
{
Config::getInstance()->remember_property_assignments_after_call = false;
$this->addFile(
'somefile.php',
'<?php
class Foo {
public array $a = [];
public array $b = [];
public array $c = [];
public function five(): bool {
$has_changes = false;
while ($this->a || ($this->b && $this->c)) {
$has_changes = true;
$this->alter();
}
return $has_changes;
}
public function alter() : void {
if (rand(0, 1)) {
array_pop($this->a);
}
if (rand(0, 1)) {
array_pop($this->b);
}
if (rand(0, 1)) {
array_pop($this->c);
}
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testUniversalObjectCrates(): void
{
2021-12-03 21:40:18 +01:00
Config::getInstance()->addUniversalObjectCrate(DateTime::class);
$this->addFile(
'somefile.php',
'<?php
$f = new \DateTime();
// reads are fine
$f->bar;
// sets are fine
$f->buzz = false;
2022-12-18 17:15:15 +01:00
',
);
$this->analyzeFile('somefile.php', new Context());
}
public function testForgetPropertyAssignmentsInBranchWithThrowNormally(): void
{
$this->addFile(
'somefile.php',
'<?php
class XCollector {
/** @var X[] */
private static array $xs = [];
public static function modify() : void {
foreach (self::$xs as $x) {
$x->x = null;
}
}
}
class X {
/** @var ?int **/
public $x;
public function getX(bool $b): int {
$this->x = 5;
if ($b) {
XCollector::modify();
throw new \Exception("bad");
}
return $this->x;
}
2022-12-18 17:15:15 +01:00
}',
);
$this->analyzeFile('somefile.php', new Context());
}
public function providerValidCodeParse(): iterable
{
return [
'newVarInIf' => [
'code' => '<?php
class A {
/**
* @var mixed
*/
public $foo = "hello";
/** @return void */
public function barBar()
{
if (rand(0,10) === 5) {
$this->foo = [];
}
if (!is_array($this->foo)) {
// do something
}
}
2017-05-27 02:05:57 +02:00
}',
],
'propertyWithoutTypeSuppressingIssue' => [
'code' => '<?php
class A {
public $foo = "hello";
}
$a = (new A)->foo;',
'assertions' => [],
'ignored_issues' => [
'MissingPropertyType',
2017-05-27 02:05:57 +02:00
'MixedAssignment',
],
],
'propertyWithoutTypeSuppressingIssueAndAssertingNull' => [
'code' => '<?php
class A {
/** @return void */
function foo() {
$boop = $this->foo === null && rand(0,1);
echo $this->foo->baz;
}
}',
'assertions' => [],
'ignored_issues' => [
'UndefinedThisPropertyFetch',
'MixedAssignment',
2018-05-09 21:49:22 +02:00
'MixedArgument',
'MixedMethodCall',
2017-05-27 02:05:57 +02:00
'MixedPropertyFetch',
],
],
'sharedPropertyInIf' => [
'code' => '<?php
class A {
/** @var int */
public $foo = 0;
}
class B {
/** @var string */
public $foo = "";
}
$a = rand(0, 10) ? new A(): (rand(0, 10) ? new B() : null);
$b = null;
if ($a instanceof A || $a instanceof B) {
$b = $a->foo;
}',
'assertions' => [
'$b' => 'int|null|string',
2017-05-27 02:05:57 +02:00
],
],
'sharedPropertyInElseIf' => [
'code' => '<?php
class A {
/** @var int */
public $foo = 0;
}
class B {
/** @var string */
public $foo = "";
}
$a = rand(0, 10) ? new A() : new B();
if (rand(0, 1)) {
$a = null;
}
$b = null;
if (rand(0, 10) === 4) {
// do nothing
}
elseif ($a instanceof A || $a instanceof B) {
$b = $a->foo;
}',
'assertions' => [
'$b' => 'int|null|string',
2017-05-27 02:05:57 +02:00
],
],
'nullablePropertyCheck' => [
'code' => '<?php
class A {
/** @var string */
public $aa = "";
}
class B {
/** @var A|null */
public $bb;
}
$b = rand(0, 10) ? new A() : new B();
if ($b instanceof B && isset($b->bb) && $b->bb->aa === "aa") {
echo $b->bb->aa;
2017-05-27 02:05:57 +02:00
}',
],
'nullablePropertyAfterGuard' => [
'code' => '<?php
class A {
/** @var string|null */
public $aa;
}
$a = new A();
if (!$a->aa) {
$a->aa = "hello";
}
2017-05-27 02:05:57 +02:00
echo substr($a->aa, 1);',
],
'nullableStaticPropertyWithIfCheck' => [
'code' => '<?php
class A {
/** @var A|null */
public static $fooFoo;
2018-01-11 21:50:45 +01:00
public static function getFoo(): A {
if (!self::$fooFoo) {
self::$fooFoo = new A();
}
return self::$fooFoo;
}
2017-05-27 02:05:57 +02:00
}',
],
'reflectionProperties' => [
'code' => '<?php
class Foo {
}
2020-06-11 18:19:27 +02:00
$a = new \ReflectionMethod(Foo::class, "__construct");
2017-05-27 02:05:57 +02:00
echo $a->name . " - " . $a->class;',
],
'grandparentReflectedProperties' => [
'code' => '<?php
$a = new DOMElement("foo");
$owner = $a->ownerDocument;',
'assertions' => [
2020-04-06 19:17:02 +02:00
'$owner' => 'DOMDocument|null',
2017-05-27 02:05:57 +02:00
],
],
'propertyMapHydration' => [
'code' => '<?php
function foo(DOMElement $e) : void {
echo $e->attributes->length;
}',
],
'genericTypeFromPropertyMap' => [
'code' => '<?php
function foo(DOMElement $e) : ?DOMAttr {
return $e->attributes->item(0);
2022-12-18 17:15:15 +01:00
}',
],
'goodArrayProperties' => [
'code' => '<?php
interface I1 {}
class A1 implements I1{}
class B1 implements I1 {}
class C1 {
/** @var array<I1> */
public $is = [];
}
$c = new C1;
$c->is = [new A1];
$c->is = [new A1, new A1];
$c->is = [new A1, new B1];',
'assertions' => [],
'ignored_issues' => ['MixedAssignment'],
],
'issetPropertyDoesNotExist' => [
'code' => '<?php
class A {
}
$a = new A();
if (isset($a->bar)) {
2017-05-27 02:05:57 +02:00
}',
],
'notSetInConstructorButHasDefault' => [
'code' => '<?php
class A {
/** @var int */
public $a = 0;
public function __construct() { }
2017-05-27 02:05:57 +02:00
}',
],
'propertySetInPrivateMethod' => [
'code' => '<?php
class A {
/** @var int */
public $a;
public function __construct() {
$this->foo();
}
2018-01-11 21:50:45 +01:00
private function foo(): void {
$this->a = 5;
}
2017-05-27 02:05:57 +02:00
}',
],
'definedInTraitSetInConstructor' => [
'code' => '<?php
trait A {
/** @var string **/
public $a;
}
class B {
use A;
public function __construct() {
$this->a = "hello";
}
2017-05-27 02:05:57 +02:00
}',
],
'propertySetInNestedPrivateMethod' => [
'code' => '<?php
class A {
/** @var int */
public $a;
public function __construct() {
$this->foo();
}
2018-01-11 21:50:45 +01:00
private function foo(): void {
$this->bar();
}
2018-01-11 21:50:45 +01:00
private function bar(): void {
$this->a = 5;
}
2017-05-27 02:05:57 +02:00
}',
],
'propertyArrayIssetAssertion' => [
'code' => '<?php
2018-01-11 21:50:45 +01:00
function bar(string $s): void { }
class A {
/** @var array<string, string> */
public $a = [];
2018-01-11 21:50:45 +01:00
private function foo(): void {
if (isset($this->a["hello"])) {
bar($this->a["hello"]);
}
}
2017-05-27 02:05:57 +02:00
}',
],
'propertyArrayIssetAssertionWithVariableOffset' => [
'code' => '<?php
2018-01-11 21:50:45 +01:00
function bar(string $s): void { }
class A {
/** @var array<string, string> */
public $a = [];
2018-01-11 21:50:45 +01:00
private function foo(): void {
$b = "hello";
if (!isset($this->a[$b])) {
return;
}
bar($this->a[$b]);
}
2017-05-27 02:05:57 +02:00
}',
],
'staticPropertyArrayIssetAssertionWithVariableOffset' => [
'code' => '<?php
2018-01-11 21:50:45 +01:00
function bar(string $s): void { }
class A {
/** @var array<string, string> */
public static $a = [];
}
2018-01-11 21:50:45 +01:00
function foo(): void {
$b = "hello";
if (!isset(A::$a[$b])) {
return;
}
bar(A::$a[$b]);
2017-05-27 02:05:57 +02:00
}',
],
'staticPropertyArrayIssetAssertionWithVariableOffsetAndElse' => [
'code' => '<?php
2018-01-11 21:50:45 +01:00
function bar(string $s): void { }
class A {
/** @var array<string, string> */
public static $a = [];
}
2018-01-11 21:50:45 +01:00
function foo(): void {
$b = "hello";
if (!isset(A::$a[$b])) {
$g = "bar";
} else {
bar(A::$a[$b]);
$g = "foo";
}
bar($g);
}',
],
'traitConstructor' => [
'code' => '<?php
trait T {
/** @var string **/
public $foo;
public function __construct() {
$this->foo = "hello";
}
}
class A {
use T;
}',
],
'abstractClassWithNoConstructor' => [
'code' => '<?php
abstract class A {
/** @var string */
public $foo;
}',
],
'abstractClassConstructorAndChildConstructor' => [
'code' => '<?php
abstract class A {
/** @var string */
public $foo;
public function __construct() {
$this->foo = "";
}
}
class B extends A {
public function __construct() {
parent::__construct();
}
}',
],
2017-07-09 03:19:16 +02:00
'abstractClassConstructorAndImplicitChildConstructor' => [
'code' => '<?php
abstract class A {
/** @var string */
public $foo;
2017-07-09 03:19:16 +02:00
public function __construct(int $bar) {
$this->foo = (string)$bar;
}
}
2017-07-09 03:19:16 +02:00
class B extends A {}
class E extends \Exception{}',
],
'notSetInEmptyConstructor' => [
'code' => '<?php
/** @psalm-suppress PropertyNotSetInConstructor */
class A {
/** @var int */
public $a;
public function __construct() { }
}',
],
'extendsClassWithPrivateConstructorSet' => [
'code' => '<?php
namespace Q;
class Base
{
/**
* @var string
*/
private $aString;
public function __construct()
{
$this->aString = "aa";
echo($this->aString);
}
}
class Descendant extends Base
{
/**
* @var bool
*/
private $aBool;
public function __construct()
{
parent::__construct();
$this->aBool = true;
}
}',
],
'extendsClassWithPrivateAndException' => [
'code' => '<?php
abstract class A extends \Exception {
/** @var string **/
private $p;
/** @param string $p **/
final public function __construct($p) {
$this->p = $p;
}
}
final class B extends A {}',
],
'setInAbstractMethod' => [
'code' => '<?php
interface I {
2018-01-11 21:50:45 +01:00
public function foo(): void;
}
2022-01-13 20:38:17 +01:00
/** @psalm-suppress PropertyNotSetInConstructor */
abstract class A implements I {
/** @var string */
public $bar;
public function __construct() {
$this->foo();
}
}
class B extends A {
public function foo(): void {
$this->bar = "hello";
}
}',
],
'callsPrivateParentMethodThenUsesParentInitializedProperty' => [
'code' => '<?php
abstract class A {
/** @var string */
public $bar;
public function __construct() {
$this->setBar();
}
private function setBar(): void {
$this->bar = "hello";
}
}
class B extends A {
public function __construct() {
parent::__construct();
echo $this->bar;
}
}',
],
'setInFinalMethod' => [
'code' => '<?php
class C
{
/**
* @var string
*/
private $a;
/**
* @var string
*/
private $b;
/**
* @param string[] $opts
* @psalm-param array{a:string,b:string} $opts
*/
public function __construct(array $opts)
{
$this->setOptions($opts);
}
/**
* @param string[] $opts
* @psalm-param array{a:string,b:string} $opts
*/
final public function setOptions(array $opts): void
{
$this->a = $opts["a"] ?? "defaultA";
$this->b = $opts["b"] ?? "defaultB";
}
}',
],
'setInFinalClass' => [
'code' => '<?php
final class C
{
/**
* @var string
*/
private $a;
/**
* @var string
*/
private $b;
/**
* @param string[] $opts
* @psalm-param array{a:string,b:string} $opts
*/
public function __construct(array $opts)
{
$this->setOptions($opts);
}
/**
* @param string[] $opts
* @psalm-param array{a:string,b:string} $opts
*/
public function setOptions(array $opts): void
{
$this->a = $opts["a"] ?? "defaultA";
$this->b = $opts["b"] ?? "defaultB";
}
}',
],
2017-12-14 04:29:38 +01:00
'selfPropertyType' => [
'code' => '<?php
2017-12-14 04:29:38 +01:00
class Node
{
/** @var self|null */
public $next;
public function __construct() {
if (rand(0, 1)) {
$this->next = new Node();
}
}
}
$node = new Node();
2018-09-17 18:15:45 +02:00
$next = $node->next;',
2017-12-14 04:29:38 +01:00
'assertions' => [
'$next' => 'Node|null',
2017-12-14 04:29:38 +01:00
],
],
'perPropertySuppress' => [
'code' => '<?php
class A {
/**
* @var int
* @psalm-suppress PropertyNotSetInConstructor
*/
public $a;
public function __construct() { }
}',
],
'analyzePropertyMappedClass' => [
'code' => '<?php
namespace PhpParser\Node\Stmt;
use PhpParser\Node;
class Finally_ extends Node\Stmt
{
2021-03-23 06:30:51 +01:00
/** @var list<Node\Stmt> Statements */
public $stmts;
/**
* Constructs a finally node.
*
2021-03-23 06:30:51 +01:00
* @param list<Node\Stmt> $stmts Statements
* @param array $attributes Additional attributes
*/
public function __construct(array $stmts = array(), array $attributes = array()) {
parent::__construct($attributes);
$this->stmts = $stmts;
}
public function getSubNodeNames() : array {
return array("stmts");
}
public function getType() : string {
return "Stmt_Finally";
}
}',
'assertions' => [],
],
'privatePropertyAccessible' => [
'code' => '<?php
class A {
/** @var string */
private $foo;
public function __construct(string $foo) {
$this->foo = $foo;
}
private function bar() : void {}
}
class B extends A {
/** @var string */
private $foo;
public function __construct(string $foo) {
$this->foo = $foo;
parent::__construct($foo);
}
}',
],
'privatePropertyAccessibleDifferentType' => [
'code' => '<?php
class A {
/** @var int */
private $foo;
public function __construct(string $foo) {
$this->foo = 5;
}
private function bar() : void {}
}
class B extends A {
/** @var string */
private $foo;
public function __construct(string $foo) {
$this->foo = $foo;
parent::__construct($foo);
}
}',
],
'privatePropertyAccessibleInTwoSubclasses' => [
'code' => '<?php
class A {
public function __construct() {}
}
class B extends A {
/**
* @var int
*/
private $prop;
public function __construct()
{
parent::__construct();
$this->prop = 1;
}
}
class C extends A {
/**
* @var int
*/
private $prop;
public function __construct()
{
parent::__construct();
$this->prop = 2;
}
}',
],
'noIssueWhenSuppressingMixedAssignmentForProperty' => [
'code' => '<?php
class A {
/** @var string|null */
public $foo;
/** @param mixed $a */
public function barBar($a): void
{
$this->foo = $a;
}
}',
'assertions' => [],
'ignored_issues' => [
'MixedAssignment',
],
],
'propertyAssignmentToMixed' => [
'code' => '<?php
class C {
/** @var string|null */
public $foo;
}
/** @param mixed $a */
function barBar(C $c, $a): void
{
$c->foo = $a;
}',
'assertions' => [],
'ignored_issues' => [
'MixedAssignment',
],
],
'propertySetInBothIfBranches' => [
'code' => '<?php
class Foo
{
/** @var int */
private $status;
public function __construct(int $in)
{
if (rand(0, 1)) {
$this->status = 1;
} else {
$this->status = $in;
}
}
}',
],
'propertySetInPrivateMethodWithIfAndElse' => [
'code' => '<?php
class A {
/** @var int */
public $a;
public function __construct() {
if (rand(0, 1)) {
$this->foo();
} else {
$this->bar();
}
}
private function foo(): void {
$this->a = 5;
}
private function bar(): void {
$this->a = 5;
}
}',
],
'allowMixedAssignmetWhenDesired' => [
'code' => '<?php
class A {
/**
* @var mixed
*/
private $mixed = "hello";
/**
* @param mixed $value
*/
public function setMixed($value): void
{
$this->mixed = $value;
}
}',
],
'suppressUndefinedThisPropertyFetch' => [
'code' => '<?php
class A {
public function __construct() {
/** @psalm-suppress UndefinedThisPropertyAssignment */
$this->bar = rand(0, 1) ? "hello" : null;
}
/** @psalm-suppress UndefinedThisPropertyFetch */
public function foo() : void {
if ($this->bar === null && rand(0, 1)) {}
}
}',
],
'suppressUndefinedPropertyFetch' => [
'code' => '<?php
class A {
public function __construct() {
/** @psalm-suppress UndefinedThisPropertyAssignment */
$this->bar = rand(0, 1) ? "hello" : null;
}
}
$a = new A();
/** @psalm-suppress UndefinedPropertyFetch */
if ($a->bar === null && rand(0, 1)) {}',
],
'setPropertiesOfSpecialObjects' => [
'code' => '<?php
$a = new stdClass();
$a->b = "c";
$d = new SimpleXMLElement("<person><child role=\"son\"></child></person>");
$d->e = "f";',
'assertions' => [
'$a' => 'stdClass',
'$a->b' => 'string',
'$d' => 'SimpleXMLElement',
'$d->e' => 'mixed',
],
],
'allowLessSpecificReturnTypeForOverriddenMethod' => [
'code' => '<?php
class A {
public function aa(): ?string {
return "bar";
}
}
class B extends A {
public static function aa(): ?string {
return rand(0, 1) ? "bar" : null;
}
}
class C extends A {
public static function aa(): ?string {
return "bar";
}
2019-03-23 19:27:54 +01:00
}',
],
'allowLessSpecificReturnTypeForInterfaceMethod' => [
'code' => '<?php
interface Foo {
public static function foo(): ?string;
}
class Bar implements Foo {
public static function foo(): ?string
{
return "bar";
}
}
class Baz implements Foo {
/**
* @return string $baz
*/
public static function foo(): ?string
{
return "baz";
}
}
class Bax implements Foo {
/**
* @return null|string $baz
*/
public static function foo(): ?string
{
return "bax";
}
}
class Baw implements Foo {
/**
* @return null|string $baz
*/
public static function foo(): ?string
{
/** @var null|string $val */
$val = "baw";
return $val;
}
}',
],
'staticPropertyMethodCall' => [
'code' => '<?php
class A {
/** @var self|null */
public static $instance;
/** @var string|null */
public $bat;
public function foo() : void {
if (self::$instance) {
self::$instance->bar();
echo self::$instance->bat;
}
}
public function bar() : void {}
}
if (A::$instance) {
A::$instance->bar();
echo A::$instance->bat;
}',
],
'nonStaticPropertyMethodCall' => [
'code' => '<?php
class A {
/** @var self|null */
public $instance;
/** @var string|null */
public $bat;
public function foo() : void {
if ($this->instance) {
$this->instance->bar();
echo $this->instance->bat;
}
}
public function bar() : void {}
}
$a = new A();
if ($a->instance) {
$a->instance->bar();
echo $a->instance->bat;
2019-03-23 19:27:54 +01:00
}',
],
'staticPropertyOfStaticTypeMethodCall' => [
'code' => '<?php
class A {
/** @var static|null */
public $instance;
}
class B extends A {
/** @var string|null */
public $bat;
public function foo() : void {
if ($this->instance) {
$this->instance->bar();
echo $this->instance->bat;
}
}
public function bar() : void {}
2019-03-23 19:27:54 +01:00
}',
],
'classStringPropertyType' => [
'code' => '<?php
class C {
/** @psalm-var array<class-string, int> */
public $member = [
InvalidArgumentException::class => 1,
];
2019-03-23 19:27:54 +01:00
}',
],
'allowPrivatePropertySetAfterInstanceof' => [
'code' => '<?php
class A {
/** @var string|null */
private $foo;
public function bar() : void {
if (!$this instanceof B) {
return;
}
$this->foo = "hello";
}
}
class B extends A {}',
],
2019-01-09 14:35:53 +01:00
'noCrashForAbstractConstructorWithInstanceofInterface' => [
'code' => '<?php
2019-01-09 14:35:53 +01:00
abstract class A {
/** @var int */
public $a;
public function __construct() {
if ($this instanceof I) {
$this->a = $this->bar();
} else {
$this->a = 6;
}
}
}
interface I {
public function bar() : int;
}',
],
'SKIPPED-abstractConstructorWithInstanceofClass' => [
'code' => '<?php
2019-01-09 14:35:53 +01:00
abstract class A {
/** @var int */
public $a;
public function __construct() {
if ($this instanceof B) {
$this->a = $this->bar();
} else {
$this->a = 6;
}
}
}
class B extends A {
public function bar() : int {
return 3;
}
}',
'assertions' => [],
'ignored_issues' => [],
2019-01-09 14:35:53 +01:00
],
'inheritDocPropertyTypes' => [
'code' => '<?php
class X {
/**
* @var string|null
*/
public $a;
/**
* @var string|null
*/
public static $b;
}
class Y extends X {
public $a = "foo";
public static $b = "foo";
}
(new Y)->a = "hello";
echo (new Y)->a;
Y::$b = "bar";
echo Y::$b;',
],
'subclassPropertySetInParentConstructor' => [
'code' => '<?php
class Base {
/** @var string */
protected $prop;
public function __construct(string $s) {
$this->prop = $s;
}
}
class Child extends Base {
/** @var string */
protected $prop;
}',
],
2019-01-31 23:48:48 +01:00
'callInParentContext' => [
'code' => '<?php
2019-01-31 23:48:48 +01:00
class A {
2019-02-01 00:05:21 +01:00
/** @var int */
public $i = 1;
2019-01-31 23:48:48 +01:00
}
abstract class B
{
/**
* @var string
*/
protected $foo;
/**
* @var A[]
*/
private $as = [];
public function __construct()
{
$this->foo = "";
$this->bar();
}
public function bar(): void
{
\usort($this->as, function (A $a, A $b): int {
return $b->i <=> $a->i;
});
}
}
class C extends B
{
public function __construct()
{
parent::__construct();
}
}',
],
'staticVarSelf' => [
'code' => '<?php
class Foo {
/** @var self */
public static $current;
}
$a = Foo::$current;',
'assertions' => [
'$a' => 'Foo',
2019-03-23 19:27:54 +01:00
],
],
'noMixedErrorWhenAssignmentExpectsMixed' => [
'code' => '<?php
class A {
/** @var array<string, mixed> $bar */
public $bar = [];
/** @param mixed $b */
public function foo($b) : void {
$this->bar["a"] = $b;
}
2019-03-23 19:27:54 +01:00
}',
],
'propertySetInGrandparentExplicitly' => [
'code' => '<?php
class A {
/**
* @var string
*/
public $s;
public function __construct(string $s) {
$this->s = $s;
}
}
class B extends A {}
class C extends B {
public function __construct(string $s) {
A::__construct($s);
}
2019-03-23 19:27:54 +01:00
}',
],
'propertySetInGrandparentImplicitly' => [
'code' => '<?php
class A {
/**
* @var string
*/
public $s;
public function __construct(string $s) {
$this->s = $s;
}
}
class B extends A {}
2019-03-23 19:27:54 +01:00
class C extends B {}',
],
'unitializedPropertySuppressPropertyNotSetInConstructor' => [
'code' => '<?php
class A {
/** @var string */
public $foo;
public function __construct() {
$this->setFoo(); // public method that circumvents checks
echo strlen($this->foo);
}
public function setFoo() : void {
$this->foo = "foo";
}
}',
'assertions' => [],
'ignored_issues' => ['PropertyNotSetInConstructor'],
],
'unitializedPropertySuppressPropertyNotSetInAbstractConstructor' => [
'code' => '<?php
abstract class A {
/** @readonly */
public string $s;
abstract public function __construct(string $s);
}',
],
'setTKeyedArrayPropertyType' => [
'code' => '<?php
class Foo {
/**
* @psalm-var array{from:bool, to:bool}
*/
protected $changed = [
"from" => false,
"to" => false,
];
/**
* @psalm-param "from"|"to" $property
*/
public function ChangeThing(string $property) : void {
$this->changed[$property] = true;
}
2019-03-23 19:27:54 +01:00
}',
],
'noRedundantConditionWhenCheckingInitializations' => [
'code' => '<?php
final class Clazz {
/**
* @var bool
*/
public $x;
/**
* @var int
*/
public $y = 0;
public function func1 (): bool {
if ($this->y) {
return true;
}
return false;
}
public function func2 (): int {
if ($this->y) {
return 1;
}
return 2;
}
2019-02-19 03:31:08 +01:00
public function __construct () {
$this->x = false;
if ($this->func1()) {
$this->y = $this->func2();
}
$this->func2();
}
2019-03-23 19:27:54 +01:00
}',
2019-02-19 03:31:08 +01:00
],
'noRedundantConditionWhenCheckingInitializationsEdgeCases' => [
'code' => '<?php
2019-02-19 03:31:08 +01:00
final class Clazz {
/**
* @var bool
*/
public $x;
/**
* @var int
*/
public $y = 0;
public function func1 (): bool {
if ($this->y !== 0) {
return true;
}
return false;
}
public function func2 (): int {
if ($this->y !== 0) {
return $this->y;
}
return 2;
}
public function __construct () {
$this->x = false;
if ($this->func1()) {
$this->y = $this->func2();
}
$this->func2();
}
2019-03-23 19:27:54 +01:00
}',
],
'propertySetInProtectedMethodWithConstant' => [
'code' => '<?php
class A {
/** @var int */
public $a;
public function __construct() {
$this->foo();
}
protected function foo(): void {
$this->a = 5;
}
}
class B extends A {
const HELLO = "HELLO";
protected function foo() : void {
$this->a = 6;
echo self::HELLO;
}
}',
],
'setPropertyInParentProtectedMethodExplicitCall' => [
'code' => '<?php
abstract class A {
public function __construct() {
$this->overriddenByB();
}
protected function overriddenByB(): void {
// do nothing
}
}
class B extends A {
/** @var int */
private $foo;
/** @var int */
protected $bar;
public function __construct() {
parent::__construct();
}
protected final function overriddenByB(): void {
$this->foo = 1;
$this->bar = 1;
}
}',
],
'setPropertyInParentProtectedMethodImplicitCall' => [
'code' => '<?php
abstract class A {
public function __construct() {
$this->overriddenByB();
}
protected function overriddenByB(): void {
// do nothing
}
}
class B extends A {
/** @var int */
private $foo;
/** @var int */
protected $bar;
protected final function overriddenByB(): void {
$this->foo = 1;
$this->bar = 1;
}
}',
],
'setPropertyInParentWithPrivateConstructor' => [
'code' => '<?php
namespace NS;
class Base
{
/**
* @var int
*/
protected $a;
final private function __construct()
{
$this->setA();
}
private function setA() : void {
$this->a = 5;
}
public static function getInstance(): self { return new static; }
}
2019-07-05 22:24:00 +02:00
class Concrete extends Base {}',
],
'preventCrashWhenCallingInternalMethodInPropertyInitialisationChecks' => [
'code' => '<?php
class Foo extends \RuntimeException {
/** @var array */
protected $serializableTrace;
public function __construct() {
parent::__construct("hello", 0);
$this->serializableTrace = $this->getTrace();
}
}
class Bar extends Foo {}',
],
'inferPropertyTypesForSimpleConstructors' => [
'code' => '<?php
class A {
private $foo;
private $bar;
public function __construct(int $foot, string $bart) {
$this->foo = $foot;
$this->bar = $bart;
}
public function getFoo() : int {
return $this->foo;
}
public function getBar() : string {
return $this->bar;
}
}',
],
'nullableDocblockTypedPropertyNoConstructor' => [
'code' => '<?php
class A {
/** @var ?bool */
private $foo;
}',
],
'nullableDocblockTypedPropertyEmptyConstructor' => [
'code' => '<?php
class A {
/** @var ?bool */
private $foo;
public function __construct() {}
}',
],
'nullableDocblockTypedPropertyUseBeforeInitialised' => [
'code' => '<?php
class A {
/** @var ?bool */
private $foo;
public function __construct() {
echo $this->foo;
}
}',
],
'dontAlterClosureParams' => [
'code' => '<?php
class C {
/** @var array */
public $i;
public function __construct() {
$this->i = [
function (Exception $e): void {},
function (LogicException $e): void {},
];
}
2019-07-05 22:24:00 +02:00
}',
],
'inferSpreadParamType' => [
'code' => '<?php
class Tag {}
class EntityTags {
private $tags;
/** @no-named-arguments */
public function __construct(Tag ...$tags) {
$this->tags = $tags;
}
2019-07-05 22:24:00 +02:00
}',
],
'staticPropertyDefaultWithStaticType' => [
'code' => '<?php
class Test {
/** @var array<int, static> */
private static $t1 = [];
/** @var array<int, static> */
private $t2 = [];
2022-12-18 17:15:15 +01:00
}',
],
'propagateIgnoreNullableOnPropertyFetch' => [
'code' => '<?php
class A {
public string $s = "hey";
}
/**
* @psalm-ignore-nullable-return
*/
function foo() : ?A {
return rand(0, 1) ? new A : null;
}
function takesString(string $_s) : void {}
$foo = foo();
if ($foo->s !== null) {}
echo $foo->s ?? "bar";
takesString($foo->s);',
],
'noMissingPropertyWhenArrayTypeProvided' => [
'code' => '<?php
class Foo {
private $bar;
/** @psalm-param array{key: string} $bar */
public function __construct(array $bar) {
$this->bar = $bar;
}
public function bar(): void {
echo $this->bar["key"];
}
}',
],
'rememberThisPropertyAsssignmentsInMethod' => [
'code' => '<?php
class A {
public bool $foo = false;
public function bar() : void {
$this->foo = false;
$this->maybeChange();
if ($this->foo) {}
}
public function maybeChange() : void {
if (rand(0, 1)) {
$this->foo = true;
}
}
2022-12-18 17:15:15 +01:00
}',
],
'testRemoveClauseAfterReassignment' => [
'code' => '<?php
class Test {
/** @var ?bool */
private $foo;
public function run(): void {
$this->foo = false;
$this->bar();
if ($this->foo === true) {}
}
private function bar(): void {
if (mt_rand(0, 1)) {
$this->foo = true;
}
}
}',
],
'allowIssetOnTypedProperty' => [
'code' => '<?php
class A {
public string $a;
public function __construct(bool $b) {
if ($b) {
$this->a = "hello";
}
if (isset($this->a)) {
echo $this->a;
$this->a = "bello";
}
$this->a = "bar";
}
2022-12-18 17:15:15 +01:00
}',
],
'allowGoodArrayPushOnArrayValue' => [
'code' => '<?php
class MyClass {
/**
* @var int[]
*/
private $prop = [];
/**
* @return void
*/
public function foo() {
array_push($this->prop, 5);
}
}',
],
'someConditionalCallToParentConstructor' => [
'code' => '<?php
class GrandParentClassDoesNotDefine {
public function __construct() {}
}
class ParentClassDefinesVar extends GrandParentClassDoesNotDefine {
protected string $val;
public function __construct() {
$this->val = "hello";
if (rand(0, 1)) {
parent::__construct();
}
}
}
class ChildClass extends ParentClassDefinesVar {
public function __construct() {
parent::__construct();
}
2022-12-18 17:15:15 +01:00
}',
],
'noConditionalCallToParentConstructor' => [
'code' => '<?php
class GrandParentClassDoesNotDefine {
public function __construct() {}
}
class ParentClassDefinesVar extends GrandParentClassDoesNotDefine {
protected string $val;
public function __construct() {
$this->val = "hello";
parent::__construct();
}
}
class ChildClass extends ParentClassDefinesVar {
public function __construct() {
parent::__construct();
}
2022-12-18 17:15:15 +01:00
}',
],
'allowByReferenceAssignmentToUninitializedNullableProperty' => [
'code' => '<?php
class C {
private ?\Closure $onCancel;
public function __construct() {
$this->foo($this->onCancel);
}
/**
* @param mixed $onCancel
* @param-out \Closure $onCancel
*/
public function foo(&$onCancel) : void {
$onCancel = function (): void {};
}
2022-12-18 17:15:15 +01:00
}',
],
'dontCarryAssertionsOver' => [
'code' => '<?php
class A
{
private string $network;
public function __construct(string $s)
{
$this->network = $s;
$this->firstCheck();
$this->secondCheck();
}
public function firstCheck(): void
{
if ($this->network === "x") {
return;
}
}
public function secondCheck(): void
{
if ($this->network === "x") {
return;
}
}
2022-12-18 17:15:15 +01:00
}',
],
'useVariableAccessInStatic' => [
'code' => '<?php
class A2 {
public static string $title = "foo";
public static string $label = "bar";
}
$model = new A2();
$message = $model::$title;
$message .= $model::$label;
2022-12-18 17:15:15 +01:00
echo $message;',
],
'staticPropertyInFinalMethod' => [
'code' => '<?php
abstract class Foo {
/** @var static */
protected Foo $foo;
}
final class Bar extends Foo {
public function __construct(Bar $bar) {
$this->foo = $bar;
}
public function baz(): Bar {
return $this->foo;
}
2022-12-18 17:15:15 +01:00
}',
],
'aliasedFinalMethod' => [
'code' => '<?php
trait A {
private int $prop;
public final function setProp(int $prop): void {
$this->prop = $prop;
}
}
class B {
use A {
setProp as setPropFinal;
}
public function __construct() {
$this->setPropFinal(1);
}
2022-12-18 17:15:15 +01:00
}',
],
'aliasedAsFinalMethod' => [
'code' => '<?php
trait A {
private int $prop;
public function setProp(int $prop): void {
$this->prop = $prop;
}
}
class B {
use A {
setProp as final setPropFinal;
}
public function __construct() {
$this->setPropFinal(1);
}
2022-12-18 17:15:15 +01:00
}',
],
'staticPropertyAssertion' => [
'code' => '<?php
class Foo {
/** @var int */
private static $transactionDepth;
function bar(): void {
if (self::$transactionDepth === 0) {
} else {
--self::$transactionDepth;
if (self::$transactionDepth === 0) {
}
}
}
2022-12-18 17:15:15 +01:00
}',
],
'dontMemoizePropertyTypeAfterRootVarAssertion' => [
'code' => '<?php
class A {
public string $i = "";
}
class B {
public int $i = 0;
}
/** @param A|B $o */
function takesAorB(object $o) : void {
echo $o->i;
if ($o instanceof A) {
echo strlen($o->i);
}
2022-12-18 17:15:15 +01:00
}',
],
2020-08-30 19:16:37 +02:00
'unionPropertyType' => [
'code' => '<?php
2020-08-30 19:16:37 +02:00
class A {
public string|int $i;
public function __construct() {
$this->i = 5;
$this->i = "hello";
}
}
$a = new A();
if ($a->i === 3) {}
2022-01-22 01:00:45 +01:00
if ($a->i === "foo") {}',
'assertions' => [],
'ignored_issues' => [],
'php_version' => '8.0',
2020-08-30 19:16:37 +02:00
],
'setClassStringOfStatic' => [
'code' => '<?php
class A {
public static array $stack = [];
public static function foo() : void {
$class = get_called_class();
$class::$stack[] = 1;
}
2022-12-18 17:15:15 +01:00
}',
],
'promotedPublicPropertyWithDefault' => [
'code' => '<?php
class A {
public function __construct(public int $foo = 5) {}
}
2022-12-18 17:15:15 +01:00
echo (new A)->foo;',
],
'promotedPublicPropertyWitoutDefault' => [
'code' => '<?php
class A {
public function __construct(public int $foo) {}
}
2022-12-18 17:15:15 +01:00
echo (new A(5))->foo;',
],
'promotedProtectedProperty' => [
'code' => '<?php
class A {
public function __construct(protected int $foo) {}
}
class AChild extends A {
public function bar() : int {
return $this->foo;
}
2022-12-18 17:15:15 +01:00
}',
],
'skipConstructor' => [
'code' => '<?php
class A {
protected string $s;
public function __construct() {
$this->s = "hello";
}
}
class B extends A {}
class C extends B {
public function __construct()
{
parent::__construct();
echo $this->s;
}
2022-12-18 17:15:15 +01:00
}',
],
2020-11-29 15:26:39 +01:00
'getPropertyThatMayNotBeSet' => [
'code' => '<?php
2020-11-29 15:26:39 +01:00
/**
* @psalm-suppress MissingConstructor
*/
class A {
public string $bar;
public function getBar() : string {
/** @psalm-suppress RedundantPropertyInitializationCheck */
2020-11-29 15:26:39 +01:00
if (!isset($this->bar)) {
return "hello";
}
return $this->bar;
}
public function getBarAgain() : string {
/** @psalm-suppress RedundantPropertyInitializationCheck */
2020-11-29 15:26:39 +01:00
if (isset($this->bar)) {
return $this->bar;
}
return "hello";
}
}',
],
2020-12-01 23:25:45 +01:00
'memoizePropertyAfterSetting' => [
'code' => '<?php
2020-12-01 23:25:45 +01:00
class A {
public function foo() : void {
/** @psalm-suppress UndefinedThisPropertyAssignment */
$this->b = "c";
echo strlen($this->b);
}
2022-12-18 17:15:15 +01:00
}',
2020-12-01 23:25:45 +01:00
],
'noErrorForSplatArgs' => [
'code' => '<?php
class Foo {
protected array $b;
protected function __construct(?string ...$bb) {
$this->b = $bb;
}
}
2022-12-18 17:15:15 +01:00
class Bar extends Foo {}',
],
2020-12-03 17:13:10 +01:00
'noUndefinedPropertyIssueAfterSuppressingOnInterface' => [
'code' => '<?php
2020-12-03 17:13:10 +01:00
interface I {}
function bar(I $i) : void {
/**
* @psalm-suppress NoInterfaceProperties
* @psalm-suppress MixedArgument
*/
echo $i->foo;
2022-12-18 17:15:15 +01:00
}',
2020-12-03 17:13:10 +01:00
],
'noRedundantCastWhenCheckingProperties' => [
'code' => '<?php
class Foo
{
public array $map;
public function __construct()
{
$this->map = [];
$this->map["test"] = "test";
$this->useMap();
}
public function useMap(): void
{
$keys = array_keys($this->map);
$key = reset($keys);
echo (string) $key;
}
2022-12-18 17:15:15 +01:00
}',
],
2020-12-10 07:29:07 +01:00
'ignoreUndefinedMethodOnUnion' => [
'code' => '<?php
2020-12-10 07:29:07 +01:00
class NullObject {
/**
* @return null
*/
public function __get(string $s) {
return null;
}
}
class User {
public string $name = "Dave";
}
2020-12-10 07:40:20 +01:00
function takesNullableUser(User|NullObject $user) : ?string {
$name = $user->name;
if ($name === null) {}
return $name;
}',
'assertions' => [],
'ignored_issues' => [],
2022-12-18 17:15:15 +01:00
'php_version' => '8.0',
2020-12-10 07:29:07 +01:00
],
'dynamicPropertyFetch' => [
'code' => '<?php
class Foo {
public int $a = 0;
}
function takesFoo(?Foo $foo, string $b) : void {
2021-04-25 21:22:32 +02:00
/** @psalm-suppress MixedArgument */
echo $foo->{$b} ?? null;
2022-12-18 17:15:15 +01:00
}',
],
'nullCoalesceWithNullablePropertyAccess' => [
'code' => '<?php
class Bar {
public ?string $a = null;
}
function takesBar(?Bar $bar) : string {
return $bar?->a ?? "default";
}',
'assertions' => [],
'ignored_issues' => [],
2022-12-18 17:15:15 +01:00
'php_version' => '8.0',
],
2021-02-08 04:56:37 +01:00
'possiblyNullOnFunctionCallCoalesced' => [
'code' => '<?php
2021-02-08 04:56:37 +01:00
class Foo
{
/** @var int */
public $a = 0;
}
function accessOnVar(?Foo $bar, string $b) : void {
2021-04-25 21:22:32 +02:00
/** @psalm-suppress MixedArgument */
2021-02-08 04:56:37 +01:00
echo $bar->{$b} ?? null;
}',
],
'dontMemoizeConditionalAssignment' => [
'code' => '<?php
class A {}
class B {
protected ?A $a = null;
public function test(): void {
if (!$this->a) {
$this->mayBeSetA();
}
if ($this->a instanceof A) {
}
}
protected function mayBeSetA(): void {
if (mt_rand(0, 1)) {
$this->a = new A();
}
}
2022-12-18 17:15:15 +01:00
}',
],
'allowDefaultForTemplatedProperty' => [
'code' => '<?php
/**
* @template T as string|null
*/
abstract class A {
/** @var list<T> */
public $foo = [];
}
/**
* @extends A<string>
*/
class AChild extends A {
public $foo = ["hello"];
2022-12-18 17:15:15 +01:00
}',
],
'allowBuiltinPropertyDocblock' => [
'code' => '<?php
class FooException extends LogicException {
/** @var int */
protected $code = 404;
2022-12-18 17:15:15 +01:00
}',
],
'dontMemoizeFinalMutationFreeInferredMethod' => [
'code' => '<?php
final class ExecutionMode
{
private bool $isAutoCommitEnabled = true;
public function enableAutoCommit(): void
{
$this->isAutoCommitEnabled = true;
}
public function disableAutoCommit(): void
{
$this->isAutoCommitEnabled = false;
}
public function isAutoCommitEnabled(): bool
{
return $this->isAutoCommitEnabled;
}
}
$mode = new ExecutionMode();
$mode->disableAutoCommit();
assert($mode->isAutoCommitEnabled() === false);
$mode->enableAutoCommit();
2022-12-18 17:15:15 +01:00
assert($mode->isAutoCommitEnabled() === true);',
],
'promotedInheritedPropertyWithDocblock' => [
'code' => '<?php
abstract class A {
/** @var array */
public array $arr;
}
final class B extends A {
/** @param array $arr */
protected function __construct(public array $arr){}
2022-12-18 17:15:15 +01:00
}',
],
'nullsafeShortCircuit' => [
'code' => '<?php
class Foo {
private ?self $nullableSelf = null;
public function __construct(private self $self) {}
public function doBar(): ?self
{
return $this->nullableSelf?->self->self;
}
}',
'assertions' => [],
'ignored_issues' => [],
2022-12-18 17:15:15 +01:00
'php_version' => '8.0',
],
2022-12-04 20:31:56 +01:00
'impossibleIntersection' => [
'code' => '<?php
class Foo {}
class Bar {}
/** @psalm-suppress MissingConstructor */
class Baz
{
private Foo&Bar $foobar;
}
',
'assertions' => [],
'ignored_issues' => [],
2022-12-18 17:15:15 +01:00
'php_version' => '8.1',
2022-12-04 20:31:56 +01:00
],
2023-04-21 12:33:08 +02:00
'intersectionPropertyAccess' => [
'code' => '<?php
/** @property int $test1 */
class a {
public function __get(string $name)
{
return 0;
}
}
/** @var a&object{test2: "lmao"} */
$r = null;
$test1 = $r->test1;
$test2 = $r->test2;',
'assertions' => [
'$test1===' => 'int',
'$test2===' => "'lmao'",
],
],
];
}
public function providerInvalidCodeParse(): iterable
{
return [
'undefinedPropertyAssignment' => [
'code' => '<?php
class A {
}
(new A)->foo = "cool";',
2017-05-27 02:05:57 +02:00
'error_message' => 'UndefinedPropertyAssignment',
],
'undefinedPropertyFetch' => [
'code' => '<?php
class A {
}
echo (new A)->foo;',
2017-05-27 02:05:57 +02:00
'error_message' => 'UndefinedPropertyFetch',
],
'undefinedThisPropertyAssignment' => [
'code' => '<?php
class A {
2018-01-11 21:50:45 +01:00
public function fooFoo(): void {
$this->foo = "cool";
}
}',
2017-05-27 02:05:57 +02:00
'error_message' => 'UndefinedThisPropertyAssignment',
],
'undefinedStaticPropertyAssignment' => [
'code' => '<?php
class A {
public static function barBar(): void
{
/** @psalm-suppress UndefinedPropertyFetch */
self::$foo = 5;
}
}',
'error_message' => 'UndefinedPropertyAssignment',
],
'undefinedThisPropertyFetch' => [
'code' => '<?php
class A {
2018-01-11 21:50:45 +01:00
public function fooFoo(): void {
echo $this->foo;
}
}',
2017-05-27 02:05:57 +02:00
'error_message' => 'UndefinedThisPropertyFetch',
],
'missingPropertyType' => [
'code' => '<?php
class A {
public $foo = null;
2018-01-11 21:50:45 +01:00
public function assignToFoo(): void {
$this->foo = 5;
}
}',
2019-02-27 22:00:44 +01:00
'error_message' => 'MissingPropertyType - src' . DIRECTORY_SEPARATOR . 'somefile.php:3:32 - Property A::$foo does not have a ' .
'declared type - consider int|null',
],
'missingPropertyTypeWithConstructorInit' => [
'code' => '<?php
class A {
public $foo;
2018-04-21 23:20:21 +02:00
public function __construct() {
$this->foo = 5;
}
}',
2019-02-27 22:00:44 +01:00
'error_message' => 'MissingPropertyType - src' . DIRECTORY_SEPARATOR . 'somefile.php:3:32 - Property A::$foo does not have a ' .
2017-05-27 02:05:57 +02:00
'declared type - consider int',
],
'missingPropertyTypeWithConstructorInitAndNull' => [
'code' => '<?php
class A {
public $foo;
2018-04-21 23:20:21 +02:00
public function __construct() {
$this->foo = 5;
}
2018-01-11 21:50:45 +01:00
public function makeNull(): void {
$this->foo = null;
}
}',
2019-02-27 22:00:44 +01:00
'error_message' => 'MissingPropertyType - src' . DIRECTORY_SEPARATOR . 'somefile.php:3:32 - Property A::$foo does not have a ' .
'declared type - consider int|null',
],
'missingPropertyTypeWithConstructorInitAndNullDefault' => [
'code' => '<?php
class A {
public $foo = null;
2018-04-21 23:20:21 +02:00
public function __construct() {
$this->foo = 5;
}
}',
2019-02-27 22:00:44 +01:00
'error_message' => 'MissingPropertyType - src' . DIRECTORY_SEPARATOR . 'somefile.php:3:32 - Property A::$foo does not have a ' .
2017-05-27 02:05:57 +02:00
'declared type - consider int|null',
],
'missingPropertyTypeWithConstructorInitConditionallySet' => [
'code' => '<?php
class A {
public $foo;
public function __construct() {
if (rand(0, 1)) {
$this->foo = 5;
}
}
}',
'error_message' => 'MissingPropertyType - src' . DIRECTORY_SEPARATOR . 'somefile.php:3:32 - Property A::$foo does not have a ' .
'declared type - consider int|null',
],
'badAssignment' => [
'code' => '<?php
class A {
/** @var string */
public $foo;
2018-01-11 21:50:45 +01:00
public function barBar(): void
{
$this->foo = 5;
}
}',
'error_message' => 'InvalidPropertyAssignmentValue',
],
'badStaticAssignment' => [
'code' => '<?php
class A {
/** @var string */
public static $foo = "a";
public static function barBar(): void
{
self::$foo = 5;
}
}',
'error_message' => 'InvalidPropertyAssignmentValue',
],
'typeCoercion' => [
'code' => '<?php
class A {
/** @var B|null */
public $foo;
public function barBar(A $a): void
{
$this->foo = $a;
}
}
class B extends A {}',
'error_message' => 'PropertyTypeCoercion',
],
'mixedTypeCoercion' => [
'code' => '<?php
class A {
/** @var array<int, A> */
public $foo = [];
/** @param A[] $arr */
public function barBar(array $arr): void
{
$this->foo = $arr;
}
}',
'error_message' => 'MixedPropertyTypeCoercion',
],
'staticTypeCoercion' => [
'code' => '<?php
class A {
/** @var B|null */
public static $foo;
public static function barBar(A $a): void
{
self::$foo = $a;
}
}
class B extends A {}',
'error_message' => 'PropertyTypeCoercion',
],
'staticMixedTypeCoercion' => [
'code' => '<?php
class A {
/** @var array<int, A> */
public static $foo = [];
/** @param A[] $arr */
public static function barBar(array $arr): void
{
self::$foo = $arr;
}
}',
'error_message' => 'MixedPropertyTypeCoercion',
],
2018-03-21 04:55:26 +01:00
'possiblyBadAssignment' => [
'code' => '<?php
2018-03-21 04:55:26 +01:00
class A {
/** @var string */
public $foo;
public function barBar(): void
{
$this->foo = rand(0, 1) ? 5 : "hello";
}
}',
'error_message' => 'PossiblyInvalidPropertyAssignmentValue',
],
'possiblyBadStaticAssignment' => [
'code' => '<?php
class A {
/** @var string */
public static $foo = "a";
public function barBar(): void
{
self::$foo = rand(0, 1) ? 5 : "hello";
}
}',
'error_message' => 'PossiblyInvalidPropertyAssignmentValue',
],
'badAssignmentAsWell' => [
'code' => '<?php
$a = "hello";
$a->foo = "bar";',
2017-05-27 02:05:57 +02:00
'error_message' => 'InvalidPropertyAssignment',
],
'badFetch' => [
'code' => '<?php
$a = "hello";
echo $a->foo;',
2017-05-27 02:05:57 +02:00
'error_message' => 'InvalidPropertyFetch',
],
'possiblyBadFetch' => [
'code' => '<?php
$a = rand(0, 5) > 3 ? "hello" : new stdClass;
echo $a->foo;',
'error_message' => 'PossiblyInvalidPropertyFetch',
],
'mixedPropertyFetch' => [
'code' => '<?php
class Foo {
/** @var string */
public $foo = "";
}
/** @var mixed */
$a = (new Foo());
echo $a->foo;',
'error_message' => 'MixedPropertyFetch',
'ignored_issues' => [
'MissingPropertyType',
2017-05-27 02:05:57 +02:00
'MixedAssignment',
],
],
'mixedPropertyAssignment' => [
'code' => '<?php
class Foo {
/** @var string */
public $foo = "";
}
/** @var mixed */
$a = (new Foo());
$a->foo = "hello";',
'error_message' => 'MixedPropertyAssignment',
'ignored_issues' => [
'MissingPropertyType',
2017-05-27 02:05:57 +02:00
'MixedAssignment',
],
],
'possiblyNullablePropertyAssignment' => [
'code' => '<?php
class Foo {
/** @var string */
public $foo = "";
}
$a = rand(0, 10) ? new Foo() : null;
$a->foo = "hello";',
2017-05-27 02:05:57 +02:00
'error_message' => 'PossiblyNullPropertyAssignment',
],
'nullablePropertyAssignment' => [
'code' => '<?php
$a = null;
$a->foo = "hello";',
2017-05-27 02:05:57 +02:00
'error_message' => 'NullPropertyAssignment',
],
'possiblyNullablePropertyFetch' => [
'code' => '<?php
class Foo {
/** @var string */
public $foo = "";
}
$a = rand(0, 10) ? new Foo() : null;
echo $a->foo;',
2017-05-27 02:05:57 +02:00
'error_message' => 'PossiblyNullPropertyFetch',
],
'nullablePropertyFetch' => [
'code' => '<?php
$a = null;
echo $a->foo;',
2017-05-27 02:05:57 +02:00
'error_message' => 'NullPropertyFetch',
],
'badArrayProperty' => [
'code' => '<?php
class A {}
class B {}
class C {
/** @var array<B> */
public $bb;
}
$c = new C;
$c->bb = [new A, new B];',
'error_message' => 'InvalidPropertyAssignmentValue',
],
'possiblyBadArrayProperty' => [
'code' => '<?php
class A {
/** @var int[] */
public $bb = [];
}
class B {
/** @var string[] */
public $bb;
}
$c = rand(0, 1) ? new A : new B;
$c->bb = ["hello", "world"];',
'error_message' => 'PossiblyInvalidPropertyAssignmentValue',
],
'notSetInEmptyConstructor' => [
'code' => '<?php
class A {
/** @var int */
public $a;
public function __construct() { }
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:4',
],
'noConstructor' => [
'code' => '<?php
class A {
/** @var int */
public $a;
}',
2017-05-27 02:05:57 +02:00
'error_message' => 'MissingConstructor',
],
2017-07-09 03:19:16 +02:00
'abstractClassInheritsNoConstructor' => [
'code' => '<?php
2017-07-09 03:19:16 +02:00
abstract class A {
/** @var string */
public $foo;
}
class B extends A {}',
'error_message' => 'MissingConstructor',
],
'abstractClassInheritsPrivateConstructor' => [
'code' => '<?php
abstract class A {
/** @var string */
public $foo;
private function __construct() {
$this->foo = "hello";
}
}
2019-04-09 03:51:18 +02:00
class B extends A {}
$b = new B();',
'error_message' => 'InaccessibleMethod - src' . DIRECTORY_SEPARATOR . 'somefile.php:13',
],
'classInheritsPrivateConstructorWithImplementedConstructor' => [
'code' => '<?php
abstract class A {
/** @var string */
public $foo;
private function __construct() {
$this->foo = "hello";
}
}
class B extends A {
public function __construct() {}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:11',
],
'notSetInAllBranchesOfIf' => [
'code' => '<?php
class A {
/** @var int */
public $a;
public function __construct() {
if (rand(0, 1)) {
$this->a = 5;
}
}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:4',
],
'propertySetInProtectedMethod' => [
'code' => '<?php
class A {
/** @var int */
public $a;
public function __construct() {
$this->foo();
}
2018-01-11 21:50:45 +01:00
protected function foo(): void {
$this->a = 5;
}
}
class B extends A {
protected function foo() : void {}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:15',
],
'definedInTraitNotSetInEmptyConstructor' => [
'code' => '<?php
trait A {
/** @var string **/
public $a;
}
class B {
use A;
public function __construct() {
}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:6',
],
'propertySetInPrivateMethodWithIf' => [
'code' => '<?php
class A {
/** @var int */
public $a;
public function __construct() {
if (rand(0, 1)) {
$this->foo();
}
}
2018-01-11 21:50:45 +01:00
private function foo(): void {
$this->a = 5;
}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:4',
],
'privatePropertySameNameNotSetInConstructor' => [
'code' => '<?php
class A {
/** @var string */
private $b;
public function __construct() {
$this->b = "foo";
}
}
class B extends A {
/** @var string */
private $b;
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:13',
],
'privateMethodCalledInParentConstructor' => [
'code' => '<?php
class C extends B {}
abstract class B extends A {
/** @var string */
private $b;
/** @var string */
protected $c;
}
class A {
public function __construct() {
$this->publicMethod();
}
2018-07-22 02:29:39 +02:00
public function publicMethod() : void {
$this->privateMethod();
}
private function privateMethod() : void {}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:2',
],
'privatePropertySetInParentConstructorReversedOrder' => [
'code' => '<?php
class B extends A {
/** @var string */
private $b;
}
class A {
public function __construct() {
if ($this instanceof B) {
$this->b = "foo";
}
}
}',
'error_message' => 'InaccessibleProperty',
],
'privatePropertySetInParentConstructor' => [
'code' => '<?php
class A {
public function __construct() {
if ($this instanceof B) {
$this->b = "foo";
}
}
}
class B extends A {
/** @var string */
private $b;
}
',
'error_message' => 'InaccessibleProperty',
],
'undefinedPropertyClass' => [
'code' => '<?php
class A {
/** @var B */
public $foo;
}',
'error_message' => 'UndefinedDocblockClass',
2017-05-27 02:05:57 +02:00
],
'abstractClassWithNoConstructorButChild' => [
'code' => '<?php
abstract class A {
/** @var string */
public $foo;
}
class B extends A {
public function __construct() {}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:7',
],
'badAssignmentToUndefinedVars' => [
'code' => '<?php
$x->$y = 4;',
'error_message' => 'UndefinedGlobalVariable',
],
'echoUndefinedPropertyFetch' => [
'code' => '<?php
echo $x->$y;',
'error_message' => 'UndefinedGlobalVariable',
],
'toStringPropertyAssignment' => [
'code' => '<?php
class A {
/** @var ?string */
public $foo;
}
class B {
public function __toString() {
return "bar";
}
}
$a = new A();
$a->foo = new B;',
'error_message' => 'ImplicitToStringCast',
],
'noInfiniteLoop' => [
'code' => '<?php
class A {
/** @var string */
public $foo;
public function __construct() {
$this->doThing();
}
private function doThing(): void {
if (rand(0, 1)) {
$this->doOtherThing();
}
}
private function doOtherThing(): void {
if (rand(0, 1)) {
$this->doThing();
}
}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:4',
],
2018-01-29 03:43:26 +01:00
'invalidPropertyDefault' => [
'code' => '<?php
2018-01-29 03:43:26 +01:00
class A {
/** @var int */
public $a = "hello";
}',
'error_message' => 'InvalidPropertyAssignmentValue',
],
'prohibitMixedAssignmentNormally' => [
'code' => '<?php
class A {
/**
* @var string
*/
private $mixed;
/**
* @param mixed $value
*/
public function setMixed($value): void
{
$this->mixed = $value;
}
}',
'error_message' => 'MixedAssignment',
],
'assertPropertyTypeHasImpossibleType' => [
'code' => '<?php
class A {
/** @var ?B */
public $foo;
}
class B {}
$a = new A();
if (is_string($a->foo)) {}',
'error_message' => 'DocblockTypeContradiction',
],
'impossiblePropertyCheck' => [
'code' => '<?php
class Bar {}
class Foo {
/** @var Bar */
private $bar;
public function __construct() {
$this->bar = new Bar();
}
public function getBar(): void {
if (!$this->bar) {}
}
}',
'error_message' => 'DocblockTypeContradiction',
],
'staticPropertyOfStaticTypeMethodCallWithUndefinedMethod' => [
'code' => '<?php
class A {
/** @var static|null */
public $instance;
public function foo() : void {
if ($this->instance) {
$this->instance->bar();
}
}
}
class B extends A {
public function bar() : void {}
}',
'error_message' => 'UndefinedMethod',
],
'misnamedPropertyByVariable' => [
'code' => '<?php
class B {
/** @var string|null */
public $foo;
public function bar(string $var_name) : ?string {
if ($var_name === "bar") {
return $this->$var_name;
}
return null;
}
}',
'error_message' => 'UndefinedThisPropertyFetch',
],
'inheritDocPropertyTypesIncorrectAssignmentToInstanceProperty' => [
'code' => '<?php
class X {
/**
* @var string|null
*/
public $a;
}
class Y extends X {
public $a = "foo";
}
(new Y)->a = 5;
echo (new Y)->a;
Y::$b = "bar";
echo Y::$b;',
'error_message' => 'InvalidPropertyAssignmentValue',
],
'inheritDocPropertyTypesIncorrectAssignmentToStaticProperty' => [
'code' => '<?php
class X {
/**
* @var string|null
*/
public static $b;
}
class Y extends X {
public static $b = "foo";
}
Y::$b = 5;',
'error_message' => 'InvalidPropertyAssignmentValue',
],
'unitializedProperty' => [
'code' => '<?php
class A {
/** @var string */
public $foo;
public function __construct() {
echo strlen($this->foo);
$this->foo = "foo";
}
}',
2019-03-23 19:27:54 +01:00
'error_message' => 'UninitializedProperty',
],
'unitializedPropertyWithoutType' => [
'code' => '<?php
class A {
public $foo;
public function __construct() {
echo strlen($this->foo);
$this->foo = "foo";
}
}',
'error_message' => 'UninitializedProperty',
2022-12-18 17:15:15 +01:00
'ignored_issues' => ['MixedArgument', 'MissingPropertyType'],
],
'unitializedObjectProperty' => [
'code' => '<?php
class Foo {
/** @var int */
public $bar = 5;
}
function takesInt(int $i) : void {}
class A {
/** @var Foo */
public $foo;
public function __construct(Foo $foo) {
takesInt($this->foo->bar);
$this->foo = $foo;
}
}',
2019-03-23 19:27:54 +01:00
'error_message' => 'UninitializedProperty',
],
'possiblyNullArg' => [
'code' => '<?php
class A {
/** @var ?string */
public $foo;
public function __construct() {
echo strlen($this->foo);
$this->foo = "foo";
}
}',
2019-03-23 19:27:54 +01:00
'error_message' => 'PossiblyNullArgument',
],
'noCrashOnMagicCall' => [
'code' => '<?php
class A {
/** @var string */
private $a;
public function __construct() {
$this->setA();
}
public function __call(string $var, array $args) {}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:4',
],
'reportGoodLocationForPropertyError' => [
'code' => '<?php
class C {
2019-04-14 19:12:21 +02:00
/** @var string */
public $s;
public function __construct() {
$this->setS();
}
public function setS() : void {
$this->s = "hello";
}
}
class D extends C {
public function setS() : void {
// nothing happens here
}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor - src' . DIRECTORY_SEPARATOR . 'somefile.php:15',
],
'noCrashWhenUnsettingPropertyWithoutDefaultInConstructor' => [
'code' => '<?php
class A {
/** @var bool */
private $foo;
public function __construct() {
unset($this->foo);
}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor',
],
'nullableTypedPropertyNoConstructor' => [
'code' => '<?php
class A {
private ?bool $foo;
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'MissingConstructor',
],
'nullableTypedPropertyEmptyConstructor' => [
'code' => '<?php
class A {
private ?bool $foo;
public function __construct() {}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor',
],
'nullableTypedPropertyUseBeforeInitialised' => [
'code' => '<?php
class A {
private ?bool $foo;
public function __construct() {
echo $this->foo;
}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'UninitializedProperty',
],
'nullableTypedPropertyNoConstructorWithDocblock' => [
'code' => '<?php
class A {
/** @var ?bool */
private ?bool $foo;
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'MissingConstructor',
],
'nullableTypedPropertyEmptyConstructorWithDocblock' => [
'code' => '<?php
class A {
/** @var ?bool */
private ?bool $foo;
public function __construct() {}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'PropertyNotSetInConstructor',
],
'nullableTypedPropertyUseBeforeInitialisedWithDocblock' => [
'code' => '<?php
class A {
/** @var ?bool */
private ?bool $foo;
public function __construct() {
echo $this->foo;
}
}',
2019-07-05 22:24:00 +02:00
'error_message' => 'UninitializedProperty',
],
'badStaticPropertyDefault' => [
'code' => '<?php
class TestStatic {
/**
* @var array<string, bool>
*/
public static $test = ["string-key" => 1];
}',
2022-12-18 17:15:15 +01:00
'error_message' => 'InvalidPropertyAssignmentValue',
],
'addNullToMixedAfterNullablePropertyFetch' => [
'code' => '<?php
class A {
/**
* @var mixed
*/
public $foo;
}
function takesString(string $s) : void {}
function takesA(?A $a) : void {
/**
* @psalm-suppress PossiblyNullPropertyFetch
* @psalm-suppress MixedArgument
*/
takesString($a->foo);
}',
'error_message' => 'PossiblyNullArgument',
],
'catchBadArrayStaticProperty' => [
'code' => '<?php
namespace Bar;
class Foo {}
class A {
/** @var array<string, object> */
public array $map = [];
/**
* @param string $class
*/
public function get(string $class) : void {
$this->map[$class] = 5;
}
}',
2022-12-18 17:15:15 +01:00
'error_message' => 'InvalidPropertyAssignmentValue',
],
'preventArrayPushOnArrayValue' => [
'code' => '<?php
class MyClass {
/**
* @var int[]
*/
private $prop = [];
/**
* @return void
*/
public function foo() {
array_push($this->prop, "bad");
}
}',
2022-12-18 17:15:15 +01:00
'error_message' => 'InvalidPropertyAssignmentValue',
],
'overriddenConstructorCalledMethod' => [
'code' => '<?php
class ParentClass {
private string $prop;
public function __construct() {
$this->init();
}
public function init(): void {
$this->prop = "zxc";
}
}
class ChildClass extends ParentClass {
public function init(): void {}
}',
2022-12-18 17:15:15 +01:00
'error_message' => 'PropertyNotSetInConstructor',
],
'propertyWithSameNameUndefined' => [
'code' => '<?php
class Foo {}
class Bar {
public int $id = 3;
public function __construct(Foo $model) {
echo $model->id;
}
}',
'error_message' => 'UndefinedPropertyFetch',
],
'missingPropertyTypeWithDocblock' => [
'code' => '<?php
class C {
/**
* @varr int
*/
public $var;
}',
'error_message' => 'MissingPropertyType',
],
'promotedPrivateProperty' => [
'code' => '<?php
class A {
public function __construct(private int $foo = 5) {}
}
echo (new A)->foo;',
'error_message' => 'InaccessibleProperty',
],
'overwritePropertyType' => [
'code' => '<?php
class A {
/** @var array */
public string $s = [];
}',
'error_message' => 'MismatchingDocblockPropertyType',
],
2021-02-08 04:56:37 +01:00
'possiblyNullOnFunctionCallNotCoalesced' => [
'code' => '<?php
2021-02-08 04:56:37 +01:00
function getC() : ?C {
return rand(0, 1) ? new C() : null;
}
function foo() : void {
echo getC()->id;
}
class C {
public int $id = 1;
}',
'error_message' => 'PossiblyNullPropertyFetch',
],
'noCrashWhenCallingMagicSet' => [
'code' => '<?php
class A {
public function __set(string $s, mixed $value) : void {}
}
(new A)->__set("foo");',
'error_message' => 'TooFewArguments',
],
'noCrashWhenCallingMagicGet' => [
'code' => '<?php
class A {
public function __get(string $s) : mixed {}
}
(new A)->__get();',
'error_message' => 'TooFewArguments',
],
'staticReadOfNonStaticProperty' => [
'code' => '<?php
class A {
/** @var int */
public $prop = 1;
}
echo A::$prop;
',
'error_message' => 'UndefinedPropertyFetch',
],
'staticWriteToNonStaticProperty' => [
'code' => '<?php
class A {
/** @var int */
public $prop = 1;
}
A::$prop = 42;
',
'error_message' => 'UndefinedPropertyAssignment',
],
'nonStaticReadOfStaticProperty' => [
'code' => '<?php
class A {
/** @var int */
public static $prop = 1;
}
echo (new A)->prop;
',
'error_message' => 'UndefinedPropertyFetch',
],
'nonStaticWriteToStaticProperty' => [
'code' => '<?php
class A {
/** @var int */
public static $prop = 1;
}
(new A)->prop = 42;
',
'error_message' => 'UndefinedPropertyAssignment',
2022-12-18 17:15:15 +01:00
],
];
}
2016-07-12 06:53:36 +02:00
}