Skip to content

Commit 2f6f562

Browse files
committed
[rector] Rector fixes
1 parent 92c1cbd commit 2f6f562

2 files changed

Lines changed: 10 additions & 5 deletions

File tree

rules-tests/CodeQuality/Rector/Expression/ConfiguredMockEntityToSetterObjectRector/config/configured_rule.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,9 @@
22

33
declare(strict_types=1);
44

5+
use Rector\PHPUnit\CodeQuality\Rector\Expression\ConfiguredMockEntityToSetterObjectRector;
56
use Rector\Config\RectorConfig;
67

78
return static function (RectorConfig $rectorConfig): void {
8-
$rectorConfig->rule(\Rector\PHPUnit\CodeQuality\Rector\Expression\ConfiguredMockEntityToSetterObjectRector::class);
9+
$rectorConfig->rule(ConfiguredMockEntityToSetterObjectRector::class);
910
};

rules/CodeQuality/Rector/Expression/ConfiguredMockEntityToSetterObjectRector.php

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,10 @@
44

55
namespace Rector\PHPUnit\CodeQuality\Rector\Expression;
66

7+
use PhpParser\Node\Expr\Assign;
8+
use PhpParser\Node\Expr\Array_;
9+
use PhpParser\Node\Expr;
10+
use PhpParser\Node\Arg;
711
use PhpParser\Node;
812
use PhpParser\Node\Expr\MethodCall;
913
use PhpParser\Node\Expr\New_;
@@ -85,7 +89,7 @@ public function refactor(Node $node): ?array
8589
return null;
8690
}
8791

88-
if (! $node->expr instanceof Node\Expr\Assign) {
92+
if (! $node->expr instanceof Assign) {
8993
return null;
9094
}
9195

@@ -117,7 +121,7 @@ public function refactor(Node $node): ?array
117121
}
118122

119123
$definedGettersArg = $methodCall->getArgs()[1];
120-
if (! $definedGettersArg->value instanceof Node\Expr\Array_) {
124+
if (! $definedGettersArg->value instanceof Array_) {
121125
return null;
122126
}
123127

@@ -128,7 +132,7 @@ public function refactor(Node $node): ?array
128132
$assign->expr = new New_(new FullyQualified($mockedClassValue));
129133

130134
foreach ($definedGettersArray->items as $arrayItem) {
131-
if (! $arrayItem->key instanceof Node\Expr) {
135+
if (! $arrayItem->key instanceof Expr) {
132136
continue;
133137
}
134138

@@ -144,7 +148,7 @@ public function refactor(Node $node): ?array
144148

145149
$setterName = 'set' . substr($getterName, 3);
146150

147-
$setterMethodCall = new MethodCall($objectVariable, $setterName, [new Node\Arg($arrayItem->value)]);
151+
$setterMethodCall = new MethodCall($objectVariable, $setterName, [new Arg($arrayItem->value)]);
148152
$standaloneExpressions[] = new Expression($setterMethodCall);
149153
}
150154

0 commit comments

Comments
 (0)