Generic/UpperCaseConstantName: false positives when constant name is "DEFINE"

This commit fixes false positives when the sniff encounters a constant
named "DEFINE". When looking for calls to `define()`, the code was
checking only if there was a non-empty token after `define`. Instead, it
should check if the next non-empty token after `define` is
`T_OPEN_PARENTHESIS`.

Author: rodrigoprimo

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

@@ -99,7 +99,7 @@ public function process(File $phpcsFile, $stackPtr)
         // 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);
-        if ($openBracket === false) {
+        if ($openBracket === false || $tokens[$openBracket]['code'] !== T_OPEN_PARENTHESIS) {
             return;
         }
 

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

@@ -56,3 +56,5 @@ $foo-> /* comment */ define('bar');
 $foo?->
 // phpcs:ignore Stnd.Cat.SniffName -- for reasons.
 define('bar');
+
+const DEFINE = 'value';