Tests/Tokenizer: fix conditions checking range in T_DEFAULT test

Among other things, the RecurseScopeMapDefaultKeywordConditionsTest
::testSwitchDefault() method checks if tokens within the scope of a
`T_DEFAULT` token have `T_DEFAULT` set as one of its `conditions` array.

The test was incorrectly checking token `conditions` due to an
off-by-one error in the loop range when the `T_DEFAULT` uses curly
braces.

For a `T_DEFAULT` with curly braces, the tokenizer adds `T_DEFAULT` to
the `conditions` array of all the tokens within its scope up to the
`T_BREAK|T_RETURN|T_CONTINUE`, but the test was checking only until the
token before the `T_BREAK|T_RETURN|T_CONTINUE`. While for a `T_DEFAULT`
without curly braces, it adds to all the tokens within the scope until
the token before the scope closer.

This commit updates the test to ensure that all tokens within a
default case scope that should have the `T_DEFAULT` token in their
`conditions` array are properly checked. To achieve that the loop was
modified:

- The end point ($end) is now set to `$closer - 1` when curly braces are
 not used.
- The loop condition uses inclusive comparison (<=) to ensure that the
`$end` is also checked.

Author: rodrigoprimo

tests/Core/Tokenizers/Tokenizer/RecurseScopeMapDefaultKeywordConditionsTest.php

@@ -157,12 +157,12 @@ public function testSwitchDefault($testMarker, $openerOffset, $closerOffset, $co
         $this->assertSame($expectedScopeCloser, $tokens[$closer]['scope_closer'], 'T_DEFAULT closer scope closer token incorrect');
 
         if (($opener + 1) !== $closer) {
-            $end = $closer;
+            $end = ($closer - 1);
             if (isset($conditionStop) === true) {
                 $end = ($token + $conditionStop);
             }
 
-            for ($i = ($opener + 1); $i < $end; $i++) {
+            for ($i = ($opener + 1); $i <= $end; $i++) {
                 $this->assertArrayHasKey(
                     $token,
                     $tokens[$i]['conditions'],