Changeset 51566 for trunk/tests/phpunit/tests/post/template.php

Timestamp:

08/06/2021 09:55:31 PM (2 years ago)

Author:

SergeyBiryukov

Message:

Tests: Replace assertNotRegExp() with assertDoesNotMatchRegularExpression().

The assertRegExp() and assertNotRegExp() methods were hard deprecated in PHPUnit 9.1 and the functionality will be removed in PHPUnit 10.0.

The assertMatchesRegularExpression() and assertDoesNotMatchRegularExpression() methods were introduced as a replacement in PHPUnit 9.1.

These new PHPUnit methods are polyfilled by the PHPUnit Polyfills and switching to them will future-proof the tests some more.

References:

• ​https://github.com/sebastianbergmann/phpunit/blob/9.1.5/ChangeLog-9.1.md#910---2020-04-03
• ​https://github.com/sebastianbergmann/phpunit/issues/4085
• ​https://github.com/sebastianbergmann/phpunit/issues/4088

Follow-up to [51559-51565].

Props jrf.
See #46149.

File:

• trunk/tests/phpunit/tests/post/template.php (modified) (3 diffs)

trunk/tests/phpunit/tests/post/template.php

@@ -320 +320 @@
         );
 
-        $this->assertNotRegExp( '/<select[^>]+class=\'/', $found );
+        $this->assertDoesNotMatchRegularExpression( '/<select[^>]+class=\'/', $found );
     }
 
@@ -427 +427 @@
         // After falling back, the markup should include whitespace around <li>'s.
         $this->assertMatchesRegularExpression( '/\s<li.*>|<\/li>\s/U', $menu );
-        $this->assertNotRegExp( '/><li.*>|<\/li></U', $menu );
+        $this->assertDoesNotMatchRegularExpression( '/><li.*>|<\/li></U', $menu );
 
         // No menus + wp_nav_menu() falls back to wp_page_menu(), this time without a container.
@@ -449 +449 @@
 
         // After falling back, the markup should not include whitespace around <li>'s.
-        $this->assertNotRegExp( '/\s<li.*>|<\/li>\s/U', $menu );
+        $this->assertDoesNotMatchRegularExpression( '/\s<li.*>|<\/li>\s/U', $menu );
         $this->assertMatchesRegularExpression( '/><li.*>|<\/li></U', $menu );