WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/03/2021 12:48:42 AM (3 years ago)
Author:
SergeyBiryukov
Message:

Tests: Split WP_Posts_List_Table and WP_Comments_List_Table tests into two separate files for clarity.

These were previously combined in the includesListTable.php file. Since the tests were specific neither to the _get_list_table() function nor the parent WP_List_Table class, the naming was confusing, which should now be resolved.

Follow-up to [31730], [38854], [40297], [48151], [48521], [49190], [51993].

See #53363.

File:
1 copied

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/admin/wpCommentsListTable.php

    r51995 r51997  
    44 * @group admin
    55 */
    6 class Tests_Admin_IncludesListTable extends WP_UnitTestCase {
    7     protected static $top           = array();
    8     protected static $children      = array();
    9     protected static $grandchildren = array();
    10     protected static $post_ids      = array();
     6class Tests_Admin_wpCommentsListTable extends WP_UnitTestCase {
    117
    128    /**
    13      * @var WP_Posts_List_Table
     9     * @var WP_Comments_List_Table
    1410     */
    1511    protected $table;
    … …  
    1713    function set_up() {
    1814        parent::set_up();
    19         $this->table = _get_list_table( 'WP_Posts_List_Table', array( 'screen' => 'edit-page' ) );
    20     }
    21 
    22     public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
    23         // Note that our top/children/grandchildren arrays are 1-indexed.
    24 
    25         // Create top-level pages.
    26         $num_posts = 5;
    27         foreach ( range( 1, $num_posts ) as $i ) {
    28             $p = $factory->post->create_and_get(
    29                 array(
    30                     'post_type'  => 'page',
    31                     'post_title' => sprintf( 'Top Level Page %d', $i ),
    32                 )
    33             );
    34 
    35             self::$top[ $i ]  = $p;
    36             self::$post_ids[] = $p->ID;
    37         }
    38 
    39         // Create child pages.
    40         $num_children = 3;
    41         foreach ( self::$top as $top => $top_page ) {
    42             foreach ( range( 1, $num_children ) as $i ) {
    43                 $p = $factory->post->create_and_get(
    44                     array(
    45                         'post_type'   => 'page',
    46                         'post_parent' => $top_page->ID,
    47                         'post_title'  => sprintf( 'Child %d', $i ),
    48                     )
    49                 );
    50 
    51                 self::$children[ $top ][ $i ] = $p;
    52                 self::$post_ids[]             = $p->ID;
    53             }
    54         }
    55 
    56         // Create grand-child pages for the third and fourth top-level pages.
    57         $num_grandchildren = 3;
    58         foreach ( range( 3, 4 ) as $top ) {
    59             foreach ( self::$children[ $top ] as $child => $child_page ) {
    60                 foreach ( range( 1, $num_grandchildren ) as $i ) {
    61                     $p = $factory->post->create_and_get(
    62                         array(
    63                             'post_type'   => 'page',
    64                             'post_parent' => $child_page->ID,
    65                             'post_title'  => sprintf( 'Grandchild %d', $i ),
    66                         )
    67                     );
    68 
    69                     self::$grandchildren[ $top ][ $child ][ $i ] = $p;
    70                     self::$post_ids[]                            = $p->ID;
    71                 }
    72             }
    73         }
    74     }
    75 
    76     /**
    77      * @ticket 15459
    78      *
    79      * @covers WP_Posts_List_Table::display_rows
    80      * @covers WP_Posts_List_Table::set_hierarchical_display
    81      */
    82     function test_list_hierarchical_pages_first_page() {
    83         $this->_test_list_hierarchical_page(
    84             array(
    85                 'paged'          => 1,
    86                 'posts_per_page' => 2,
    87             ),
    88             array(
    89                 self::$top[1]->ID,
    90                 self::$children[1][1]->ID,
    91             )
    92         );
    93     }
    94 
    95     /**
    96      * @ticket 15459
    97      *
    98      * @covers WP_Posts_List_Table::display_rows
    99      * @covers WP_Posts_List_Table::set_hierarchical_display
    100      */
    101     function test_list_hierarchical_pages_second_page() {
    102         $this->_test_list_hierarchical_page(
    103             array(
    104                 'paged'          => 2,
    105                 'posts_per_page' => 2,
    106             ),
    107             array(
    108                 self::$top[1]->ID,
    109                 self::$children[1][2]->ID,
    110                 self::$children[1][3]->ID,
    111             )
    112         );
    113     }
    114 
    115     /**
    116      * @ticket 15459
    117      *
    118      * @covers WP_Posts_List_Table::display_rows
    119      * @covers WP_Posts_List_Table::set_hierarchical_display
    120      */
    121     function test_search_hierarchical_pages_first_page() {
    122         $this->_test_list_hierarchical_page(
    123             array(
    124                 'paged'          => 1,
    125                 'posts_per_page' => 2,
    126                 's'              => 'Child',
    127             ),
    128             array(
    129                 self::$children[1][1]->ID,
    130                 self::$children[1][2]->ID,
    131             )
    132         );
    133     }
    134 
    135     /**
    136      * @ticket 15459
    137      *
    138      * @covers WP_Posts_List_Table::display_rows
    139      * @covers WP_Posts_List_Table::set_hierarchical_display
    140      */
    141     function test_search_hierarchical_pages_second_page() {
    142         $this->_test_list_hierarchical_page(
    143             array(
    144                 'paged'          => 2,
    145                 'posts_per_page' => 2,
    146                 's'              => 'Top',
    147             ),
    148             array(
    149                 self::$top[3]->ID,
    150                 self::$top[4]->ID,
    151             )
    152         );
    153     }
    154 
    155     /**
    156      * @ticket 15459
    157      *
    158      * @covers WP_Posts_List_Table::display_rows
    159      * @covers WP_Posts_List_Table::set_hierarchical_display
    160      */
    161     function test_grandchildren_hierarchical_pages_first_page() {
    162         // Page 6 is the first page with grandchildren.
    163         $this->_test_list_hierarchical_page(
    164             array(
    165                 'paged'          => 6,
    166                 'posts_per_page' => 2,
    167             ),
    168             array(
    169                 self::$top[3]->ID,
    170                 self::$children[3][1]->ID,
    171                 self::$grandchildren[3][1][1]->ID,
    172                 self::$grandchildren[3][1][2]->ID,
    173             )
    174         );
    175     }
    176 
    177     /**
    178      * @ticket 15459
    179      *
    180      * @covers WP_Posts_List_Table::display_rows
    181      * @covers WP_Posts_List_Table::set_hierarchical_display
    182      */
    183     function test_grandchildren_hierarchical_pages_second_page() {
    184         // Page 7 is the second page with grandchildren.
    185         $this->_test_list_hierarchical_page(
    186             array(
    187                 'paged'          => 7,
    188                 'posts_per_page' => 2,
    189             ),
    190             array(
    191                 self::$top[3]->ID,
    192                 self::$children[3][1]->ID,
    193                 self::$grandchildren[3][1][3]->ID,
    194                 self::$children[3][2]->ID,
    195             )
    196         );
    197     }
    198 
    199     /**
    200      * Helper function to test the output of a page which uses `WP_Posts_List_Table`.
    201      *
    202      * @param array $args         Query args for the list of pages.
    203      * @param array $expected_ids Expected IDs of pages returned.
    204      */
    205     protected function _test_list_hierarchical_page( array $args, array $expected_ids ) {
    206         if ( PHP_VERSION_ID >= 80100 ) {
    207             /*
    208              * For the time being, ignoring PHP 8.1 "null to non-nullable" deprecations coming in
    209              * via hooked in filter functions until a more structural solution to the
    210              * "missing input validation" conundrum has been architected and implemented.
    211              */
    212             $this->expectDeprecation();
    213             $this->expectDeprecationMessageMatches( '`Passing null to parameter \#[0-9]+ \(\$[^\)]+\) of type [^ ]+ is deprecated`' );
    214         }
    215 
    216         $matches = array();
    217 
    218         $_REQUEST['paged']   = $args['paged'];
    219         $GLOBALS['per_page'] = $args['posts_per_page'];
    220 
    221         $args = array_merge(
    222             array(
    223                 'post_type' => 'page',
    224             ),
    225             $args
    226         );
    227 
    228         // Mimic the behaviour of `wp_edit_posts_query()`:
    229         if ( ! isset( $args['orderby'] ) ) {
    230             $args['orderby']                = 'menu_order title';
    231             $args['order']                  = 'asc';
    232             $args['posts_per_page']         = -1;
    233             $args['posts_per_archive_page'] = -1;
    234         }
    235 
    236         // Effectively ignore the output until retrieving it later via `getActualOutput()`.
    237         $this->expectOutputRegex( '`.`' );
    238 
    239         $pages = new WP_Query( $args );
    240 
    241         $this->table->set_hierarchical_display( true );
    242         $this->table->display_rows( $pages->posts );
    243         $output = $this->getActualOutput();
    244 
    245         // Clean up.
    246         unset( $_REQUEST['paged'] );
    247         unset( $GLOBALS['per_page'] );
    248 
    249         preg_match_all( '|<tr[^>]*>|', $output, $matches );
    250 
    251         $this->assertCount( count( $expected_ids ), array_keys( $matches[0] ) );
    252 
    253         foreach ( $expected_ids as $id ) {
    254             $this->assertStringContainsString( sprintf( 'id="post-%d"', $id ), $output );
    255         }
    256     }
    257 
    258     /**
    259      * @ticket 37407
    260      *
    261      * @covers WP_Posts_List_Table::extra_tablenav
    262      */
    263     function test_filter_button_should_not_be_shown_if_there_are_no_posts() {
    264         // Set post type to a non-existent one.
    265         $this->table->screen->post_type = 'foo';
    266 
    267         ob_start();
    268         $this->table->extra_tablenav( 'top' );
    269         $output = ob_get_clean();
    270 
    271         $this->assertStringNotContainsString( 'id="post-query-submit"', $output );
    272     }
    273 
    274     /**
    275      * @ticket 37407
    276      *
    277      * @covers WP_Posts_List_Table::extra_tablenav
    278      */
    279     function test_months_dropdown_should_not_be_shown_if_there_are_no_posts() {
    280         // Set post type to a non-existent one.
    281         $this->table->screen->post_type = 'foo';
    282 
    283         ob_start();
    284         $this->table->extra_tablenav( 'top' );
    285         $output = ob_get_clean();
    286 
    287         $this->assertStringNotContainsString( 'id="filter-by-date"', $output );
    288     }
    289 
    290     /**
    291      * @ticket 37407
    292      *
    293      * @covers WP_Posts_List_Table::extra_tablenav
    294      */
    295     function test_category_dropdown_should_not_be_shown_if_there_are_no_posts() {
    296         // Set post type to a non-existent one.
    297         $this->table->screen->post_type = 'foo';
    298 
    299         ob_start();
    300         $this->table->extra_tablenav( 'top' );
    301         $output = ob_get_clean();
    302 
    303         $this->assertStringNotContainsString( 'id="cat"', $output );
    304     }
    305 
    306     /**
    307      * @ticket 38341
    308      *
    309      * @covers WP_Posts_List_Table::extra_tablenav
    310      */
    311     public function test_empty_trash_button_should_not_be_shown_if_there_are_no_posts() {
    312         // Set post type to a non-existent one.
    313         $this->table->screen->post_type = 'foo';
    314 
    315         ob_start();
    316         $this->table->extra_tablenav( 'top' );
    317         $output = ob_get_clean();
    318 
    319         $this->assertStringNotContainsString( 'id="delete_all"', $output );
     15        $this->table = _get_list_table( 'WP_Comments_List_Table', array( 'screen' => 'edit-comments' ) );
    32016    }
    32117
    … …  
    32622     */
    32723    public function test_filter_button_should_not_be_shown_if_there_are_no_comments() {
    328         $table = _get_list_table( 'WP_Comments_List_Table', array( 'screen' => 'edit-comments' ) );
    329 
    33024        ob_start();
    331         $table->extra_tablenav( 'top' );
     25        $this->table->extra_tablenav( 'top' );
    33226        $output = ob_get_clean();
    33327
    … …  
    34943        );
    35044
    351         $table = _get_list_table( 'WP_Comments_List_Table', array( 'screen' => 'edit-comments' ) );
    352         $table->prepare_items();
     45        $this->table->prepare_items();
    35346
    35447        ob_start();
    355         $table->extra_tablenav( 'top' );
     48        $this->table->extra_tablenav( 'top' );
    35649        $output = ob_get_clean();
    35750
    … …  
    37366        );
    37467
    375         $table = _get_list_table( 'WP_Comments_List_Table', array( 'screen' => 'edit-comments' ) );
    376         $table->prepare_items();
     68        $this->table->prepare_items();
    37769
    37870        ob_start();
    379         $table->extra_tablenav( 'top' );
     71        $this->table->extra_tablenav( 'top' );
    38072        $output = ob_get_clean();
    38173
    … …  
    39082     */
    39183    public function test_empty_trash_button_should_not_be_shown_if_there_are_no_comments() {
    392         $table = _get_list_table( 'WP_Comments_List_Table', array( 'screen' => 'edit-comments' ) );
    393 
    39484        ob_start();
    395         $table->extra_tablenav( 'top' );
     85        $this->table->extra_tablenav( 'top' );
    39686        $output = ob_get_clean();
    39787
    … …  
    40595     */
    40696    public function test_bulk_action_menu_supports_options_and_optgroups() {
    407         $table = _get_list_table( 'WP_Comments_List_Table', array( 'screen' => 'edit-comments' ) );
    408 
    40997        add_filter(
    41098            'bulk_actions-edit-comments',
    … …  
    421109
    422110        ob_start();
    423         $table->bulk_actions();
     111        $this->table->bulk_actions();
    424112        $output = ob_get_clean();
    425113
    … …  
    442130     */
    443131    public function test_sortable_columns() {
    444         require_once ABSPATH . 'wp-admin/includes/class-wp-comments-list-table.php';
    445 
    446132        $override_sortable_columns = array(
    447133            'author'   => array( 'comment_author', true ),
    … …  
    478164     */
    479165    public function test_sortable_columns_with_current_ordering() {
    480         require_once ABSPATH . 'wp-admin/includes/class-wp-comments-list-table.php';
    481 
    482166        $override_sortable_columns = array(
    483167            'author'   => array( 'comment_author', false ),
Note: See TracChangeset for help on using the changeset viewer.