Changeset 57508 for trunk/tests/phpunit/tests/html-api/wpHtmlSupportRequiredHtmlProcessor.php

Timestamp:

02/01/2024 01:00:56 AM (23 months ago)

Author:

dmsnell

Message:

HTML API: Test cleanup

Rename $p variable to $processor in tests for clarity.

Use static data providers. A mix of static and non-static data providers were
used in HTML API tests. Data providers are required to be static in the next
PHPUnit version and there's no harm in using them consistently now.

Follow-up to [57507]

Props jonsurrell
See #59647

File:

• trunk/tests/phpunit/tests/html-api/wpHtmlSupportRequiredHtmlProcessor.php (modified) (1 diff)

trunk/tests/phpunit/tests/html-api/wpHtmlSupportRequiredHtmlProcessor.php

@@ -43 +43 @@
      */
     private function ensure_support_is_added_everywhere( $tag_name ) {
-        $p = WP_HTML_Processor::create_fragment( "<$tag_name>" );
+        $processor = WP_HTML_Processor::create_fragment( "<$tag_name>" );
 
-        $this->assertFalse( $p->step(), "Must support terminating elements in specific scope check before adding support for the {$tag_name} element." );
+        $this->assertFalse( $processor->step(), "Must support terminating elements in specific scope check before adding support for the {$tag_name} element." );
     }