WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
03/19/2023 12:03:30 PM (21 months ago)
Author:
SergeyBiryukov
Message:

Tests: Use the data_ prefix for various data provider methods.

This aims to bring more consistency to the test suite, as the vast majority of data providers already use that prefix.

Includes moving some data providers next to the tests they are used in.

Follow-up to [55464].

See #57841.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/formatting/balanceTags.php

    r54727 r55562  
    88class Tests_Formatting_BalanceTags extends WP_UnitTestCase {
    99
    10     public function nestable_tags() {
    11         return array(
    12             array( 'article' ),
    13             array( 'aside' ),
    14             array( 'blockquote' ),
    15             array( 'details' ),
    16             array( 'div' ),
    17             array( 'figure' ),
    18             array( 'object' ),
    19             array( 'q' ),
    20             array( 'section' ),
    21             array( 'span' ),
    22         );
    23     }
    24 
    25     // This is a complete(?) listing of valid single/self-closing tags.
    26     public function single_tags() {
    27         return array(
    28             array( 'area' ),
    29             array( 'base' ),
    30             array( 'basefont' ),
    31             array( 'br' ),
    32             array( 'col' ),
    33             array( 'command' ),
    34             array( 'embed' ),
    35             array( 'frame' ),
    36             array( 'hr' ),
    37             array( 'img' ),
    38             array( 'input' ),
    39             array( 'isindex' ),
    40             array( 'link' ),
    41             array( 'meta' ),
    42             array( 'param' ),
    43             array( 'source' ),
    44             array( 'track' ),
    45             array( 'wbr' ),
    46         );
    47     }
    48 
    49     public function supported_traditional_tag_names() {
     10    /**
     11     * @ticket 47014
     12     * @dataProvider data_supported_traditional_tag_names
     13     */
     14    public function test_detects_traditional_tag_names( $tag ) {
     15        $normalized = strtolower( $tag );
     16
     17        $this->assertSame( "<$normalized>inside</$normalized>", balanceTags( "<$tag>inside", true ) );
     18    }
     19
     20    public function data_supported_traditional_tag_names() {
    5021        return array(
    5122            array( 'a' ),
    … …  
    5930    }
    6031
    61     public function supported_custom_element_tag_names() {
     32    /**
     33     * @ticket 47014
     34     * @dataProvider data_supported_custom_element_tag_names
     35     */
     36    public function test_detects_supported_custom_element_tag_names( $tag ) {
     37        $this->assertSame( "<$tag>inside</$tag>", balanceTags( "<$tag>inside", true ) );
     38    }
     39
     40    public function data_supported_custom_element_tag_names() {
    6241        return array(
    6342            array( 'custom-element' ),
    … …  
    7150    }
    7251
    73     public function invalid_tag_names() {
     52    /**
     53     * @ticket 47014
     54     * @dataProvider data_invalid_tag_names
     55     */
     56    public function test_ignores_invalid_tag_names( $input, $output ) {
     57        $this->assertSame( $output, balanceTags( $input, true ) );
     58    }
     59
     60    public function data_invalid_tag_names() {
    7461        return array(
    7562            array( '<0-day>inside', '&lt;0-day>inside' ), // Can't start with a number - handled by the "<3" fix.
    … …  
    7966
    8067    /**
     68     * @ticket 47014
     69     * @dataProvider data_unsupported_valid_tag_names
     70     */
     71    public function test_ignores_unsupported_custom_tag_names( $tag ) {
     72        $this->assertSame( "<$tag>inside", balanceTags( "<$tag>inside", true ) );
     73    }
     74
     75    /**
    8176     * These are valid custom elements but we don't support them yet.
    8277     *
    8378     * @see https://html.spec.whatwg.org/multipage/custom-elements.html#valid-custom-element-name
    8479     */
    85     public function unsupported_valid_tag_names() {
     80    public function data_unsupported_valid_tag_names() {
    8681        return array(
    8782            // We don't allow ending in a dash.
    … …  
    139134
    140135    /**
     136     * @ticket 47014
     137     * @dataProvider data_supported_invalid_tag_names
     138     */
     139    public function test_detects_supported_invalid_tag_names( $tag ) {
     140        $this->assertSame( "<$tag>inside</$tag>", balanceTags( "<$tag>inside", true ) );
     141    }
     142
     143    /**
    141144     * These are invalid custom elements but we support them right now in order to keep the parser simpler.
    142145     *
    143146     * @see https://w3c.github.io/webcomponents/spec/custom/#valid-custom-element-name
    144147     */
    145     public function supported_invalid_tag_names() {
     148    public function data_supported_invalid_tag_names() {
    146149        return array(
    147150            // Reserved names for custom elements.
    … …  
    158161
    159162    /**
    160      * @ticket 47014
    161      * @dataProvider supported_traditional_tag_names
    162      */
    163     public function test_detects_traditional_tag_names( $tag ) {
    164         $normalized = strtolower( $tag );
    165 
    166         $this->assertSame( "<$normalized>inside</$normalized>", balanceTags( "<$tag>inside", true ) );
    167     }
    168 
    169     /**
    170      * @ticket 47014
    171      * @dataProvider supported_custom_element_tag_names
    172      */
    173     public function test_detects_supported_custom_element_tag_names( $tag ) {
    174         $this->assertSame( "<$tag>inside</$tag>", balanceTags( "<$tag>inside", true ) );
    175     }
    176 
    177     /**
    178      * @ticket 47014
    179      * @dataProvider invalid_tag_names
    180      */
    181     public function test_ignores_invalid_tag_names( $input, $output ) {
    182         $this->assertSame( $output, balanceTags( $input, true ) );
    183     }
    184 
    185     /**
    186      * @ticket 47014
    187      * @dataProvider unsupported_valid_tag_names
    188      */
    189     public function test_ignores_unsupported_custom_tag_names( $tag ) {
    190         $this->assertSame( "<$tag>inside", balanceTags( "<$tag>inside", true ) );
    191     }
    192 
    193     /**
    194      * @ticket 47014
    195      * @dataProvider supported_invalid_tag_names
    196      */
    197     public function test_detects_supported_invalid_tag_names( $tag ) {
    198         $this->assertSame( "<$tag>inside</$tag>", balanceTags( "<$tag>inside", true ) );
    199     }
    200 
    201     /**
    202163     * If a recognized valid single tag appears unclosed, it should get self-closed
    203164     *
    204165     * @ticket 1597
    205      * @dataProvider single_tags
     166     * @dataProvider data_single_tags
    206167     */
    207168    public function test_selfcloses_unclosed_known_single_tags( $tag ) {
    … …  
    214175     *
    215176     * @ticket 1597
    216      * @dataProvider single_tags
     177     * @dataProvider data_single_tags
    217178     */
    218179    public function test_selfcloses_known_single_tags_having_closing_tag( $tag ) {
    219180        $this->assertSame( "<$tag />", balanceTags( "<$tag></$tag>", true ) );
     181    }
     182
     183    // This is a complete(?) listing of valid single/self-closing tags.
     184    public function data_single_tags() {
     185        return array(
     186            array( 'area' ),
     187            array( 'base' ),
     188            array( 'basefont' ),
     189            array( 'br' ),
     190            array( 'col' ),
     191            array( 'command' ),
     192            array( 'embed' ),
     193            array( 'frame' ),
     194            array( 'hr' ),
     195            array( 'img' ),
     196            array( 'input' ),
     197            array( 'isindex' ),
     198            array( 'link' ),
     199            array( 'meta' ),
     200            array( 'param' ),
     201            array( 'source' ),
     202            array( 'track' ),
     203            array( 'wbr' ),
     204        );
    220205    }
    221206
    … …  
    275260
    276261    /**
    277      * @dataProvider nestable_tags
     262     * @dataProvider data_nestable_tags
    278263     */
    279264    public function test_balances_nestable_tags( $tag ) {
    … …  
    292277            $this->assertSame( $expected[ $key ], balanceTags( $inputs[ $key ], true ) );
    293278        }
     279    }
     280
     281    public function data_nestable_tags() {
     282        return array(
     283            array( 'article' ),
     284            array( 'aside' ),
     285            array( 'blockquote' ),
     286            array( 'details' ),
     287            array( 'div' ),
     288            array( 'figure' ),
     289            array( 'object' ),
     290            array( 'q' ),
     291            array( 'section' ),
     292            array( 'span' ),
     293        );
    294294    }
    295295
Note: See TracChangeset for help on using the changeset viewer.