Skip to content

Commit 53578cb

Browse files
VincentLangletVincentLanglet
committed
Migrate legacy tests
1 parent 74b909a commit 53578cb

9 files changed

+321
-222
lines changed

‎tests/PHPStan/Analyser/LegacyNodeScopeResolverTest.php

Lines changed: 0 additions & 176 deletions
Original file line numberDiff line numberDiff line change
@@ -5273,14 +5273,6 @@ public function testArrayFunctions(
52735273

52745274
public function dataFunctions(): array
52755275
{
5276-
$strSplitDefaultReturnType = 'non-empty-list<string>|false';
5277-
if (PHP_VERSION_ID >= 80000) {
5278-
$strSplitDefaultReturnType = 'non-empty-list<string>';
5279-
}
5280-
if (PHP_VERSION_ID >= 80200) {
5281-
$strSplitDefaultReturnType = 'list<string>';
5282-
}
5283-
52845276
return [
52855277
[
52865278
'string',
@@ -5470,42 +5462,6 @@ public function dataFunctions(): array
54705462
'(array{sec: int, usec: int, minuteswest: int, dsttime: int}|float)',
54715463
'$gettimeofdayBenevolent',
54725464
],
5473-
[
5474-
$strSplitDefaultReturnType,
5475-
'$strSplitConstantStringWithoutDefinedParameters',
5476-
],
5477-
[
5478-
'array{\'a\', \'b\', \'c\', \'d\', \'e\', \'f\'}',
5479-
'$strSplitConstantStringWithoutDefinedSplitLength',
5480-
],
5481-
[
5482-
PHP_VERSION_ID < 80200 ? 'non-empty-list<string>' : 'list<string>',
5483-
'$strSplitStringWithoutDefinedSplitLength',
5484-
],
5485-
[
5486-
'array{\'a\', \'b\', \'c\', \'d\', \'e\', \'f\'}',
5487-
'$strSplitConstantStringWithOneSplitLength',
5488-
],
5489-
[
5490-
'array{\'abcdef\'}',
5491-
'$strSplitConstantStringWithGreaterSplitLengthThanStringLength',
5492-
],
5493-
[
5494-
'false',
5495-
'$strSplitConstantStringWithFailureSplitLength',
5496-
],
5497-
[
5498-
$strSplitDefaultReturnType,
5499-
'$strSplitConstantStringWithInvalidSplitLengthType',
5500-
],
5501-
[
5502-
"array{'a', 'b', 'c', 'd', 'e', 'f'}|array{'g', 'h', 'i', 'j', 'k', 'l'}",
5503-
'$strSplitConstantStringWithVariableStringAndConstantSplitLength',
5504-
],
5505-
[
5506-
$strSplitDefaultReturnType,
5507-
'$strSplitConstantStringWithVariableStringAndVariableSplitLength',
5508-
],
55095465
// parse_url
55105466
[
55115467
'array|int|string|false|null',
@@ -8956,138 +8912,6 @@ public function testPhp73Functions(
89568912
);
89578913
}
89588914

8959-
public function dataPhp74Functions(): array
8960-
{
8961-
return [
8962-
[
8963-
PHP_VERSION_ID < 80000 ? 'list<string>|false' : 'list<string>',
8964-
'$mbStrSplitConstantStringWithoutDefinedParameters',
8965-
],
8966-
[
8967-
'array{\'a\', \'b\', \'c\', \'d\', \'e\', \'f\'}',
8968-
'$mbStrSplitConstantStringWithoutDefinedSplitLength',
8969-
],
8970-
[
8971-
'list<string>',
8972-
'$mbStrSplitStringWithoutDefinedSplitLength',
8973-
],
8974-
[
8975-
'array{\'a\', \'b\', \'c\', \'d\', \'e\', \'f\'}',
8976-
'$mbStrSplitConstantStringWithOneSplitLength',
8977-
],
8978-
[
8979-
'array{\'abcdef\'}',
8980-
'$mbStrSplitConstantStringWithGreaterSplitLengthThanStringLength',
8981-
],
8982-
[
8983-
'false',
8984-
'$mbStrSplitConstantStringWithFailureSplitLength',
8985-
],
8986-
[
8987-
PHP_VERSION_ID < 80000 ? 'list<string>|false' : 'list<string>',
8988-
'$mbStrSplitConstantStringWithInvalidSplitLengthType',
8989-
],
8990-
[
8991-
"array{'a', 'b', 'c', 'd', 'e', 'f'}|array{'g', 'h', 'i', 'j', 'k', 'l'}",
8992-
'$mbStrSplitConstantStringWithVariableStringAndConstantSplitLength',
8993-
],
8994-
[
8995-
PHP_VERSION_ID < 80000 ? 'list<string>|false' : 'list<string>',
8996-
'$mbStrSplitConstantStringWithVariableStringAndVariableSplitLength',
8997-
],
8998-
[
8999-
"array{'a', 'b', 'c', 'd', 'e', 'f'}",
9000-
'$mbStrSplitConstantStringWithOneSplitLengthAndValidEncoding',
9001-
],
9002-
[
9003-
'false',
9004-
'$mbStrSplitConstantStringWithOneSplitLengthAndInvalidEncoding',
9005-
],
9006-
[
9007-
PHP_VERSION_ID < 80000 ? 'list<string>|false' : 'list<string>',
9008-
'$mbStrSplitConstantStringWithOneSplitLengthAndVariableEncoding',
9009-
],
9010-
[
9011-
"array{'abcdef'}",
9012-
'$mbStrSplitConstantStringWithGreaterSplitLengthThanStringLengthAndValidEncoding',
9013-
],
9014-
[
9015-
'false',
9016-
'$mbStrSplitConstantStringWithGreaterSplitLengthThanStringLengthAndInvalidEncoding',
9017-
],
9018-
[
9019-
PHP_VERSION_ID < 80000 ? 'list<string>|false' : 'list<string>',
9020-
'$mbStrSplitConstantStringWithGreaterSplitLengthThanStringLengthAndVariableEncoding',
9021-
],
9022-
[
9023-
'false',
9024-
'$mbStrSplitConstantStringWithFailureSplitLengthAndValidEncoding',
9025-
],
9026-
[
9027-
'false',
9028-
'$mbStrSplitConstantStringWithFailureSplitLengthAndInvalidEncoding',
9029-
],
9030-
[
9031-
'false',
9032-
'$mbStrSplitConstantStringWithFailureSplitLengthAndVariableEncoding',
9033-
],
9034-
[
9035-
PHP_VERSION_ID < 80000 ? 'list<string>|false' : 'list<string>',
9036-
'$mbStrSplitConstantStringWithInvalidSplitLengthTypeAndValidEncoding',
9037-
],
9038-
[
9039-
'false',
9040-
'$mbStrSplitConstantStringWithInvalidSplitLengthTypeAndInvalidEncoding',
9041-
],
9042-
[
9043-
PHP_VERSION_ID < 80000 ? 'list<string>|false' : 'list<string>',
9044-
'$mbStrSplitConstantStringWithInvalidSplitLengthTypeAndVariableEncoding',
9045-
],
9046-
[
9047-
"array{'a', 'b', 'c', 'd', 'e', 'f'}|array{'g', 'h', 'i', 'j', 'k', 'l'}",
9048-
'$mbStrSplitConstantStringWithVariableStringAndConstantSplitLengthAndValidEncoding',
9049-
],
9050-
[
9051-
'false',
9052-
'$mbStrSplitConstantStringWithVariableStringAndConstantSplitLengthAndInvalidEncoding',
9053-
],
9054-
[
9055-
PHP_VERSION_ID < 80000 ? 'list<string>|false' : 'list<string>',
9056-
'$mbStrSplitConstantStringWithVariableStringAndConstantSplitLengthAndVariableEncoding',
9057-
],
9058-
[
9059-
PHP_VERSION_ID < 80000 ? 'list<string>|false' : 'list<string>',
9060-
'$mbStrSplitConstantStringWithVariableStringAndVariableSplitLengthAndValidEncoding',
9061-
],
9062-
[
9063-
'false',
9064-
'$mbStrSplitConstantStringWithVariableStringAndVariableSplitLengthAndInvalidEncoding',
9065-
],
9066-
[
9067-
PHP_VERSION_ID < 80000 ? 'list<string>|false' : 'list<string>',
9068-
'$mbStrSplitConstantStringWithVariableStringAndVariableSplitLengthAndVariableEncoding',
9069-
],
9070-
];
9071-
}
9072-
9073-
/**
9074-
* @dataProvider dataPhp74Functions
9075-
*/
9076-
public function testPhp74Functions(
9077-
string $description,
9078-
string $expression,
9079-
): void
9080-
{
9081-
if (PHP_VERSION_ID < 70400) {
9082-
$this->markTestSkipped('Test requires PHP 7.4');
9083-
}
9084-
$this->assertTypes(
9085-
__DIR__ . '/data/php74_functions.php',
9086-
$description,
9087-
$expression,
9088-
);
9089-
}
9090-
90918915
public function dataUnionMethods(): array
90928916
{
90938917
return [

‎tests/PHPStan/Analyser/NodeScopeResolverTest.php

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,19 @@ private static function findTestFiles(): iterable
4949
}
5050
}
5151

52+
if (PHP_VERSION_ID >= 80200) {
53+
yield __DIR__ . '/data/str-split-php82.php';
54+
} elseif (PHP_VERSION_ID >= 80000) {
55+
yield __DIR__ . '/data/str-split-php80.php';
56+
} else {
57+
yield __DIR__ . '/data/str-split-php74.php';
58+
}
59+
if (PHP_VERSION_ID >= 80000) {
60+
yield __DIR__ . '/data/mb-str-split-php80.php';
61+
} elseif (PHP_VERSION_ID >= 74000) {
62+
yield __DIR__ . '/data/mb-str-split-php74.php';
63+
}
64+
5265
yield __DIR__ . '/../Rules/Methods/data/bug-6856.php';
5366

5467
if (PHP_VERSION_ID < 80000) {

‎tests/PHPStan/Analyser/data/functions.php

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -56,19 +56,6 @@
5656
$gettimeofdayDefault = gettimeofday(null);
5757
$gettimeofdayBenevolent = gettimeofday($undefined);
5858

59-
// str_split
60-
/** @var string $string */
61-
$string = doFoo();
62-
$strSplitConstantStringWithoutDefinedParameters = str_split();
63-
$strSplitConstantStringWithoutDefinedSplitLength = str_split('abcdef');
64-
$strSplitStringWithoutDefinedSplitLength = str_split($string);
65-
$strSplitConstantStringWithOneSplitLength = str_split('abcdef', 1);
66-
$strSplitConstantStringWithGreaterSplitLengthThanStringLength = str_split('abcdef', 999);
67-
$strSplitConstantStringWithFailureSplitLength = str_split('abcdef', 0);
68-
$strSplitConstantStringWithInvalidSplitLengthType = str_split('abcdef', []);
69-
$strSplitConstantStringWithVariableStringAndConstantSplitLength = str_split(doFoo() ? 'abcdef' : 'ghijkl', 1);
70-
$strSplitConstantStringWithVariableStringAndVariableSplitLength = str_split(doFoo() ? 'abcdef' : 'ghijkl', doFoo() ? 1 : 2);
71-
7259
// parse_url
7360
/** @var int $integer */
7461
$integer = doFoo();

‎tests/PHPStan/Analyser/data/mb-str-split-php74.php

Lines changed: 94 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,94 @@
1+
<?php
2+
3+
namespace MbStrSplitPHP74;
4+
5+
use function PHPStan\Testing\assertType;
6+
7+
class MbStrSplit {
8+
public function legacyTest(): void
9+
{
10+
/** @var string $string */
11+
$string = doFoo();
12+
13+
$mbStrSplitConstantStringWithoutDefinedParameters = mb_str_split();
14+
assertType('list<string>|false', $mbStrSplitConstantStringWithoutDefinedParameters);
15+
16+
$mbStrSplitConstantStringWithoutDefinedSplitLength = mb_str_split('abcdef');
17+
assertType('array{\'a\', \'b\', \'c\', \'d\', \'e\', \'f\'}', $mbStrSplitConstantStringWithoutDefinedSplitLength);
18+
19+
$mbStrSplitStringWithoutDefinedSplitLength = mb_str_split($string);
20+
assertType('list<string>', $mbStrSplitStringWithoutDefinedSplitLength);
21+
22+
$mbStrSplitConstantStringWithOneSplitLength = mb_str_split('abcdef', 1);
23+
assertType('array{\'a\', \'b\', \'c\', \'d\', \'e\', \'f\'}', $mbStrSplitConstantStringWithOneSplitLength);
24+
25+
$mbStrSplitConstantStringWithGreaterSplitLengthThanStringLength = mb_str_split('abcdef', 999);
26+
assertType('array{\'abcdef\'}', $mbStrSplitConstantStringWithGreaterSplitLengthThanStringLength);
27+
28+
$mbStrSplitConstantStringWithFailureSplitLength = mb_str_split('abcdef', 0);
29+
assertType('false', $mbStrSplitConstantStringWithFailureSplitLength);
30+
31+
$mbStrSplitConstantStringWithInvalidSplitLengthType = mb_str_split('abcdef', []);
32+
assertType('list<string>|false', $mbStrSplitConstantStringWithInvalidSplitLengthType);
33+
34+
$mbStrSplitConstantStringWithVariableStringAndConstantSplitLength = mb_str_split(doFoo() ? 'abcdef' : 'ghijkl', 1);
35+
assertType("array{'a', 'b', 'c', 'd', 'e', 'f'}|array{'g', 'h', 'i', 'j', 'k', 'l'}", $mbStrSplitConstantStringWithVariableStringAndConstantSplitLength);
36+
37+
$mbStrSplitConstantStringWithVariableStringAndVariableSplitLength = mb_str_split(doFoo() ? 'abcdef' : 'ghijkl', doFoo() ? 1 : 2);
38+
assertType('list<string>|false', $mbStrSplitConstantStringWithVariableStringAndVariableSplitLength);
39+
40+
$mbStrSplitConstantStringWithOneSplitLengthAndValidEncoding = mb_str_split('abcdef', 1, 'UTF-8');
41+
assertType("array{'a', 'b', 'c', 'd', 'e', 'f'}", $mbStrSplitConstantStringWithOneSplitLengthAndValidEncoding);
42+
43+
$mbStrSplitConstantStringWithOneSplitLengthAndInvalidEncoding = mb_str_split('abcdef', 1, 'FAKE');
44+
assertType('false', $mbStrSplitConstantStringWithOneSplitLengthAndInvalidEncoding);
45+
46+
$mbStrSplitConstantStringWithOneSplitLengthAndVariableEncoding = mb_str_split('abcdef', 1, doFoo());
47+
assertType('list<string>|false', $mbStrSplitConstantStringWithOneSplitLengthAndVariableEncoding);
48+
49+
$mbStrSplitConstantStringWithGreaterSplitLengthThanStringLengthAndValidEncoding = mb_str_split('abcdef', 999, 'UTF-8');
50+
assertType("array{'abcdef'}", $mbStrSplitConstantStringWithGreaterSplitLengthThanStringLengthAndValidEncoding);
51+
52+
$mbStrSplitConstantStringWithGreaterSplitLengthThanStringLengthAndInvalidEncoding = mb_str_split('abcdef', 999, 'FAKE');
53+
assertType('false', $mbStrSplitConstantStringWithGreaterSplitLengthThanStringLengthAndInvalidEncoding);
54+
55+
$mbStrSplitConstantStringWithGreaterSplitLengthThanStringLengthAndVariableEncoding = mb_str_split('abcdef', 999, doFoo());
56+
assertType('list<string>|false', $mbStrSplitConstantStringWithGreaterSplitLengthThanStringLengthAndVariableEncoding);
57+
58+
$mbStrSplitConstantStringWithFailureSplitLengthAndValidEncoding = mb_str_split('abcdef', 0, 'UTF-8');
59+
assertType('false', $mbStrSplitConstantStringWithFailureSplitLengthAndValidEncoding);
60+
61+
$mbStrSplitConstantStringWithFailureSplitLengthAndInvalidEncoding = mb_str_split('abcdef', 0, 'FAKE');
62+
assertType('false', $mbStrSplitConstantStringWithFailureSplitLengthAndInvalidEncoding);
63+
64+
$mbStrSplitConstantStringWithFailureSplitLengthAndVariableEncoding = mb_str_split('abcdef', 0, doFoo());
65+
assertType('false', $mbStrSplitConstantStringWithFailureSplitLengthAndVariableEncoding);
66+
67+
$mbStrSplitConstantStringWithInvalidSplitLengthTypeAndValidEncoding = mb_str_split('abcdef', [], 'UTF-8');
68+
assertType('list<string>|false', $mbStrSplitConstantStringWithInvalidSplitLengthTypeAndValidEncoding);
69+
70+
$mbStrSplitConstantStringWithInvalidSplitLengthTypeAndInvalidEncoding = mb_str_split('abcdef', [], 'FAKE');
71+
assertType('false', $mbStrSplitConstantStringWithInvalidSplitLengthTypeAndInvalidEncoding);
72+
73+
$mbStrSplitConstantStringWithInvalidSplitLengthTypeAndVariableEncoding = mb_str_split('abcdef', [], doFoo());
74+
assertType('list<string>|false', $mbStrSplitConstantStringWithInvalidSplitLengthTypeAndVariableEncoding);
75+
76+
$mbStrSplitConstantStringWithVariableStringAndConstantSplitLengthAndValidEncoding = mb_str_split(doFoo() ? 'abcdef' : 'ghijkl', 1, 'UTF-8');
77+
assertType("array{'a', 'b', 'c', 'd', 'e', 'f'}|array{'g', 'h', 'i', 'j', 'k', 'l'}", $mbStrSplitConstantStringWithVariableStringAndConstantSplitLengthAndValidEncoding);
78+
79+
$mbStrSplitConstantStringWithVariableStringAndConstantSplitLengthAndInvalidEncoding = mb_str_split(doFoo() ? 'abcdef' : 'ghijkl', 1, 'FAKE');
80+
assertType('false', $mbStrSplitConstantStringWithVariableStringAndConstantSplitLengthAndInvalidEncoding);
81+
82+
$mbStrSplitConstantStringWithVariableStringAndConstantSplitLengthAndVariableEncoding = mb_str_split(doFoo() ? 'abcdef' : 'ghijkl', 1, doFoo());
83+
assertType('list<string>|false', $mbStrSplitConstantStringWithVariableStringAndConstantSplitLengthAndVariableEncoding);
84+
85+
$mbStrSplitConstantStringWithVariableStringAndVariableSplitLengthAndValidEncoding = mb_str_split(doFoo() ? 'abcdef' : 'ghijkl', doFoo() ? 1 : 2, 'UTF-8');
86+
assertType('list<string>|false', $mbStrSplitConstantStringWithVariableStringAndVariableSplitLengthAndValidEncoding);
87+
88+
$mbStrSplitConstantStringWithVariableStringAndVariableSplitLengthAndInvalidEncoding = mb_str_split(doFoo() ? 'abcdef' : 'ghijkl', doFoo() ? 1 : 2, 'FAKE');
89+
assertType('false', $mbStrSplitConstantStringWithVariableStringAndVariableSplitLengthAndInvalidEncoding);
90+
91+
$mbStrSplitConstantStringWithVariableStringAndVariableSplitLengthAndVariableEncoding = mb_str_split(doFoo() ? 'abcdef' : 'ghijkl', doFoo() ? 1 : 2, doFoo());
92+
assertType('list<string>|false', $mbStrSplitConstantStringWithVariableStringAndVariableSplitLengthAndVariableEncoding);
93+
}
94+
}

0 commit comments

Comments
 (0)