Changeset 57649 for trunk/tests/phpunit/tests/interactivity-api/wpInteractivityAPIDirectivesProcessor.php

Timestamp:

02/17/2024 03:26:43 PM (10 months ago)

Author:

swissspidy

Message:

Interactivity API: Skip instead of bail out if HTML contains SVG or MATH.

Addresses an issue with server-side processing of directives when there is e.g. an SVG icon a navigation menu.

Props cbravobernal, westonruter, dmsnell, swissspidy.
Fixes #60517.

File:

• trunk/tests/phpunit/tests/interactivity-api/wpInteractivityAPIDirectivesProcessor.php (modified) (2 diffs)

trunk/tests/phpunit/tests/interactivity-api/wpInteractivityAPIDirectivesProcessor.php

@@ -12 +12 @@
  * @coversDefaultClass WP_Interactivity_API_Directives_Processor
  */
-class Tests_WP_Interactivity_API_Directives_Processor extends WP_UnitTestCase {
+class Tests_Interactivity_API_WpInteractivityAPIDirectivesProcessor extends WP_UnitTestCase {
     /**
      * Tests the `get_content_between_balanced_template_tags` method on template
@@ -779 +779 @@
         $this->assertFalse( $p->next_balanced_tag_closer_tag() );
     }
+
+    /**
+     * Tests that skip_to_tag_closer skips to the next tag,
+     * independant of the content.
+     *
+     * @ticket 60517
+     *
+     * @covers ::skip_to_tag_closer
+     */
+    public function test_skip_to_tag_closer() {
+        $content = '<div><span>Not closed</div>';
+        $p       = new WP_Interactivity_API_Directives_Processor( $content );
+        $p->next_tag();
+        $this->assertTrue( $p->skip_to_tag_closer() );
+        $this->assertTrue( $p->is_tag_closer() );
+        $this->assertEquals( 'DIV', $p->get_tag() );
+    }
+
+    /**
+     * Tests that skip_to_tag_closer does not skip to the
+     * next tag if there is no closing tag.
+     *
+     * @ticket 60517
+     *
+     * @covers ::skip_to_tag_closer
+     */
+    public function test_skip_to_tag_closer_bails_not_closed() {
+        $content = '<div>Not closed parent';
+        $p       = new WP_Interactivity_API_Directives_Processor( $content );
+        $p->next_tag();
+        $this->assertFalse( $p->skip_to_tag_closer() );
+    }
+
+    /**
+     * Tests that skip_to_tag_closer does not skip to the next
+     * tag if the closing tag is different from the current tag.
+     *
+     * @ticket 60517
+     *
+     * @covers ::skip_to_tag_closer
+     */
+    public function test_skip_to_tag_closer_bails_different_tags() {
+        $content = '<div></span>';
+        $p       = new WP_Interactivity_API_Directives_Processor( $content );
+        $p->next_tag();
+        $this->assertFalse( $p->skip_to_tag_closer() );
+    }
 }