fix array processing in another post/returns analysis

Author: mvorisek

src/StaticAnalysis/ExecutableLinesFindingVisitor.php

@@ -137,6 +137,19 @@ private function computeReturns(): void
      */
     private function getLines(NodeAbstract $node, bool $fromReturns): array
     {
+        // ugly fix for non-fully AST based processing
+        // self::afterTraverse()/self::computeReturns() should be rewritten using self::leaveNode()
+        if ($node instanceof Array_ && [] !== $node->items) {
+            $parentNode = $node->getAttribute('parent');
+            if ($parentNode instanceof Return_ ||
+                $parentNode instanceof Expression ||
+                $parentNode instanceof Assign ||
+                $parentNode instanceof Array_
+            ) {
+                    return [];
+            }
+        }
+
         if (!$fromReturns && (
             $node instanceof Return_ ||
             $node instanceof Expression ||

tests/tests/Data/RawCodeCoverageDataTest.php

@@ -492,7 +492,6 @@ public function testReturnStatementWithConstantExprOnlyReturnTheLineOfLast(): vo
                 404,
                 414,
                 416,
-                423, // should be 424 as reported by xdebug
                 425,
                 437,
                 442,