Tests/Tokenizer: use markers for the testSwitchDefault() test

Using markers instead of offsets help stabilize the tests.

Author: rodrigoprimo

tests/Core/Tokenizers/Tokenizer/RecurseScopeMapDefaultKeywordConditionsTest.inc

@@ -28,8 +28,10 @@ function switchWithDefaultAndCurlies($i) {
         case 2:
             return 2;
         /* testSimpleSwitchDefaultWithCurlies */
-        default: {
+        default: /* testSimpleSwitchDefaultWithCurliesScopeOpener */ {
+            /* testSimpleSwitchDefaultWithCurliesConditionStop */
             return 'default';
+        /* testSimpleSwitchDefaultWithCurliesScopeCloser */
         }
     }
 }
@@ -60,6 +62,7 @@ function matchWithDefaultInSwitch() {
                 /* testMatchDefaultNestedInSwitchDefault */
                 default, => 'default',
             };
+            /* testSwitchDefaultCloserMarker */
             break;
     }
 }
@@ -207,6 +210,7 @@ function switchAndDefaultSharingScopeCloser($i) {
         /* testSwitchAndDefaultSharingScopeCloser */
         default:
             echo 'one';
+    /* testSwitchAndDefaultSharingScopeCloserScopeCloser */
     endswitch;
 }
 
@@ -221,6 +225,7 @@ function switchDefaultNestedIfWithAndWithoutBraces($i, $foo, $baz) {
             else {
                 echo 'else';
             }
+            /* testSwitchDefaultNestedIfWithAndWithoutBracesScopeCloser */
             break;
     }
 }

tests/Core/Tokenizers/Tokenizer/RecurseScopeMapDefaultKeywordConditionsTest.php

@@ -125,29 +125,39 @@ public static function dataMatchDefault()
      * Note: Cases and default structures within a switch control structure *do* get case/default scope
      * conditions.
      *
-     * @param string   $testMarker        The comment prefacing the target token.
-     * @param int      $openerOffset      The expected offset of the scope opener in relation to the testMarker.
-     * @param int      $closerOffset      The expected offset of the scope closer in relation to the testMarker.
-     * @param int|null $conditionStop     The expected offset at which tokens stop having T_DEFAULT as a scope condition.
-     * @param string   $testContent       The token content to look for.
-     * @param bool     $sharedScopeCloser Whether to skip checking for the `scope_condition` of the
-     *                                    scope closer. Needed when the default and switch
-     *                                    structures share a scope closer. See
-     *                                    https://github.com/PHPCSStandards/PHP_CodeSniffer/issues/810.
+     * @param string      $testMarker          The comment prefacing the target token.
+     * @param string|null $testOpenerMarker    The comment prefacing the scope opener token.
+     * @param string|null $testCloserMarker    The comment prefacing the scope closer token.
+     * @param string|null $conditionStopMarker The expected offset at which tokens stop having T_DEFAULT as a scope condition.
+     * @param string      $testContent         The token content to look for.
+     * @param bool        $sharedScopeCloser   Whether to skip checking for the `scope_condition` of the
+     *                                         scope closer. Needed when the default and switch
+     *                                         structures share a scope closer. See
+     *                                         https://github.com/PHPCSStandards/PHP_CodeSniffer/issues/810.
      *
      * @dataProvider dataSwitchDefault
      * @covers       PHP_CodeSniffer\Tokenizers\Tokenizer::recurseScopeMap
      *
      * @return void
      */
