Generic/UselessOverridingMethod: improve sniff handling of a parse error

This commit improves how the sniff handles code that is missing the
closing parenthesis in the parent method call. Now, the code will
intentionally bail early in such cases.

Before, the sniff would unintentionally bail when such cases were found
at a later point and would incorrectly consider everything after the
opening parenthesis to be the first parameter of the parent method.

The commit includes a test.

Author: rodrigoprimo

src/Standards/Generic/Sniffs/CodeAnalysis/UselessOverridingMethodSniff.php

@@ -109,7 +109,7 @@ public function process(File $phpcsFile, $stackPtr)
         $next = $phpcsFile->findNext(Tokens::$emptyTokens, ($next + 1), null, true);
 
         // Skip for invalid code.
-        if ($next === false || $tokens[$next]['code'] !== T_OPEN_PARENTHESIS) {
+        if ($next === false || $tokens[$next]['code'] !== T_OPEN_PARENTHESIS || isset($tokens[$next]['parenthesis_closer']) === false) {
             return;
         }
 

src/Standards/Generic/Tests/CodeAnalysis/UselessOverridingMethodUnitTest.6.inc

@@ -0,0 +1,10 @@
+<?php
+
+// Intentional parse error (missing closing parenthesis in parent method call).
+// Testing that the sniff is *not* triggered in this case.
+
+class FooBar {
+    public function __construct() {
+        parent::__construct(
+    }
+}