Changeset 51565 for trunk/tests/phpunit/tests/post/nav-menu.php

Timestamp:

08/06/2021 09:52:06 PM (3 years ago)

Author:

SergeyBiryukov

Message:

Tests: Replace assertRegExp() with assertMatchesRegularExpression().

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-51564].

Props jrf.
See #46149.

File:

• trunk/tests/phpunit/tests/post/nav-menu.php (modified) (2 diffs)

trunk/tests/phpunit/tests/post/nav-menu.php

@@ -461 +461 @@
 
         // The markup should include whitespace between <li>'s.
-        $this->assertRegExp( '/\s<li.*>|<\/li>\s/U', $menu );
+        $this->assertMatchesRegularExpression( '/\s<li.*>|<\/li>\s/U', $menu );
         $this->assertNotRegExp( '/<\/li><li.*>/U', $menu );
 
@@ -475 +475 @@
         // The markup should not include whitespace around <li>'s.
         $this->assertNotRegExp( '/\s<li.*>|<\/li>\s/U', $menu );
-        $this->assertRegExp( '/><li.*>|<\/li></U', $menu );
+        $this->assertMatchesRegularExpression( '/><li.*>|<\/li></U', $menu );
     }