Changeset 43571 for trunk/tests/phpunit/tests/formatting/EscUrl.php

Timestamp:

08/17/2018 01:50:26 AM (6 years ago)

Author:

pento

Message:

Coding Standards: Upgrade WPCS to 1.0.0

WPCS 1.0.0 includes a bunch of new auto-fixers, which drops the number of coding standards issues across WordPress significantly. Prior to running the auto-fixers, there were 15,312 issues detected. With this commit, we now drop to 4,769 issues.

This change includes three notable additions:

• Multiline function calls must now put each parameter on a new line.
• Auto-formatting files is now part of the grunt precommit script.
• Auto-fixable coding standards issues will now cause Travis failures.

Fixes #44600.

File:

• trunk/tests/phpunit/tests/formatting/EscUrl.php (modified) (6 diffs)

trunk/tests/phpunit/tests/formatting/EscUrl.php

@@ -53 +53 @@
                 'query'    => 'query=2&r[]=3',
                 'fragment' => 'fragment',
-            ), parse_url( $url )
+            ),
+            parse_url( $url )
         );
         $this->assertEquals( 'https://user:pass@host.example.com:1234/path;p=1?query=2&r%5B%5D=3#fragment', esc_url_raw( $url ) );
@@ -84 +85 @@
         $this->assertEquals( '', esc_url( 'nasty://example.com/' ) );
         $this->assertEquals(
-            '', esc_url(
-                'example.com', array(
+            '',
+            esc_url(
+                'example.com',
+                array(
                     'https',
                 )
@@ -91 +94 @@
         );
         $this->assertEquals(
-            '', esc_url(
-                'http://example.com', array(
+            '',
+            esc_url(
+                'http://example.com',
+                array(
                     'https',
                 )
@@ -98 +103 @@
         );
         $this->assertEquals(
-            'https://example.com', esc_url(
-                'https://example.com', array(
+            'https://example.com',
+            esc_url(
+                'https://example.com',
+                array(
                     'http',
                     'https',
@@ -109 +116 @@
             $this->assertEquals( "{$scheme}://example.com", esc_url( "{$scheme}://example.com" ), $scheme );
             $this->assertEquals(
-                "{$scheme}://example.com", esc_url(
-                    "{$scheme}://example.com", array(
+                "{$scheme}://example.com",
+                esc_url(
+                    "{$scheme}://example.com",
+                    array(
                         $scheme,
                     )
-                ), $scheme
+                ),
+                $scheme
             );
         }
@@ -122 +132 @@
         $this->assertTrue( ! in_array( 'foo', wp_allowed_protocols(), true ) );
         $this->assertEquals(
-            'foo://example.com', esc_url(
-                'foo://example.com', array(
+            'foo://example.com',
+            esc_url(
+                'foo://example.com',
+                array(
                     'foo',
                 )