REPO-70: [EQP][Sniffs Consolidation] Assign severity to each sniff

- review fixes

Author: lenaorobei

Magento/Sniffs/Functions/StaticFunctionSniff.php

@@ -0,0 +1,59 @@
+<?php
+/**
+ * Copyright © Magento. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+namespace Magento\Sniffs\Functions;
+
+use PHP_CodeSniffer\Sniffs\Sniff;
+use PHP_CodeSniffer\Files\File;
+
+/**
+ * Detects static function definitions.
+ */
+class StaticFunctionSniff implements Sniff
+{
+    /**
+     * String representation of warning.
+     *
+     * @var string
+     */
+    protected $warningMessage = 'Static method cannot be intercepted and its use is discouraged.';
+
+    /**
+     * Warning violation code.
+     *
+     * @var string
+     */
+    protected $warningCode = 'StaticFunction';
+
+    /**
+     * @inheritDoc
+     */
+    public function register()
+    {
+        return [T_STATIC];
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function process(File $phpcsFile, $stackPtr)
+    {
+        $posOfFunction = $phpcsFile->findNext(T_FUNCTION, $stackPtr) + 1;
+        $tokens = array_slice($phpcsFile->getTokens(), $stackPtr, $posOfFunction - $stackPtr);
+
+        $allowedTypes = [T_STATIC => true, T_WHITESPACE => true, T_FUNCTION => true];
+        foreach ($tokens as $token) {
+            $code = $token['code'];
+            if (!array_key_exists($code, $allowedTypes)) {
+                break;
+            }
+
+            if ($code === T_FUNCTION) {
+                $phpcsFile->addWarning($this->warningMessage, $posOfFunction, $this->warningCode);
+            }
+        }
+    }
+}

Magento/Tests/Functions/StaticFunctionUnitTest.inc

@@ -0,0 +1,26 @@
+<?php
+
+namespace Foo\Bar;
+
+interface  FooInterface
+{
+    public static
+    function aStaticMethod();
+
+    public function normalMethod();
+}
+
+class Foo implements FooInterface
+{
+    private static $property;
+
+    public static function aStaticMethod()
+    {
+        return self::$property;
+    }
+
+    public function normalMethod()
+    {
+        return $this;
+    }
+}

Magento/Tests/Functions/StaticFunctionUnitTest.php

@@ -0,0 +1,33 @@
+<?php
+/**
+ * Copyright © Magento. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+namespace Magento\Tests\Functions;
+
+use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
+
+/**
+ * Class StaticFunctionSniffTest
+ */
+class StaticFunctionUnitTest extends AbstractSniffUnitTest
+{
+    /**
+     * @inheritdoc
+     */
+    protected function getErrorList()
+    {
+        return [];
+    }
+
+    /**
+     * @inheritdoc
+     */
+    protected function getWarningList()
+    {
+        return [
+            8 => 1,
+            17 => 1
+        ];
+    }
+}

Magento/ruleset.xml

@@ -117,6 +117,9 @@
         <severity>8</severity>
         <type>warning</type>
     </rule>
+    <rule ref="Magento.Functions.StaticFunction">
+        <severity>6</severity>
+    </rule>
     <rule ref="Magento.Files.LineLength">
         <properties>
             <property name="lineLimit" value="120"/>
@@ -398,6 +401,10 @@
         <severity>6</severity>
         <type>warning</type>
     </rule>
+    <rule ref="Squiz.Operators.ValidLogicalOperators">
+        <severity>6</severity>
+        <type>warning</type>
+    </rule>
     <rule ref="Squiz.WhiteSpace.ScopeClosingBrace">
         <severity>6</severity>
         <type>warning</type>