Skip to content

Commit 7f1306e

Browse files
committed
Update AssertSameBooleanExpectedRule.php
1 parent 5bc3006 commit 7f1306e

File tree

1 file changed

+15
-8
lines changed

1 file changed

+15
-8
lines changed

src/Rules/PHPUnit/AssertSameBooleanExpectedRule.php

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,7 @@ public function processNode(Node $node, Scope $scope): array
5151
->identifier('phpunit.assertTrue')
5252
->fixNode($node, static function (CallLike $node) {
5353
$node->name = new Node\Identifier('assertTrue');
54-
55-
$args = $node->getArgs();
56-
unset($args[0]);
57-
$node->args = $args;
54+
$node->args = self::rewriteArgs($node->args);
5855

5956
return $node;
6057
})
@@ -68,10 +65,7 @@ public function processNode(Node $node, Scope $scope): array
6865
->identifier('phpunit.assertFalse')
6966
->fixNode($node, static function (CallLike $node) {
7067
$node->name = new Node\Identifier('assertFalse');
71-
72-
$args = $node->getArgs();
73-
unset($args[0]);
74-
$node->args = $args;
68+
$node->args = self::rewriteArgs($node->args);
7569

7670
return $node;
7771
})
@@ -82,4 +76,17 @@ public function processNode(Node $node, Scope $scope): array
8276
return [];
8377
}
8478

79+
/**
80+
* @param array<Node\Arg|Node\VariadicPlaceholder> $args
81+
* @return list<Node\Arg|Node\VariadicPlaceholder>
82+
*/
83+
private static function rewriteArgs(array $args): array
84+
{
85+
$newArgs = [];
86+
for ($i = 1; $i < count($args); $i++) {
87+
$newArgs[] = $args[$i];
88+
}
89+
return $newArgs;
90+
}
91+
8592
}

0 commit comments

Comments
 (0)