Tests: remove unreachable code

`break` after a `return` statement is redundant. As a side-note: I did have to chuckle when I saw that the `NonExecutableCode` test file also included this 😂
squizlabs · Oct 29, 2023 · cbd6fd5 · cbd6fd5
1 parent 321d252
commit cbd6fd5
Show file tree

Hide file tree

Showing 23 changed files with 54 additions and 77 deletions.
diff --git a/src/Standards/Generic/Tests/Classes/DuplicateClassNameUnitTest.php b/src/Standards/Generic/Tests/Classes/DuplicateClassNameUnitTest.php
@@ -50,27 +50,26 @@ public function getWarningList($testFile='')
                 12 => 1,
                 13 => 1,
             ];
-            break;
+
         case 'DuplicateClassNameUnitTest.2.inc':
             return [
                 2 => 1,
                 3 => 1,
                 4 => 1,
                 5 => 1,
             ];
-            break;
+
         case 'DuplicateClassNameUnitTest.5.inc':
             return [
                 3 => 1,
                 7 => 1,
             ];
-            break;
+
         case 'DuplicateClassNameUnitTest.6.inc':
             return [10 => 1];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getWarningList()

diff --git a/src/Standards/Generic/Tests/Files/InlineHTMLUnitTest.php b/src/Standards/Generic/Tests/Files/InlineHTMLUnitTest.php
@@ -30,16 +30,15 @@ public function getErrorList($testFile='')
         switch ($testFile) {
         case 'InlineHTMLUnitTest.3.inc':
             return [4 => 1];
-            break;
+
         case 'InlineHTMLUnitTest.4.inc':
             return [1 => 1];
-            break;
+
         case 'InlineHTMLUnitTest.7.inc':
             return [1 => 1];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()

diff --git a/src/Standards/Generic/Tests/Files/LineLengthUnitTest.php b/src/Standards/Generic/Tests/Files/LineLengthUnitTest.php
@@ -50,14 +50,13 @@ public function getErrorList($testFile='')
                 45 => 1,
                 82 => 1,
             ];
-            break;
+
         case 'LineLengthUnitTest.2.inc':
         case 'LineLengthUnitTest.3.inc':
             return [7 => 1];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()
@@ -89,20 +88,19 @@ public function getWarningList($testFile='')
                 75 => 1,
                 84 => 1,
             ];
-            break;
+
         case 'LineLengthUnitTest.2.inc':
         case 'LineLengthUnitTest.3.inc':
             return [6 => 1];
-            break;
+
         case 'LineLengthUnitTest.4.inc':
             return [
                 10 => 1,
                 14 => 1,
             ];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getWarningList()

diff --git a/src/Standards/Generic/Tests/Formatting/MultipleStatementAlignmentUnitTest.php b/src/Standards/Generic/Tests/Formatting/MultipleStatementAlignmentUnitTest.php
@@ -122,7 +122,7 @@ public function getWarningList($testFile='MultipleStatementAlignmentUnitTest.inc
                 499 => 1,
                 500 => 1,
             ];
-        break;
+
         case 'MultipleStatementAlignmentUnitTest.js':
             return [
                 11  => 1,
@@ -154,10 +154,9 @@ public function getWarningList($testFile='MultipleStatementAlignmentUnitTest.inc
                 114 => 1,
                 117 => 1,
             ];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getWarningList()

diff --git a/src/Standards/Generic/Tests/PHP/CharacterBeforePHPOpeningTagUnitTest.php b/src/Standards/Generic/Tests/PHP/CharacterBeforePHPOpeningTagUnitTest.php
@@ -30,10 +30,9 @@ public function getErrorList($testFile='')
         switch ($testFile) {
         case 'CharacterBeforePHPOpeningTagUnitTest.1.inc':
             return [2 => 1];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()

diff --git a/src/Standards/Generic/Tests/PHP/ClosingPHPTagUnitTest.php b/src/Standards/Generic/Tests/PHP/ClosingPHPTagUnitTest.php
@@ -30,12 +30,12 @@ public function getErrorList($testFile='')
         switch ($testFile) {
         case 'ClosingPHPTagUnitTest.1.inc':
             return [9 => 1];
+
         case 'ClosingPHPTagUnitTest.2.inc':
             return [5 => 1];
-            break;
+
         default:
             return [];
-            break;
         }
 
     }//end getErrorList()

diff --git a/src/Standards/Generic/Tests/PHP/LowerCaseConstantUnitTest.php b/src/Standards/Generic/Tests/PHP/LowerCaseConstantUnitTest.php
@@ -52,7 +52,7 @@ public function getErrorList($testFile='LowerCaseConstantUnitTest.inc')
                 95  => 1,
                 100 => 2,
             ];
