Skip to content

Commit 6938446

Browse files
committed
fix
1 parent f510ea5 commit 6938446

3 files changed

Lines changed: 12 additions & 0 deletions

File tree

‎src/Rules/PHPUnit/AssertEqualsIsDiscouragedRule.php‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,10 @@ public function processNode(Node $node, Scope $scope): array
7373
),
7474
)->identifier('phpunit.assertEquals')
7575
->fixNode($node, static function (CallLike $node) use ($correctName) {
76+
if (AssertRuleHelper::hasNamedOrUnpackedArguments($node)) {
77+
return $node;
78+
}
79+
7680
$node->name = new Node\Identifier($correctName);
7781

7882
return $node;

‎src/Rules/PHPUnit/AssertSameBooleanExpectedRule.php‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,10 @@ public function processNode(Node $node, Scope $scope): array
5151
RuleErrorBuilder::message('You should use assertTrue() instead of assertSame() when expecting "true"')
5252
->identifier('phpunit.assertTrue')
5353
->fixNode($node, static function (CallLike $node) {
54+
if (AssertRuleHelper::hasNamedOrUnpackedArguments($node)) {
55+
return $node;
56+
}
57+
5458
$node->name = new Node\Identifier('assertTrue');
5559
$node->args = self::rewriteArgs($node->args);
5660

‎src/Rules/PHPUnit/AssertSameNullExpectedRule.php‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,10 @@ public function processNode(Node $node, Scope $scope): array
5151
RuleErrorBuilder::message('You should use assertNull() instead of assertSame(null, $actual).')
5252
->identifier('phpunit.assertNull')
5353
->fixNode($node, static function (CallLike $node) {
54+
if (AssertRuleHelper::hasNamedOrUnpackedArguments($node)) {
55+
return $node;
56+
}
57+
5458
$node->name = new Node\Identifier('assertNull');
5559
$node->args = self::rewriteArgs($node->args);
5660

0 commit comments

Comments
 (0)