mirrored from git://develop.git.wordpress.org/
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
2 changed files
with
190 additions
and
23 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,123 @@ | ||
| <?php | ||
| /** | ||
| * Unit tests covering WP_HTML_Processor::set_inner_markup() | ||
| * | ||
| * @package WordPress | ||
| * @subpackage HTML-API | ||
| * | ||
| * @since 6.4.0 | ||
| * | ||
| * @group html-api | ||
| * | ||
| * @coversDefaultClass WP_HTML_Processor | ||
| */ | ||
| class Tests_HtmlApi_WpHtmlProcessorSetInnerMarkup extends WP_UnitTestCase { | ||
| /** | ||
| * @ticket {TICKET_NUMBER} | ||
| * | ||
| * @covers WP_HTML_Processor::set_inner_markup | ||
| * | ||
| * @dataProvider data_html_with_inner_markup_changes | ||
| * | ||
| * @since 6.4.0 | ||
| * | ||
| * @param string $html_with_target_node HTML containing a node with the `target` attribute set. | ||
| * @param string $new_markup HTML for replacing the inner markup of the target node. | ||
| * @param string $expected_output New HTMl after replacing inner markup. | ||
| */ | ||
| public function test_replaces_inner_html_appropriately( $html_with_target_node, $new_markup, $expected_output ) { | ||
| $p = WP_HTML_Processor::createFragment( $html_with_target_node ); | ||
|
|
||
| while ( $p->next_tag() && null === $p->get_attribute( 'target' ) ) { | ||
| continue; | ||
| } | ||
|
|
||
| $this->assertTrue( $p->set_inner_markup( $new_markup ), 'Failed to set inner markup.' ); | ||
| $this->assertSame( $expected_output, $p->get_updated_html(), 'Failed to appropriately set inner markup.' ); | ||
| } | ||
|
|
||
| /** | ||
| * Data provider. | ||
| * | ||
| * @return array[] | ||
| */ | ||
| public function data_html_with_inner_markup_changes() { | ||
| $data = array( | ||
| 'Void element' => array( '<img target>', '', '<img target>' ), | ||
| 'Void element inside text' => array( 'before<img src="atat.png" loading=lazy target>after', '', 'before<img src="atat.png" loading=lazy target>after' ), | ||
| 'Void element inside another element' => array( '<p>Look at this <img target> graph.</p>', '', '<p>Look at this <img target> graph.</p>' ), | ||
| 'Empty elements' => array( '<div target></div>', '', '<div target></div>' ), | ||
| 'Element with nested tags' => array( '<div target>inside <span>the</span> div</div>', '', '<div target></div>' ), | ||
| 'Element inside another element' => array( '<div>inside <span target>the</span> div</div>', '', '<div>inside <span target></span> div</div>' ), | ||
| 'Unclosed element' => array( '<div target>This is <em>all</em> inside the DIV', '', '<div target>' ), | ||
| 'Unclosed nested element' => array( '<div><p target>One thought<p>And another', '', '<div><p target><p>And another' ), | ||
| 'Partially-closed element' => array( '<div target>This is <em>all</em> inside the DIV</div', '', '<div target>' ), | ||
| 'Implicitly-closed element' => array( '<div><p target>Inside the P</div>Outside the P</p>', '', '<div><p target></div>Outside the P</p>' ), | ||
| ); | ||
|
|
||
| $inner_html = <<<HTML | ||
| <p>This is inside the <strong>Match</strong></p> | ||
| <p><img></p> | ||
| <div> | ||
| <figure> | ||
| <img> | ||
| <figcaption>Look at the <strike>picture</strike> photograph.</figcaption> | ||
| </figure> | ||
| </div> | ||
| HTML; | ||
|
|
||
| $prefix = <<<HTML | ||
| <div> | ||
| <p>This is not in the match. | ||
| <p>This is another paragraph not <a href="#">in</a> the match. | ||
| </div> | ||
| <div target> | ||
| HTML; | ||
|
|
||
| /* | ||
| * Removing the indent on this first line keeps the test easy to reason about, | ||
| * otherwise extra indents appear after removing the inner content, because | ||
| * that indentation before and after is whitespace and not part of the tag. | ||
| */ | ||
| $suffix = <<<HTML | ||
| </div> | ||
| <div> | ||
| <p>This is also note in the match.</p> | ||
| </div> | ||
| HTML; | ||
|
|
||
| $data['Complicated inner nesting'] = array( $prefix . $inner_html . $suffix, '', $prefix . $suffix ); | ||
|
|
||
| return $data; | ||
| } | ||
|
|
||
| /** | ||
| * Ensures that the cursor isn't moved when setting the inner markup. It should | ||
| * remain at the same location as the tag opener where it was called. | ||
| * | ||
| * @ticket {TICKET_NUMBER} | ||
| * | ||
| * @covers WP_HTML_Processor::set_inner_markup | ||
| * | ||
| * @since 6.4.0 | ||
| */ | ||
| public function test_preserves_cursor() { | ||
| $p = WP_HTML_Processor::createFragment( '<div><p><span>The <code target>cursor</code> should not move <em next-target>unexpectedly</em>.</span></p></div>' ); | ||
|
|
||
| while ( $p->next_tag() && null === $p->get_attribute( 'target' ) ) { | ||
| continue; | ||
| } | ||
|
|
||
| $this->assertTrue( $p->set_inner_markup( '<img next-target>' ) ); | ||
| $this->assertSame( | ||
| '<div><p><span>The <code target><img next-target></code> should not move <em next-target>unexpectedly</em>.</span></p></div>', | ||
| $p->get_updated_html(), | ||
| 'Failed to replace appropriate inner markup.' | ||
| ); | ||
|
|
||
| $this->assertSame( 'CODE', $p->get_tag(), "Should have remained on CODE, but found {$p->get_tag()} instead." ); | ||
|
|
||
| $p->next_tag(); | ||
| $this->assertNotNull( $p->get_attribute( 'next-target' ), "Expected to move to inserted IMG element, but found {$p->get_tag()} instead." ); | ||
| } | ||
| } |