-    public function testSwitchDefault($testMarker, $openerOffset, $closerOffset, $conditionStop=null, $testContent='default', $sharedScopeCloser=false)
-    {
+    public function testSwitchDefault(
+        $testMarker, $testOpenerMarker=null, $testCloserMarker=null, $conditionStopMarker=null, $testContent='default', $sharedScopeCloser=false
+    ) {
         $tokens = $this->phpcsFile->getTokens();
 
         $token      = $this->getTargetToken($testMarker, [T_MATCH_DEFAULT, T_DEFAULT, T_STRING], $testContent);
         $tokenArray = $tokens[$token];
-        $expectedScopeOpener = ($token + $openerOffset);
-        $expectedScopeCloser = ($token + $closerOffset);
+
+        if ($testOpenerMarker === null) {
+            $testOpenerMarker = $testMarker;
+        }
+
+        if ($testCloserMarker === null) {
+            $testCloserMarker = $testMarker;
+        }
+
+        $expectedScopeOpener = $this->getTargetToken($testOpenerMarker, [T_COLON, T_OPEN_CURLY_BRACKET, T_SEMICOLON]);
+        $expectedScopeCloser = $this->getTargetToken($testCloserMarker, [T_BREAK, T_CLOSE_CURLY_BRACKET, T_RETURN, T_ENDSWITCH]);
 
         // Make sure we're looking at the right token.
         $this->assertSame(
@@ -179,86 +189,86 @@ public function testSwitchDefault($testMarker, $openerOffset, $closerOffset, $co
         $this->assertSame(
             $expectedScopeOpener,
             $tokenArray['scope_opener'],
-            sprintf('Scope opener of the T_DEFAULT token incorrect. Marker: %s.', $testMarker)
+            sprintf('Scope opener of the T_DEFAULT token incorrect. Marker: %s.', $testOpenerMarker)
         );
         $this->assertSame(
             $expectedScopeCloser,
             $tokenArray['scope_closer'],
-            sprintf('Scope closer of the T_DEFAULT token incorrect. Marker: %s.', $testMarker)
+            sprintf('Scope closer of the T_DEFAULT token incorrect. Marker: %s.', $testCloserMarker)
         );
 
         $opener = $tokenArray['scope_opener'];
         $this->assertArrayHasKey(
             'scope_condition',
             $tokens[$opener],
-            sprintf('Opener scope condition is not set. Marker: %s.', $testMarker)
+            sprintf('Opener scope condition is not set. Marker: %s.', $testOpenerMarker)
         );
         $this->assertArrayHasKey(
             'scope_opener',
             $tokens[$opener],
-            sprintf('Opener scope opener is not set. Marker: %s.', $testMarker)
+            sprintf('Opener scope opener is not set. Marker: %s.', $testOpenerMarker)
         );
         $this->assertArrayHasKey(
             'scope_closer',
             $tokens[$opener],
-            sprintf('Opener scope closer is not set. Marker: %s.', $testMarker)
+            sprintf('Opener scope closer is not set. Marker: %s.', $testOpenerMarker)
         );
         $this->assertSame(
             $token,
             $tokens[$opener]['scope_condition'],
-            sprintf('Opener scope condition is not the T_DEFAULT token. Marker: %s.', $testMarker)
+            sprintf('Opener scope condition is not the T_DEFAULT token. Marker: %s.', $testOpenerMarker)
         );
         $this->assertSame(
             $expectedScopeOpener,
             $tokens[$opener]['scope_opener'],
-            sprintf('T_DEFAULT opener scope opener token incorrect. Marker: %s.', $testMarker)
+            sprintf('T_DEFAULT opener scope opener token incorrect. Marker: %s.', $testOpenerMarker)
         );
         $this->assertSame(
             $expectedScopeCloser,
             $tokens[$opener]['scope_closer'],
-            sprintf('T_DEFAULT opener scope closer token incorrect. Marker: %s.', $testMarker)
+            sprintf('T_DEFAULT opener scope closer token incorrect. Marker: %s.', $testOpenerMarker)
         );
 
         if ($sharedScopeCloser === false) {
             $closer = $tokenArray['scope_closer'];
             $this->assertArrayHasKey(
                 'scope_condition',
                 $tokens[$closer],
-                sprintf('Closer scope condition is not set. Marker: %s.', $testMarker)
+                sprintf('Closer scope condition is not set. Marker: %s.', $testCloserMarker)
             );
             $this->assertArrayHasKey(
                 'scope_opener',
                 $tokens[$closer],
-                sprintf('Closer scope opener is not set. Marker: %s.', $testMarker)
+                sprintf('Closer scope opener is not set. Marker: %s.', $testCloserMarker)
             );
             $this->assertArrayHasKey(
                 'scope_closer',
                 $tokens[$closer],
-                sprintf('Closer scope closer is not set. Marker: %s.', $testMarker)
+                sprintf('Closer scope closer is not set. Marker: %s.', $testCloserMarker)
             );
             $this->assertSame(
                 $token,
                 $tokens[$closer]['scope_condition'],
-                sprintf('Closer scope condition is not the T_DEFAULT token. Marker: %s.', $testMarker)
+                sprintf('Closer scope condition is not the T_DEFAULT token. Marker: %s.', $testCloserMarker)
             );
             $this->assertSame(
                 $expectedScopeOpener,
                 $tokens[$closer]['scope_opener'],
-                sprintf('T_DEFAULT closer scope opener token incorrect. Marker: %s.', $testMarker)
+                sprintf('T_DEFAULT closer scope opener token incorrect. Marker: %s.', $testCloserMarker)
             );
             $this->assertSame(
                 $expectedScopeCloser,
                 $tokens[$closer]['scope_closer'],
-                sprintf('T_DEFAULT closer scope closer token incorrect. Marker: %s.', $testMarker)
+                sprintf('T_DEFAULT closer scope closer token incorrect. Marker: %s.', $testCloserMarker)
             );
         } else {
             $closer = $expectedScopeCloser;
         }//end if
 
         if (($opener + 1) !== $closer) {
             $end = $closer;
-            if (isset($conditionStop) === true) {
-                $end = ($opener + $conditionStop);
+            if (isset($conditionStopMarker) === true) {
+                $end = $this->getTargetToken($conditionStopMarker, [T_RETURN]);
             }
 
             for ($i = ($opener + 1); $i < $end; $i++) {
@@ -284,47 +294,47 @@ public static function dataSwitchDefault()
     {
         return [
             'simple_switch_default'                                     => [
-                'testMarker'   => '/* testSimpleSwitchDefault */',
-                'openerOffset' => 1,
-                'closerOffset' => 4,
+                'testMarker'       => '/* testSimpleSwitchDefault */',
+                'testOpenerMarker' => null,
+                'testCloserMarker' => null,
             ],
             'simple_switch_default_with_curlies'                        => [
                 // For a default structure with curly braces, the scope opener
                 // will be the open curly and the closer the close curly.
                 // However, scope conditions will not be set for open to close,
                 // but only for the open token up to the "break/return/continue" etc.
-                'testMarker'    => '/* testSimpleSwitchDefaultWithCurlies */',
-                'openerOffset'  => 3,
-                'closerOffset'  => 12,
-                'conditionStop' => 3,
+                'testMarker'          => '/* testSimpleSwitchDefaultWithCurlies */',
+                'testOpenerMarker'    => '/* testSimpleSwitchDefaultWithCurliesScopeOpener */',
+                'testCloserMarker'    => '/* testSimpleSwitchDefaultWithCurliesScopeCloser */',
+                'conditionStopMarker' => '/* testSimpleSwitchDefaultWithCurliesConditionStop */',
             ],
             'switch_default_toplevel'                                   => [
-                'testMarker'   => '/* testSwitchDefault */',
-                'openerOffset' => 1,
-                'closerOffset' => 43,
+                'testMarker'       => '/* testSwitchDefault */',
+                'testOpenerMarker' => null,
+                'testCloserMarker' => '/* testSwitchDefaultCloserMarker */',
             ],
             'switch_default_nested_in_match_case'                       => [
-                'testMarker'   => '/* testSwitchDefaultNestedInMatchCase */',
-                'openerOffset' => 1,
-                'closerOffset' => 20,
+                'testMarker'       => '/* testSwitchDefaultNestedInMatchCase */',
+                'testOpenerMarker' => null,
+                'testCloserMarker' => null,
             ],
             'switch_default_nested_in_match_default'                    => [
-                'testMarker'   => '/* testSwitchDefaultNestedInMatchDefault */',
-                'openerOffset' => 1,
-                'closerOffset' => 18,
+                'testMarker'       => '/* testSwitchDefaultNestedInMatchDefault */',
+                'testOpenerMarker' => null,
+                'testCloserMarker' => null,
             ],
             'switch_and_default_sharing_scope_closer'                   => [
                 'testMarker'        => '/* testSwitchAndDefaultSharingScopeCloser */',
-                'openerOffset'      => 1,
-                'closerOffset'      => 10,
+                'testOpenerMarker'  => null,
+                'testCloserMarker'  => '/* testSwitchAndDefaultSharingScopeCloserScopeCloser */',
                 'conditionStop'     => null,
                 'testContent'       => 'default',
                 'sharedScopeCloser' => true,
             ],
             'switch_and_default_with_nested_if_with_and_without_braces' => [
-                'testMarker'   => '/* testSwitchDefaultNestedIfWithAndWithoutBraces */',
-                'openerOffset' => 1,
-                'closerOffset' => 48,
+                'testMarker'       => '/* testSwitchDefaultNestedIfWithAndWithoutBraces */',
+                'testOpenerMarker' => null,
+                'testCloserMarker' => '/* testSwitchDefaultNestedIfWithAndWithoutBracesScopeCloser */',
             ],
         ];