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

Timestamp:

07/01/2019 03:18:28 AM (7 years ago)

Author:

pento

Message:

Formatting: Don't add <p> tags inside <svg> tags.

Inline <svg>s should generally work, as browsers should just ignore <p> or <br/> tags that shouldn't be inside the <svg>. To keep things neat, however, it's better not add them in the first place.

Props jared_smith, nacin, pento.
Fixes #9437.

File:

• trunk/tests/phpunit/tests/formatting/Autop.php (modified) (1 diff)

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

@@ -567 +567 @@
         $this->assertEquals( $expected, trim( wpautop( $content ) ) );
     }
+
+    /**
+     * wpautop() should ignore inline SVG graphics
+     *
+     * @ticket 9437
+     */
+    function test_that_wpautop_ignores_inline_svgs() {
+        $content =
+            '<svg xmlns="http://www.w3.org/2000/svg">
+                <circle cx="50" cy="50" r="30" fill="blue">
+                    <animateTransform attributeName="transform" type="scale" to="1.5" dur="2s" fill="freeze"/>
+                </circle>
+            </svg>';
+
+        $expected = '<p>' . $content . '</p>';
+
+        $this->assertEquals( $expected, trim( wpautop( $content ) ) );
+    }
+
+    /**
+     * wpautop() should ignore inline scripts
+     *
+     * @ticket 9437
+     */
+    function test_that_wpautop_ignores_inline_scripts() {
+        $content =
+            '<script type="text/javascript">
+                var dummy = 1;
+            </script>';
+
+        $expected = '<p>' . $content . '</p>';
+
+        $this->assertEquals( $expected, trim( wpautop( $content ) ) );
+    }
 }