-
Notifications
You must be signed in to change notification settings - Fork 65
Expand file tree
/
Copy pathAssertCompareOnCountableWithMethodToAssertCountRector.php
More file actions
127 lines (107 loc) · 3.71 KB
/
AssertCompareOnCountableWithMethodToAssertCountRector.php
File metadata and controls
127 lines (107 loc) · 3.71 KB
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
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
<?php
declare(strict_types=1);
namespace Rector\PHPUnit\CodeQuality\Rector\MethodCall;
use PhpParser\Node;
use PhpParser\Node\Arg;
use PhpParser\Node\Expr\FuncCall;
use PhpParser\Node\Expr\MethodCall;
use PhpParser\Node\Expr\StaticCall;
use PhpParser\Node\Identifier;
use PHPStan\Type\ObjectType;
use Rector\PHPUnit\NodeAnalyzer\TestsNodeAnalyzer;
use Rector\Rector\AbstractRector;
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
/**
* @see \Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\AssertCompareOnCountableWithMethodToAssertCountRector\AssertCompareOnCountableWithMethodToAssertCountRectorTest
*/
final class AssertCompareOnCountableWithMethodToAssertCountRector extends AbstractRector
{
public function __construct(
private readonly TestsNodeAnalyzer $testsNodeAnalyzer
) {
}
public function getRuleDefinition(): RuleDefinition
{
return new RuleDefinition(
'Replaces use of assertSame and assertEquals on Countable objects with count method',
[
new CodeSample(
<<<'CODE_SAMPLE'
$this->assertSame(1, $countable->count());
CODE_SAMPLE
,
<<<'CODE_SAMPLE'
$this->assertCount(1, $countable);
CODE_SAMPLE
),
new CodeSample(
'$this->assertSame(10, count($anything), "message");',
'$this->assertCount(10, $anything, "message");'
),
]
);
}
/**
* @return array<class-string<MethodCall|StaticCall>>
*/
public function getNodeTypes(): array
{
return [MethodCall::class, StaticCall::class];
}
/**
* @param MethodCall|StaticCall $node
*/
public function refactor(Node $node): MethodCall|StaticCall|null
{
if (! $this->testsNodeAnalyzer->isPHPUnitMethodCallNames(
$node,
['assertSame', 'assertNotSame', 'assertEquals', 'assertNotEquals']
)) {
return null;
}
if ($node->isFirstClassCallable()) {
return null;
}
$assertArgs = $node->getArgs();
if (count($assertArgs) < 2) {
return null;
}
$comparedExpr = $assertArgs[1]->value;
if (
$comparedExpr instanceof FuncCall
&& $this->isNames($comparedExpr->name, ['count', 'sizeof', 'iterator_count'])
) {
$countArg = $comparedExpr->getArgs()[0];
if ($this->isObjectType($countArg->value, new ObjectType('Generator'))) {
return null;
}
$assertArgs[1] = new Arg($countArg->value);
$node->args = $assertArgs;
$this->renameMethod($node);
return $node;
}
if (
($comparedExpr instanceof MethodCall)
&& $this->isName($comparedExpr->name, 'count')
&& $comparedExpr->getArgs() === []
) {
$type = $this->getType($comparedExpr->var);
if ((new ObjectType('Countable'))->isSuperTypeOf($type)->yes()) {
$args = $assertArgs;
$args[1] = new Arg($comparedExpr->var);
$node->args = $args;
$this->renameMethod($node);
}
}
return null;
}
private function renameMethod(MethodCall|StaticCall $node): void
{
if ($this->isNames($node->name, ['assertSame', 'assertEquals'])) {
$node->name = new Identifier('assertCount');
} elseif ($this->isNames($node->name, ['assertNotSame', 'assertNotEquals'])) {
$node->name = new Identifier('assertNotCount');
}
}
}