-
-
Notifications
You must be signed in to change notification settings - Fork 7
/
AssertEqualsFixer.php
91 lines (73 loc) · 2.67 KB
/
AssertEqualsFixer.php
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
<?php
declare(strict_types=1);
/*
* This file is part of Contao.
*
* (c) Leo Feyer
*
* @license LGPL-3.0-or-later
*/
namespace Contao\EasyCodingStandard\Fixer;
use PhpCsFixer\AbstractFixer;
use PhpCsFixer\FixerDefinition\CodeSample;
use PhpCsFixer\FixerDefinition\FixerDefinition;
use PhpCsFixer\FixerDefinition\FixerDefinitionInterface;
use PhpCsFixer\Tokenizer\Analyzer\ArgumentsAnalyzer;
use PhpCsFixer\Tokenizer\Token;
use PhpCsFixer\Tokenizer\Tokens;
final class AssertEqualsFixer extends AbstractFixer
{
private array $mapper = [
'assertEquals' => 'assertSame',
'assertNotEquals' => 'assertNotSame',
];
public function getDefinition(): FixerDefinitionInterface
{
return new FixerDefinition(
'Unless comparing objects, assertSame() should be used instead of asserEquals().',
[
new CodeSample(
<<<'EOT'
<?php
public function testFoo(): void
{
$obj = new Foo();
$this->assertSame("foo", $obj->getName());
$this->assertEquals(new Foo(), $obj);
}
EOT,
),
],
);
}
public function isCandidate(Tokens $tokens): bool
{
return $tokens->isTokenKindFound(T_STRING);
}
protected function applyFix(\SplFileInfo $file, Tokens $tokens): void
{
for ($index = 1, $count = \count($tokens); $index < $count; ++$index) {
if (!$tokens[$index]->isGivenKind(T_STRING)) {
continue;
}
$name = $tokens[$index]->getContent();
if (!isset($this->mapper[$name]) || !$this->isMethodCall($index, $tokens)) {
continue;
}
$end = $tokens->findBlockEnd(Tokens::BLOCK_TYPE_PARENTHESIS_BRACE, $index + 1);
$argumentsAnalyzer = new ArgumentsAnalyzer();
$argumentsIndexes = $argumentsAnalyzer->getArguments($tokens, $index + 1, $end);
$argStart = array_key_first($argumentsIndexes);
$argEnd = $argumentsIndexes[$argStart];
// Continue if the expected value is a or contains an object
if ($tokens->findGivenKind(T_NEW, $argStart, $argEnd)) {
continue;
}
$tokens->offsetSet($index, new Token([T_STRING, $this->mapper[$name]]));
}
}
private function isMethodCall(int $index, Tokens $tokens): bool
{
return $tokens[$index - 1]->isGivenKind([T_OBJECT_OPERATOR, T_PAAMAYIM_NEKUDOTAYIM]) && $tokens[$index + 1]->equals('(');
}
}