WordPress.org
Get WordPress

Make WordPress Core

Ticket #53363: 53363-GH-1519-move-incorrectly-placed-test.patch

File 53363-GH-1519-move-incorrectly-placed-test.patch, 13.8 KB (added by jrf, 3 years ago)

The three commits of GH PR 1519 as one patch ready for commit

  • tests/phpunit/includes/bootstrap.php 

    From 139dc0d44c2735edd442322163cc4be1dcc7448c Mon Sep 17 00:00:00 2001
From: jrfnl <jrfnl@users.noreply.github.com>
Date: Tue, 20 Jul 2021 16:03:16 +0200
Subject: [PATCH] QA: move incorrectly placed test file

The `Block_Supported_Styles_Test` class is not a `TestCase` to be extended, but an actual concrete test class.

Note: the tests as-is are failing. This should be fixed before this can be merged.

Includes minor CS fixes and adding the `$message` parameter to assertions to aid in debugging.

* Block_Supported_Styles_Test: update the style related test to current expected format
* Block_Supported_Styles_Test: remove two outdated tests
    The functionality which these tests were testing is no longer available in this manner, so these tests are redundant.
---
 tests/phpunit/includes/bootstrap.php          |   1 -
 .../blocks/block-supported-styles.php}        | 176 +++++++-----------
 2 files changed, 67 insertions(+), 110 deletions(-)
 rename tests/phpunit/{includes/testcase-block-supports.php => tests/blocks/block-supported-styles.php} (86%)

diff --git a/tests/phpunit/includes/bootstrap.php b/tests/phpunit/includes/bootstrap.php
index ce50a89534..367d445790 100644
    a b require __DIR__ . '/testcase-rest-controller.php'; 
    203203require __DIR__ . '/testcase-rest-post-type-controller.php';
    204204require __DIR__ . '/testcase-xmlrpc.php';
    205205require __DIR__ . '/testcase-ajax.php';
    206 require __DIR__ . '/testcase-block-supports.php';
    207206require __DIR__ . '/testcase-canonical.php';
    208207require __DIR__ . '/testcase-xml.php';
    209208require __DIR__ . '/exceptions.php';

  • .php 

    diff --git a/tests/phpunit/includes/testcase-block-supports.php b/tests/phpunit/tests/blocks/block-supported-styles.php
