mirror of
https://github.com/danog/psalm.git
synced 2024-11-27 04:45:20 +01:00
802 lines
20 KiB
PHP
802 lines
20 KiB
PHP
<?php
|
|
namespace Psalm\Tests;
|
|
|
|
use PhpParser\ParserFactory;
|
|
use PHPUnit_Framework_TestCase;
|
|
use Psalm\Checker\FileChecker;
|
|
use Psalm\Config;
|
|
use Psalm\Context;
|
|
|
|
class ScopeTest extends PHPUnit_Framework_TestCase
|
|
{
|
|
/** @var \PhpParser\Parser */
|
|
protected static $parser;
|
|
protected static $file_filter;
|
|
|
|
public static function setUpBeforeClass()
|
|
{
|
|
self::$parser = (new ParserFactory)->create(ParserFactory::PREFER_PHP7);
|
|
|
|
$config = new TestConfig();
|
|
$config->throw_exception = true;
|
|
|
|
self::$file_filter = null;
|
|
}
|
|
|
|
public function setUp()
|
|
{
|
|
FileChecker::clearCache();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage PossiblyUndefinedVariable - somefile.php:6 - Possibly undefined variable $b, first seen
|
|
* on line 3
|
|
*/
|
|
public function testPossiblyUndefinedVarInIf()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
if (rand(0,100) === 10) {
|
|
$b = "s";
|
|
}
|
|
|
|
echo $b;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testNewVarInIf()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
if (rand(0,100) === 10) {
|
|
$badge = "hello";
|
|
}
|
|
else {
|
|
$badge = "goodbye";
|
|
}
|
|
|
|
echo $badge;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testNewVarInIfWithElseReturn()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
if (rand(0,100) === 10) {
|
|
$badge = "hello";
|
|
}
|
|
else {
|
|
throw new \Exception();
|
|
}
|
|
|
|
echo $badge;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage PossiblyUndefinedVariable - somefile.php:3 - Possibly undefined variable $array, first
|
|
* seen on line 3
|
|
*/
|
|
public function testPossiblyUndefinedArrayInIf()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
if (rand(0,100) === 10) {
|
|
$array[] = "hello";
|
|
}
|
|
|
|
echo $array;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage PossiblyUndefinedVariable - somefile.php:3 - Possibly undefined variable $array, first
|
|
* seen on line 3
|
|
*/
|
|
public function testPossiblyUndefinedArrayInForeach()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
foreach ([1, 2, 3, 4] as $b) {
|
|
$array[] = "hello";
|
|
}
|
|
|
|
echo $array;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage PossiblyUndefinedVariable - somefile.php:4 - Possibly undefined variable $array, first
|
|
* seen on line 4
|
|
*/
|
|
public function testPossiblyUndefinedArrayInWhileAndForeach()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
for ($i = 0; $i < 4; $i++) {
|
|
while (rand(0,10) === 5) {
|
|
$array[] = "hello";
|
|
}
|
|
}
|
|
|
|
echo $array;
|
|
');
|
|
|
|
Config::getInstance()->setIssueHandler('PossiblyUndefinedVariable', self::$file_filter);
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage PossiblyUndefinedVariable - somefile.php:6 - Possibly undefined variable $car, first
|
|
* seen on line 3
|
|
*/
|
|
public function testPossiblyUndefinedVariableInForeach()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
foreach ([1, 2, 3, 4] as $b) {
|
|
$car = "Volvo";
|
|
}
|
|
|
|
echo $car;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testSwitchVariableWithContinue()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
foreach ([\'a\', \'b\', \'c\'] as $letter) {
|
|
switch ($letter) {
|
|
case \'a\':
|
|
$foo = 1;
|
|
break;
|
|
case \'b\':
|
|
$foo = 2;
|
|
break;
|
|
default:
|
|
continue;
|
|
}
|
|
|
|
$moo = $foo;
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testSwitchVariableWithContinueAndIfs()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
foreach ([\'a\', \'b\', \'c\'] as $letter) {
|
|
switch ($letter) {
|
|
case \'a\':
|
|
if (rand(0, 10) === 1) {
|
|
continue;
|
|
}
|
|
$foo = 1;
|
|
break;
|
|
case \'b\':
|
|
if (rand(0, 10) === 1) {
|
|
continue;
|
|
}
|
|
$foo = 2;
|
|
break;
|
|
default:
|
|
continue;
|
|
}
|
|
|
|
$moo = $foo;
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testSwitchVariableWithFallthrough()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
foreach ([\'a\', \'b\', \'c\'] as $letter) {
|
|
switch ($letter) {
|
|
case \'a\':
|
|
case \'b\':
|
|
$foo = 2;
|
|
break;
|
|
|
|
default:
|
|
$foo = 3;
|
|
break;
|
|
}
|
|
|
|
$moo = $foo;
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testSwitchVariableWithFallthroughStatement()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
foreach ([\'a\', \'b\', \'c\'] as $letter) {
|
|
switch ($letter) {
|
|
case \'a\':
|
|
$bar = 1;
|
|
|
|
case \'b\':
|
|
$foo = 2;
|
|
break;
|
|
|
|
default:
|
|
$foo = 3;
|
|
break;
|
|
}
|
|
|
|
$moo = $foo;
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testTryCatchVar()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
try {
|
|
$worked = true;
|
|
}
|
|
catch (\Exception $e) {
|
|
$worked = false;
|
|
}
|
|
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$context = new Context('somefile.php');
|
|
$file_checker->check(true, true, $context);
|
|
$this->assertEquals('bool', (string) $context->vars_in_scope['$worked']);
|
|
}
|
|
|
|
public function testWhileVar()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
|
|
$worked = false;
|
|
|
|
while (rand(0,100) === 10) {
|
|
$worked = true;
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$context = new Context('somefile.php');
|
|
$file_checker->check(true, true, $context);
|
|
$this->assertEquals('bool', (string) $context->vars_in_scope['$worked']);
|
|
}
|
|
|
|
public function testDoWhileVar()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
$worked = false;
|
|
|
|
do {
|
|
$worked = true;
|
|
}
|
|
while (rand(0,100) === 10);
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$context = new Context('somefile.php');
|
|
$file_checker->check(true, true, $context);
|
|
$this->assertEquals('bool', (string) $context->vars_in_scope['$worked']);
|
|
}
|
|
|
|
public function testAssignmentInIf()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
if ($row = (rand(0, 10) ? [5] : null)) {
|
|
echo $row[0];
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testNegatedAssignmentInIf()
|
|
{
|
|
$this->markTestIncomplete('This currently fails');
|
|
$stmts = self::$parser->parse('<?php
|
|
if (!($row = (rand(0, 10) ? [5] : null))) {
|
|
// do nothing
|
|
}
|
|
else {
|
|
echo $row[0];
|
|
}
|
|
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testAssignInElseIf()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
if (rand(0, 10) > 5) {
|
|
echo "hello";
|
|
} elseif ($row = (rand(0, 10) ? [5] : null)) {
|
|
echo $row[0];
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testIfNotEqualFalse()
|
|
{
|
|
$this->markTestIncomplete('This currently fails');
|
|
$stmts = self::$parser->parse('<?php
|
|
if (($row = rand(0,10) ? [] : false) !== false) {
|
|
$row[0] = "good";
|
|
echo $row[0];
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testPassByRefInIf()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
if (preg_match("/bad/", "badger", $matches)) {
|
|
echo (string)$matches[0];
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testPassByRefInIfCheckAfter()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
if (!preg_match("/bad/", "badger", $matches)) {
|
|
exit();
|
|
}
|
|
echo (string)$matches[0];
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testPassByRefInIfWithBoolean()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
$a = true;
|
|
if ($a && preg_match("/bad/", "badger", $matches)) {
|
|
echo (string)$matches[0];
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage PossiblyUndefinedVariable - somefile.php:9 - Possibly undefined variable $a, first
|
|
* seen on line 4
|
|
*/
|
|
public function testPossiblyUndefinedVariableInForeachAndIf()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
foreach ([1,2,3,4] as $i) {
|
|
if ($i === 1) {
|
|
$a = true;
|
|
break;
|
|
}
|
|
}
|
|
|
|
echo $a;
|
|
');
|
|
|
|
Config::getInstance()->setIssueHandler('PossiblyUndefinedVariable', self::$file_filter);
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testFunctionExists()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
if (true && function_exists("flabble")) {
|
|
flabble();
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testNestedPropertyFetchInElseif()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
/** @var A|null */
|
|
public $foo;
|
|
|
|
public function __toString() : string {
|
|
return "boop";
|
|
}
|
|
}
|
|
|
|
$a = rand(0, 10) === 5 ? new A() : null;
|
|
|
|
if (false) {
|
|
|
|
}
|
|
elseif ($a && $a->foo) {
|
|
echo $a;
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testGlobalReturn()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
$foo = "foo";
|
|
|
|
function a() : string {
|
|
global $foo;
|
|
|
|
return $foo;
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testStatic()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
function a() : string {
|
|
static $foo = "foo";
|
|
|
|
return $foo;
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage InaccessibleMethod
|
|
*/
|
|
public function testInaccessiblePrivateMethod()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
private function foo() : void {
|
|
|
|
}
|
|
}
|
|
|
|
(new A())->foo();
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage InaccessibleMethod
|
|
*/
|
|
public function testInaccessibleProtectedMethod()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
protected function foo() : void {
|
|
|
|
}
|
|
}
|
|
|
|
(new A())->foo();
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testAccessiblePrivateMethodFromSubclass()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
private function foo() : void {
|
|
|
|
}
|
|
|
|
private function bar() : void {
|
|
$this->foo();
|
|
}
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage UndefinedMethod
|
|
*/
|
|
public function testInaccessiblePrivateMethodFromSubclass()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
private function foo() : void {
|
|
|
|
}
|
|
}
|
|
|
|
class B extends A {
|
|
public function doFoo() : void {
|
|
$this->foo();
|
|
}
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testAccessibleProtectedMethodFromSubclass()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
protected function foo() : void {
|
|
}
|
|
}
|
|
|
|
class B extends A {
|
|
public function doFoo() : void {
|
|
$this->foo();
|
|
}
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage InaccessibleProperty
|
|
*/
|
|
public function testInaccessiblePrivateProperty()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
/** @var string */
|
|
private $foo;
|
|
}
|
|
|
|
echo (new A())->foo;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage InaccessibleProperty
|
|
*/
|
|
public function testInaccessibleProtectedProperty()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
/** @var string */
|
|
protected $foo;
|
|
}
|
|
|
|
echo (new A())->foo;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage UndefinedThisPropertyFetch
|
|
*/
|
|
public function testInaccessiblePrivatePropertyFromSubclass()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
/** @var string */
|
|
private $foo;
|
|
}
|
|
|
|
class B extends A {
|
|
public function doFoo() : void {
|
|
echo $this->foo;
|
|
}
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage InaccessibleProperty
|
|
*/
|
|
public function testInaccessibleStaticPrivateProperty()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
/** @var string */
|
|
private static $foo;
|
|
}
|
|
|
|
echo A::$foo;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage InaccessibleProperty
|
|
*/
|
|
public function testInaccessibleStaticProtectedProperty()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
/** @var string */
|
|
protected static $foo;
|
|
}
|
|
|
|
echo A::$foo;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage InaccessibleProperty
|
|
*/
|
|
public function testInaccessibleStaticPrivatePropertyFromSubclass()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
/** @var string */
|
|
private static $foo;
|
|
}
|
|
|
|
class B extends A {
|
|
public function doFoo() : void {
|
|
echo A::$foo;
|
|
}
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testAccessibleProtectedPropertyFromSubclass()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
/** @var string */
|
|
protected $foo;
|
|
}
|
|
|
|
class B extends A {
|
|
public function doFoo() : void {
|
|
echo $this->foo;
|
|
}
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
public function testAccessibleStaticPropertyFromSubclass()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
/** @var string */
|
|
protected static $foo;
|
|
|
|
public function bar() : void {
|
|
echo self::$foo;
|
|
}
|
|
}
|
|
|
|
class B extends A {
|
|
public function doFoo() : void {
|
|
echo A::$foo;
|
|
}
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage InvalidGlobal
|
|
*/
|
|
public function testInvalidGlobal()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
$a = "heli";
|
|
|
|
global $a;
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
|
|
/**
|
|
* @expectedException \Psalm\Exception\CodeException
|
|
* @expectedExceptionMessage InvalidStaticVariable
|
|
*/
|
|
public function testThisInStatic()
|
|
{
|
|
$stmts = self::$parser->parse('<?php
|
|
class A {
|
|
public static function foo() {
|
|
echo $this;
|
|
}
|
|
}
|
|
');
|
|
|
|
$file_checker = new FileChecker('somefile.php', $stmts);
|
|
$file_checker->check();
|
|
}
|
|
}
|