2017-12-05 18:14:10 +01:00
|
|
|
<?php
|
|
|
|
namespace Psalm\Tests;
|
|
|
|
|
|
|
|
class EmptyTest extends TestCase
|
|
|
|
{
|
2018-11-06 03:57:36 +01:00
|
|
|
use Traits\InvalidCodeAnalysisTestTrait;
|
|
|
|
use Traits\ValidCodeAnalysisTestTrait;
|
2017-12-05 18:14:10 +01:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @return array
|
|
|
|
*/
|
2018-11-06 03:57:36 +01:00
|
|
|
public function providerValidCodeParse()
|
2017-12-05 18:14:10 +01:00
|
|
|
{
|
|
|
|
return [
|
2018-12-08 19:18:55 +01:00
|
|
|
'ifNotUndefinedAndEmpty' => [
|
2017-12-05 18:14:10 +01:00
|
|
|
'<?php
|
|
|
|
$a = !empty($b) ? $b : null;',
|
|
|
|
'assertions' => [
|
2018-12-08 19:18:55 +01:00
|
|
|
'$a' => 'mixed|null',
|
2017-12-05 18:14:10 +01:00
|
|
|
],
|
|
|
|
'error_levels' => ['MixedAssignment'],
|
|
|
|
],
|
|
|
|
'emptyArrayVar' => [
|
|
|
|
'<?php
|
|
|
|
function a(array $in): void
|
|
|
|
{
|
|
|
|
$r = [];
|
|
|
|
foreach ($in as $entry) {
|
|
|
|
if (!empty($entry["a"])) {
|
|
|
|
$r[] = [];
|
|
|
|
}
|
|
|
|
if (empty($entry["a"])) {
|
|
|
|
$r[] = [];
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
function b(array $in): void
|
|
|
|
{
|
|
|
|
$i = 0;
|
|
|
|
foreach ($in as $entry) {
|
|
|
|
if (!empty($entry["a"])) {
|
|
|
|
$i--;
|
|
|
|
}
|
|
|
|
if (empty($entry["a"])) {
|
|
|
|
$i++;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
function c(array $in): void
|
|
|
|
{
|
|
|
|
foreach ($in as $entry) {
|
|
|
|
if (!empty($entry["a"])) {}
|
|
|
|
}
|
|
|
|
foreach ($in as $entry) {
|
|
|
|
if (empty($entry["a"])) {}
|
|
|
|
}
|
|
|
|
}',
|
|
|
|
'assertions' => [],
|
2018-01-10 01:33:39 +01:00
|
|
|
'error_levels' => ['MixedAssignment', 'MixedArrayAccess'],
|
2017-12-05 18:14:10 +01:00
|
|
|
],
|
|
|
|
'removeEmptyArray' => [
|
|
|
|
'<?php
|
|
|
|
$arr_or_string = [];
|
|
|
|
|
|
|
|
if (rand(0, 1)) {
|
|
|
|
$arr_or_string = "hello";
|
|
|
|
}
|
|
|
|
|
|
|
|
/** @return void **/
|
|
|
|
function foo(string $s) {}
|
|
|
|
|
|
|
|
if (!empty($arr_or_string)) {
|
|
|
|
foo($arr_or_string);
|
|
|
|
}',
|
|
|
|
],
|
|
|
|
'emptyArrayReconciliationThenIf' => [
|
|
|
|
'<?php
|
|
|
|
/**
|
|
|
|
* @param string|string[] $a
|
|
|
|
*/
|
2018-01-11 21:50:45 +01:00
|
|
|
function foo($a): string {
|
2017-12-05 18:14:10 +01:00
|
|
|
if (is_string($a)) {
|
|
|
|
return $a;
|
|
|
|
} elseif (empty($a)) {
|
|
|
|
return "goodbye";
|
|
|
|
}
|
|
|
|
|
|
|
|
if (isset($a[0])) {
|
|
|
|
return $a[0];
|
|
|
|
};
|
|
|
|
|
|
|
|
return "not found";
|
|
|
|
}',
|
|
|
|
],
|
|
|
|
'emptyStringReconciliationThenIf' => [
|
|
|
|
'<?php
|
|
|
|
/**
|
|
|
|
* @param Exception|string|string[] $a
|
|
|
|
*/
|
2018-01-11 21:50:45 +01:00
|
|
|
function foo($a): string {
|
2017-12-05 18:14:10 +01:00
|
|
|
if (is_array($a)) {
|
|
|
|
return "hello";
|
|
|
|
} elseif (empty($a)) {
|
|
|
|
return "goodbye";
|
|
|
|
}
|
|
|
|
|
|
|
|
if (is_string($a)) {
|
|
|
|
return $a;
|
|
|
|
};
|
|
|
|
|
|
|
|
return "an exception";
|
|
|
|
}',
|
|
|
|
],
|
|
|
|
'emptyExceptionReconciliationAfterIf' => [
|
|
|
|
'<?php
|
|
|
|
/**
|
|
|
|
* @param Exception|null $a
|
|
|
|
*/
|
2018-01-11 21:50:45 +01:00
|
|
|
function foo($a): string {
|
2017-12-05 18:14:10 +01:00
|
|
|
if ($a && $a->getMessage() === "hello") {
|
|
|
|
return "hello";
|
|
|
|
} elseif (empty($a)) {
|
|
|
|
return "goodbye";
|
|
|
|
}
|
|
|
|
|
|
|
|
return $a->getMessage();
|
|
|
|
}',
|
|
|
|
],
|
|
|
|
'noFalsyLeak' => [
|
|
|
|
'<?php
|
2018-01-11 21:50:45 +01:00
|
|
|
function foo(string $s): void {
|
2017-12-05 18:14:10 +01:00
|
|
|
if (empty($s) || $s === "hello") {}
|
|
|
|
}',
|
|
|
|
],
|
|
|
|
'noRedundantConditionOnMixed' => [
|
|
|
|
'<?php
|
2018-01-11 21:50:45 +01:00
|
|
|
function testarray(array $data): void {
|
2017-12-05 18:14:10 +01:00
|
|
|
foreach ($data as $item) {
|
2018-12-19 22:15:19 +01:00
|
|
|
if (!empty($item["a"]) && !empty($item["b"]["c"])) {
|
2017-12-05 18:14:10 +01:00
|
|
|
echo "Found\n";
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}',
|
|
|
|
'assertions' => [],
|
2018-01-10 01:33:39 +01:00
|
|
|
'error_levels' => ['MixedAssignment', 'MixedArrayAccess'],
|
2017-12-05 18:14:10 +01:00
|
|
|
],
|
2017-12-10 20:56:51 +01:00
|
|
|
'dontBleedEmptyAfterExtract' => [
|
|
|
|
'<?php
|
2018-01-11 21:50:45 +01:00
|
|
|
function foo(array $args): void {
|
2017-12-10 20:56:51 +01:00
|
|
|
extract($args);
|
|
|
|
if ((empty($arr) && empty($a)) || $c === 0) {
|
|
|
|
} else {
|
|
|
|
foreach ($arr as $b) {}
|
|
|
|
}
|
|
|
|
}',
|
|
|
|
'assertions' => [],
|
2018-04-07 21:16:46 +02:00
|
|
|
'error_levels' => ['MixedAssignment', 'MixedArgument'],
|
2017-12-10 20:56:51 +01:00
|
|
|
],
|
2018-03-18 00:28:01 +01:00
|
|
|
'emptyObjectLike' => [
|
|
|
|
'<?php
|
|
|
|
$arr = [
|
|
|
|
"profile" => [
|
|
|
|
"foo" => "bar",
|
|
|
|
],
|
|
|
|
"groups" => [
|
|
|
|
"foo" => "bar",
|
|
|
|
"hide" => rand(0, 5),
|
|
|
|
],
|
|
|
|
];
|
|
|
|
|
|
|
|
foreach ($arr as $item) {
|
|
|
|
if (empty($item["hide"]) || $item["hide"] === 3) {}
|
|
|
|
}',
|
|
|
|
],
|
2018-04-09 16:19:23 +02:00
|
|
|
'alwaysBoolResult' => [
|
|
|
|
'<?php
|
|
|
|
function takesBool(bool $p): void {}
|
|
|
|
takesBool(empty($q));'
|
|
|
|
],
|
2018-05-08 22:34:08 +02:00
|
|
|
'noRedundantConditionAfterFalsyIntChecks' => [
|
|
|
|
'<?php
|
|
|
|
function foo(int $t) : void {
|
|
|
|
if (!$t) {
|
|
|
|
foreach ([0, 1, 2] as $a) {
|
|
|
|
if (!$t) {
|
|
|
|
$t = $a;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}',
|
|
|
|
],
|
|
|
|
'noRedundantConditionAfterEmptyMixedChecks' => [
|
2018-05-08 06:57:18 +02:00
|
|
|
'<?php
|
|
|
|
function foo($t) : void {
|
|
|
|
if (empty($t)) {
|
|
|
|
foreach ($_GET["u"] as $a) {
|
|
|
|
if (empty($t)) {
|
|
|
|
$t = $a;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}',
|
|
|
|
'assertions' => [],
|
|
|
|
'error_levels' => ['MixedAssignment', 'MissingParamType'],
|
|
|
|
],
|
2018-12-08 19:18:55 +01:00
|
|
|
'canBeNonEmptyArray' => [
|
2018-05-31 21:07:03 +02:00
|
|
|
'<?php
|
2018-12-08 19:18:55 +01:00
|
|
|
function _processScopes($scopes) : void {
|
|
|
|
if (!is_array($scopes) && !empty($scopes)) {
|
|
|
|
$scopes = explode(" ", trim($scopes));
|
|
|
|
} else {
|
|
|
|
// false is allowed here
|
|
|
|
}
|
2018-05-31 21:07:03 +02:00
|
|
|
|
2018-12-08 19:18:55 +01:00
|
|
|
if (empty($scopes)){}
|
2018-05-31 21:07:03 +02:00
|
|
|
}',
|
|
|
|
'assertions' => [],
|
2018-12-08 19:18:55 +01:00
|
|
|
'error_levels' => ['MixedAssignment', 'MissingParamType', 'MixedArgument'],
|
2018-05-31 21:07:03 +02:00
|
|
|
],
|
2018-12-11 19:50:26 +01:00
|
|
|
'multipleEmptiesInCondition' => [
|
|
|
|
'<?php
|
|
|
|
/** @param array<int, int> $o */
|
|
|
|
function foo(array $o) : void {
|
|
|
|
if (empty($o[0]) && empty($o[1])) {}
|
|
|
|
}',
|
|
|
|
],
|
|
|
|
'multipleEmptiesInConditionWithMixedOffset' => [
|
|
|
|
'<?php
|
|
|
|
/** @param array $o */
|
|
|
|
function foo(array $o) : void {
|
|
|
|
if (empty($o[0]) && empty($o[1])) {}
|
|
|
|
}',
|
|
|
|
],
|
2018-12-20 22:03:21 +01:00
|
|
|
'unsetChangesArrayEmptiness' => [
|
|
|
|
'<?php
|
|
|
|
function foo(array $n): void {
|
|
|
|
if (empty($n)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
while (!empty($n)) {
|
|
|
|
unset($n[rand(0, 10)]);
|
|
|
|
}
|
|
|
|
}',
|
|
|
|
],
|
|
|
|
'unsetChangesComplicatedArrayEmptiness' => [
|
|
|
|
'<?php
|
|
|
|
function contains(array $data, array $needle): bool {
|
|
|
|
if (empty($data) || empty($needle)) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
$stack = [];
|
|
|
|
|
|
|
|
while (!empty($needle)) {
|
|
|
|
$key = key($needle);
|
|
|
|
$val = $needle[$key];
|
|
|
|
unset($needle[$key]);
|
|
|
|
|
|
|
|
if (array_key_exists($key, $data) && is_array($val)) {
|
|
|
|
$next = $data[$key];
|
|
|
|
unset($data[$key]);
|
|
|
|
|
|
|
|
if (!empty($val)) {
|
|
|
|
$stack[] = [$val, $next];
|
|
|
|
}
|
|
|
|
} elseif (!array_key_exists($key, $data) || $data[$key] != $val) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (empty($needle) && !empty($stack)) {
|
|
|
|
list($needle, $data) = array_pop($stack);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return true;
|
|
|
|
}',
|
|
|
|
'assertions' => [],
|
|
|
|
'error_levels' => ['MixedAssignment', 'MissingParamType', 'MixedArgument', 'MixedArrayOffset'],
|
|
|
|
],
|
2018-12-26 11:52:37 +01:00
|
|
|
'possiblyEmptyIterable' => [
|
|
|
|
'<?php
|
|
|
|
function foo(iterable $i) : void {
|
|
|
|
if (empty($i)) {}
|
|
|
|
if (!empty($i)) {}
|
|
|
|
}',
|
|
|
|
],
|
2019-02-07 22:24:31 +01:00
|
|
|
'allowEmptyClassConstantOffsetCheck' => [
|
|
|
|
'<?php
|
|
|
|
class Foo {
|
|
|
|
const BAR = "bar";
|
|
|
|
const ONE = 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* @param array<string,string> $data
|
|
|
|
*/
|
|
|
|
function bat(array $data) : void {
|
|
|
|
if (!empty($data["foo"])) {
|
|
|
|
if (empty($data[Foo::BAR])) {}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* @param array<int,string> $data
|
|
|
|
*/
|
|
|
|
function baz(array $data) : void {
|
|
|
|
if (!empty($data[0])) {
|
|
|
|
if (empty($data[Foo::ONE])) {}
|
|
|
|
}
|
|
|
|
}',
|
|
|
|
],
|
2019-02-18 18:53:55 +01:00
|
|
|
'doubleEmptyCheckTwoArrays' => [
|
|
|
|
'<?php
|
|
|
|
function foo(array $a, array $b) : void {
|
|
|
|
if (empty($a) && empty($b)) {}
|
|
|
|
}'
|
|
|
|
],
|
|
|
|
'SKIPPED-doubleEmptyCheckOnObjectLike' => [
|
|
|
|
'<?php
|
|
|
|
/**
|
|
|
|
* @param array{a: array, b: array} $arr
|
|
|
|
*/
|
|
|
|
function foo(array $arr) : void {
|
|
|
|
if (empty($arr["a"]) && empty($arr["b"])) {}
|
|
|
|
}'
|
|
|
|
],
|
2017-12-05 18:14:10 +01:00
|
|
|
];
|
|
|
|
}
|
2018-05-01 04:13:13 +02:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @return array
|
|
|
|
*/
|
2018-11-06 03:57:36 +01:00
|
|
|
public function providerInvalidCodeParse()
|
2018-05-01 04:13:13 +02:00
|
|
|
{
|
|
|
|
return [
|
|
|
|
'preventImpossibleEmpty' => [
|
|
|
|
'<?php
|
|
|
|
function foo(array $arr) : void {
|
|
|
|
if (empty($ar)) {
|
|
|
|
// do something
|
|
|
|
}
|
|
|
|
}',
|
|
|
|
'error_message' => 'UndefinedVariable',
|
|
|
|
],
|
2018-12-08 19:18:55 +01:00
|
|
|
'reconciliationForMixed' => [
|
|
|
|
'<?php
|
|
|
|
function foo(array $arr) : void {
|
|
|
|
$a = empty($arr["a"]) ? "" : $arr["a"];
|
|
|
|
|
|
|
|
if ($a) {
|
|
|
|
if ($a) {}
|
|
|
|
}
|
|
|
|
}',
|
|
|
|
'error_message' => 'RedundantCondition',
|
|
|
|
'error_levels' => ['MixedAssignment', 'MissingParamType'],
|
|
|
|
],
|
2018-05-01 04:13:13 +02:00
|
|
|
];
|
|
|
|
}
|
2017-12-05 18:14:10 +01:00
|
|
|
}
|