Generic/UpperCaseConstantName: fix attributes false positive

This commit fixes false positives in the sniff when handling
attributes called `define`. Before this change the sniff was not
taking into account that `define` is a valid name for an attribute,
and it would incorrectly handle these cases as calls to the `define()`
function.

Author: rodrigoprimo

src/Standards/Generic/Sniffs/NamingConventions/UpperCaseConstantNameSniff.php

@@ -96,6 +96,11 @@ public function process(File $phpcsFile, $stackPtr)
             return;
         }
 
+        // Make sure this is not an attribute.
+        if (empty($tokens[$stackPtr]['nested_attributes']) === false) {
+            return;
+        }
+
         // If the next non-whitespace token after this token
         // is not an opening parenthesis then it is not a function call.
         $openBracket = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);

src/Standards/Generic/Tests/NamingConventions/UpperCaseConstantNameUnitTest.inc

@@ -58,3 +58,12 @@ $foo?->
 define('bar');
 
 const DEFINE = 'value';
+
+#[Define('some param')]
+class MyClass {}
+
+#[
+    AttributeA,
+    define('some param')
+]
+class MyClass {}