Squiz/NonExecutableCode: fix bug with alternative switch control stru…

…ctures

The `T_ENDSWITCH` token wasn't being taken into account as a potential end token.

Includes unit tests.

Fixes 2512

src/Standards/Squiz/Sniffs/PHP/NonExecutableCodeSniff.php

@@ -105,6 +105,7 @@ public function process(File $phpcsFile, $stackPtr)
                         T_CASE,
                         T_DEFAULT,
                         T_CLOSE_CURLY_BRACKET,
+                        T_ENDSWITCH,
                     ],
                     ($end + 1)
                 );

src/Standards/Squiz/Tests/PHP/NonExecutableCodeUnitTest.1.inc

@@ -257,3 +257,42 @@ switch ($var) {
 
 end:
 echo 'j hit 17';
+
+// Issue 2512.
+class TestAlternativeControlStructures {
+
+    public function alternative_switch_in_function( $var ) {
+
+        switch ( $var ) :
+            case 'value1':
+                do_something();
+                break;
+
+            default:
+            case 'value2':
+                do_something_else();
+                break;
+        endswitch;
+    }
+
+    public function various_alternative_control_structures() {
+        $_while = 1;
+
+        for ( $a = 0; $a++ < 1; ) :
+            foreach ( [ 1 ] as $b ) :
+                while ( $_while-- ) :
+                    if ( 1 ) :
+                        switch ( 1 ) :
+                            default:
+                                echo 'yay, we made it!';
+                                break;
+                        endswitch;
+                    endif;
+                endwhile;
+            endforeach;
+        endfor;
+    }
+}
+
+$var_after_class_in_global_space = 1;
+do_something_else();