Merge pull request #315 from PHPCSStandards/feature/tokenizer-php-add-tests-for-resolvesimpletoken

Tokenizer/PHP::resolveSimpleToken(): add dedicated tests

Author: jrfnl

tests/Core/Tokenizer/AbstractTokenizerTestCase.php

@@ -17,6 +17,7 @@
 use PHP_CodeSniffer\Tests\ConfigDouble;
 use PHP_CodeSniffer\Tests\Core\AbstractMethodUnitTest;
 use PHPUnit\Framework\TestCase;
+use ReflectionProperty;
 
 abstract class AbstractTokenizerTestCase extends TestCase
 {
@@ -102,4 +103,22 @@ protected function getTargetToken($commentString, $tokenType, $tokenContent=null
     }//end getTargetToken()
 
 
+    /**
+     * Clear the static "resolved tokens" cache property on the Tokenizer\PHP class.
+     *
+     * This method should be used selectively by tests to ensure the code under test is actually hit
+     * by the test testing the code.
+     *
+     * @return void
+     */
+    public static function clearResolvedTokensCache()
+    {
+        $property = new ReflectionProperty('PHP_CodeSniffer\Tokenizers\PHP', 'resolveTokenCache');
+        $property->setAccessible(true);
+        $property->setValue(null, []);
+        $property->setAccessible(false);
+
+    }//end clearResolvedTokensCache()
+
+
 }//end class

tests/Core/Tokenizer/ResolveSimpleTokenTest.inc

@@ -0,0 +1,51 @@
+<?php
+
+/* testBracesAndColon */
+switch ($var[10]) {
+    case TEST_COLON:
+        break;
+}
+
+/* testNamedParamColon */
+callMe(name: $var);
+
+/* testReturnTypeColon */
+$closure = function(): Type {};
+
+/* testConcat */
+echo 'text' . $var;
+
+/* testSimpleMathTokens */
+$a = 10 * 3 / 2 + 5 - 4 % 2;
+
+/* testUnaryPlusMinus */
+$a = +10 / -1;
+
+/* testBitwiseTokens */
+$a = CONST_A ^ CONST_B & CONST_C | CONST_D ~ CONST_E;
+
+try {
+/* testBitwiseOrInCatch */
+} catch ( Exception_A | Exception_B $e ) {
+}
+
+/* testLessThan */
+$a = 10 < $var;
+
+/* testGreaterThan */
+$a = 10 > $var;
+
+/* testBooleanNot */
+$a = ! $var;
+
+/* testComma */
+echo $a, $b, $c;
+
+/* testAsperand */
+$a = @callMe();
+
+/* testDollarAndCurlies */
+echo ${$var};
+
+/* testBacktick */
+$a = `ls -e`;