Changeset 58048 for trunk/tests/phpunit/tests/html-api/wpHtmlTagProcessor-bookmark.php

Timestamp:

04/26/2024 01:51:06 PM (2 years ago)

Author:

dmsnell

Message:

HTML API: Tests should use expectedIncorrectUsage phpunit annotation

Some tests in the HTML API test suite use setExpectedIncorrectUsage.
The annotation @expectedIncorrectUsage can be used to simplify the tests.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/6449
Discussed in https://core.trac.wordpress.org/ticket/61080

Fixes #61080.
Props dmsnell, jonsurrell.

File:

• trunk/tests/phpunit/tests/html-api/wpHtmlTagProcessor-bookmark.php (modified) (2 diffs)

trunk/tests/phpunit/tests/html-api/wpHtmlTagProcessor-bookmark.php

@@ -415 +415 @@
      *
      * @covers WP_HTML_Tag_Processor::set_bookmark
+     * @expectedIncorrectUsage WP_HTML_Tag_Processor::set_bookmark
      */
     public function test_limits_the_number_of_bookmarks() {
@@ -424 +425 @@
         }
 
-        $this->setExpectedIncorrectUsage( 'WP_HTML_Tag_Processor::set_bookmark' );
         $this->assertFalse( $processor->set_bookmark( 'final bookmark' ), "Allocated $i bookmarks, which is one above the limit" );
     }