Move unit test method Missing Docblock to new code

This allows it to be ignored in phpcs.xml.

Author: andrewnicols

moodle/Sniffs/Commenting/MissingDocblockSniff.php

@@ -180,7 +180,9 @@ protected function processFunctions(File $phpcsFile, int $stackPtr): void {
             $objectName = TokenUtil::getObjectName($phpcsFile, $typePtr);
             $objectType = TokenUtil::getObjectType($phpcsFile, $typePtr);
 
-            if ($extendsOrImplements) {
+            if ($isUnitTestFile) {
+                $phpcsFile->addWarning('Missing unit test docblock for %s %s', $typePtr, 'MissingTestDescription', [$objectType, $objectName]);
+            } elseif ($extendsOrImplements) {
                 $phpcsFile->addWarning('Missing docblock for %s %s', $typePtr, 'Missing', [$objectType, $objectName]);
             } else {
                 $phpcsFile->addError('Missing docblock for %s %s', $typePtr, 'Missing', [$objectType, $objectName]);

moodle/Tests/Sniffs/Commenting/MissingDocblockSniffTest.php

@@ -136,7 +136,7 @@ public static function docblockCorrectnessProvider(): array {
                     3 => 'Missing docblock for class example_test',
                 ],
                 'warnings' => [
-                    12 => 'Missing docblock for function test_the_thing',
+                    12 => 'Missing unit test docblock for function test_the_thing',
                 ],
             ],
         ];