diff --git a/core/modules/aggregator/tests/src/Unit/Menu/AggregatorLocalTasksTest.php b/core/modules/aggregator/tests/src/Unit/Menu/AggregatorLocalTasksTest.php
index 80a2db5..a68c368 100644
--- a/core/modules/aggregator/tests/src/Unit/Menu/AggregatorLocalTasksTest.php
+++ b/core/modules/aggregator/tests/src/Unit/Menu/AggregatorLocalTasksTest.php
@@ -27,7 +27,7 @@ protected function setUp() {
   /**
    * Tests local task existence.
    *
-   * @dataProvider getAggregatorAdminRoutes
+   * @dataProvider providerAggregatorAdminRoutes
    */
   public function testAggregatorAdminLocalTasks($route) {
     $this->assertLocalTasks($route, array(
@@ -38,7 +38,7 @@ public function testAggregatorAdminLocalTasks($route) {
   /**
    * Provides a list of routes to test.
    */
-  public function getAggregatorAdminRoutes() {
+  public function providerAggregatorAdminRoutes() {
     return array(
       array('aggregator.admin_overview'),
       array('aggregator.admin_settings'),
@@ -48,7 +48,7 @@ public function getAggregatorAdminRoutes() {
   /**
    * Checks aggregator source tasks.
    *
-   * @dataProvider getAggregatorSourceRoutes
+   * @dataProvider providerAggregatorSourceRoutes
    */
   public function testAggregatorSourceLocalTasks($route) {
     $this->assertLocalTasks($route, array(
@@ -60,7 +60,7 @@ public function testAggregatorSourceLocalTasks($route) {
   /**
    * Provides a list of source routes to test.
    */
-  public function getAggregatorSourceRoutes() {
+  public function providerAggregatorSourceRoutes() {
     return array(
       array('entity.aggregator_feed.canonical'),
       array('entity.aggregator_feed.edit_form'),
diff --git a/core/modules/block_content/tests/src/Unit/Menu/BlockContentLocalTasksTest.php b/core/modules/block_content/tests/src/Unit/Menu/BlockContentLocalTasksTest.php
index 83cbcf8..88ef98f 100644
--- a/core/modules/block_content/tests/src/Unit/Menu/BlockContentLocalTasksTest.php
+++ b/core/modules/block_content/tests/src/Unit/Menu/BlockContentLocalTasksTest.php
@@ -58,7 +58,7 @@ protected function setUp() {
   /**
    * Checks block_content listing local tasks.
    *
-   * @dataProvider getBlockContentListingRoutes
+   * @dataProvider providerBlockContentListingRoutes
    */
   public function testBlockContentListLocalTasks($route) {
     $this->assertLocalTasks($route, array(
@@ -76,7 +76,7 @@ public function testBlockContentListLocalTasks($route) {
   /**
    * Provides a list of routes to test.
    */
-  public function getBlockContentListingRoutes() {
+  public function providerBlockContentListingRoutes() {
     return array(
       array('entity.block_content.collection', 'entity.block_content_type.collection'),
     );
diff --git a/core/modules/book/tests/src/Unit/Menu/BookLocalTasksTest.php b/core/modules/book/tests/src/Unit/Menu/BookLocalTasksTest.php
index 1702855..3f010c3 100644
--- a/core/modules/book/tests/src/Unit/Menu/BookLocalTasksTest.php
+++ b/core/modules/book/tests/src/Unit/Menu/BookLocalTasksTest.php
@@ -27,7 +27,7 @@ protected function setUp() {
   /**
    * Tests local task existence.
    *
-   * @dataProvider getBookAdminRoutes
+   * @dataProvider providerBookAdminRoutes
    */
   public function testBookAdminLocalTasks($route) {
 
@@ -39,7 +39,7 @@ public function testBookAdminLocalTasks($route) {
   /**
    * Provides a list of routes to test.
    */
-  public function getBookAdminRoutes() {
+  public function providerBookAdminRoutes() {
     return array(
       array('book.admin'),
       array('book.settings'),
@@ -49,7 +49,7 @@ public function getBookAdminRoutes() {
   /**
    * Tests local task existence.
    *
-   * @dataProvider getBookNodeRoutes
+   * @dataProvider providerBookNodeRoutes
    */
   public function testBookNodeLocalTasks($route) {
     $this->assertLocalTasks($route, array(
@@ -60,7 +60,7 @@ public function testBookNodeLocalTasks($route) {
   /**
    * Provides a list of routes to test.
    */
-  public function getBookNodeRoutes() {
+  public function providerBookNodeRoutes() {
     return array(
       array('entity.node.canonical'),
       array('entity.node.book_outline_form'),
diff --git a/core/modules/comment/tests/src/Unit/CommentLinkBuilderTest.php b/core/modules/comment/tests/src/Unit/CommentLinkBuilderTest.php
index 202f6a4..923c7a7 100644
--- a/core/modules/comment/tests/src/Unit/CommentLinkBuilderTest.php
+++ b/core/modules/comment/tests/src/Unit/CommentLinkBuilderTest.php
@@ -102,7 +102,7 @@ protected function setUp() {
    *   Array of expected links keyed by link ID. Can be either string (link
    *   title) or array of link properties.
    *
-   * @dataProvider getLinkCombinations
+   * @dataProvider providerLinkCombinations
    *
    * @covers ::buildCommentedEntityLinks
    */
@@ -163,7 +163,7 @@ public function testCommentLinkBuilder(NodeInterface $node, $context, $has_acces
   /**
    * Data provider for ::testCommentLinkBuilder.
    */
-  public function getLinkCombinations() {
+  public function providerLinkCombinations() {
     $cases = array();
     // No links should be created if the entity doesn't have the field.
     $cases[] = array(
diff --git a/core/modules/config/tests/src/Unit/Menu/ConfigLocalTasksTest.php b/core/modules/config/tests/src/Unit/Menu/ConfigLocalTasksTest.php
index a757378..9fce923 100644
--- a/core/modules/config/tests/src/Unit/Menu/ConfigLocalTasksTest.php
+++ b/core/modules/config/tests/src/Unit/Menu/ConfigLocalTasksTest.php
@@ -24,7 +24,7 @@ protected function setUp() {
   /**
    * Tests config local tasks existence.
    *
-   * @dataProvider getConfigAdminRoutes
+   * @dataProvider providerConfigAdminRoutes
    */
   public function testConfigAdminLocalTasks($route, $expected) {
     $this->assertLocalTasks($route, $expected);
@@ -33,7 +33,7 @@ public function testConfigAdminLocalTasks($route, $expected) {
   /**
    * Provides a list of routes to test.
    */
-  public function getConfigAdminRoutes() {
+  public function providerConfigAdminRoutes() {
     return array(
       array('config.sync', array(array('config.sync', 'config.full', 'config.single'))),
       array('config.export_full', array(array('config.sync', 'config.full', 'config.single'), array('config.export_full', 'config.import_full'))),
diff --git a/core/modules/contact/tests/src/Unit/MailHandlerTest.php b/core/modules/contact/tests/src/Unit/MailHandlerTest.php
index 97d22b9..5cfdaa5 100644
--- a/core/modules/contact/tests/src/Unit/MailHandlerTest.php
+++ b/core/modules/contact/tests/src/Unit/MailHandlerTest.php
@@ -132,7 +132,7 @@ public function testInvalidRecipient() {
   /**
    * Tests the sendMailMessages method.
    *
-   * @dataProvider getSendMailMessages
+   * @dataProvider providerSendMailMessages
    *
    * @covers ::sendMailMessages
    */
@@ -160,7 +160,7 @@ function($module, $key, $to, $langcode, $params, $from) use (&$results) {
   /**
    * Data provider for ::testSendMailMessages.
    */
-  public function getSendMailMessages() {
+  public function providerSendMailMessages() {
     $data = array();
     $recipients = array('admin@drupal.org', 'user@drupal.org');
     $default_result = array(
diff --git a/core/modules/language/tests/src/Unit/Menu/LanguageLocalTasksTest.php b/core/modules/language/tests/src/Unit/Menu/LanguageLocalTasksTest.php
index 6e35abb..c525965 100644
--- a/core/modules/language/tests/src/Unit/Menu/LanguageLocalTasksTest.php
+++ b/core/modules/language/tests/src/Unit/Menu/LanguageLocalTasksTest.php
@@ -26,7 +26,7 @@ protected function setUp() {
   /**
    * Tests language admin overview local tasks existence.
    *
-   * @dataProvider getLanguageAdminOverviewRoutes
+   * @dataProvider providerLanguageAdminOverviewRoutes
    */
   public function testLanguageAdminLocalTasks($route, $expected) {
     $this->assertLocalTasks($route, $expected);
@@ -35,7 +35,7 @@ public function testLanguageAdminLocalTasks($route, $expected) {
   /**
    * Provides a list of routes to test.
    */
-  public function getLanguageAdminOverviewRoutes() {
+  public function providerLanguageAdminOverviewRoutes() {
     return array(
       array('entity.configurable_language.collection', array(array('entity.configurable_language.collection', 'language.negotiation'))),
       array('language.negotiation', array(array('entity.configurable_language.collection', 'language.negotiation'))),
diff --git a/core/modules/locale/tests/src/Unit/LocaleLookupTest.php b/core/modules/locale/tests/src/Unit/LocaleLookupTest.php
index fafcb45..dd280c4 100644
--- a/core/modules/locale/tests/src/Unit/LocaleLookupTest.php
+++ b/core/modules/locale/tests/src/Unit/LocaleLookupTest.php
@@ -134,7 +134,7 @@ public function testResolveCacheMissWithoutFallback() {
    *
    * @covers ::resolveCacheMiss
    *
-   * @dataProvider resolveCacheMissWithFallbackProvider
+   * @dataProvider providerResolveCacheMissWithFallback
    */
   public function testResolveCacheMissWithFallback($langcode, $string, $context, $expected) {
     // These are fake words!
@@ -192,7 +192,7 @@ public function testResolveCacheMissWithFallback($langcode, $string, $context, $
   /**
    * Provides test data for testResolveCacheMissWithFallback().
    */
-  public function resolveCacheMissWithFallbackProvider() {
+  public function providerResolveCacheMissWithFallback() {
     return array(
       array('cs', 'test', 'irrelevant', 'test v české'),
       array('cs', 'fake', 'irrelevant', 'falešný'),
diff --git a/core/modules/locale/tests/src/Unit/Menu/LocaleLocalTasksTest.php b/core/modules/locale/tests/src/Unit/Menu/LocaleLocalTasksTest.php
index e30f676..10c34af 100644
--- a/core/modules/locale/tests/src/Unit/Menu/LocaleLocalTasksTest.php
+++ b/core/modules/locale/tests/src/Unit/Menu/LocaleLocalTasksTest.php
@@ -29,7 +29,7 @@ protected function setUp() {
   /**
    * Checks locale listing local tasks.
    *
-   * @dataProvider getLocalePageRoutes
+   * @dataProvider providerLocalePageRoutes
    */
   public function testLocalePageLocalTasks($route) {
     $tasks = array(
@@ -41,7 +41,7 @@ public function testLocalePageLocalTasks($route) {
   /**
    * Provides a list of routes to test.
    */
-  public function getLocalePageRoutes() {
+  public function providerLocalePageRoutes() {
     return array(
       array('locale.translate_page'),
       array('locale.translate_import'),
diff --git a/core/modules/migrate/tests/src/Unit/SqlBaseTest.php b/core/modules/migrate/tests/src/Unit/SqlBaseTest.php
index b526b7c..32fff19 100644
--- a/core/modules/migrate/tests/src/Unit/SqlBaseTest.php
+++ b/core/modules/migrate/tests/src/Unit/SqlBaseTest.php
@@ -29,7 +29,7 @@ class SqlBaseTest extends UnitTestCase {
    * @param array $idmap_options
    *   An array of connection options for the id map connection.
    *
-   * @dataProvider sqlBaseTestProvider
+   * @dataProvider providerSqlBaseTest
    */
   public function testMapJoinable($expected_result, $id_map_is_sql, $with_id_map, $source_options = [], $idmap_options = []) {
     // Setup a connection object.
@@ -83,7 +83,7 @@ public function testMapJoinable($expected_result, $id_map_is_sql, $with_id_map,
    * @return array
    *   An array of data per test run.
    */
-  public function sqlBaseTestProvider() {
+  public function providerSqlBaseTest() {
     return [
       // Source ids are empty so mapJoinable() is false.
       [FALSE, FALSE, FALSE],
diff --git a/core/modules/shortcut/tests/src/Unit/Menu/ShortcutLocalTasksTest.php b/core/modules/shortcut/tests/src/Unit/Menu/ShortcutLocalTasksTest.php
index 526b680..fdd529b 100644
--- a/core/modules/shortcut/tests/src/Unit/Menu/ShortcutLocalTasksTest.php
+++ b/core/modules/shortcut/tests/src/Unit/Menu/ShortcutLocalTasksTest.php
@@ -27,7 +27,7 @@ protected function setUp() {
   /**
    * Checks shortcut listing local tasks.
    *
-   * @dataProvider getShortcutPageRoutes
+   * @dataProvider providerShortcutPageRoutes
    */
   public function testShortcutPageLocalTasks($route) {
     $tasks = array(
@@ -39,7 +39,7 @@ public function testShortcutPageLocalTasks($route) {
   /**
    * Provides a list of routes to test.
    */
-  public function getShortcutPageRoutes() {
+  public function providerShortcutPageRoutes() {
     return array(
       array('entity.user.canonical'),
       array('entity.user.edit_form'),
diff --git a/core/modules/simpletest/tests/src/Unit/TestBaseTest.php b/core/modules/simpletest/tests/src/Unit/TestBaseTest.php
index 6fa9c55..b9151a2 100644
--- a/core/modules/simpletest/tests/src/Unit/TestBaseTest.php
+++ b/core/modules/simpletest/tests/src/Unit/TestBaseTest.php
@@ -79,6 +79,15 @@ public function providerRandomStringValidate() {
   }
 
   /**
+   * Tests the random strings validation rules.
+   *
+   * @param string $string
+   *   The string to validate.
+   * @param bool $expected
+   *   The expected result of the validation.
+   *
+   * @see \Drupal\simpletest\TestBase::randomStringValidate().
+   *
    * @covers ::randomStringValidate
    * @dataProvider providerRandomStringValidate
    */
diff --git a/core/modules/system/tests/src/Unit/Menu/SystemLocalTasksTest.php b/core/modules/system/tests/src/Unit/Menu/SystemLocalTasksTest.php
index e1b48e9..a510f1c 100644
--- a/core/modules/system/tests/src/Unit/Menu/SystemLocalTasksTest.php
+++ b/core/modules/system/tests/src/Unit/Menu/SystemLocalTasksTest.php
@@ -50,7 +50,7 @@ protected function setUp() {
   /**
    * Tests local task existence.
    *
-   * @dataProvider getSystemAdminRoutes
+   * @dataProvider providerSystemAdminRoutes
    */
   public function testSystemAdminLocalTasks($route, $expected) {
     $this->assertLocalTasks($route, $expected);
@@ -59,7 +59,7 @@ public function testSystemAdminLocalTasks($route, $expected) {
   /**
    * Provides a list of routes to test.
    */
-  public function getSystemAdminRoutes() {
+  public function providerSystemAdminRoutes() {
     return array(
       array('system.admin_content', array(array('system.admin_content'))),
       array('system.theme_settings_theme', array(
diff --git a/core/modules/taxonomy/tests/src/Unit/Menu/TaxonomyLocalTasksTest.php b/core/modules/taxonomy/tests/src/Unit/Menu/TaxonomyLocalTasksTest.php
index be3dd53..52715ee 100644
--- a/core/modules/taxonomy/tests/src/Unit/Menu/TaxonomyLocalTasksTest.php
+++ b/core/modules/taxonomy/tests/src/Unit/Menu/TaxonomyLocalTasksTest.php
@@ -24,7 +24,7 @@ protected function setUp() {
   /**
    * Checks taxonomy edit local tasks.
    *
-   * @dataProvider getTaxonomyPageRoutes
+   * @dataProvider providerTaxonomyPageRoutes
    */
   public function testTaxonomyPageLocalTasks($route, $subtask = array()) {
     $tasks = array(
@@ -37,7 +37,7 @@ public function testTaxonomyPageLocalTasks($route, $subtask = array()) {
   /**
    * Provides a list of routes to test.
    */
-  public function getTaxonomyPageRoutes() {
+  public function providerTaxonomyPageRoutes() {
     return array(
       array('entity.taxonomy_term.canonical'),
       array('entity.taxonomy_term.edit_form'),
diff --git a/core/modules/tour/tests/src/Unit/Entity/TourTest.php b/core/modules/tour/tests/src/Unit/Entity/TourTest.php
index a6625c8..e9a8700 100644
--- a/core/modules/tour/tests/src/Unit/Entity/TourTest.php
+++ b/core/modules/tour/tests/src/Unit/Entity/TourTest.php
@@ -28,7 +28,7 @@ class TourTest extends UnitTestCase {
    *
    * @covers ::hasMatchingRoute
    *
-   * @dataProvider routeProvider
+   * @dataProvider providerRoute
    */
   public function testHasMatchingRoute($routes, $route_name, $route_params, $result) {
     $tour = $this->getMockBuilder('\Drupal\tour\Entity\Tour')
@@ -45,10 +45,10 @@ public function testHasMatchingRoute($routes, $route_name, $route_params, $resul
     $tour->resetKeyedRoutes();
   }
 
-  /*
+  /**
    * Provides sample routes for testing.
    */
-  public function routeProvider() {
+  public function providerRoute() {
     return array(
       // Simple match.
       array(
diff --git a/core/modules/update/tests/src/Unit/Menu/UpdateLocalTasksTest.php b/core/modules/update/tests/src/Unit/Menu/UpdateLocalTasksTest.php
index b82ec79..a7161cd 100644
--- a/core/modules/update/tests/src/Unit/Menu/UpdateLocalTasksTest.php
+++ b/core/modules/update/tests/src/Unit/Menu/UpdateLocalTasksTest.php
@@ -24,7 +24,7 @@ protected function setUp() {
   /**
    * Checks update report tasks.
    *
-   * @dataProvider getUpdateReportRoutes
+   * @dataProvider providerUpdateReportRoutes
    */
   public function testUpdateReportLocalTasks($route) {
     $this->assertLocalTasks($route, array(
@@ -35,7 +35,7 @@ public function testUpdateReportLocalTasks($route) {
   /**
    * Provides a list of report routes to test.
    */
-  public function getUpdateReportRoutes() {
+  public function providerUpdateReportRoutes() {
     return array(
       array('update.status'),
       array('update.settings'),
@@ -46,7 +46,7 @@ public function getUpdateReportRoutes() {
   /**
    * Checks update module tasks.
    *
-   * @dataProvider getUpdateModuleRoutes
+   * @dataProvider providerUpdateModuleRoutes
    */
   public function testUpdateModuleLocalTasks($route) {
     $this->assertLocalTasks($route, array(
@@ -58,7 +58,7 @@ public function testUpdateModuleLocalTasks($route) {
   /**
    * Provides a list of module routes to test.
    */
-  public function getUpdateModuleRoutes() {
+  public function providerUpdateModuleRoutes() {
     return array(
       array('update.module_update'),
     );
@@ -67,7 +67,7 @@ public function getUpdateModuleRoutes() {
   /**
    * Checks update theme tasks.
    *
-   * @dataProvider getUpdateThemeRoutes
+   * @dataProvider providerUpdateThemeRoutes
    */
   public function testUpdateThemeLocalTasks($route) {
     $this->assertLocalTasks($route, array(
@@ -79,7 +79,7 @@ public function testUpdateThemeLocalTasks($route) {
   /**
    * Provides a list of theme routes to test.
    */
-  public function getUpdateThemeRoutes() {
+  public function providerUpdateThemeRoutes() {
     return array(
       array('update.theme_update'),
     );
diff --git a/core/modules/user/tests/src/Unit/Menu/UserLocalTasksTest.php b/core/modules/user/tests/src/Unit/Menu/UserLocalTasksTest.php
index fb7bb82..63dcb57 100644
--- a/core/modules/user/tests/src/Unit/Menu/UserLocalTasksTest.php
+++ b/core/modules/user/tests/src/Unit/Menu/UserLocalTasksTest.php
@@ -24,7 +24,7 @@ protected function setUp() {
   /**
    * Tests local task existence.
    *
-   * @dataProvider getUserAdminRoutes
+   * @dataProvider providerUserAdminRoutes
    */
   public function testUserAdminLocalTasks($route, $expected) {
     $this->assertLocalTasks($route, $expected);
@@ -33,7 +33,7 @@ public function testUserAdminLocalTasks($route, $expected) {
   /**
    * Provides a list of routes to test.
    */
-  public function getUserAdminRoutes() {
+  public function providerUserAdminRoutes() {
     return array(
       array('entity.user.collection', array(array('entity.user.collection', 'user.admin_permissions', 'entity.user_role.collection'))),
       array('user.admin_permissions', array(array('entity.user.collection', 'user.admin_permissions', 'entity.user_role.collection'))),
@@ -45,7 +45,7 @@ public function getUserAdminRoutes() {
   /**
    * Checks user listing local tasks.
    *
-   * @dataProvider getUserLoginRoutes
+   * @dataProvider providerUserLoginRoutes
    */
   public function testUserLoginLocalTasks($route) {
     $tasks = array(
@@ -57,7 +57,7 @@ public function testUserLoginLocalTasks($route) {
   /**
    * Provides a list of routes to test.
    */
-  public function getUserLoginRoutes() {
+  public function providerUserLoginRoutes() {
     return array(
       array('user.login'),
       array('user.register'),
@@ -68,7 +68,7 @@ public function getUserLoginRoutes() {
   /**
    * Checks user listing local tasks.
    *
-   * @dataProvider getUserPageRoutes
+   * @dataProvider providerUserPageRoutes
    */
   public function testUserPageLocalTasks($route, $subtask = array()) {
     $tasks = array(
@@ -81,7 +81,7 @@ public function testUserPageLocalTasks($route, $subtask = array()) {
   /**
    * Provides a list of routes to test.
    */
-  public function getUserPageRoutes() {
+  public function providerUserPageRoutes() {
     return array(
       array('entity.user.canonical'),
       array('entity.user.edit_form'),
diff --git a/core/modules/user/tests/src/Unit/UserAccessControlHandlerTest.php b/core/modules/user/tests/src/Unit/UserAccessControlHandlerTest.php
index 04cdef9..9fbaaa4 100644
--- a/core/modules/user/tests/src/Unit/UserAccessControlHandlerTest.php
+++ b/core/modules/user/tests/src/Unit/UserAccessControlHandlerTest.php
@@ -139,7 +139,7 @@ public function assertFieldAccess($field, $viewer, $target, $view, $edit) {
   /**
    * Ensures user name access is working properly.
    *
-   * @dataProvider userNameProvider
+   * @dataProvider providerUserName
    */
   public function testUserNameAccess($viewer, $target, $view, $edit) {
     $this->assertFieldAccess('name', $viewer, $target, $view, $edit);
@@ -148,7 +148,7 @@ public function testUserNameAccess($viewer, $target, $view, $edit) {
   /**
    * Provides test data for estUserNameAccess().
    */
-  public function userNameProvider() {
+  public function providerUserName() {
     $name_access = array(
       // The viewer user is allowed to see user names on all accounts.
       array(
@@ -190,7 +190,7 @@ public function userNameProvider() {
   /**
    * Tests that private user settings cannot be viewed by other users.
    *
-   * @dataProvider hiddenUserSettingsProvider
+   * @dataProvider providerHiddenUserSettings
    */
   public function testHiddenUserSettings($field, $viewer, $target, $view, $edit) {
     $this->assertFieldAccess($field, $viewer, $target, $view, $edit);
@@ -199,7 +199,7 @@ public function testHiddenUserSettings($field, $viewer, $target, $view, $edit) {
   /**
    * Provides test data for testHiddenUserSettings().
    */
-  public function hiddenUserSettingsProvider() {
+  public function providerHiddenUserSettings() {
     $access_info = array();
 
     $fields = array(
@@ -251,7 +251,7 @@ public function hiddenUserSettingsProvider() {
   /**
    * Tests that private user settings cannot be viewed by other users.
    *
-   * @dataProvider adminFieldAccessProvider
+   * @dataProvider providerAdminFieldAccess
    */
   public function testAdminFieldAccess($field, $viewer, $target, $view, $edit) {
     $this->assertFieldAccess($field, $viewer, $target, $view, $edit);
@@ -260,7 +260,7 @@ public function testAdminFieldAccess($field, $viewer, $target, $view, $edit) {
   /**
    * Provides test data for testAdminFieldAccess().
    */
-  public function adminFieldAccessProvider() {
+  public function providerAdminFieldAccess() {
     $access_info = array();
 
     $fields = array(
@@ -301,7 +301,7 @@ public function adminFieldAccessProvider() {
   /**
    * Tests that passwords cannot be viewed, just edited.
    *
-   * @dataProvider passwordAccessProvider
+   * @dataProvider providerPasswordAccess
    */
   public function testPasswordAccess($viewer, $target, $view, $edit) {
     $this->assertFieldAccess('pass', $viewer, $target, $view, $edit);
@@ -310,7 +310,7 @@ public function testPasswordAccess($viewer, $target, $view, $edit) {
   /**
    * Provides test data for passwordAccessProvider().
    */
-  public function passwordAccessProvider() {
+  public function providerPasswordAccess() {
     $pass_access = array(
       array(
         'viewer' => 'viewer',
@@ -346,7 +346,7 @@ public function passwordAccessProvider() {
   /**
    * Tests the user created field access.
    *
-   * @dataProvider createdAccessProvider
+   * @dataProvider providerCreatedAccess
    */
   public function testCreatedAccess($viewer, $target, $view, $edit) {
     $this->assertFieldAccess('created', $viewer, $target, $view, $edit);
@@ -355,7 +355,7 @@ public function testCreatedAccess($viewer, $target, $view, $edit) {
   /**
    * Provides test data for testCreatedAccess().
    */
-  public function createdAccessProvider() {
+  public function providerCreatedAccess() {
     $created_access = array(
       array(
         'viewer' => 'viewer',
@@ -382,7 +382,7 @@ public function createdAccessProvider() {
   /**
    * Tests access to a non-existing base field.
    *
-   * @dataProvider NonExistingFieldAccessProvider
+   * @dataProvider providerNonExistingFieldAccess
    */
   public function testNonExistingFieldAccess($viewer, $target, $view, $edit) {
     // By default everyone has access to all fields that do not have explicit
@@ -394,7 +394,7 @@ public function testNonExistingFieldAccess($viewer, $target, $view, $edit) {
   /**
    * Provides test data for testNonExistingFieldAccess().
    */
-  public function NonExistingFieldAccessProvider() {
+  public function providerNonExistingFieldAccess() {
     $created_access = array(
       array(
         'viewer' => 'viewer',
diff --git a/core/tests/Drupal/Tests/Core/Access/AccessResultTest.php b/core/tests/Drupal/Tests/Core/Access/AccessResultTest.php
index 42d2d2a..2691bbc 100644
--- a/core/tests/Drupal/Tests/Core/Access/AccessResultTest.php
+++ b/core/tests/Drupal/Tests/Core/Access/AccessResultTest.php
@@ -520,7 +520,7 @@ public function testInheritCacheability() {
    *    does not implement CacheableInterface, then the result won't either.
    *    This is the case for items 3, 6 and 9 in the set above.
    */
-  public function andOrCacheabilityPropagationProvider() {
+  public function providerAndOrCacheabilityPropagation() {
     // ct: cacheable=true, cf: cacheable=false, un: uncacheable.
     // Note: the test cases that have a "un" access result as the first operand
     // test UncacheableTestAccessResult, not AccessResult. However, we
@@ -793,7 +793,7 @@ public function andOrCacheabilityPropagationProvider() {
    * @covers ::orIf
    * @covers ::inheritCacheability
    *
-   * @dataProvider andOrCacheabilityPropagationProvider
+   * @dataProvider providerAndOrCacheabilityPropagation
    */
   public function testAndOrCacheabilityPropagation(AccessResultInterface $first, $op, AccessResultInterface $second, $implements_cacheable_interface, $is_cacheable) {
     if ($op === 'OR') {
diff --git a/core/tests/Drupal/Tests/Core/Cache/CacheTest.php b/core/tests/Drupal/Tests/Core/Cache/CacheTest.php
index 261aa80..9ca8287 100644
--- a/core/tests/Drupal/Tests/Core/Cache/CacheTest.php
+++ b/core/tests/Drupal/Tests/Core/Cache/CacheTest.php
@@ -21,7 +21,7 @@ class CacheTest extends UnitTestCase {
    *
    * @return array
    */
-  public function validateTagsProvider() {
+  public function providerValidateTags() {
     return [
       [[], FALSE],
       [['foo'], FALSE],
@@ -47,7 +47,7 @@ public function validateTagsProvider() {
   /**
    * @covers ::validateTags
    *
-   * @dataProvider validateTagsProvider
+   * @dataProvider providerValidateTags
    */
   public function testValidateTags(array $tags, $expected_exception_message) {
     if ($expected_exception_message !== FALSE) {
@@ -62,7 +62,7 @@ public function testValidateTags(array $tags, $expected_exception_message) {
    *
    * @return array
    */
-  public function mergeTagsProvider() {
+  public function providerMergeTags() {
     return [
       [[], [], []],
       [['bar'], ['foo'], ['bar', 'foo']],
@@ -77,7 +77,7 @@ public function mergeTagsProvider() {
   /**
    * @covers ::mergeTags
    *
-   * @dataProvider mergeTagsProvider
+   * @dataProvider providerMergeTags
    */
   public function testMergeTags(array $a, array $b, array $expected) {
     $this->assertEquals($expected, Cache::mergeTags($a, $b));
@@ -120,7 +120,7 @@ public function testMergeMaxAges($a, $b, $expected) {
    *
    * @return array
    */
-  public function buildTagsProvider() {
+  public function providerBuildTags() {
     return [
       ['node', [1], ['node:1']],
       ['node', [1, 2, 3], ['node:1', 'node:2', 'node:3']],
@@ -145,7 +145,7 @@ public function buildTagsProvider() {
   /**
    * @covers ::buildTags
    *
-   * @dataProvider buildTagsProvider
+   * @dataProvider providerBuildTags
    */
   public function testBuildTags($prefix, array $suffixes, array $expected, $glue = ':') {
     $this->assertEquals($expected, Cache::buildTags($prefix, $suffixes, $glue));
diff --git a/core/tests/Drupal/Tests/Core/Config/ConfigTest.php b/core/tests/Drupal/Tests/Core/Config/ConfigTest.php
index 936287a..82237b7 100644
--- a/core/tests/Drupal/Tests/Core/Config/ConfigTest.php
+++ b/core/tests/Drupal/Tests/Core/Config/ConfigTest.php
@@ -72,7 +72,7 @@ public function setUp() {
 
   /**
    * @covers ::setName
-   * @dataProvider setNameProvider
+   * @dataProvider providerSetName
    */
   public function testSetName($name) {
     // Set the name.
@@ -91,7 +91,7 @@ public function testSetName($name) {
    *
    * @see \Drupal\Tests\Core\Config\ConfigTest::testSetName()
    */
-  public function setNameProvider() {
+  public function providerSetName() {
     return array(
       // Valid name with dot.
       array(
@@ -118,7 +118,7 @@ public function testIsNew() {
 
   /**
    * @covers ::setData
-   * @dataProvider nestedDataProvider
+   * @dataProvider providerNestedData
    */
   public function testSetData($data) {
     $this->config->setData($data);
@@ -128,7 +128,7 @@ public function testSetData($data) {
 
   /**
    * @covers ::save
-   * @dataProvider nestedDataProvider
+   * @dataProvider providerNestedData
    */
   public function testSaveNew($data) {
     $this->cacheTagsInvalidator->expects($this->never())
@@ -154,7 +154,7 @@ public function testSaveNew($data) {
 
   /**
    * @covers ::save
-   * @dataProvider nestedDataProvider
+   * @dataProvider providerNestedData
    */
   public function testSaveExisting($data) {
     $this->cacheTagsInvalidator->expects($this->once())
@@ -177,7 +177,7 @@ public function testSaveExisting($data) {
    * @covers ::setModuleOverride
    * @covers ::setSettingsOverride
    * @covers ::getOriginal
-   * @dataProvider overrideDataProvider
+   * @dataProvider providerOverrideData
    */
   public function testOverrideData($data, $module_data, $setting_data) {
     // Set initial data.
@@ -224,7 +224,7 @@ public function testOverrideData($data, $module_data, $setting_data) {
 
   /**
    * @covers ::set
-   * @dataProvider nestedDataProvider
+   * @dataProvider providerNestedData
    */
   public function testSetValue($data) {
     foreach ($data as $key => $value) {
@@ -255,7 +255,7 @@ public function testSetIllegalOffsetValue() {
 
   /**
    * @covers ::initWithData
-   * @dataProvider nestedDataProvider
+   * @dataProvider providerNestedData
    */
   public function testInitWithData($data) {
     $config = $this->config->initWithData($data);
@@ -278,7 +278,7 @@ public function testInitWithData($data) {
 
   /**
    * @covers ::clear
-   * @dataProvider simpleDataProvider
+   * @dataProvider providerSimpleData
    */
   public function testClear($data) {
     foreach ($data as $key => $value) {
@@ -292,7 +292,7 @@ public function testClear($data) {
 
   /**
    * @covers ::clear
-   * @dataProvider nestedDataProvider
+   * @dataProvider providerNestedData
    */
   public function testNestedClear($data) {
     foreach ($data as $key => $value) {
@@ -310,7 +310,7 @@ public function testNestedClear($data) {
 
   /**
    * @covers ::delete
-   * @dataProvider overrideDataProvider
+   * @dataProvider providerOverrideData
    */
   public function testDelete($data, $module_data) {
     $this->cacheTagsInvalidator->expects($this->once())
@@ -355,7 +355,7 @@ public function testDelete($data, $module_data) {
 
   /**
    * @covers ::merge
-   * @dataProvider mergeDataProvider
+   * @dataProvider providerMergeData
    */
   public function testMerge($data, $data_to_merge, $merged_data) {
     // Set initial data.
@@ -373,7 +373,7 @@ public function testMerge($data, $data_to_merge, $merged_data) {
    *
    * @see \Drupal\Tests\Core\Config\ConfigTest::testMerge()
    */
-  public function mergeDataProvider() {
+  public function providerMergeData() {
     return array(
       array(
         // Data.
@@ -389,7 +389,7 @@ public function mergeDataProvider() {
   /**
    * @covers ::validateName
    * @expectedException \Drupal\Core\Config\ConfigNameException
-   * @dataProvider validateNameProvider
+   * @dataProvider providerValidateName
    */
   public function testValidateNameException($name, $exception_message) {
     $this->setExpectedException('\Drupal\Core\Config\ConfigNameException', $exception_message);
@@ -408,7 +408,7 @@ public function testGetCacheTags() {
    *
    * @see \Drupal\Tests\Core\Config\ConfigTest::testValidateNameException()
    */
-  public function validateNameProvider() {
+  public function providerValidateName() {
     $return = array(
       // Name missing namespace (dot).
       array(
@@ -445,7 +445,7 @@ public function validateNameProvider() {
    * @see \Drupal\Tests\Core\Config\ConfigTest::testOverrideData()
    * @see \Drupal\Tests\Core\Config\ConfigTest::testDelete()
    */
-  public function overrideDataProvider() {
+  public function providerOverrideData() {
     return array(
       array(
         // Original data.
@@ -469,7 +469,7 @@ public function overrideDataProvider() {
    *
    * @see \Drupal\Tests\Core\Config\ConfigTest::testClear()
    */
-  public function simpleDataProvider() {
+  public function providerSimpleData() {
     return array(
       array(
         array(
@@ -490,7 +490,7 @@ public function simpleDataProvider() {
    * @see \Drupal\Tests\Core\Config\ConfigTest::testInitWithData()
    * @see \Drupal\Tests\Core\Config\ConfigTest::testNestedClear()
    */
-  public function nestedDataProvider() {
+  public function providerNestedData() {
     return array(
       array(
         array(
diff --git a/core/tests/Drupal/Tests/Core/Extension/ModuleHandlerTest.php b/core/tests/Drupal/Tests/Core/Extension/ModuleHandlerTest.php
index 1a52527..c0814fe 100644
--- a/core/tests/Drupal/Tests/Core/Extension/ModuleHandlerTest.php
+++ b/core/tests/Drupal/Tests/Core/Extension/ModuleHandlerTest.php
@@ -471,7 +471,7 @@ public function testResetImplementations() {
   }
 
   /**
-   * @dataProvider dependencyProvider
+   * @dataProvider providerDependency
    * @covers ::parseDependency
    */
   public function testDependencyParsing($dependency, $expected) {
@@ -482,7 +482,7 @@ public function testDependencyParsing($dependency, $expected) {
   /**
    * Provider for testing dependency parsing.
    */
-  public function dependencyProvider() {
+  public function providerDependency() {
     return array(
       array('system', array('name' => 'system')),
       array('taxonomy', array('name' => 'taxonomy')),
diff --git a/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php b/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php
index ce7bfee..63c7bcb 100644
--- a/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php
+++ b/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php
@@ -107,7 +107,7 @@ public function testGetFormIdWithBaseForm() {
   /**
    * Tests the handling of FormStateInterface::$response.
    *
-   * @dataProvider formStateResponseProvider
+   * @dataProvider providerFormStateResponse
    */
   public function testHandleFormStateResponse($class, $form_state_key) {
     $form_id = 'test_form_id';
@@ -140,7 +140,7 @@ public function testHandleFormStateResponse($class, $form_state_key) {
   /**
    * Provides test data for testHandleFormStateResponse().
    */
-  public function formStateResponseProvider() {
+  public function providerFormStateResponse() {
     return array(
       array('Symfony\Component\HttpFoundation\Response', 'response'),
       array('Symfony\Component\HttpFoundation\RedirectResponse', 'redirect'),
diff --git a/core/tests/Drupal/Tests/Core/Menu/LocalActionManagerTest.php b/core/tests/Drupal/Tests/Core/Menu/LocalActionManagerTest.php
index 85731c7..113eea4 100644
--- a/core/tests/Drupal/Tests/Core/Menu/LocalActionManagerTest.php
+++ b/core/tests/Drupal/Tests/Core/Menu/LocalActionManagerTest.php
@@ -141,7 +141,7 @@ public function testGetTitle() {
   /**
    * @covers ::getActionsForRoute
    *
-   * @dataProvider getActionsForRouteProvider
+   * @dataProvider providerActionsForRoute
    */
   public function testGetActionsForRoute($route_appears, array $plugin_definitions, array $expected_actions) {
     $this->discovery->expects($this->any())
@@ -180,7 +180,7 @@ public function testGetActionsForRoute($route_appears, array $plugin_definitions
     $this->assertEquals($expected_actions, $this->localActionManager->getActionsForRoute($route_appears));
   }
 
-  public function getActionsForRouteProvider() {
+  public function providerActionsForRoute() {
     // Single available and single expected plugins.
     $data[] = array(
       'test_route',
diff --git a/core/tests/Drupal/Tests/Core/Path/PathMatcherTest.php b/core/tests/Drupal/Tests/Core/Path/PathMatcherTest.php
index 01c6ab9..0dce87f 100644
--- a/core/tests/Drupal/Tests/Core/Path/PathMatcherTest.php
+++ b/core/tests/Drupal/Tests/Core/Path/PathMatcherTest.php
@@ -44,7 +44,7 @@ protected function setUp() {
   /**
    * Test that standard paths works with multiple patterns.
    *
-   * @dataProvider getMatchPathData
+   * @dataProvider providerMatchPathData
    */
   public function testMatchPath($patterns, $paths) {
     foreach ($paths as $path => $expected_result) {
@@ -64,7 +64,7 @@ public function testMatchPath($patterns, $paths) {
    * @return array
    *   A nested array of pattern arrays and path arrays.
    */
-  public function getMatchPathData() {
+  public function providerMatchPathData() {
     return array(
       array(
         // Single absolute paths.
diff --git a/core/tests/Drupal/Tests/Core/Route/RoleAccessCheckTest.php b/core/tests/Drupal/Tests/Core/Route/RoleAccessCheckTest.php
index ec07d8e..781a85f 100644
--- a/core/tests/Drupal/Tests/Core/Route/RoleAccessCheckTest.php
+++ b/core/tests/Drupal/Tests/Core/Route/RoleAccessCheckTest.php
@@ -88,7 +88,7 @@ protected function getTestRouteCollection() {
    *
    * @see \Drupal\Tests\Core\Route\RouterRoleTest::testRoleAccess
    */
-  public function roleAccessProvider() {
+  public function providerRoleAccess() {
     // Setup two different roles used in the test.
     $rid_1 = 'role_test_1';
     $rid_2 = 'role_test_2';
@@ -138,9 +138,9 @@ public function roleAccessProvider() {
    *   A list of accounts which should not have access to the given path.
    *
    * @see \Drupal\Tests\Core\Route\RouterRoleTest::getTestRouteCollection
-   * @see \Drupal\Tests\Core\Route\RouterRoleTest::roleAccessProvider
+   * @see \Drupal\Tests\Core\Route\RouterRoleTest::providerRoleAccess
    *
-   * @dataProvider roleAccessProvider
+   * @dataProvider providerRoleAccess
    */
   public function testRoleAccess($path, $grant_accounts, $deny_accounts) {
     $role_access_check = new RoleAccessCheck();
diff --git a/core/tests/Drupal/Tests/Core/Routing/AcceptHeaderMatcherTest.php b/core/tests/Drupal/Tests/Core/Routing/AcceptHeaderMatcherTest.php
index d0c811c..ea61ea2 100644
--- a/core/tests/Drupal/Tests/Core/Routing/AcceptHeaderMatcherTest.php
+++ b/core/tests/Drupal/Tests/Core/Routing/AcceptHeaderMatcherTest.php
@@ -71,7 +71,7 @@ public function acceptFilterProvider() {
    * @param string $excluded_route
    *   The route name that should be filtered out during matching.
    *
-   * @dataProvider acceptFilterProvider
+   * @dataProvider providerAcceptFilter
    */
   public function testAcceptFiltering($accept_header, $format, $included_route, $excluded_route) {
     $collection = $this->fixtures->sampleRouteCollection();
diff --git a/core/tests/Drupal/Tests/Core/Routing/RouteMatchBaseTest.php b/core/tests/Drupal/Tests/Core/Routing/RouteMatchBaseTest.php
index 7994a2d..a3fd44c 100644
--- a/core/tests/Drupal/Tests/Core/Routing/RouteMatchBaseTest.php
+++ b/core/tests/Drupal/Tests/Core/Routing/RouteMatchBaseTest.php
@@ -38,7 +38,7 @@
   /**
    * Provide sets of parameters and expected parameters for parameter tests.
    */
-  public function routeMatchProvider() {
+  public function providerRouteMatch() {
     $base_data = array(
       array(
         new Route(
@@ -86,7 +86,7 @@ public function routeMatchProvider() {
 
   /**
    * @covers ::getRouteName
-   * @dataProvider routeMatchProvider
+   * @dataProvider providerRouteMatch
    */
   public function testGetRouteName(RouteMatchInterface $route_match) {
     $this->assertSame('test_route', $route_match->getRouteName());
@@ -94,7 +94,7 @@ public function testGetRouteName(RouteMatchInterface $route_match) {
 
   /**
    * @covers ::getRouteObject
-   * @dataProvider routeMatchProvider
+   * @dataProvider providerRouteMatch
    */
   public function testGetRouteObject(RouteMatchInterface $route_match, Route $route) {
     $this->assertSame($route, $route_match->getRouteObject());
@@ -103,7 +103,7 @@ public function testGetRouteObject(RouteMatchInterface $route_match, Route $rout
   /**
    * @covers ::getParameter
    * @covers \Drupal\Core\Routing\RouteMatch::getParameterNames
-   * @dataProvider routeMatchProvider
+   * @dataProvider providerRouteMatch
    */
   public function testGetParameter(RouteMatchInterface $route_match, Route $route, $parameters, $expected_filtered_parameters) {
     foreach ($expected_filtered_parameters as $name => $expected_value) {
@@ -117,7 +117,7 @@ public function testGetParameter(RouteMatchInterface $route_match, Route $route,
   /**
    * @covers ::getParameters
    * @covers \Drupal\Core\Routing\RouteMatch::getParameterNames
-   * @dataProvider routeMatchProvider
+   * @dataProvider providerRouteMatch
    */
   public function testGetParameters(RouteMatchInterface $route_match, Route $route, $parameters, $expected_filtered_parameters) {
     $this->assertSame($expected_filtered_parameters, $route_match->getParameters()->all());
@@ -126,7 +126,7 @@ public function testGetParameters(RouteMatchInterface $route_match, Route $route
   /**
    * @covers ::getRawParameter
    * @covers \Drupal\Core\Routing\RouteMatch::getParameterNames
-   * @dataProvider routeMatchProvider
+   * @dataProvider providerRouteMatch
    */
   public function testGetRawParameter(RouteMatchInterface $route_match, Route $route, $parameters, $expected_filtered_parameters) {
     foreach ($expected_filtered_parameters as $name => $expected_value) {
@@ -140,7 +140,7 @@ public function testGetRawParameter(RouteMatchInterface $route_match, Route $rou
   /**
    * @covers ::getRawParameters
    * @covers \Drupal\Core\Routing\RouteMatch::getParameterNames
-   * @dataProvider routeMatchProvider
+   * @dataProvider providerRouteMatch
    */
   public function testGetRawParameters(RouteMatchInterface $route_match, Route $route, $parameters, $expected_filtered_parameters) {
     $this->assertSame($expected_filtered_parameters, $route_match->getRawParameters()->all());
diff --git a/core/tests/Drupal/Tests/Core/StackMiddleware/ReverseProxyMiddlewareTest.php b/core/tests/Drupal/Tests/Core/StackMiddleware/ReverseProxyMiddlewareTest.php
index f92109b..b7826ca 100644
--- a/core/tests/Drupal/Tests/Core/StackMiddleware/ReverseProxyMiddlewareTest.php
+++ b/core/tests/Drupal/Tests/Core/StackMiddleware/ReverseProxyMiddlewareTest.php
@@ -51,7 +51,7 @@ public function testNoProxy() {
   /**
    * Tests that subscriber sets trusted headers when reverse proxy is set.
    *
-   * @dataProvider reverseProxyEnabledProvider
+   * @dataProvider providerReverseProxyEnabled
    */
   public function testReverseProxyEnabled($provided_settings) {
       // Enable reverse proxy and add test values.
@@ -62,7 +62,7 @@ public function testReverseProxyEnabled($provided_settings) {
   /**
    * Data provider for testReverseProxyEnabled.
    */
-  public function reverseProxyEnabledProvider() {
+  public function providerReverseProxyEnabled() {
     return array(
       array(
         array(
diff --git a/core/tests/Drupal/Tests/Core/UrlTest.php b/core/tests/Drupal/Tests/Core/UrlTest.php
index dcc2c66..0f07875 100644
--- a/core/tests/Drupal/Tests/Core/UrlTest.php
+++ b/core/tests/Drupal/Tests/Core/UrlTest.php
@@ -452,7 +452,7 @@ public function testGetOptions($urls) {
    *
    * @covers ::access
    * @covers ::accessManager
-   * @dataProvider accessProvider
+   * @dataProvider providerAccess
    */
   public function testAccessRouted($access) {
     $account = $this->getMock('Drupal\Core\Session\AccountInterface');
@@ -482,7 +482,7 @@ public function testAccessUnrouted() {
    * @param bool $access
    *
    * @covers ::renderAccess
-   * @dataProvider accessProvider
+   * @dataProvider providerAccess
    */
   public function testRenderAccess($access) {
     $element = array(
@@ -792,7 +792,7 @@ protected function getMockAccessManager($access, $account = NULL) {
   /**
    * Data provider for the access test methods.
    */
-  public function accessProvider() {
+  public function providerAccess() {
     return array(
       array(TRUE),
       array(FALSE),