[DeadCode] Skip pipe operator on RemoveDeadStmtRector (#7941)

* [patch-3] Create void-return-functions-with-side-effects.php.inc

the examples assert and passthru are void return functions with side effects and should not be removed

* [DeadCode] Skip pipe operator on RemoveDeadStmtRector

---------

Co-authored-by: Anna Damm <anna.damm@check24.de>

Author: samsonasik

rules-tests/DeadCode/Rector/Expression/RemoveDeadStmtRector/Fixture/skip_pipe_operator.php.inc

@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\Expression\RemoveDeadStmtRector\Fixture;
+
+is_string('foo')
+    |> assert(...);
+
+sprintf('echo "%s"', 'foo')
+    |> passthru(...);
+
+?>

rules-tests/DeadCode/Rector/Expression/RemoveDeadStmtRector/RemoveDeadStmtRectorTest.php

@@ -13,6 +13,10 @@ final class RemoveDeadStmtRectorTest extends AbstractRectorTestCase
     #[DataProvider('provideData')]
     public function test(string $filePath): void
     {
+        if (str_ends_with($filePath, 'skip_pipe_operator.php.inc') && PHP_VERSION_ID < 80500) {
+            $this->markTestSkipped('test contains php 8.5 syntax early before transformation');
+        }
+
         $this->doTestFile($filePath);
     }
 

rules/DeadCode/NodeManipulator/LivingCodeManipulator.php

@@ -13,6 +13,7 @@
 use PhpParser\Node\Expr\BinaryOp\Coalesce;
 use PhpParser\Node\Expr\BinaryOp\LogicalAnd;
 use PhpParser\Node\Expr\BinaryOp\LogicalOr;
+use PhpParser\Node\Expr\BinaryOp\Pipe;
 use PhpParser\Node\Expr\BitwiseNot;
 use PhpParser\Node\Expr\BooleanNot;
 use PhpParser\Node\Expr\Cast;
@@ -120,7 +121,8 @@ private function isBinaryOpWithoutChange(Expr $expr): bool
             $expr instanceof BooleanAnd ||
             $expr instanceof LogicalOr ||
             $expr instanceof BooleanOr ||
-            $expr instanceof Coalesce
+            $expr instanceof Coalesce ||
+            $expr instanceof Pipe
         );
     }