similarity index 86%
rename from tests/phpunit/includes/testcase-block-supports.php
rename to tests/phpunit/tests/blocks/block-supported-styles.php
index 981bfcd190..faf26e59c4 100644
    old new  
    11<?php
     2/**
     3 * Block Tests
     4 *
     5 * @package WordPress
     6 * @subpackage Blocks
     7 * @since 5.6.0
     8 */
     9
    210/**
    311 * Test block supported styles.
    412 *
    5  * @package    WordPress
    6  * @subpackage UnitTests
    7  * @since      5.6.0
     13 * @since 5.6.0
     14 *
     15 * @group blocks
    816 */
    917class Block_Supported_Styles_Test extends WP_UnitTestCase {
    1018
     19        /**
     20         * Block content to test with (i.e. what's wrapped by the block wrapper `<div />`).
     21         *
     22         * @var string
     23         */
     24        const BLOCK_CONTENT = '
     25                <p data-image-description="&lt;p&gt;Test!&lt;/p&gt;">Test</p>
     26                <p>äöü</p>
     27                <p>ß</p>
     28                <p>系の家庭に</p>
     29                <p>Example &lt;p&gt;Test!&lt;/p&gt;</p>
     30        ';
     31
    1132        /**
    1233         * Registered block names.
    1334         *
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    7293                return substr( $split_arr, 0, $end_index ); // String between first '>' and last '<'.
    7394        }
    7495
    75         /**
    76          * Block content to test with (i.e. what's wrapped by the block wrapper `<div />`).
    77          *
    78          * @var string
    79          */
    80         const BLOCK_CONTENT = '
    81                 <p data-image-description="&lt;p&gt;Test!&lt;/p&gt;">Test</p>
    82                 <p>äöü</p>
    83                 <p>ß</p>
    84                 <p>系の家庭に</p>
    85                 <p>Example &lt;p&gt;Test!&lt;/p&gt;</p>
    86         ';
    87 
    8896        /**
    8997         * Returns the rendered output for the current block.
    9098         *
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    116124                $class_list   = $this->get_attribute_from_block( 'class', $styled_block );
    117125                $style_list   = $this->get_attribute_from_block( 'style', $styled_block );
    118126
    119                 $this->assertSame( $expected_classes, $class_list );
    120                 $this->assertSame( $expected_styles, $style_list );
     127                $this->assertSame( $expected_classes, $class_list, 'Class list does not match expected classes' );
     128                $this->assertSame( $expected_styles, $style_list, 'Style list does not match expected styles' );
    121129        }
    122130
    123131        /**
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    137145                $class_list = $this->get_attribute_from_block( 'class', $styled_block );
    138146                $style_list = $this->get_attribute_from_block( 'style', $styled_block );
    139147
    140                 $this->assertSame( self::BLOCK_CONTENT, $content );
     148                $this->assertSame( self::BLOCK_CONTENT, $content, 'Block content does not match expected content' );
    141149                $this->assertSameSets(
    142150                        explode( ' ', $expected_classes ),
    143                         explode( ' ', $class_list )
     151                        explode( ' ', $class_list ),
     152                        'Class list does not match expected classes'
    144153                );
    145154                $this->assertSame(
    146155                        array_map( 'trim', explode( ';', $expected_styles ) ),
    147                         array_map( 'trim', explode( ';', $style_list ) )
     156                        array_map( 'trim', explode( ';', $style_list ) ),
     157                        'Style list does not match expected styles'
    148158                );
    149159        }
    150160
    151161        /**
    152162         * Tests color support for named color support for named colors.
    153163         */
    154         function test_named_color_support() {
     164        public function test_named_color_support() {
    155165                $block_type_settings = array(
    156166                        'attributes'      => array(),
    157167                        'supports'        => array(
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    183193        /**
    184194         * Tests color support for custom colors.
    185195         */
    186         function test_custom_color_support() {
     196        public function test_custom_color_support() {
    187197                $block_type_settings = array(
    188198                        'attributes'      => array(),
    189199                        'supports'        => array(
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    217227                $this->assert_content_and_styles_and_classes_match( $block, $expected_classes, $expected_styles );
    218228        }
    219229
    220         /**
    221          * Tests link color support for named colors.
    222          */
    223         function test_named_link_color_support() {
    224                 $block_type_settings = array(
    225                         'attributes'      => array(),
    226                         'supports'        => array(
    227                                 'color' => array(
    228                                         'link' => true,
    229                                 ),
    230                         ),
    231                         'render_callback' => true,
    232                 );
    233                 $this->register_block_type( 'core/example', $block_type_settings );
    234 
    235                 $block = array(
    236                         'blockName'    => 'core/example',
    237                         'attrs'        => array(
    238                                 'style' => array( 'color' => array( 'link' => 'var:preset|color|red' ) ),
    239                         ),
    240                         'innerBlock'   => array(),
    241                         'innerContent' => array(),
    242                         'innerHTML'    => array(),
    243                 );
    244 
    245                 $expected_classes = 'foo-bar-class wp-block-example has-link-color';
    246                 $expected_styles  = 'test: style; --wp--style--color--link: var(--wp--preset--color--red);';
    247 
    248                 $this->assert_content_and_styles_and_classes_match( $block, $expected_classes, $expected_styles );
    249         }
    250 
    251         /**
    252          * Tests link color support for custom colors.
    253          */
    254         function test_custom_link_color_support() {
    255                 $block_type_settings = array(
    256                         'attributes'      => array(),
    257                         'supports'        => array(
    258                                 'color' => array(
    259                                         'link' => true,
    260                                 ),
    261                         ),
    262                         'render_callback' => true,
    263                 );
    264                 $this->register_block_type( 'core/example', $block_type_settings );
    265 
    266                 $block = array(
    267                         'blockName'    => 'core/example',
    268                         'attrs'        => array(
    269                                 'style' => array( 'color' => array( 'link' => '#fff' ) ),
    270                         ),
    271                         'innerBlock'   => array(),
    272                         'innerContent' => array(),
    273                         'innerHTML'    => array(),
    274                 );
    275 
    276                 $expected_classes = 'foo-bar-class wp-block-example has-link-color';
    277                 $expected_styles  = 'test: style; --wp--style--color--link: #fff;';
    278 
    279                 $this->assert_content_and_styles_and_classes_match( $block, $expected_classes, $expected_styles );
    280         }
    281 
    282230        /**
    283231         * Tests gradient color support for named gradients.
    284232         */
    285         function test_named_gradient_support() {
     233        public function test_named_gradient_support() {
    286234                $block_type_settings = array(
    287235                        'attributes'      => array(),
    288236                        'supports'        => array(
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    313261        /**
    314262         * Tests gradient color support for custom gradients.
    315263         */
    316         function test_custom_gradient_support() {
     264        public function test_custom_gradient_support() {
    317265                $block_type_settings = array(
    318266                        'attributes'      => array(),
    319267                        'supports'        => array(
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    344292        /**
    345293         * Tests that style attributes for colors are not applied without the support flag.
    346294         */
    347         function test_color_unsupported() {
     295        public function test_color_unsupported() {
    348296                $block_type_settings = array(
    349297                        'attributes'      => array(),
    350298                        'supports'        => array(),
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    380328        /**
    381329         * Tests support for named font sizes.
    382330         */
    383         function test_named_font_size() {
     331        public function test_named_font_size() {
    384332                $block_type_settings = array(
    385333                        'attributes' => array(),
    386334                        'supports'   => array(
    387                                 'fontSize' => true,
     335                                'typography' => array(
     336                                        'fontSize' => true,
     337                                ),
    388338                        ),
    389339                );
    390340                $this->register_block_type( 'core/example', $block_type_settings );
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    408358        /**
    409359         * Tests support for custom font sizes.
    410360         */
    411         function test_custom_font_size() {
     361        public function test_custom_font_size() {
    412362                $block_type_settings = array(
    413363                        'attributes' => array(),
    414364                        'supports'   => array(
    415                                 'fontSize' => true,
     365                                'typography' => array(
     366                                        'fontSize' => true,
     367                                ),
    416368                        ),
    417369                );
    418370                $this->register_block_type( 'core/example', $block_type_settings );
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    420372                $block = array(
    421373                        'blockName'    => 'core/example',
    422374                        'attrs'        => array(
    423                                 'style' => array( 'typography' => array( 'fontSize' => '10' ) ),
     375                                'style' => array( 'typography' => array( 'fontSize' => '10px' ) ),
    424376                        ),
    425377                        'innerBlock'   => array(),
    426378                        'innerContent' => array(),
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    436388        /**
    437389         * Tests that font size attributes are not applied without support flag.
    438390         */
    439         function test_font_size_unsupported() {
     391        public function test_font_size_unsupported() {
    440392                $block_type_settings = array(
    441393                        'attributes' => array(),
    442394                        'supports'   => array(),
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    463415        /**
    464416         * Tests line height support.
    465417         */
    466         function test_line_height() {
     418        public function test_line_height() {
    467419                $block_type_settings = array(
    468420                        'attributes' => array(),
    469421                        'supports'   => array(
    470                                 'lineHeight' => true,
     422                                'typography' => array(
     423                                        'lineHeight' => true,
     424                                ),
    471425                        ),
    472426                );
    473427                $this->register_block_type( 'core/example', $block_type_settings );
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    491445        /**
    492446         * Tests line height not applied without support flag.
    493447         */
    494         function test_line_height_unsupported() {
     448        public function test_line_height_unsupported() {
    495449                $block_type_settings = array(
    496450                        'attributes' => array(),
    497451                        'supports'   => array(),
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    517471        /**
    518472         * Tests support for block alignment.
    519473         */
    520         function test_block_alignment() {
     474        public function test_block_alignment() {
    521475                $block_type_settings = array(
    522476                        'attributes' => array(),
    523477                        'supports'   => array(
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    545499        /**
    546500         * Tests block alignment requires support to be added.
    547501         */
    548         function test_block_alignment_unsupported() {
     502        public function test_block_alignment_unsupported() {
    549503                $block_type_settings = array(
    550504                        'attributes' => array(),
    551505                        'supports'   => array(),
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    571525        /**
    572526         * Tests all support flags together to ensure they work together as expected.
    573527         */
    574         function test_all_supported() {
     528        public function test_all_supported() {
    575529                $block_type_settings = array(
    576530                        'attributes' => array(),
    577531                        'supports'   => array(
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    579533                                        'gradients' => true,
    580534                                        'link'      => true,
    581535                                ),
    582                                 'fontSize'   => true,
    583                                 'lineHeight' => true,
     536                                'typography' => array(
     537                                        'fontSize'   => true,
     538                                        'lineHeight' => true,
     539                                ),
    584540                                'align'      => true,
    585541                        ),
    586542                );
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    598554                                        ),
    599555                                        'typography' => array(
    600556                                                'lineHeight' => '20',
    601                                                 'fontSize'   => '10',
     557                                                'fontSize'   => '10px',
    602558                                        ),
    603559                                ),
    604560                        ),
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    617573         * Tests that only styles for the supported flag are added.
    618574         * Verify one support enabled does not imply multiple supports enabled.
    619575         */
    620         function test_one_supported() {
     576        public function test_one_supported() {
    621577                $block_type_settings = array(
    622578                        'attributes' => array(),
    623579                        'supports'   => array(
    624                                 'fontSize' => true,
     580                                'typography' => array(
     581                                        'fontSize' => true,
     582                                ),
    625583                        ),
    626584                );
    627585                $this->register_block_type( 'core/example', $block_type_settings );
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    639597                                        ),
    640598                                        'typography' => array(
    641599                                                'lineHeight' => '20',
    642                                                 'fontSize'   => '10',
     600                                                'fontSize'   => '10px',
    643601                                        ),
    644602                                ),
    645603                        ),
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    657615        /**
    658616         * Tests custom classname server-side block support.
    659617         */
    660         function test_custom_classnames_support() {
     618        public function test_custom_classnames_support() {
    661619                $block_type_settings = array(
    662620                        'attributes' => array(),
    663621                        'supports'   => array(),
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    683641        /**
    684642         * Tests custom classname server-side block support opt-out.
    685643         */
    686         function test_custom_classnames_support_opt_out() {
     644        public function test_custom_classnames_support_opt_out() {
    687645                $block_type_settings = array(
    688646                        'attributes' => array(),
    689647                        'supports'   => array(
    … … class Block_Supported_Styles_Test extends WP_UnitTestCase { 
    711669        /**
    712670         * Tests generated classname server-side block support opt-out.
    713671         */
    714         function test_generatted_classnames_support_opt_out() {
     672        public function test_generated_classnames_support_opt_out() {
    715673                $block_type_settings = array(
    716674                        'attributes' => array(),
    717675                        'supports'   => array(