-        break;
+
         case 'LowerCaseConstantUnitTest.js':
             return [
                 2  => 1,
@@ -64,10 +64,9 @@ public function getErrorList($testFile='LowerCaseConstantUnitTest.inc')
                 13 => 1,
                 14 => 1,
             ];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()

diff --git a/src/Standards/Generic/Tests/PHP/RequireStrictTypesUnitTest.php b/src/Standards/Generic/Tests/PHP/RequireStrictTypesUnitTest.php
@@ -30,7 +30,6 @@ public function getErrorList($testFile='')
         switch ($testFile) {
         case 'RequireStrictTypesUnitTest.1.inc':
             return [];
-            break;
         }
 
         return [1 => 1];

diff --git a/src/Standards/Generic/Tests/PHP/SyntaxUnitTest.php b/src/Standards/Generic/Tests/PHP/SyntaxUnitTest.php
@@ -32,10 +32,9 @@ public function getErrorList($testFile='')
         case 'SyntaxUnitTest.1.inc':
         case 'SyntaxUnitTest.2.inc':
             return [3 => 1];
-            break;
+
         default:
             return [];
-            break;
         }
 
     }//end getErrorList()

diff --git a/src/Standards/Generic/Tests/Strings/UnnecessaryStringConcatUnitTest.php b/src/Standards/Generic/Tests/Strings/UnnecessaryStringConcatUnitTest.php
@@ -37,7 +37,7 @@ public function getErrorList($testFile='UnnecessaryStringConcatUnitTest.inc')
                 19 => 1,
                 20 => 1,
             ];
-            break;
+
         case 'UnnecessaryStringConcatUnitTest.js':
             return [
                 1  => 1,
@@ -46,10 +46,9 @@ public function getErrorList($testFile='UnnecessaryStringConcatUnitTest.inc')
                 14 => 1,
                 15 => 1,
             ];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()

diff --git a/src/Standards/Generic/Tests/WhiteSpace/DisallowSpaceIndentUnitTest.php b/src/Standards/Generic/Tests/WhiteSpace/DisallowSpaceIndentUnitTest.php
@@ -85,7 +85,7 @@ public function getErrorList($testFile='DisallowSpaceIndentUnitTest.1.inc')
                 117 => 1,
                 118 => 1,
             ];
-            break;
+
         case 'DisallowSpaceIndentUnitTest.3.inc':
             return [
                 2  => 1,
@@ -96,16 +96,15 @@ public function getErrorList($testFile='DisallowSpaceIndentUnitTest.1.inc')
                 14 => 1,
                 15 => 1,
             ];
-            break;
+
         case 'DisallowSpaceIndentUnitTest.js':
             return [3 => 1];
-            break;
+
         case 'DisallowSpaceIndentUnitTest.css':
             return [2 => 1];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()

diff --git a/src/Standards/Squiz/Tests/Commenting/LongConditionClosingCommentUnitTest.php b/src/Standards/Squiz/Tests/Commenting/LongConditionClosingCommentUnitTest.php
@@ -60,7 +60,7 @@ public function getErrorList($testFile='LongConditionClosingCommentUnitTest.inc'
                 1008 => 1,
                 1032 => 1,
             ];
-            break;
+
         case 'LongConditionClosingCommentUnitTest.js':
             return [
                 47  => 1,
@@ -76,10 +76,9 @@ public function getErrorList($testFile='LongConditionClosingCommentUnitTest.inc'
                 439 => 1,
                 444 => 1,
             ];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()

diff --git a/src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.php b/src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.php
@@ -73,7 +73,7 @@ public function getErrorList($testFile='OperatorBracketUnitTest.inc')
                 193 => 1,
                 194 => 2,
             ];
