[fix] skip HTML + PHP mix in DeclareStrictTypesTestsRector to avoid mallformed code (#600)

* [fix] skip HTML + PHP mix in DeclareStrictTypesTestsRector to avoid mallformed code

* cleanup RemoveOverrideFinalConstructTestCaseRector

Author: TomasVotruba

rules-tests/CodeQuality/Rector/StmtsAwareInterface/DeclareStrictTypesTestsRector/Fixture/skip_html_php_mix.php.inc

@@ -0,0 +1,12 @@
+<img src="asdf">
+
+<?php
+
+use PHPUnit\Framework\TestCase;
+
+final class SkipHtmlPHPMix extends TestCase
+{
+    public function test()
+    {
+    }
+}

rules/CodeQuality/Rector/StmtsAwareInterface/DeclareStrictTypesTestsRector.php

@@ -74,6 +74,10 @@ public function beforeTraverse(array $nodes): ?array
     {
         parent::beforeTraverse($nodes);
 
+        if ($this->file->containsHTML()) {
+            return null;
+        }
+
         if ($this->shouldSkipNodes($nodes, $this->file)) {
             return null;
         }

rules/PHPUnit120/Rector/Class_/RemoveOverrideFinalConstructTestCaseRector.php

@@ -71,17 +71,17 @@ public function refactor(Node $node): Node|null
             return null;
         }
 
-        $constructClassMethod = $node->getMethod(MethodName::CONSTRUCT);
-
-        if ($constructClassMethod instanceof ClassMethod) {
-            foreach ($node->stmts as $key => $stmt) {
-                if ($stmt instanceof ClassMethod && $this->isName($stmt, MethodName::CONSTRUCT)) {
-                    unset($node->stmts[$key]);
+        foreach ($node->stmts as $key => $stmt) {
+            if (! $stmt instanceof ClassMethod) {
+                continue;
+            }
 
-                    $node->setAttribute('hasRemovedFinalConstruct', true);
-                    return $node;
-                }
+            if (! $this->isName($stmt, MethodName::CONSTRUCT)) {
+                continue;
             }
+
+            unset($node->stmts[$key]);
+            return $node;
         }
 
         return null;