WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/04/2021 03:22:47 PM (5 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/db/charset.php

    r51568 r52010  
    3636         * @ticket 21212
    3737         */
    38         function data_strip_invalid_text() {
     38        public function data_strip_invalid_text() {
    3939                $fields = array(
    4040                        'latin1'                                => array(
    … …  
    480480         * @covers wpdb::strip_invalid_text
    481481         */
    482         function test_strip_invalid_text( $data, $expected, $message ) {
     482        public function test_strip_invalid_text( $data, $expected, $message ) {
    483483                $charset = self::$_wpdb->charset;
    484484                if ( isset( $data[0]['connection_charset'] ) ) {
    … …  
    513513         * @covers wpdb::process_fields
    514514         */
    515         function test_process_fields_failure() {
     515        public function test_process_fields_failure() {
    516516                global $wpdb;
    517517
    … …  
    529529         * @ticket 21212
    530530         */
    531         function data_process_field_charsets() {
     531        public function data_process_field_charsets() {
    532532                if ( $GLOBALS['wpdb']->charset ) {
    533533                        $charset = $GLOBALS['wpdb']->charset;
    … …  
    600600         * @covers wpdb::process_field_charsets
    601601         */
    602         function test_process_field_charsets( $data, $expected, $message ) {
     602        public function test_process_field_charsets( $data, $expected, $message ) {
    603603                $actual = self::$_wpdb->process_field_charsets( $data, $GLOBALS['wpdb']->posts );
    604604                $this->assertSame( $expected, $actual, $message );
    … …  
    612612         * @depends test_process_field_charsets
    613613         */
    614         function test_process_field_charsets_on_nonexistent_table() {
     614        public function test_process_field_charsets_on_nonexistent_table() {
    615615                $data = array(
    616616                        'post_content' => array(
    … …  
    629629         * @covers wpdb::check_ascii
    630630         */
    631         function test_check_ascii() {
     631        public function test_check_ascii() {
    632632                $ascii = "\0\t\n\r '" . '!"#$%&()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\]^_`abcdefghijklmnopqrstuvwxyz{|}~';
    633633                $this->assertTrue( self::$_wpdb->check_ascii( $ascii ) );
    … …  
    639639         * @covers wpdb::check_ascii
    640640         */
    641         function test_check_ascii_false() {
     641        public function test_check_ascii_false() {
    642642                $this->assertFalse( self::$_wpdb->check_ascii( 'ABCDEFGHIJKLMNOPQRSTUVWXYZ¡©«' ) );
    643643        }
    … …  
    648648         * @covers wpdb::strip_invalid_text_for_column
    649649         */
    650         function test_strip_invalid_text_for_column() {
     650        public function test_strip_invalid_text_for_column() {
    651651                global $wpdb;
    652652
    … …  
    738738         * @ticket 21212
    739739         */
    740         function data_test_get_table_charset() {
     740        public function data_test_get_table_charset() {
    741741                $table_name = 'test_get_table_charset';
    742742
    … …  
    758758         * @covers wpdb::get_table_charset
    759759         */
    760         function test_get_table_charset( $drop, $create, $table, $expected_charset ) {
     760        public function test_get_table_charset( $drop, $create, $table, $expected_charset ) {
    761761                self::$_wpdb->query( $drop );
    762762
    … …  
    779779         * @ticket 21212
    780780         */
    781         function data_test_get_column_charset() {
     781        public function data_test_get_column_charset() {
    782782                $table_name = 'test_get_column_charset';
    783783
    … …  
    799799         * @covers wpdb::get_col_charset
    800800         */
    801         function test_get_column_charset( $drop, $create, $table, $expected_charset ) {
     801        public function test_get_column_charset( $drop, $create, $table, $expected_charset ) {
    802802                self::$_wpdb->query( $drop );
    803803
    … …  
    822822         * @covers wpdb::get_col_charset
    823823         */
    824         function test_get_column_charset_non_mysql( $drop, $create, $table, $columns ) {
     824        public function test_get_column_charset_non_mysql( $drop, $create, $table, $columns ) {
    825825                self::$_wpdb->query( $drop );
    826826
    … …  
    849849         * @covers wpdb::get_col_charset
    850850         */
    851         function test_get_column_charset_is_mysql_undefined( $drop, $create, $table, $columns ) {
     851        public function test_get_column_charset_is_mysql_undefined( $drop, $create, $table, $columns ) {
    852852                self::$_wpdb->query( $drop );
    853853
    … …  
    873873         * @ticket 21212
    874874         */
    875         function data_strip_invalid_text_from_query() {
     875        public function data_strip_invalid_text_from_query() {
    876876                $table_name = 'strip_invalid_text_from_query_table';
    877877                $data       = array(
    … …  
    909909         * @covers wpdb::strip_invalid_text_from_query
    910910         */
    911         function test_strip_invalid_text_from_query( $create, $query, $expected, $drop ) {
     911        public function test_strip_invalid_text_from_query( $create, $query, $expected, $drop ) {
    912912                self::$_wpdb->query( $drop );
    913913
    … …  
    927927         * @ticket 32104
    928928         */
    929         function data_dont_strip_text_from_schema_queries() {
     929        public function data_dont_strip_text_from_schema_queries() {
    930930                // An obviously invalid and fake table name.
    931931                $table_name = "\xff\xff\xff\xff";
    … …  
    953953         * @covers wpdb::strip_invalid_text_from_query
    954954         */
    955         function test_dont_strip_text_from_schema_queries( $query ) {
     955        public function test_dont_strip_text_from_schema_queries( $query ) {
    956956                $return = self::$_wpdb->strip_invalid_text_from_query( $query );
    957957                $this->assertSame( $query, $return );
    … …  
    963963         * @covers wpdb::query
    964964         */
    965         function test_invalid_characters_in_query() {
     965        public function test_invalid_characters_in_query() {
    966966                global $wpdb;
    967967
    … …  
    977977         * @ticket 21212
    978978         */
    979         function data_table_collation_check() {
     979        public function data_table_collation_check() {
    980980                $table_name = 'table_collation_check';
    981981                $data       = array(
    … …  
    10351035         * @covers wpdb::check_safe_collation
    10361036         */
    1037         function test_table_collation_check( $create, $expected, $query, $drop, $always_true ) {
     1037        public function test_table_collation_check( $create, $expected, $query, $drop, $always_true ) {
    10381038                self::$_wpdb->query( $drop );
    10391039
    … …  
    10541054         * @covers wpdb::strip_invalid_text_for_column
    10551055         */
    1056         function test_strip_invalid_text_for_column_bails_if_ascii_input_too_long() {
     1056        public function test_strip_invalid_text_for_column_bails_if_ascii_input_too_long() {
    10571057                global $wpdb;
    10581058
    … …  
    10711071         * @covers wpdb::strip_invalid_text_from_query
    10721072         */
    1073         function test_strip_invalid_text_from_query_cp1251_is_safe() {
     1073        public function test_strip_invalid_text_from_query_cp1251_is_safe() {
    10741074                $tablename = 'test_cp1251_query_' . rand_str( 5 );
    10751075                if ( ! self::$_wpdb->query( "CREATE TABLE $tablename ( a VARCHAR(50) ) DEFAULT CHARSET 'cp1251'" ) ) {
    … …  
    10901090         * @covers wpdb::strip_invalid_text_from_query
    10911091         */
    1092         function test_no_db_charset_defined() {
     1092        public function test_no_db_charset_defined() {
    10931093                $tablename = 'test_cp1251_query_' . rand_str( 5 );
    10941094                if ( ! self::$_wpdb->query( "CREATE TABLE $tablename ( a VARCHAR(50) ) DEFAULT CHARSET 'cp1251'" ) ) {
    … …  
    11141114         * @covers wpdb::set_charset
    11151115         */
    1116         function test_set_charset_changes_the_connection_collation() {
     1116        public function test_set_charset_changes_the_connection_collation() {
    11171117                self::$_wpdb->set_charset( self::$_wpdb->dbh, 'utf8', 'utf8_general_ci' );
    11181118                $results = self::$_wpdb->get_results( "SHOW VARIABLES WHERE Variable_name='collation_connection'" );
Note: See TracChangeset for help on using the changeset viewer.