-            break;
+
         case 'OperatorBracketUnitTest.js':
             return [
                 5   => 1,
@@ -90,10 +90,9 @@ public function getErrorList($testFile='OperatorBracketUnitTest.inc')
                 63  => 1,
                 108 => 1,
             ];
-             break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()

diff --git a/src/Standards/Squiz/Tests/Operators/ComparisonOperatorUsageUnitTest.php b/src/Standards/Squiz/Tests/Operators/ComparisonOperatorUsageUnitTest.php
@@ -59,7 +59,7 @@ public function getErrorList($testFile='ComparisonOperatorUsageUnitTest.inc')
                 131 => 1,
                 135 => 1,
             ];
-            break;
+
         case 'ComparisonOperatorUsageUnitTest.js':
             return [
                 5  => 1,
@@ -74,10 +74,9 @@ public function getErrorList($testFile='ComparisonOperatorUsageUnitTest.inc')
                 67 => 1,
                 71 => 1,
             ];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()

diff --git a/src/Standards/Squiz/Tests/PHP/CommentedOutCodeUnitTest.php b/src/Standards/Squiz/Tests/PHP/CommentedOutCodeUnitTest.php
@@ -58,16 +58,15 @@ public function getWarningList($testFile='CommentedOutCodeUnitTest.inc')
                 147 => 1,
                 158 => 1,
             ];
-            break;
+
         case 'CommentedOutCodeUnitTest.css':
             return [
                 7  => 1,
                 16 => 1,
             ];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getWarningList()

diff --git a/src/Standards/Squiz/Tests/PHP/DisallowInlineIfUnitTest.php b/src/Standards/Squiz/Tests/PHP/DisallowInlineIfUnitTest.php
@@ -33,13 +33,12 @@ public function getErrorList($testFile='DisallowInlineIfUnitTest.inc')
                 8  => 1,
                 18 => 1,
             ];
-            break;
+
         case 'DisallowInlineIfUnitTest.js':
             return [1 => 1];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()

diff --git a/src/Standards/Squiz/Tests/PHP/DisallowSizeFunctionsInLoopsUnitTest.php b/src/Standards/Squiz/Tests/PHP/DisallowSizeFunctionsInLoopsUnitTest.php
@@ -43,13 +43,12 @@ public function getErrorList($testFile='DisallowSizeFunctionsInLoopsUnitTest.inc
                 44 => 1,
                 46 => 1,
             ];
-            break;
+
         case 'DisallowSizeFunctionsInLoopsUnitTest.js':
             return [1 => 1];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()

diff --git a/src/Standards/Squiz/Tests/PHP/NonExecutableCodeUnitTest.php b/src/Standards/Squiz/Tests/PHP/NonExecutableCodeUnitTest.php
@@ -83,7 +83,7 @@ public function getWarningList($testFile='')
                 391 => 1,
                 396 => 1,
             ];
-            break;
+
         case 'NonExecutableCodeUnitTest.2.inc':
             return [
                 7  => 1,
@@ -97,7 +97,7 @@ public function getWarningList($testFile='')
                 70 => 2,
                 71 => 2,
             ];
-            break;
+
         case 'NonExecutableCodeUnitTest.3.inc':
             return [
                 27 => 1,
@@ -108,7 +108,6 @@ public function getWarningList($testFile='')
             ];
         default:
             return [];
-            break;
         }//end switch
 
     }//end getWarningList()

diff --git a/src/Standards/Squiz/Tests/WhiteSpace/ControlStructureSpacingUnitTest.php b/src/Standards/Squiz/Tests/WhiteSpace/ControlStructureSpacingUnitTest.php
@@ -61,7 +61,7 @@ public function getErrorList($testFile='ControlStructureSpacingUnitTest.inc')
                 261 => 1,
                 262 => 1,
             ];
-            break;
+
         case 'ControlStructureSpacingUnitTest.js':
             return [
                 3  => 1,
@@ -76,10 +76,9 @@ public function getErrorList($testFile='ControlStructureSpacingUnitTest.inc')
                 74 => 2,
                 75 => 2,
             ];
-            break;
+
         default:
             return [];
-            break;
         }//end switch
 
     }//end getErrorList()