Issue #3477599 by quietone: Fix Drupal.Commenting.VariableComment.Missing

core/modules/block/tests/src/Functional/BlockUiTest.php

@@ -34,8 +34,6 @@ class BlockUiTest extends BrowserTestBase {
    */
   protected $defaultTheme = 'stark';
 
-  protected $regions;
-
   /**
    * The submitted block values used by this test.
    *

core/modules/locale/tests/src/Functional/LocaleUpdateCronTest.php

@@ -16,7 +16,6 @@
 class LocaleUpdateCronTest extends LocaleUpdateBase {
 
   use CronRunTrait;
-  protected $batchOutput = [];
 
   /**
    * {@inheritdoc}

core/modules/navigation/tests/src/Functional/NavigationTopBarContentModerationTest.php

@@ -14,6 +14,9 @@
  */
 class NavigationTopBarContentModerationTest extends ModerationStateTestBase {
 
+  /**
+   * {@inheritdoc}
+   */
   protected $defaultTheme = 'stark';
 
   /**

core/modules/package_manager/tests/src/Functional/FailureMarkerRequirementTest.php

@@ -51,7 +51,7 @@ public function __construct() {}
       /**
        * {@inheritdoc}
        */
-      // phpcs:ignore DrupalPractice.CodeAnalysis.VariableAnalysis.UnusedVariable
+      // phpcs:ignore DrupalPractice.CodeAnalysis.VariableAnalysis.UnusedVariable, Drupal.Commenting.VariableComment.Missing
       protected string $type = 'test';
     };
     $failure_marker->write($stage, $message);

core/modules/package_manager/tests/src/Kernel/PathExcluder/SiteConfigurationExcluderTest.php

@@ -115,6 +115,9 @@ public function testDefaultSiteDirectoryPermissions(): void {
  */
 class TestSiteConfigurationExcluder extends SiteConfigurationExcluder {
 
+  /**
+   * The site path.
+   */
   public string $sitePath;
 
 }

core/modules/system/tests/src/Functional/Extension/LegacyHookTest.php

@@ -13,8 +13,14 @@
  */
 class LegacyHookTest extends BrowserTestBase {
 
+  /**
+   * {@inheritdoc}
+   */
   protected static $modules = ['legacy_hook_test'];
 
+  /**
+   * {@inheritdoc}
+   */
   protected $defaultTheme = 'stark';
 
   public function testLegacyHook(): void {

core/modules/system/tests/src/Functional/System/PageTitleTest.php

@@ -30,7 +30,6 @@ class PageTitleTest extends BrowserTestBase {
    * The test user.
    */
   protected $contentUser;
-  protected $savedTitle;
 
   /**
    * Implement setUp().

core/modules/user/tests/src/Kernel/Views/HandlerFilterPermissionTest.php

@@ -21,7 +21,6 @@ class HandlerFilterPermissionTest extends UserKernelTestBase {
    * @var array
    */
   public static $testViews = ['test_filter_permission'];
-  protected $columnMap;
 
   /**
    * Tests the permission filter handler.

core/phpcs.xml.dist

@@ -109,11 +109,6 @@
     <!-- Sniff for: DuplicateVar, EmptyVar, IncorrectVarType, InlineVariableName, WrongStyle -->
     <exclude name="Drupal.Commenting.VariableComment.MissingVar"/>
   </rule>
-  <rule ref="Drupal.Commenting.VariableComment.Missing">
-    <include-pattern>./core/tests/*</include-pattern>
-    <exclude-pattern>./core/*/tests/*</exclude-pattern>
-    <include-pattern>core/modules/*/tests/modules/*</include-pattern>
-  </rule>
   <rule ref="Drupal.ControlStructures.ControlSignature"/>
   <rule ref="Drupal.ControlStructures.ElseIf"/>
   <rule ref="Drupal.ControlStructures.InlineControlStructure"/>