Changeset 37239

Timestamp:

04/18/2016 03:26:48 AM (9 years ago)

Author:

jeremyfelt

Message:

Tests: Add missing params to doc blocks

• These should have been included in [37236] and [37238].
• Simplifies parameter names in test_get_network_by_path()

See #36566.

File:

• trunk/tests/phpunit/tests/multisite/bootstrap.php (modified) (2 diffs)

trunk/tests/phpunit/tests/multisite/bootstrap.php

@@ -64 +64 @@
      * @ticket 27003
      * @dataProvider data_get_network_by_path
-     */
-    function test_get_network_by_path( $expected_key, $request_domain, $request_path, $message ) {
-        $network = get_network_by_path( $request_domain, $request_path );
+     *
+     * @param string $expected_key The array key associated with expected data for the test.
+     * @param string $domain       The requested domain.
+     * @param string $path         The requested path.
+     * @param string $message      The message to pass for failed tests.
+     */
+    function test_get_network_by_path( $expected_key, $domain, $path, $message ) {
+        $network = get_network_by_path( $domain, $path );
         $this->assertEquals( self::$network_ids[ $expected_key ], $network->id, $message );
     }
@@ -140 +145 @@
      * @ticket 27884
      * @dataProvider data_multisite_bootstrap
+     *
+     * @param string $site_key    The array key associated with the expected site for the test.
+     * @param string $network_key The array key associated with the expected network for the test.
+     * @param string $domain      The requested domain.
+     * @param string $path        The requested path.
      */
     function test_multisite_bootstrap( $site_key, $network_key, $domain, $path ) {