Make WordPress Core


Ignore:
Timestamp:
11/04/2021 03:22:47 PM (3 years ago)
Author:
hellofromTonya
Message:

Coding Standards: Add visibility to methods in tests/phpunit/tests/.

Adds a public visibility to test fixtures, tests, data providers, and callbacks methods.

Adds a private visibility to helper methods within test classes.

Renames callbacks and helpers that previously started with a _ prefix. Why? For consistency and to leverage using the method visibility. Further naming standardizations is beyond the scope of this commit.

Props costdev, jrf, hellofromTonya.
Fixes #54177.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/canonical/noRewrite.php

    r51568 r52010  
    2525     * @dataProvider data
    2626     */
    27     function test( $test_url, $expected, $ticket = 0, $expected_doing_it_wrong = array() ) {
     27    public function test( $test_url, $expected, $ticket = 0, $expected_doing_it_wrong = array() ) {
    2828        $this->assertCanonical( $test_url, $expected, $ticket, $expected_doing_it_wrong );
    2929    }
    3030
    31     function data() {
     31    public function data() {
    3232        /*
    3333         * Test URL.
Note: See TracChangeset for help on using the changeset viewer.