Issue #2822382 by voleger, pfrenssen, erozqba, RytoEX, borisson_, sidharthap,...

Issue #2822382 by voleger, pfrenssen, erozqba, RytoEX, borisson_, sidharthap, Jo Fitzgerald, shashikant_chauhan: Make every  property protected on classes extending BrowserTestBase and KernelTestBase

core/modules/action/tests/src/Functional/ActionListTest.php

@@ -16,7 +16,7 @@ class ActionListTest extends BrowserTestBase {
    *
    * @var array
    */
-  public static $modules = ['action'];
+  protected static $modules = ['action'];
 
   /**
    * Tests the behavior when there are no actions to list in the admin page.

core/modules/action/tests/src/Functional/ActionUninstallTest.php

@@ -17,7 +17,7 @@ class ActionUninstallTest extends BrowserTestBase {
    *
    * @var array
    */
-  public static $modules = ['views', 'action'];
+  protected static $modules = ['views', 'action'];
 
   /**
    * Tests Action uninstall.

core/modules/action/tests/src/Functional/BulkFormTest.php

@@ -18,7 +18,7 @@ class BulkFormTest extends BrowserTestBase {
    *
    * @var array
    */
-  public static $modules = ['node', 'action_bulk_test'];
+  protected static $modules = ['node', 'action_bulk_test'];
 
   /**
    * Tests the bulk form.

core/modules/action/tests/src/Functional/ConfigurationTest.php

@@ -18,7 +18,7 @@ class ConfigurationTest extends BrowserTestBase {
    *
    * @var array
    */
-  public static $modules = ['action'];
+  protected static $modules = ['action'];
 
   /**
    * Tests configuration of advanced actions through administration interface.

core/modules/action/tests/src/Kernel/Migrate/d6/MigrateActionConfigsTest.php

@@ -17,7 +17,7 @@ class MigrateActionConfigsTest extends MigrateDrupal6TestBase {
   /**
    * {@inheritdoc}
    */
-  public static $modules = ['action'];
+  protected static $modules = ['action'];
 
   /**
    * {@inheritdoc}

core/modules/action/tests/src/Kernel/Migrate/d6/MigrateActionsTest.php

@@ -12,7 +12,7 @@
  */
 class MigrateActionsTest extends MigrateDrupal6TestBase {
 
-  public static $modules = ['action', 'comment', 'node'];
+  protected static $modules = ['action', 'comment', 'node'];
 
   /**
    * {@inheritdoc}

core/modules/action/tests/src/Kernel/Migrate/d7/MigrateActionConfigsTest.php

@@ -17,7 +17,7 @@ class MigrateActionConfigsTest extends MigrateDrupal7TestBase {
   /**
    * {@inheritdoc}
    */
-  public static $modules = ['action'];
+  protected static $modules = ['action'];
 
   /**
    * {@inheritdoc}

core/modules/action/tests/src/Kernel/Migrate/d7/MigrateActionsTest.php

@@ -12,7 +12,7 @@
  */
 class MigrateActionsTest extends MigrateDrupal7TestBase {
 
-  public static $modules = ['action', 'comment', 'node'];
+  protected static $modules = ['action', 'comment', 'node'];
 
   /**
    * {@inheritdoc}

core/modules/action/tests/src/Kernel/Plugin/Action/EmailActionTest.php

@@ -16,7 +16,7 @@ class EmailActionTest extends KernelTestBase {
   /**
    * {@inheritdoc}
    */
-  public static $modules = ['system', 'user', 'action', 'dblog'];
+  protected static $modules = ['system', 'user', 'action', 'dblog'];
 
   /**
    * {@inheritdoc}

core/modules/action/tests/src/Kernel/Plugin/migrate/source/ActionTest.php

@@ -15,7 +15,7 @@ class ActionTest extends MigrateSqlSourceTestBase {
   /**
    * {@inheritdoc}
    */
-  public static $modules = ['action', 'migrate_drupal', 'system'];
+  protected static $modules = ['action', 'migrate_drupal', 'system'];
 
   /**
    * {@inheritdoc}

core/modules/aggregator/src/Tests/AggregatorTestBase.php

@@ -27,7 +27,7 @@ abstract class AggregatorTestBase extends WebTestBase {
    *
    * @var array
    */
-  public static $modules = ['block', 'node', 'aggregator', 'aggregator_test', 'views'];
+  protected static $modules = ['block', 'node', 'aggregator', 'aggregator_test', 'views'];
 
   /**
    * {@inheritdoc}

core/modules/aggregator/tests/src/Functional/AggregatorRenderingTest.php

@@ -17,7 +17,7 @@ class AggregatorRenderingTest extends AggregatorTestBase {
    *
    * @var array
    */
-  public static $modules = ['block', 'test_page_test'];
+  protected static $modules = ['block', 'test_page_test'];
 
   protected function setUp() {
     parent::setUp();

core/modules/aggregator/tests/src/Functional/AggregatorTestBase.php

@@ -24,7 +24,7 @@ abstract class AggregatorTestBase extends BrowserTestBase {
    *
    * @var array
    */
-  public static $modules = ['block', 'node', 'aggregator', 'aggregator_test', 'views'];
+  protected static $modules = ['block', 'node', 'aggregator', 'aggregator_test', 'views'];
 
   /**
    * {@inheritdoc}

core/modules/aggregator/tests/src/Functional/DeleteFeedTest.php

@@ -14,7 +14,7 @@ class DeleteFeedTest extends AggregatorTestBase {
    *
    * @var array
    */
-  public static $modules = ['block'];
+  protected static $modules = ['block'];
 
   /**
    * Deletes a feed and ensures that all of its services are deleted.

core/modules/aggregator/tests/src/Functional/FeedCacheTagsTest.php

@@ -17,7 +17,7 @@ class FeedCacheTagsTest extends EntityWithUriCacheTagsTestBase {
   /**
    * {@inheritdoc}
    */
-  public static $modules = ['aggregator'];
+  protected static $modules = ['aggregator'];
 
   /**
    * {@inheritdoc}

core/modules/aggregator/tests/src/Functional/FeedLanguageTest.php

@@ -19,7 +19,7 @@ class FeedLanguageTest extends AggregatorTestBase {
    *
    * @var array
    */
-  public static $modules = ['language'];
+  protected static $modules = ['language'];
 
   /**
    * List of langcodes.

core/modules/aggregator/tests/src/Functional/Hal/FeedHalJsonBasicAuthTest.php

@@ -14,7 +14,7 @@ class FeedHalJsonBasicAuthTest extends FeedHalJsonTestBase {
   /**
    * {@inheritdoc}
    */
-  public static $modules = ['basic_auth'];
+  protected static $modules = ['basic_auth'];
 
   /**
    * {@inheritdoc}

core/modules/aggregator/tests/src/Functional/Hal/FeedHalJsonTestBase.php

@@ -12,7 +12,7 @@ abstract class FeedHalJsonTestBase extends FeedResourceTestBase {
   /**
    * {@inheritdoc}
    */
-  public static $modules = ['hal'];