diff --git a/core/modules/update/tests/Drupal/update/Tests/UpdateFetcherTest.php b/core/modules/update/tests/Drupal/update/Tests/UpdateFetcherTest.php index 71e6f40..4f03647 100644 --- a/core/modules/update/tests/Drupal/update/Tests/UpdateFetcherTest.php +++ b/core/modules/update/tests/Drupal/update/Tests/UpdateFetcherTest.php @@ -47,7 +47,16 @@ protected function setUp() { /** * Tests that buildFetchUrl() builds the URL correctly. * - * @dataProvider testUpdateBuildFetchUrlProvider + * @param array $project + * A keyed array of project information matching results from update_get_projects(). + * @param string $site_key + * A string to mimic an anonymous site key hash. + * @param string $expected + * The expected url returned from UpdateFetcher::buildFetchUrl() + * + * @dataProvider providerTestUpdateBuildFetchUrl + * + * @see \Drupal\update\UpdateFetcher::buildFetchUrl() */ public function testUpdateBuildFetchUrl(array $project, $site_key, $expected) { $url = $this->updateFetcher->buildFetchUrl($project, $site_key); @@ -56,8 +65,14 @@ public function testUpdateBuildFetchUrl(array $project, $site_key, $expected) { /** * Provide test data for self::testUpdateBuildFetchUrl(). + * + * @return array + * An array of arrays, each containing: + * - 'project' - An array matching a project's .info file structure. + * - 'site_key' - An arbitrary site key. + * - 'expected' - The expected url from UpdateFetcher::buildFetchUrl(). */ - public function testUpdateBuildFetchUrlProvider() { + public function providerTestUpdateBuildFetchUrl() { $data = array(); // First test that we didn't break the trivial case.