Make WordPress Core


Ignore:
Timestamp:
08/07/2021 10:29:41 AM (3 years ago)
Author:
SergeyBiryukov
Message:

Build/Test Tools: Implement use of the void solution.

PHPUnit 8.0.0 introduced a void return type declaration to the "fixture" methods – setUpBeforeClass(), setUp(), tearDown() and tearDownAfterClass(). As the void return type was not introduced until PHP 7.1, this makes it more difficult to create cross-version compatible tests when using fixtures, due to signature mismatches.

The Yoast\PHPUnitPolyfills\TestCases\TestCase overcomes the signature mismatch by having two versions. The correct one will be loaded depending on the PHPUnit version being used.

When using this TestCase, if an individual test, or another TestCase which extends this TestCase, needs to overload any of the "fixture" methods, it should do so by using a snake_case variant of the original fixture method name, i.e. set_up_before_class(), set_up(), assert_pre_conditions(), assert_post_conditions(), tear_down(), and tear_down_after_class().

The snake_case methods will automatically be called by PHPUnit.

IMPORTANT: The snake_case methods should not call the PHPUnit parent, i.e. do not use parent::setUp() from within an overloaded set_up() method. If necessary, DO call parent::set_up().

Reference: https://github.com/Yoast/PHPUnit-Polyfills#testcases

This commit renames all declared fixture methods, and calls to parent versions of those fixture methods, from camelCase to snake_case.

Follow-up to [51559-51567].

Props jrf, hellofromTonya, johnbillion, netweb, dd32, pputzer, SergeyBiryukov.
See #46149.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/ajax/PrivacyExportPersonalData.php

    r49603 r51568  
    141141     * @since 5.2.0
    142142     */
    143     public function setUp() {
    144         parent::setUp();
     143    public function set_up() {
     144        parent::set_up();
    145145
    146146        $this->key_to_unset       = '';
     
    164164     * Clean up after each test method.
    165165     */
    166     public function tearDown() {
     166    public function tear_down() {
    167167        remove_filter( 'wp_privacy_personal_data_exporters', array( $this, 'filter_register_custom_personal_data_exporter' ) );
    168168
     
    170170            revoke_super_admin( get_current_user_id() );
    171171        }
    172         parent::tearDown();
     172        parent::tear_down();
    173173    }
    174174
Note: See TracChangeset for help on using the changeset viewer.