WordPress.org
Get WordPress

Make WordPress Core

Ticket #54257: 54257.3.diff

File 54257.3.diff, 9.6 KB (added by SergeyBiryukov, 4 years ago)

Use correct variable in the last data provider

  • tests/phpunit/tests/functions/isPhpVersionCompatible.php

     
    11<?php
    22
    33/**
    4  * Tests the is_php_version_compatible function.
     4 * Tests the is_php_version_compatible() function.
    55 *
    66 * @group functions.php
    77 * @covers ::is_php_version_compatible
    88 */
    9 class Tests_Functions_isPhpVersionCompatible extends WP_UnitTestCase {
     9class Tests_Functions_IsPhpVersionCompatible extends WP_UnitTestCase {
    1010        /**
    1111         * Tests is_php_version_compatible().
    1212         *
    1313         * @dataProvider data_is_php_version_compatible
    1414         *
    15          * @param mixed $test_value
    16          * @param bool $expected
     15         * @ticket 54257
    1716         *
    18          * @ticket 54257
     17         * @param mixed $required The minimum required PHP version.
     18         * @param bool  $expected The expected result.
    1919         */
    20         public function test_is_php_version_compatible( $test_value, $expected ) {
    21                 $this->assertSame( is_php_version_compatible( $test_value ), $expected );
     20        public function test_is_php_version_compatible( $required, $expected ) {
     21                $this->assertSame( $expected, is_php_version_compatible( $required ) );
    2222        }
    2323
    2424        /**
    25          * Provides test scenarios for test_php_version_compatible.
     25         * Data provider.
    2626         *
    2727         * @return array
    2828         */
    29         function data_is_php_version_compatible() {
     29        public function data_is_php_version_compatible() {
    3030                $php_version = phpversion();
    3131
    32                 $more = explode( '.', $php_version );
    33                 $less = $more;
     32                $version_parts  = explode( '.', $php_version );
     33                $lower_version  = $version_parts;
     34                $higher_version = $version_parts;
    3435
    35                 -- $less[ count( $less ) - 1 ];
    36                 ++ $more[ count( $less ) - 1 ];
     36                // Adjust the major version numbers.
     37                --$lower_version[0];
     38                ++$higher_version[0];
    3739
     40                $lower_version  = implode( '.', $lower_version );
     41                $higher_version = implode( '.', $higher_version );
     42
    3843                return array(
    39                         'greater' => array(
    40                                 'test_value' => implode( '.', $more ),
    41                                 'expected'   => false,
     44                        // Happy paths.
     45                        'a lower required version'  => array(
     46                                'required' => $lower_version,
     47                                'expected' => true,
    4248                        ),
    43                         'same'    => array(
    44                                 'test_value' => $php_version,
    45                                 'expected'   => true,
     49                        'the same version'          => array(
     50                                'required' => $php_version,
     51                                'expected' => true,
    4652                        ),
    47                         'less'    => array(
    48                                 'test_value' => implode( '.', $less ),
    49                                 'expected'   => true,
     53                        'a higher required version' => array(
     54                                'required' => $higher_version,
     55                                'expected' => false,
    5056                        ),
     57
     58                        // Falsey values.
     59                        'false'                     => array(
     60                                'required' => false,
     61                                'expected' => true,
     62                        ),
     63                        'null'                      => array(
     64                                'required' => null,
     65                                'expected' => true,
     66                        ),
     67                        '0 int'                     => array(
     68                                'required' => 0,
     69                                'expected' => true,
     70                        ),
     71                        '0.0 float'                 => array(
     72                                'required' => 0.0,
     73                                'expected' => true,
     74                        ),
     75                        '0 string'                  => array(
     76                                'required' => '0',
     77                                'expected' => true,
     78                        ),
     79                        'empty string'              => array(
     80                                'required' => '',
     81                                'expected' => true,
     82                        ),
     83                        'empty array'               => array(
     84                                'required' => array(),
     85                                'expected' => true,
     86                        ),
    5187                );
    5288        }
    5389}

  • tests/phpunit/tests/functions/isWpVersionCompatible.php

     
    11<?php
    22
    33/**
    4  * Tests the is_php_version_compatible function.
     4 * Tests the is_wp_version_compatible() function.
    55 *
    66 * @group functions.php
    77 * @covers ::is_wp_version_compatible
    88 */
    9 class Tests_Functions_isWpVersionCompatible extends WP_UnitTestCase {
     9class Tests_Functions_IsWpVersionCompatible extends WP_UnitTestCase {
    1010        /**
    11          * Test is_wp_version_compatible().
     11         * Tests is_wp_version_compatible().
    1212         *
    1313         * @dataProvider data_is_wp_version_compatible
    1414         *
    15          * @param mixed $test_value
    16          * @param bool $expected
     15         * @ticket 54257
    1716         *
     17         * @param mixed $required The minimum required WordPress version.
     18         * @param bool  $expected The expected result.
     19         */
     20        public function test_is_wp_version_compatible( $required, $expected ) {
     21                $this->assertSame( $expected, is_wp_version_compatible( $required ) );
     22        }
     23
     24        /**
     25         * Data provider.
     26         *
     27         * @return array
     28         */
     29        public function data_is_wp_version_compatible() {
     30                global $wp_version;
     31
     32                $version_parts  = explode( '.', $wp_version );
     33                $lower_version  = $version_parts;
     34                $higher_version = $version_parts;
     35
     36                // Adjust the major version numbers.
     37                --$lower_version[0];
     38                ++$higher_version[0];
     39
     40                $lower_version  = implode( '.', $lower_version );
     41                $higher_version = implode( '.', $higher_version );
     42
     43                return array(
     44                        // Happy paths.
     45                        'the same version'          => array(
     46                                'required' => $wp_version,
     47                                'expected' => true,
     48                        ),
     49                        'a lower required version'  => array(
     50                                'required' => $lower_version,
     51                                'expected' => true,
     52                        ),
     53                        'a higher required version' => array(
     54                                'required' => $higher_version,
     55                                'expected' => false,
     56                        ),
     57
     58                        // Falsey values.
     59                        'false'                     => array(
     60                                'required' => false,
     61                                'expected' => true,
     62                        ),
     63                        'null'                      => array(
     64                                'required' => null,
     65                                'expected' => true,
     66                        ),
     67                        '0 int'                     => array(
     68                                'required' => 0,
     69                                'expected' => true,
     70                        ),
     71                        '0.0 float'                 => array(
     72                                'required' => 0.0,
     73                                'expected' => true,
     74                        ),
     75                        '0 string'                  => array(
     76                                'required' => '0',
     77                                'expected' => true,
     78                        ),
     79                        'empty string'              => array(
     80                                'required' => '',
     81                                'expected' => true,
     82                        ),
     83                        'empty array'               => array(
     84                                'required' => array(),
     85                                'expected' => true,
     86                        ),
     87                );
     88        }
     89
     90        /**
     91         * Tests is_wp_version_compatible() with development versions.
     92         *
     93         * @dataProvider data_is_wp_version_compatible_with_development_versions
     94         *
    1895         * @ticket 54257
     96         *
     97         * @param string $required  The minimum required WordPress version.
     98         * @param string $wp        The value for the $wp_version global variable.
     99         * @param bool   $expected  The expected result.
    19100         */
    20         public function test_is_wp_version_compatible( $test_value, $expected ) {
    21                 $this->assertSame( is_wp_version_compatible( $test_value ), $expected );
     101        public function test_is_wp_version_compatible_with_development_versions( $required, $wp, $expected ) {
     102                global $wp_version;
     103
     104                $original_version = $wp_version;
     105                $wp_version       = $wp;
     106                $actual           = is_wp_version_compatible( $required );
     107
     108                // Reset the version before the assertion in case of failure.
     109                $wp_version = $original_version;
     110
     111                $this->assertSame( $expected, $actual );
    22112        }
    23113
    24114        /**
    25          * Provides test scenarios test_is_wp_version_compatible.
     115         * Data provider.
    26116         *
    27117         * @return array
    28118         */
    29         function data_is_wp_version_compatible() {
    30                 $wp_version = get_bloginfo( 'version' );
     119        public function data_is_wp_version_compatible_with_development_versions() {
     120                global $wp_version;
    31121
    32                 $more = explode( '.', $wp_version );
    33                 $less = $more;
     122                // For consistent results, remove possible suffixes.
     123                list( $version ) = explode( '-', $wp_version );
    34124
    35                 -- $less[0];
    36                 ++ $more[0];
     125                $version_parts  = explode( '.', $version );
     126                $lower_version  = $version_parts;
     127                $higher_version = $version_parts;
    37128
     129                // Adjust the major version numbers.
     130                --$lower_version[0];
     131                ++$higher_version[0];
     132
     133                $lower_version  = implode( '.', $lower_version );
     134                $higher_version = implode( '.', $higher_version );
     135
    38136                return array(
    39                         'greater' => array(
    40                                 'test_value' => implode( '.', $more ),
    41                                 'expected'   => false,
     137                        'a lower required version and an alpha wordpress version' => array(
     138                                'required' => $lower_version,
     139                                'wp'       => $version . '-alpha-12341-src',
     140                                'expected' => true,
    42141                        ),
    43                         'same'    => array(
    44                                 'test_value' => $wp_version,
    45                                 'expected'   => true,
     142                        'a lower required version and a beta wordpress version'   => array(
     143                                'required' => $lower_version,
     144                                'wp'       => $version . '-beta1',
     145                                'expected' => true,
    46146                        ),
    47                         'less'    => array(
    48                                 'test_value' => implode( '.', $less ),
    49                                 'expected'   => true,
     147                        'a lower required version and a release candidate wordpress version'   => array(
     148                                'required' => $lower_version,
     149                                'wp'       => $version . '-RC1',
     150                                'expected' => true,
    50151                        ),
     152                        'the same required version and an alpha wordpress version' => array(
     153                                'required' => $version,
     154                                'wp'       => $version . '-alpha-12341-src',
     155                                'expected' => true,
     156                        ),
     157                        'the same required version and a beta wordpress version' => array(
     158                                'required' => $version,
     159                                'wp'       => $version . '-beta1',
     160                                'expected' => true,
     161                        ),
     162                        'the same required version and a release candidate wordpress version' => array(
     163                                'required' => $version,
     164                                'wp'       => $version . '-RC1',
     165                                'expected' => true,
     166                        ),
     167                        'a higher required version and an alpha wordpress version'   => array(
     168                                'required' => $higher_version,
     169                                'wp'       => $version . '-alpha-12341-src',
     170                                'expected' => false,
     171                        ),
     172                        'a higher required version and a beta wordpress version'   => array(
     173                                'required' => $higher_version,
     174                                'wp'       => $version . '-beta1',
     175                                'expected' => false,
     176                        ),
     177                        'a higher required version and a release candidate wordpress version'   => array(
     178                                'required' => $higher_version,
     179                                'wp'       => $version . '-RC1',
     180                                'expected' => false,
     181                        ),
    51182                );
    52183        }
    53184}