Changeset 46987

Timestamp:

12/18/2019 12:26:23 AM (7 years ago)

Author:

SergeyBiryukov

Message:

Docs: Improve comments in tests/formatting/redirect.php per the documentation standards.

See #48303.

File:

• trunk/tests/phpunit/tests/formatting/redirect.php (modified) (3 diffs)

trunk/tests/phpunit/tests/formatting/redirect.php

@@ -36 +36 @@
         public function get_bad_status_codes() {
                 return array(
-                        // Tests for bad arguments
+                        // Tests for bad arguments.
                         array( '/wp-admin', 404 ),
                         array( '/wp-admin', 410 ),
@@ -53 +53 @@
                 $this->assertEquals( 'http://example.com/watchtheallowedcharacters-~+_.?#=&;,/:%!*stay', wp_sanitize_redirect( 'http://example.com/watchtheallowedcharacters-~+_.?#=&;,/:%!*stay' ) );
                 $this->assertEquals( 'http://example.com/watchtheutf8convert%F0%9D%8C%86', wp_sanitize_redirect( "http://example.com/watchtheutf8convert\xf0\x9d\x8c\x86" ) );
-                //Nesting checks
+                // Nesting checks.
                 $this->assertEquals( 'http://example.com/watchthecarriagereturngo', wp_sanitize_redirect( 'http://example.com/watchthecarriagereturn%0%0ddgo' ) );
                 $this->assertEquals( 'http://example.com/watchthecarriagereturngo', wp_sanitize_redirect( 'http://example.com/watchthecarriagereturn%0%0DDgo' ) );
@@ -102 +102 @@
         function invalid_url_provider() {
                 return array(
-                        // parse_url() fails
+                        // parse_url() fails.
                         array( '' ),
                         array( 'http://:' ),
 
-                        // non-safelisted domain
+                        // Non-safelisted domain.
                         array( 'http://non-safelisted.example/' ),
 
-                        // non-safelisted domain (leading whitespace)
+                        // Non-safelisted domain (leading whitespace).
                         array( " \t\n\r\0\x08\x0Bhttp://non-safelisted.example.com" ),
                         array( " \t\n\r\0\x08\x0B//non-safelisted.example.com" ),
 
-                        // unsupported schemes
+                        // Unsupported schemes.
                         array( 'data:text/plain;charset=utf-8,Hello%20World!' ),
                         array( 'file:///etc/passwd' ),
                         array( 'ftp://example.com/' ),
 
-                        // malformed input
+                        // Malformed input.
                         array( 'http:example.com' ),
                         array( 'http:80' ),