Make WordPress Core


Ignore:
Timestamp:
09/05/2022 09:14:38 PM (23 months ago)
Author:
SergeyBiryukov
Message:

Tests: Temporarily skip the test for recommended PHP version in readme.html.

This test verifies that the WordPress readme.html file recommends a PHP version that is actively supported. However, WordPress currently still recommends PHP 7.4 due to PHP 8.0/8.1 compatibility not being fully achieved, even though PHP 7.4 is end-of-life.

As things were, the assertion in the test was commented out, leading to this test being marked as “risky” for not performing any assertions.

Instead, let’s skip the test with a clear skip notification.

Follow-up to [52260].

Props jrf.
See #55652.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/external-http/basic.php

    r53435 r54074  
    1111     */
    1212    public function test_readme_php_version() {
     13        $this->markTestSkipped(
     14            'Temporarily disabled. Test should be re-enabled once WordPress is fully compatible with PHP 8.0+.'
     15        );
     16
    1317        // This test is designed to only run on trunk.
    1418        $this->skipOnAutomatedBranches();
     
    2226        preg_match_all( '#<tr class="stable">\s*<td>\s*<a [^>]*>\s*([0-9.]*)#s', $response_body, $php_matches );
    2327
    24         // TODO: Enable this check once PHP 8.0 compatibility is achieved.
    25         // $this->assertContains( $matches[1], $php_matches[1], "readme.html's Recommended PHP version is too old. Remember to update the WordPress.org Requirements page, too." );
     28        $this->assertContains( $matches[1], $php_matches[1], "readme.html's Recommended PHP version is too old. Remember to update the WordPress.org Requirements page, too." );
    2629    }
    2730
Note: See TracChangeset for help on using the changeset viewer.