mirror of
https://github.com/danog/psalm.git
synced 2024-12-02 09:37:59 +01:00
Fix various things
This commit is contained in:
parent
d53a9313a5
commit
e9fb3dcad2
@ -20,6 +20,7 @@
|
|||||||
"ext-json": "*",
|
"ext-json": "*",
|
||||||
"ext-libxml": "*",
|
"ext-libxml": "*",
|
||||||
"ext-tokenizer": "*",
|
"ext-tokenizer": "*",
|
||||||
|
"ext-mbstring": "*",
|
||||||
"amphp/amp": "^2.1",
|
"amphp/amp": "^2.1",
|
||||||
"amphp/byte-stream": "^1.5",
|
"amphp/byte-stream": "^1.5",
|
||||||
"composer/package-versions-deprecated": "^1.8.0",
|
"composer/package-versions-deprecated": "^1.8.0",
|
||||||
|
@ -703,7 +703,7 @@ class IfAnalyzer
|
|||||||
new DocblockTypeContradiction(
|
new DocblockTypeContradiction(
|
||||||
'if (false) is impossible',
|
'if (false) is impossible',
|
||||||
new CodeLocation($statements_analyzer, $cond),
|
new CodeLocation($statements_analyzer, $cond),
|
||||||
'false falsy',
|
'false falsy'
|
||||||
),
|
),
|
||||||
$statements_analyzer->getSuppressedIssues()
|
$statements_analyzer->getSuppressedIssues()
|
||||||
)) {
|
)) {
|
||||||
@ -714,7 +714,7 @@ class IfAnalyzer
|
|||||||
new TypeDoesNotContainType(
|
new TypeDoesNotContainType(
|
||||||
'if (false) is impossible',
|
'if (false) is impossible',
|
||||||
new CodeLocation($statements_analyzer, $cond),
|
new CodeLocation($statements_analyzer, $cond),
|
||||||
'false falsy',
|
'false falsy'
|
||||||
),
|
),
|
||||||
$statements_analyzer->getSuppressedIssues()
|
$statements_analyzer->getSuppressedIssues()
|
||||||
)) {
|
)) {
|
||||||
@ -727,7 +727,7 @@ class IfAnalyzer
|
|||||||
new RedundantConditionGivenDocblockType(
|
new RedundantConditionGivenDocblockType(
|
||||||
'if (true) is redundant',
|
'if (true) is redundant',
|
||||||
new CodeLocation($statements_analyzer, $cond),
|
new CodeLocation($statements_analyzer, $cond),
|
||||||
'true falsy',
|
'true falsy'
|
||||||
),
|
),
|
||||||
$statements_analyzer->getSuppressedIssues()
|
$statements_analyzer->getSuppressedIssues()
|
||||||
)) {
|
)) {
|
||||||
@ -738,7 +738,7 @@ class IfAnalyzer
|
|||||||
new RedundantCondition(
|
new RedundantCondition(
|
||||||
'if (true) is redundant',
|
'if (true) is redundant',
|
||||||
new CodeLocation($statements_analyzer, $cond),
|
new CodeLocation($statements_analyzer, $cond),
|
||||||
'true falsy',
|
'true falsy'
|
||||||
),
|
),
|
||||||
$statements_analyzer->getSuppressedIssues()
|
$statements_analyzer->getSuppressedIssues()
|
||||||
)) {
|
)) {
|
||||||
|
@ -48,7 +48,7 @@ class StubTest extends TestCase
|
|||||||
new Provider\FakeParserCacheProvider()
|
new Provider\FakeParserCacheProvider()
|
||||||
)
|
)
|
||||||
);
|
);
|
||||||
$project_analyzer->setPhpVersion('7.3');
|
$project_analyzer->setPhpVersion('7.4');
|
||||||
|
|
||||||
$config->setIncludeCollector(new IncludeCollector());
|
$config->setIncludeCollector(new IncludeCollector());
|
||||||
$config->visitComposerAutoloadFiles($project_analyzer, null);
|
$config->visitComposerAutoloadFiles($project_analyzer, null);
|
||||||
@ -1134,13 +1134,13 @@ class StubTest extends TestCase
|
|||||||
)
|
)
|
||||||
);
|
);
|
||||||
|
|
||||||
$this->project_analyzer->setPhpVersion('7.4');
|
|
||||||
|
|
||||||
$file_path = getcwd() . '/src/somefile.php';
|
$file_path = getcwd() . '/src/somefile.php';
|
||||||
|
|
||||||
$this->addFile(
|
$this->addFile(
|
||||||
$file_path,
|
$file_path,
|
||||||
'<?php'
|
'<?php
|
||||||
|
|
||||||
|
echo "hello";'
|
||||||
);
|
);
|
||||||
|
|
||||||
$this->expectException(\Psalm\Exception\InvalidClasslikeOverrideException::class);
|
$this->expectException(\Psalm\Exception\InvalidClasslikeOverrideException::class);
|
||||||
@ -1165,13 +1165,13 @@ class StubTest extends TestCase
|
|||||||
)
|
)
|
||||||
);
|
);
|
||||||
|
|
||||||
$this->project_analyzer->setPhpVersion('7.4');
|
|
||||||
|
|
||||||
$file_path = getcwd() . '/src/somefile.php';
|
$file_path = getcwd() . '/src/somefile.php';
|
||||||
|
|
||||||
$this->addFile(
|
$this->addFile(
|
||||||
$file_path,
|
$file_path,
|
||||||
'<?php'
|
'<?php
|
||||||
|
|
||||||
|
echo "hello";'
|
||||||
);
|
);
|
||||||
|
|
||||||
$this->expectException(\Psalm\Exception\InvalidMethodOverrideException::class);
|
$this->expectException(\Psalm\Exception\InvalidMethodOverrideException::class);
|
||||||
|
Loading…
Reference in New Issue
Block a user