Squiz/ClassFileName: expand the tests

This commit adds some extra tests for the sniff.
* Document handling of unfinished OO declarations (which contain enough info to still action).
* Safeguard that comments in unexpected places doesn't cause problems.

As this sniff looks at the file name of the file containing the tests, we cannot use the _normal_ `SniffNameUnitTest.#.inc` naming conventions for extra tests files, so the `getTestFiles()` method from the `AbstractSniffUnitTest` class needs to be overloaded to retrieve the test case files in a slightly different way.

Author: jrfnl

src/Standards/Squiz/Tests/Classes/ClassFileNameLiveCodingFailUnitTest.inc

@@ -0,0 +1,6 @@
+<?php
+
+// Intentional parse error. This should be the only test in this file.
+// Live coding/missing curlies should not block the sniff from functioning.
+
+class Class_FileName_Live_Coding_Fail_UnitTest

src/Standards/Squiz/Tests/Classes/ClassFileNameLiveCodingPassUnitTest.inc

@@ -0,0 +1,6 @@
+<?php
+
+// Intentional parse error. This should be the only test in this file.
+// Live coding/missing curlies should not block the sniff from functioning.
+
+class ClassFileNameLiveCodingPassUnitTest

src/Standards/Squiz/Tests/Classes/ClassFileNameUnitTest.inc

@@ -15,7 +15,7 @@ class CLASSFILENAMEUNITTEST {}
 interface classFileNameUnitTest {}
 interface classfilenameunittest {}
 interface CLASSFILENAMEUNITTEST {}
-trait classFileNameUnitTest {}
+trait /*comment*/ classFileNameUnitTest {}
 trait classfilenameunittest {}
 trait CLASSFILENAMEUNITTEST {}
 enum classFileNameUnitTest {}
@@ -39,7 +39,6 @@ trait ExtraClassFileNameUnitTest {}
 enum CompletelyWrongClassName {}
 enum ClassFileNameUnitTestExtra {}
 enum ClassFileNameUnitTestInc {}
-enum ExtraClassFileNameUnitTest {}
-
-
-?>
+enum
+    // Comment
+    ExtraClassFileNameUnitTest {}

src/Standards/Squiz/Tests/Classes/ClassFileNameUnitTest.php

@@ -9,6 +9,7 @@
 
 namespace PHP_CodeSniffer\Standards\Squiz\Tests\Classes;
 
+use DirectoryIterator;
 use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
 
 /**
@@ -20,46 +21,99 @@ final class ClassFileNameUnitTest extends AbstractSniffUnitTest
 {
 
 
+    /**
+     * Get a list of all test files to check.
+     *
+     * These will have the same base as the sniff name but different extensions.
+     * We ignore the .php file as it is the class.
+     *
+     * @param string $testFileBase The base path that the unit tests files will have.
+     *
+     * @return string[]
+     */
+    protected function getTestFiles($testFileBase)
+    {
+        $testFiles = [];
+
+        $dir = substr($testFileBase, 0, strrpos($testFileBase, DIRECTORY_SEPARATOR));
+        $di  = new DirectoryIterator($dir);
+
+        // Strip off the path and the "UnitTest." suffix from the $testFileBase to allow
+        // for some less conventionally named test case files.
+        $fileBase = str_replace($dir, '', $testFileBase);
+        $fileBase = substr($fileBase, 1, -9);
+
+        foreach ($di as $file) {
+            $fileName  = $file->getBasename('UnitTest.inc');
+            $extension = $file->getExtension();
+            if (substr($fileName, 0, strlen($fileBase)) === $fileBase
+                && $extension === 'inc'
+            ) {
+                $testFiles[] = $file->getPathname();
+            }
+        }
+
+        // Put them in order.
+        sort($testFiles, SORT_NATURAL);
+
+        return $testFiles;
+
+    }//end getTestFiles()
+
+
     /**
      * Returns the lines where errors should occur.
      *
      * The key of the array should represent the line number and the value
      * should represent the number of errors that should occur on that line.
      *
+     * @param string $testFile The name of the file being tested.
+     *
      * @return array<int, int>
      */
-    public function getErrorList()
+    public function getErrorList($testFile='')
     {
-        return [
-            12 => 1,
-            13 => 1,
-            14 => 1,
-            15 => 1,
-            16 => 1,
-            17 => 1,
-            18 => 1,
-            19 => 1,
-            20 => 1,
-            21 => 1,
-            22 => 1,
-            23 => 1,
-            27 => 1,
-            28 => 1,
-            29 => 1,
-            30 => 1,
-            31 => 1,
-            32 => 1,
-            33 => 1,
-            34 => 1,
-            35 => 1,
-            36 => 1,
-            37 => 1,
-            38 => 1,
-            39 => 1,
-            40 => 1,
-            41 => 1,
-            42 => 1,
-        ];
+        switch ($testFile) {
+        case 'ClassFileNameUnitTest.inc':
+            return [
+                12 => 1,
+                13 => 1,
+                14 => 1,
+                15 => 1,
+                16 => 1,
+                17 => 1,
+                18 => 1,
+                19 => 1,
+                20 => 1,
+                21 => 1,
+                22 => 1,
+                23 => 1,
+                27 => 1,
+                28 => 1,
+                29 => 1,
+                30 => 1,
+                31 => 1,
+                32 => 1,
+                33 => 1,
+                34 => 1,
+                35 => 1,
+                36 => 1,
+                37 => 1,
+                38 => 1,
+                39 => 1,
+                40 => 1,
+                41 => 1,
+                42 => 1,
+            ];
+
+        case 'ClassFileNameLiveCodingFailUnitTest.inc':
+            return [
+                6 => 1,
+            ];
+
+        default:
+            return [];
+        }//end switch
 
     }//end getErrorList()