Generic/UselessOverridingMethod: bail if not checking a class method

The UselessOverridingMethod is triggered when the T_FUNCTION token is
found. This token is used for regular functions and also methods.

The sniff should run only for class methods (including abstract and
anonymous classes). Those are the only methods that can have a parent.
So this commit changes the sniff to bail early when dealing with a
regular function or a trait, interface or enum method.

Author: rodrigoprimo

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

@@ -60,6 +60,11 @@ public function process(File $phpcsFile, $stackPtr)
             return;
         }
 
+        // Skip functions and also trait, interface and enum methods.
+        if ($phpcsFile->hasCondition($stackPtr, [T_CLASS, T_ANON_CLASS]) === false) {
+            return;
+        }
+
         // Get function name.
         $methodName = $phpcsFile->getDeclarationName($stackPtr);
 

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

@@ -81,3 +81,31 @@ interface InterfaceFoo {
 trait TraitFoo {
     abstract public function sniffShouldBailEarly();
 }
+
+abstract class AbstractBar extends AbstractFoo {
+    public function uselessMethodInAbstractClass() {
+        parent::uselessMethodInAbstractClass();
+    }
+
+    public function usefulMethodInAbstractClass() {
+        $a = 1;
+        parent::usefulMethodInAbstractClass($a);
+    }
+}
+
+enum EnumFoo {
+    public function sniffShouldBailEarly() {}
+}
+
+new class extends ParentClass {
+    public function __construct() {
+        parent::__construct();
+    }
+};
+
+new class extends ParentClass {
+    public function __construct() {
+        $a = 10;
+        parent::__construct($a);
+    }
+};

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

@@ -0,0 +1,8 @@
+<?php
+
+// Intentional fatal error (calling parent inside a function).
+// Testing that the sniff is *not* triggered in this case.
+
+function __construct() {
+    parent::__construct();
+}

src/Standards/Generic/Tests/CodeAnalysis/UselessOverridingMethodUnitTest.php

@@ -50,11 +50,13 @@ public function getWarningList($testFile='')
         switch ($testFile) {
         case 'UselessOverridingMethodUnitTest.1.inc':
             return [
-                4  => 1,
-                16 => 1,
-                38 => 1,
-                56 => 1,
-                68 => 1,
+                4   => 1,
+                16  => 1,
+                38  => 1,
+                56  => 1,
+                68  => 1,
+                86  => 1,
+                101 => 1,
             ];
         default:
             return [];