Changeset 42343 for trunk/tests/phpunit/tests/formatting/Autop.php

Timestamp:

11/30/2017 11:09:33 PM (8 years ago)

Author:

pento

Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:

• trunk/tests/phpunit/tests/formatting/Autop.php (modified) (10 diffs)

trunk/tests/phpunit/tests/formatting/Autop.php

@@ -7 +7 @@
     //From ticket https://core.trac.wordpress.org/ticket/11008
     function test_first_post() {
-        $expected = '<p>Welcome to WordPress!  This post contains important information.  After you read it, you can make it private to hide it from visitors but still have the information handy for future reference.</p>
+        $expected  = '<p>Welcome to WordPress!  This post contains important information.  After you read it, you can make it private to hide it from visitors but still have the information handy for future reference.</p>
 <p>First things first:</p>
 <ul>
@@ -58 +58 @@
 
         // On windows environments, the EOL-style is \r\n
-        $expected = str_replace( "\r\n", "\n", $expected);
-
-        $this->assertEquals($expected, wpautop($test_data));
+        $expected = str_replace( "\r\n", "\n", $expected );
+
+        $this->assertEquals( $expected, wpautop( $test_data ) );
     }
 
@@ -85 +85 @@
 
         // Make sure HTML breaks are maintained if manually inserted
-        $str = "Look at this code\n\n<pre>Line1<br />Line2<br>Line3<br/>Line4\nActual Line 2\nActual Line 3</pre>\n\nCool, huh?";
+        $str      = "Look at this code\n\n<pre>Line1<br />Line2<br>Line3<br/>Line4\nActual Line 2\nActual Line 3</pre>\n\nCool, huh?";
         $expected = "<p>Look at this code</p>\n<pre>Line1<br />Line2<br>Line3<br/>Line4\nActual Line 2\nActual Line 3</pre>\n<p>Cool, huh?</p>";
         $this->assertEquals( $expected, trim( wpautop( $str ) ) );
@@ -337 +337 @@
 
         $expected = join( "\n", $content );
-        $input = join( "\n\n", $content ); // WS difference
+        $input    = join( "\n\n", $content ); // WS difference
 
         $this->assertEquals( $expected, trim( wpautop( $input ) ) );
 
-        $input = join( "", $content ); // WS difference
+        $input = join( '', $content ); // WS difference
 
         $this->assertEquals( $expected, trim( wpautop( $input ) ) );
@@ -353 +353 @@
 
         $expected = join( "\n", $content );
-        $input = join( "", $content );
+        $input    = join( '', $content );
 
         $this->assertEquals( $expected, trim( wpautop( $input ) ) );
@@ -365 +365 @@
 
         $expected = join( "\n", $content );
-        $input = join( "", $content );
+        $input    = join( '', $content );
 
         $this->assertEquals( $expected, trim( wpautop( $input ) ) );
@@ -376 +376 @@
      */
     function test_that_wpautop_does_not_wrap_blockquotes_but_does_autop_their_contents() {
-        $content  = "<blockquote>foo</blockquote>";
-        $expected = "<blockquote><p>foo</p></blockquote>";
+        $content  = '<blockquote>foo</blockquote>';
+        $expected = '<blockquote><p>foo</p></blockquote>';
 
         $this->assertEquals( $expected, trim( wpautop( $content ) ) );
@@ -420 +420 @@
 
         foreach ( $inlines as $inline ) {
-            $content[] = "<$inline>foo</$inline>";
+            $content[]  = "<$inline>foo</$inline>";
             $expected[] = "<p><$inline>foo</$inline></p>";
         }
 
-        $content = join( "\n\n", $content );
+        $content  = join( "\n\n", $content );
         $expected = join( "\n", $expected );
 
@@ -458 +458 @@
                 "<p>Hello <!-- <object>\n<param/>\n<param/>\n<embed>\n</embed>\n</object>\n --></p>\n",
             ),
-/* Block elements inside comments will fail this test in all versions, it's not a regression.
-            array(
-                "Hello <!-- <hr> a\nhref='world' -->",
-                "<p>Hello <!-- <hr> a\nhref='world' --></p>\n",
-            ),
-            array(
-                "Hello <![CDATA[ <hr> a\nhttps://youtu.be/jgz0uSaOZbE\n ]]>",
-                "<p>Hello <![CDATA[ <hr> a\nhttps://youtu.be/jgz0uSaOZbE\n ]]></p>\n",
-            ),
-*/
-            array(
-                "Hello <![CDATA[ a\nhttps://youtu.be/jgz0uSaOZbE\n ]]>",
-                "<p>Hello <![CDATA[ a\nhttps://youtu.be/jgz0uSaOZbE\n ]]></p>\n",
-            ),
+            /* Block elements inside comments will fail this test in all versions, it's not a regression.
+                array(
+                    "Hello <!-- <hr> a\nhref='world' -->",
+                    "<p>Hello <!-- <hr> a\nhref='world' --></p>\n",
+                ),
+                array(
+                    "Hello <![CDATA[ <hr> a\nhttps://youtu.be/jgz0uSaOZbE\n ]]>",
+                    "<p>Hello <![CDATA[ <hr> a\nhttps://youtu.be/jgz0uSaOZbE\n ]]></p>\n",
+                ),
+            */
+                array(
+                    "Hello <![CDATA[ a\nhttps://youtu.be/jgz0uSaOZbE\n ]]>",
+                    "<p>Hello <![CDATA[ a\nhttps://youtu.be/jgz0uSaOZbE\n ]]></p>\n",
+                ),
             array(
                 "Hello <![CDATA[ <!-- a\nhttps://youtu.be/jgz0uSaOZbE\n a\n9 ]]> -->",
@@ -530 +530 @@
      */
     function test_that_text_before_blocks_is_peed() {
-        $content = 'a<div>b</div>';
+        $content  = 'a<div>b</div>';
         $expected = "<p>a</p>\n<div>b</div>";