Do not mutate coalesce operator in the Assignment mutator (we have in…

…dependent AssignCoalesce mutator) (#1739)
infection · Oct 16, 2022 · 822e8f1 · 822e8f1
1 parent fbd8c44
commit 822e8f1
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/Mutator/Arithmetic/Assignment.php b/src/Mutator/Arithmetic/Assignment.php
@@ -79,6 +79,6 @@ public function mutate(Node $node): iterable
 
     public function canMutate(Node $node): bool
     {
-        return $node instanceof Node\Expr\AssignOp;
+        return $node instanceof Node\Expr\AssignOp && !$node instanceof Node\Expr\AssignOp\Coalesce;
     }
 }
diff --git a/tests/phpunit/Mutator/Arithmetic/AssignmentTest.php b/tests/phpunit/Mutator/Arithmetic/AssignmentTest.php
@@ -218,5 +218,14 @@ public function mutationsProvider(): iterable
 $a = $b;
 PHP
         ];
+
+        yield 'It does not mutate Coalesce operator' => [
+            <<<'PHP'
+<?php
+
+$a ??= $b;
+PHP
+            ,
+        ];
     }
 }