WordPress.org
Get WordPress

Make WordPress Core

Changeset 45607


Ignore:
Timestamp:
07/08/2019 12:55:20 AM (6 years ago)
Author:
pento
Message:

Coding Standards: Fix the remaining issues in /tests.

All PHP files in /tests now conform to the PHP coding standards, or have exceptions appropriately marked.

Travis now also runs phpcs on the /tests directory, any future changes to these files must conform entirely to the WordPress PHP coding standards. 🎉

See #47632.

Location:
trunk
Files:
91 edited

Legend:

Unmodified
Added
Removed

  • trunk/.travis.yml

    r45570 r45607  
    1616    env: WP_TRAVISCI=e2e
    1717  - php: 7.2
    18     env: WP_TRAVISCI=travis:format
     18    env: WP_TRAVISCI=travis:phpcs
    1919  - php: 7.1
    2020    env: WP_TRAVISCI=travis:js
    … …  
    8484  fi
    8585- |
    86   # We only need to run composer install on the code formatting job.
    87   if [[ "$WP_TRAVISCI" == "travis:format" ]]; then
     86  # We only need to run composer install on the PHP coding standards job.
     87  if [[ "$WP_TRAVISCI" == "travis:phpcs" ]]; then
    8888    composer --version
    8989    travis_retry composer install

  • trunk/Gruntfile.js

    r45448 r45607  
    14391439    } );
    14401440
     1441    grunt.registerTask( 'lint:php', 'Runs the code linter on changed files.', function() {
     1442        var done = this.async();
     1443        var flags = this.flags;
     1444        var args = changedFiles.php;
     1445        if ( flags.travis ) {
     1446            args = [ 'tests' ];
     1447        }
     1448        args.unshift( 'lint' );
     1449        grunt.util.spawn( {
     1450            cmd: 'composer',
     1451            args: args,
     1452            opts: { stdio: 'inherit' }
     1453        }, function( error ) {
     1454            if ( flags.error && error ) {
     1455                done( false );
     1456            } else {
     1457                done( true );
     1458            }
     1459        } );
     1460    } );
     1461
    14411462    // Travis CI tasks.
    14421463    grunt.registerTask('travis:js', 'Runs Javascript Travis CI tasks.', [ 'jshint:corejs', 'qunit:compiled' ]);
    14431464    grunt.registerTask('travis:phpunit', 'Runs PHPUnit Travis CI tasks.', [ 'build', 'phpunit' ]);
    1444     grunt.registerTask('travis:format', 'Runs Code formatting Travis CI tasks.', [ 'format:php:error' ]);
     1465    grunt.registerTask('travis:phpcs', 'Runs PHP Coding Standards Travis CI tasks.', [ 'format:php:error', 'lint:php:travis:error' ]);
    14451466
    14461467    // Patch task.

  • trunk/phpcs.xml.dist

    r45604 r45607  
    186186    <rule ref="WordPress.Files.FileName">
    187187        <properties>
    188             <property name="custom_test_class_whitelist" type="array" value="WP_UnitTestCase,WP_Ajax_UnitTestCase,WP_Canonical_UnitTestCase,WP_Test_REST_TestCase,WP_Test_REST_Controller_Testcase,WP_Test_REST_Post_Type_Controller_Testcase,WP_XMLRPC_UnitTestCase"/>
     188            <property name="custom_test_class_whitelist" type="array">
     189                <!-- Test case parent classes -->
     190                <element value="WP_UnitTestCase"/>
     191                <element value="WP_Ajax_UnitTestCase"/>
     192                <element value="WP_Canonical_UnitTestCase"/>
     193                <element value="WP_Test_REST_TestCase"/>
     194                <element value="WP_Test_REST_Controller_Testcase"/>
     195                <element value="WP_Test_REST_Post_Type_Controller_Testcase"/>
     196                <element value="WP_XMLRPC_UnitTestCase"/>
     197                <element value="WP_Filesystem_UnitTestCase"/>
     198                <element value="WP_Image_UnitTestCase"/>
     199                <element value="WP_HTTP_UnitTestCase"/>
     200                <element value="WP_Tests_Image_Resize_UnitTestCase"/>
     201                <element value="WP_Import_UnitTestCase"/>
     202                <element value="Tests_Query_Conditionals"/>
     203
     204                <!-- Mock classes -->
     205                <element value="Spy_REST_Server"/>
     206                <element value="WP_REST_Test_Controller"/>
     207                <element value="WP_Image_Editor_Mock"/>
     208                <element value="WP_Filesystem_MockFS"/>
     209                <element value="MockPHPMailer"/>
     210                <element value="MockAction"/>
     211                <element value="WP_Object_Cache"/>
     212
     213                <!-- PHPUnit helpers -->
     214                <element value="TracTickets"/>
     215                <element value="WP_PHPUnit_Util_Getopt"/>
     216                <element value="PHPUnit_Util_Test"/>
     217                <element value="WPProfiler"/>
     218                <element value="SpeedTrapListener"/>
     219                <element value="PHPUnit_Framework_Exception"/>
     220            </property>
    189221        </properties>
    190222    </rule>

  • trunk/phpunit.xml.dist

    r44705 r45607  
    4444            <arguments>
    4545                <array>
    46                     <element key="slowThreshold">
     46                    <element key="slow_threshold">
    4747                        <integer>150</integer>
    4848                    </element>

  • trunk/tests/phpunit/includes/bootstrap.php

    r45588 r45607  
    7777
    7878// Should we run in multisite mode?
    79 $multisite = '1' == getenv( 'WP_MULTISITE' );
     79$multisite = ( '1' === getenv( 'WP_MULTISITE' ) );
    8080$multisite = $multisite || ( defined( 'WP_TESTS_MULTISITE' ) && WP_TESTS_MULTISITE );
    8181$multisite = $multisite || ( defined( 'MULTISITE' ) && MULTISITE );
    … …  
    192192
    193193                    foreach ( $skipped_groups as $group_name => $skipped ) {
    194                         if ( in_array( $group_name, $groups ) ) {
     194                        if ( in_array( $group_name, $groups, true ) ) {
    195195                            $skipped_groups[ $group_name ] = false;
    196196                        }

  • trunk/tests/phpunit/includes/functions.php

    r45580 r45607  
    101101        $wpdb->termmeta,
    102102    ) as $table ) {
     103        //phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    103104        $wpdb->query( "DELETE FROM {$table}" );
    104105    }
    … …  
    108109        $wpdb->term_taxonomy,
    109110    ) as $table ) {
     111        //phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    110112        $wpdb->query( "DELETE FROM {$table} WHERE term_id != 1" );
    111113    }

  • trunk/tests/phpunit/includes/install.php

    r45588 r45607  
    5454$wpdb->query( 'SET foreign_key_checks = 0' );
    5555foreach ( $wpdb->tables() as $table => $prefixed_table ) {
     56    //phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    5657    $wpdb->query( "DROP TABLE IF EXISTS $prefixed_table" );
    5758}
    5859
    5960foreach ( $wpdb->tables( 'ms_global' ) as $table => $prefixed_table ) {
     61    //phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    6062    $wpdb->query( "DROP TABLE IF EXISTS $prefixed_table" );
    6163

  • trunk/tests/phpunit/includes/mock-fs.php

    r42343 r45607  
    6262        foreach ( $paths as $path ) {
    6363            // Allow for comments
    64             if ( '#' == $path[0] ) {
     64            if ( '#' === $path[0] ) {
    6565                continue;
    6666            }
    6767
    6868            // Directories
    69             if ( '/' == $path[ strlen( $path ) - 1 ] ) {
     69            if ( '/' === $path[ strlen( $path ) - 1 ] ) {
    7070                $this->mkdir( $path );
    7171            } else { // Files (with dummy content for now)
    … …  
    162162    function dirlist( $path = '.', $include_hidden = true, $recursive = false ) {
    163163
    164         if ( empty( $path ) || '.' == $path ) {
     164        if ( empty( $path ) || '.' === $path ) {
    165165            $path = $this->cwd();
    166166        }
    … …  
    178178        $ret = array();
    179179        foreach ( $this->fs_map[ $path ]->children as $entry ) {
    180             if ( '.' == $entry->name || '..' == $entry->name ) {
    181                 continue;
    182             }
    183 
    184             if ( ! $include_hidden && '.' == $entry->name ) {
    185                 continue;
    186             }
    187 
    188             if ( $limit_file && $entry->name != $limit_file ) {
     180            if ( '.' === $entry->name || '..' === $entry->name ) {
     181                continue;
     182            }
     183
     184            if ( ! $include_hidden && '.' === $entry->name ) {
     185                continue;
     186            }
     187
     188            if ( $limit_file && $entry->name !== $limit_file ) {
    189189                continue;
    190190            }
    … …  
    194194            $struc['type'] = $entry->type;
    195195
    196             if ( 'd' == $struc['type'] ) {
     196            if ( 'd' === $struc['type'] ) {
    197197                if ( $recursive ) {
    198198                    $struc['files'] = $this->dirlist( trailingslashit( $path ) . trailingslashit( $struc['name'] ), $include_hidden, $recursive );
    … …  
    220220
    221221    function is_file() {
    222         return $this->type == 'f';
     222        return 'f' === $this->type;
    223223    }
    224224
    225225    function is_dir() {
    226         return $this->type == 'd';
     226        return 'd' === $this->type;
    227227    }
    228228}

  • trunk/tests/phpunit/includes/object-cache.php

    r45580 r45607  
    873873     * @param   int         $expiration     The expiration time, defaults to 0.
    874874     * @param   string      $server_key     The key identifying the server to store the value on.
    875      * @param   bool        $byKey          True to store in internal cache by key; false to not store by key
     875     * @param   bool        $by_key         True to store in internal cache by key; false to not store by key
    876876     * @return  bool                        Returns TRUE on success or FALSE on failure.
    877877     */
    878     public function add( $key, $value, $group = 'default', $expiration = 0, $server_key = '', $byKey = false ) {
     878    public function add( $key, $value, $group = 'default', $expiration = 0, $server_key = '', $by_key = false ) {
    879879        /*
    880880         * Ensuring that wp_suspend_cache_addition is defined before calling, because sometimes an advanced-cache.php
    … …  
    891891
    892892        // If group is a non-Memcached group, save to runtime cache, not Memcached
    893         if ( in_array( $group, $this->no_mc_groups ) ) {
     893        if ( in_array( $group, $this->no_mc_groups, true ) ) {
    894894
    895895            // Add does not set the value if the key exists; mimic that here
    … …  
    904904
    905905        // Save to Memcached
    906         if ( $byKey ) {
     906        if ( $by_key ) {
    907907            $result = $this->m->addByKey( $server_key, $derived_key, $value, $expiration );
    908908        } else {
    … …  
    992992     * @param   string      $group          The group value appended to the $key.
    993993     * @param   string      $server_key     The key identifying the server to store the value on.
    994      * @param   bool        $byKey          True to store in internal cache by key; false to not store by key
     994     * @param   bool        $by_key         True to store in internal cache by key; false to not store by key
    995995     * @return  bool                        Returns TRUE on success or FALSE on failure.
    996996     */
    997     public function append( $key, $value, $group = 'default', $server_key = '', $byKey = false ) {
     997    public function append( $key, $value, $group = 'default', $server_key = '', $by_key = false ) {
    998998        if ( ! is_string( $value ) && ! is_int( $value ) && ! is_float( $value ) ) {
    999999            return false;
    … …  
    10031003
    10041004        // If group is a non-Memcached group, append to runtime cache value, not Memcached
    1005         if ( in_array( $group, $this->no_mc_groups ) ) {
     1005        if ( in_array( $group, $this->no_mc_groups, true ) ) {
    10061006            if ( ! isset( $this->cache[ $derived_key ] ) ) {
    10071007                return false;
    … …  
    10141014
    10151015        // Append to Memcached value
    1016         if ( $byKey ) {
     1016        if ( $by_key ) {
    10171017            $result = $this->m->appendByKey( $server_key, $derived_key, $value );
    10181018        } else {
    … …  
    10651065     * @param   int         $expiration     The expiration time, defaults to 0.
    10661066     * @param   string      $server_key     The key identifying the server to store the value on.
    1067      * @param   bool        $byKey          True to store in internal cache by key; false to not store by key
     1067     * @param   bool        $by_key         True to store in internal cache by key; false to not store by key
    10681068     * @return  bool                        Returns TRUE on success or FALSE on failure.
    10691069     */
    1070     public function cas( $cas_token, $key, $value, $group = 'default', $expiration = 0, $server_key = '', $byKey = false ) {
     1070    public function cas( $cas_token, $key, $value, $group = 'default', $expiration = 0, $server_key = '', $by_key = false ) {
    10711071        $derived_key = $this->buildKey( $key, $group );
    10721072        $expiration  = $this->sanitize_expiration( $expiration );
    … …  
    10771077         * operation is treated as a normal "add" for no_mc_groups.
    10781078         */
    1079         if ( in_array( $group, $this->no_mc_groups ) ) {
     1079        if ( in_array( $group, $this->no_mc_groups, true ) ) {
    10801080            $this->add_to_internal_cache( $derived_key, $value );
    10811081            return true;
    … …  
    10831083
    10841084        // Save to Memcached
    1085         if ( $byKey ) {
     1085        if ( $by_key ) {
    10861086            $result = $this->m->casByKey( $cas_token, $server_key, $derived_key, $value, $expiration );
    10871087        } else {
    … …  
    11311131
    11321132        // Decrement values in no_mc_groups
    1133         if ( in_array( $group, $this->no_mc_groups ) ) {
     1133        if ( in_array( $group, $this->no_mc_groups, true ) ) {
    11341134
    11351135            // Only decrement if the key already exists and value is 0 or greater (mimics memcached behavior)
    … …  
    11921192     * @param   int         $time       The amount of time the server will wait to delete the item in seconds.
    11931193     * @param   string      $server_key The key identifying the server to store the value on.
    1194      * @param   bool        $byKey      True to store in internal cache by key; false to not store by key
     1194     * @param   bool        $by_key     True to store in internal cache by key; false to not store by key
    11951195     * @return  bool                    Returns TRUE on success or FALSE on failure.
    11961196     */
    1197     public function delete( $key, $group = 'default', $time = 0, $server_key = '', $byKey = false ) {
     1197    public function delete( $key, $group = 'default', $time = 0, $server_key = '', $by_key = false ) {
    11981198        $derived_key = $this->buildKey( $key, $group );
    11991199
    12001200        // Remove from no_mc_groups array
    1201         if ( in_array( $group, $this->no_mc_groups ) ) {
     1201        if ( in_array( $group, $this->no_mc_groups, true ) ) {
    12021202            if ( isset( $this->cache[ $derived_key ] ) ) {
    12031203                unset( $this->cache[ $derived_key ] );
    … …  
    12071207        }
    12081208
    1209         if ( $byKey ) {
     1209        if ( $by_key ) {
    12101210            $result = $this->m->deleteByKey( $server_key, $derived_key, $time );
    12111211        } else {
    … …  
    13001300     * @param   null|bool       $found      Variable passed by reference to determine if the value was found or not.
    13011301     * @param   string          $server_key The key identifying the server to store the value on.
    1302      * @param   bool            $byKey      True to store in internal cache by key; false to not store by key
     1302     * @param   bool            $by_key     True to store in internal cache by key; false to not store by key
    13031303     * @param   null|callable   $cache_cb   Read-through caching callback.
    13041304     * @param   null|float      $cas_token  The variable to store the CAS token in.
    13051305     * @return  bool|mixed                  Cached object value.
    13061306     */
    1307     public function get( $key, $group = 'default', $force = false, &$found = null, $server_key = '', $byKey = false, $cache_cb = null, &$cas_token = null ) {
     1307    public function get( $key, $group = 'default', $force = false, &$found = null, $server_key = '', $by_key = false, $cache_cb = null, &$cas_token = null ) {
    13081308        $derived_key = $this->buildKey( $key, $group );
    13091309
    … …  
    13121312
    13131313        // If either $cache_db, or $cas_token is set, must hit Memcached and bypass runtime cache
    1314         if ( func_num_args() > 6 && ! in_array( $group, $this->no_mc_groups ) ) {
    1315             if ( $byKey ) {
     1314        if ( func_num_args() > 6 && ! in_array( $group, $this->no_mc_groups, true ) ) {
     1315            if ( $by_key ) {
    13161316                $value = $this->m->getByKey( $server_key, $derived_key, $cache_cb, $cas_token );
    13171317            } else {
    … …  
    13221322                $found = true;
    13231323                return is_object( $this->cache[ $derived_key ] ) ? clone $this->cache[ $derived_key ] : $this->cache[ $derived_key ];
    1324             } elseif ( in_array( $group, $this->no_mc_groups ) ) {
     1324            } elseif ( in_array( $group, $this->no_mc_groups, true ) ) {
    13251325                return false;
    13261326            } else {
    1327                 if ( $byKey ) {
     1327                if ( $by_key ) {
    13281328                    $value = $this->m->getByKey( $server_key, $derived_key );
    13291329                } else {
    … …  
    14651465
    14661466            // If order should be preserved, reorder now
    1467             if ( ! empty( $need_to_get ) && $flags === Memcached::GET_PRESERVE_ORDER ) {
     1467            if ( ! empty( $need_to_get ) && Memcached::GET_PRESERVE_ORDER === $flags ) {
    14681468                $ordered_values = array();
    14691469
    … …  
    16041604
    16051605        // Increment values in no_mc_groups
    1606         if ( in_array( $group, $this->no_mc_groups ) ) {
     1606        if ( in_array( $group, $this->no_mc_groups, true ) ) {
    16071607
    16081608            // Only increment if the key already exists and the number is currently 0 or greater (mimics memcached behavior)
    … …  
    16691669     * @param   string    $group        The group value prepended to the $key.
    16701670     * @param   string    $server_key   The key identifying the server to store the value on.
    1671      * @param   bool      $byKey        True to store in internal cache by key; false to not store by key
     1671     * @param   bool      $by_key       True to store in internal cache by key; false to not store by key
    16721672     * @return  bool                    Returns TRUE on success or FALSE on failure.
    16731673     */
    1674     public function prepend( $key, $value, $group = 'default', $server_key = '', $byKey = false ) {
     1674    public function prepend( $key, $value, $group = 'default', $server_key = '', $by_key = false ) {
    16751675        if ( ! is_string( $value ) && ! is_int( $value ) && ! is_float( $value ) ) {
    16761676            return false;
    … …  
    16801680
    16811681        // If group is a non-Memcached group, prepend to runtime cache value, not Memcached
    1682         if ( in_array( $group, $this->no_mc_groups ) ) {
     1682        if ( in_array( $group, $this->no_mc_groups, true ) ) {
    16831683            if ( ! isset( $this->cache[ $derived_key ] ) ) {
    16841684                return false;
    … …  
    16911691
    16921692        // Append to Memcached value
    1693         if ( $byKey ) {
     1693        if ( $by_key ) {
    16941694            $result = $this->m->prependByKey( $server_key, $derived_key, $value );
    16951695        } else {
    … …  
    17411741     * @param   mixed       $value          The value to store.
    17421742     * @param   string      $group          The group value appended to the $key.
    1743      * @param   bool        $byKey          True to store in internal cache by key; false to not store by key
     1743     * @param   bool        $by_key         True to store in internal cache by key; false to not store by key
    17441744     * @param   int         $expiration     The expiration time, defaults to 0.
    17451745     * @return  bool                        Returns TRUE on success or FALSE on failure.
    17461746     */
    1747     public function replace( $key, $value, $group = 'default', $expiration = 0, $server_key = '', $byKey = false ) {
     1747    public function replace( $key, $value, $group = 'default', $expiration = 0, $server_key = '', $by_key = false ) {
    17481748        $derived_key = $this->buildKey( $key, $group );
    17491749        $expiration  = $this->sanitize_expiration( $expiration );
    17501750
    17511751        // If group is a non-Memcached group, save to runtime cache, not Memcached
    1752         if ( in_array( $group, $this->no_mc_groups ) ) {
     1752        if ( in_array( $group, $this->no_mc_groups, true ) ) {
    17531753
    17541754            // Replace won't save unless the key already exists; mimic this behavior here
    … …  
    17621762
    17631763        // Save to Memcached
    1764         if ( $byKey ) {
     1764        if ( $by_key ) {
    17651765            $result = $this->m->replaceByKey( $server_key, $derived_key, $value, $expiration );
    17661766        } else {
    … …  
    18071807     * @param   int         $expiration The expiration time, defaults to 0.
    18081808     * @param   string      $server_key The key identifying the server to store the value on.
    1809      * @param   bool        $byKey      True to store in internal cache by key; false to not store by key
     1809     * @param   bool        $by_key     True to store in internal cache by key; false to not store by key
    18101810     * @return  bool                    Returns TRUE on success or FALSE on failure.
    18111811     */
    1812     public function set( $key, $value, $group = 'default', $expiration = 0, $server_key = '', $byKey = false ) {
     1812    public function set( $key, $value, $group = 'default', $expiration = 0, $server_key = '', $by_key = false ) {
    18131813        $derived_key = $this->buildKey( $key, $group );
    18141814        $expiration  = $this->sanitize_expiration( $expiration );
    18151815
    18161816        // If group is a non-Memcached group, save to runtime cache, not Memcached
    1817         if ( in_array( $group, $this->no_mc_groups ) ) {
     1817        if ( in_array( $group, $this->no_mc_groups, true ) ) {
    18181818            $this->add_to_internal_cache( $derived_key, $value );
    18191819            return true;
    … …  
    18211821
    18221822        // Save to Memcached
    1823         if ( $byKey ) {
     1823        if ( $by_key ) {
    18241824            $result = $this->m->setByKey( $server_key, $derived_key, $value, $expiration );
    18251825        } else {
    … …  
    18681868     * @param   int             $expiration     The expiration time, defaults to 0.
    18691869     * @param   string          $server_key     The key identifying the server to store the value on.
    1870      * @param   bool            $byKey          True to store in internal cache by key; false to not store by key
     1870     * @param   bool            $by_key         True to store in internal cache by key; false to not store by key
    18711871     * @return  bool                            Returns TRUE on success or FALSE on failure.
    18721872     */
    1873     public function setMulti( $items, $groups = 'default', $expiration = 0, $server_key = '', $byKey = false ) {
     1873    public function setMulti( $items, $groups = 'default', $expiration = 0, $server_key = '', $by_key = false ) {
    18741874        // Build final keys and replace $items keys with the new keys
    18751875        $derived_keys  = $this->buildKeys( array_keys( $items ), $groups );
    … …  
    18841884
    18851885            // If group is a non-Memcached group, save to runtime cache, not Memcached
    1886             if ( in_array( $key_pieces[1], $this->no_mc_groups ) ) {
     1886            if ( in_array( $key_pieces[1], $this->no_mc_groups, true ) ) {
    18871887                $this->add_to_internal_cache( $derived_key, $value );
    18881888                unset( $derived_items[ $derived_key ] );
    … …  
    18911891
    18921892        // Save to memcached
    1893         if ( $byKey ) {
     1893        if ( $by_key ) {
    18941894            $result = $this->m->setMultiByKey( $server_key, $derived_items, $expiration );
    18951895        } else {
    … …  
    19541954        }
    19551955
    1956         if ( false !== array_search( $group, $this->global_groups ) ) {
     1956        if ( false !== array_search( $group, $this->global_groups, true ) ) {
    19571957            $prefix = $this->global_prefix;
    19581958        } else {
    … …  
    19921992
    19931993        // If we have equal numbers of keys and groups, merge $keys[n] and $group[n]
    1994         if ( count( $keys ) == count( $groups ) ) {
     1994        if ( count( $keys ) === count( $groups ) ) {
    19951995            for ( $i = 0; $i < count( $keys ); $i++ ) {
    19961996                $derived_keys[] = $this->buildKey( $keys[ $i ], $groups[ $i ] );
    … …  
    20022002                if ( isset( $groups[ $i ] ) ) {
    20032003                    $derived_keys[] = $this->buildKey( $keys[ $i ], $groups[ $i ] );
    2004                 } elseif ( count( $groups ) == 1 ) {
     2004                } elseif ( count( $groups ) === 1 ) {
    20052005                    $derived_keys[] = $this->buildKey( $keys[ $i ], $groups[0] );
    20062006                } else {
    … …  
    20472047
    20482048        // Combine the values based on direction of the "pend"
    2049         if ( 'pre' == $direction ) {
     2049        if ( 'pre' === $direction ) {
    20502050            $combined = $pended . $original;
    20512051        } else {
    … …  
    20812081    public function contains_no_mc_group( $groups ) {
    20822082        if ( is_scalar( $groups ) ) {
    2083             return in_array( $groups, $this->no_mc_groups );
     2083            return in_array( $groups, $this->no_mc_groups, true );
    20842084        }
    20852085
    … …  
    20892089
    20902090        foreach ( $groups as $group ) {
    2091             if ( in_array( $group, $this->no_mc_groups ) ) {
     2091            if ( in_array( $group, $this->no_mc_groups, true ) ) {
    20922092                return true;
    20932093            }

  • trunk/tests/phpunit/includes/phpunit6/compat.php

    r45580 r45607  
    1919
    2020        // phpcs:ignore WordPress.NamingConventions.ValidFunctionName.MethodNameInvalid
    21         public static function getTickets( $className, $methodName ) {
    22             $annotations = PHPUnit\Util\Test::parseTestMethodAnnotations( $className, $methodName );
     21        public static function getTickets( $class_name, $method_name ) {
     22            $annotations = PHPUnit\Util\Test::parseTestMethodAnnotations( $class_name, $method_name );
    2323
    2424            $tickets = array();

  • trunk/tests/phpunit/includes/phpunit7/speed-trap-listener.php

    r45588 r45607  
    2323     * @var int
    2424     */
    25     protected $slowThreshold;
     25    protected $slow_threshold;
    2626
    2727    /**
    … …  
    3030     * @var int
    3131     */
    32     protected $reportLength;
     32    protected $report_length;
    3333
    3434    /**
    … …  
    167167     *
    168168     * @param int $time          Test execution time in milliseconds
    169      * @param int $slowThreshold Test execution time at which a test should be considered slow (milliseconds)
     169     * @param int $slow_threshold Test execution time at which a test should be considered slow (milliseconds)
    170170     * @return bool
    171171     */
    172     protected function isSlow( $time, $slowThreshold ) {
    173         return $time >= $slowThreshold;
     172    protected function isSlow( $time, $slow_threshold ) {
     173        return $time >= $slow_threshold;
    174174    }
    175175
    … …  
    221221     */
    222222    protected function getReportLength() {
    223         return min( count( $this->slow ), $this->reportLength );
     223        return min( count( $this->slow ), $this->report_length );
    224224    }
    225225
    … …  
    245245     */
    246246    protected function renderHeader() {
    247         echo sprintf( "\n\nYou should really fix these slow tests (>%sms)...\n", $this->slowThreshold );
     247        echo sprintf( "\n\nYou should really fix these slow tests (>%sms)...\n", $this->slow_threshold );
    248248    }
    249249
    … …  
    252252     */
    253253    protected function renderBody() {
    254         $slowTests = $this->slow;
    255 
    256         $length = $this->getReportLength( $slowTests );
     254        $slow_tests = $this->slow;
     255
     256        $length = $this->getReportLength( $slow_tests );
    257257        for ( $i = 1; $i <= $length; ++$i ) {
    258             $label = key( $slowTests );
    259             $time  = array_shift( $slowTests );
     258            $label = key( $slow_tests );
     259            $time  = array_shift( $slow_tests );
    260260
    261261            echo sprintf( " %s. %sms to run %s\n", $i, $time, $label );
    … …  
    269269        $hidden = $this->getHiddenCount( $this->slow );
    270270        if ( $hidden ) {
    271             echo sprintf( '...and there %s %s more above your threshold hidden from view', $hidden == 1 ? 'is' : 'are', $hidden );
     271            echo sprintf( '...and there %s %s more above your threshold hidden from view', 1 === $hidden ? 'is' : 'are', $hidden );
    272272        }
    273273    }
    … …  
    279279     */
    280280    protected function loadOptions( array $options ) {
    281         $this->slowThreshold = isset( $options['slowThreshold'] ) ? $options['slowThreshold'] : 500;
    282         $this->reportLength  = isset( $options['reportLength'] ) ? $options['reportLength'] : 10;
     281        $this->slow_threshold = isset( $options['slowThreshold'] ) ? $options['slowThreshold'] : 500;
     282        $this->report_length  = isset( $options['reportLength'] ) ? $options['reportLength'] : 10;
    283283    }
    284284
    … …  
    303303        $ann = $test->getAnnotations();
    304304
    305         return isset( $ann['method']['slowThreshold'][0] ) ? $ann['method']['slowThreshold'][0] : $this->slowThreshold;
     305        return isset( $ann['method']['slowThreshold'][0] ) ? $ann['method']['slowThreshold'][0] : $this->slow_threshold;
    306306    }
    307307}

  • trunk/tests/phpunit/includes/speed-trap-listener.php

    r42343 r45607  
    2323     * @var int
    2424     */
    25     protected $slowThreshold;
     25    protected $slow_threshold;
    2626
    2727    /**
    … …  
    3030     * @var int
    3131     */
    32     protected $reportLength;
     32    protected $report_length;
    3333
    3434    /**
    … …  
    167167     *
    168168     * @param int $time          Test execution time in milliseconds
    169      * @param int $slowThreshold Test execution time at which a test should be considered slow (milliseconds)
     169     * @param int $slow_threshold Test execution time at which a test should be considered slow (milliseconds)
    170170     * @return bool
    171171     */
    172     protected function isSlow( $time, $slowThreshold ) {
    173         return $time >= $slowThreshold;
     172    protected function isSlow( $time, $slow_threshold ) {
     173        return $time >= $slow_threshold;
    174174    }
    175175
    … …  
    221221     */
    222222    protected function getReportLength() {
    223         return min( count( $this->slow ), $this->reportLength );
     223        return min( count( $this->slow ), $this->report_length );
    224224    }
    225225
    … …  
    245245     */
    246246    protected function renderHeader() {
    247         echo sprintf( "\n\nYou should really fix these slow tests (>%sms)...\n", $this->slowThreshold );
     247        echo sprintf( "\n\nYou should really fix these slow tests (>%sms)...\n", $this->slow_threshold );
    248248    }
    249249
    … …  
    252252     */
    253253    protected function renderBody() {
    254         $slowTests = $this->slow;
    255 
    256         $length = $this->getReportLength( $slowTests );
     254        $slow_tests = $this->slow;
     255
     256        $length = $this->getReportLength( $slow_tests );
    257257        for ( $i = 1; $i <= $length; ++$i ) {
    258             $label = key( $slowTests );
    259             $time  = array_shift( $slowTests );
     258            $label = key( $slow_tests );
     259            $time  = array_shift( $slow_tests );
    260260
    261261            echo sprintf( " %s. %sms to run %s\n", $i, $time, $label );
    … …  
    268268    protected function renderFooter() {
    269269        if ( $hidden = $this->getHiddenCount( $this->slow ) ) {
    270             echo sprintf( '...and there %s %s more above your threshold hidden from view', $hidden == 1 ? 'is' : 'are', $hidden );
     270            echo sprintf( '...and there %s %s more above your threshold hidden from view', 1 === $hidden ? 'is' : 'are', $hidden );
    271271        }
    272272    }
    … …  
    278278     */
    279279    protected function loadOptions( array $options ) {
    280         $this->slowThreshold = isset( $options['slowThreshold'] ) ? $options['slowThreshold'] : 500;
    281         $this->reportLength  = isset( $options['reportLength'] ) ? $options['reportLength'] : 10;
     280        $this->slow_threshold = isset( $options['slowThreshold'] ) ? $options['slowThreshold'] : 500;
     281        $this->report_length  = isset( $options['reportLength'] ) ? $options['reportLength'] : 10;
    282282    }
    283283
    … …  
    302302        $ann = $test->getAnnotations();
    303303
    304         return isset( $ann['method']['slowThreshold'][0] ) ? $ann['method']['slowThreshold'][0] : $this->slowThreshold;
     304        return isset( $ann['method']['slowThreshold'][0] ) ? $ann['method']['slowThreshold'][0] : $this->slow_threshold;
    305305    }
    306306}

  • trunk/tests/phpunit/includes/trac.php

    r45580 r45607  
    4242        }
    4343
    44         return ! in_array( $ticket_id, self::$trac_ticket_cache[ $trac_url ] );
     44        return ! in_array( $ticket_id, self::$trac_ticket_cache[ $trac_url ], true );
    4545    }
    4646

  • trunk/tests/phpunit/includes/utils.php

    r45580 r45607  
    153153            $count = 0;
    154154            foreach ( $this->events as $e ) {
    155                 if ( $e['action'] == $tag ) {
     155                if ( $e['action'] === $tag ) {
    156156                    ++$count;
    157157                }
    … …  
    223223
    224224    function data_handler( $parser, $data ) {
    225         $index                             = count( $this->data ) - 1;
    226         @$this->data[ $index ]['content'] .= $data;
     225        $index = count( $this->data ) - 1;
     226
     227        if ( ! isset( $this->data[ $index ]['content'] ) ) {
     228            $this->data[ $index ]['content'] = '';
     229        }
     230        $this->data[ $index ]['content'] .= $data;
    227231    }
    228232
    … …  
    254258        #       echo "checking '{$tree[$i][name]}' == '{$a[0]}'\n";
    255259        #       var_dump($tree[$i]['name'], $a[0]);
    256         if ( $tree[ $i ]['name'] == $a[0] ) {
     260        if ( $tree[ $i ]['name'] === $a[0] ) {
    257261            #           echo "n == {$n}\n";
    258             if ( $n == 1 ) {
     262            if ( 1 === $n ) {
    259263                $out[] = $tree[ $i ];
    260264            } else {
    … …  
    349353    $tables = $wpdb->get_col( 'SHOW TABLES;' );
    350354    foreach ( $tables as $table ) {
     355        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    351356        $wpdb->query( "DROP TABLE IF EXISTS {$table}" );
    352357    }
    … …  
    439444 * Special class for exposing protected wpdb methods we need to access
    440445 */
    441 class wpdb_exposed_methods_for_testing extends wpdb {
     446class WpdbExposedMethodsForTesting extends wpdb {
    442447    public function __construct() {
    443448        global $wpdb;

  • trunk/tests/phpunit/includes/wp-profiler.php

    r44915 r45607  
    115115        if ( $this->stack ) {
    116116            global $wp_actions;
    117             if ( $tag == end( $wp_actions ) ) {
    118                 @$this->stack[ count( $this->stack ) - 1 ]['actions'][ $tag ] ++;
     117            if ( end( $wp_actions ) === $tag ) {
     118                $this->stack[ count( $this->stack ) - 1 ]['actions'][ $tag ]++;
    119119            } else {
    120                 @$this->stack[ count( $this->stack ) - 1 ]['filters'][ $tag ] ++;
     120                $this->stack[ count( $this->stack ) - 1 ]['filters'][ $tag ]++;
    121121            }
    122122        }
    … …  
    126126    public function log_action( $tag ) {
    127127        if ( $this->stack ) {
    128             @$this->stack[ count( $this->stack ) - 1 ]['actions'][ $tag ] ++;
     128            $this->stack[ count( $this->stack ) - 1 ]['actions'][ $tag ]++;
    129129        }
    130130    }
    … …  
    145145            $sql = preg_replace( '/(WHERE \w+ =) \'\[-\w]+\'/', '$1 \'xxx\'', $sql );
    146146
    147             @$out[ $sql ] ++;
     147            $out[ $sql ] ++;
    148148        }
    149149        asort( $out );
    … …  
    156156        foreach ( $queries as $q ) {
    157157            if ( empty( $q[2] ) ) {
    158                 @$out['unknown'] ++;
     158                $out['unknown']++;
    159159            } else {
    160                 @$out[ $q[2] ] ++;
     160                $out[ $q[2] ]++;
    161161            }
    162162        }

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

    r45588 r45607  
    103103
    104104        $sub_dir = trailingslashit( dirname( $plugin[1] ) ) . 'subdir';
    105         @mkdir( $sub_dir );
    106         @file_put_contents( $sub_dir . '/subfile.php', '<?php // Silence.' );
     105        mkdir( $sub_dir );
     106        file_put_contents( $sub_dir . '/subfile.php', '<?php // Silence.' );
    107107
    108108        $plugin_files = get_plugin_files( plugin_basename( $plugin[1] ) );
    … …  
    416416            }
    417417
    418             @closedir( $mu_plugins );
     418            closedir( $mu_plugins );
    419419
    420420            foreach ( $files_to_move as $file_to_move ) {
    … …  
    443443        }
    444444
    445         @closedir( $mu_plugins );
     445        closedir( $mu_plugins );
    446446
    447447        foreach ( $files_to_move as $file_to_move ) {

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

    r43641 r45607  
    2929        $max_index_length = 191;
    3030
     31        // phpcs:disable WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    3132        $wpdb->query(
    3233            "
    … …  
    6768            "
    6869        );
     70        // phpcs:enable
    6971    }
    7072
    … …  
    7981        $sitemeta = self::$sitemeta;
    8082
     83        // phpcs:disable WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    8184        $wpdb->query( "DROP TABLE IF EXISTS {$options}" );
    8285        $wpdb->query( "DROP TABLE IF EXISTS {$blogmeta}" );
    8386        $wpdb->query( "DROP TABLE IF EXISTS {$sitemeta}" );
     87        // phpcs:enable
    8488    }
    8589

  • trunk/tests/phpunit/tests/adminbar.php

    r45588 r45607  
    745745        $nodes = $wp_admin_bar->get_nodes();
    746746        foreach ( $this->get_my_sites_network_menu_items() as $id => $cap ) {
    747             if ( in_array( $cap, $network_user_caps ) ) {
     747            if ( in_array( $cap, $network_user_caps, true ) ) {
    748748                $this->assertTrue( isset( $nodes[ $id ] ), sprintf( 'Menu item %1$s must display for a user with the %2$s cap.', $id, $cap ) );
    749749            } else {

  • trunk/tests/phpunit/tests/ajax/CustomizeMenus.php

    r43571 r45607  
    5353    function test_ajax_load_available_items_cap_check( $role, $expected_results ) {
    5454
    55         if ( 'administrator' != $role ) {
     55        if ( 'administrator' !== $role ) {
    5656            // If we're not an admin, we should get a wp_die(-1).
    5757            $this->setExpectedException( 'WPAjaxDieStopException' );
    … …  
    442442    function test_ajax_search_available_items_caps_check( $role, $expected_results ) {
    443443
    444         if ( 'administrator' != $role ) {
     444        if ( 'administrator' !== $role ) {
    445445            // If we're not an admin, we should get a wp_die(-1).
    446446            $this->setExpectedException( 'WPAjaxDieStopException' );

  • trunk/tests/phpunit/tests/ajax/DeleteComment.php

    r42343 r45607  
    9595
    9696        // trash, spam, delete should make the total go down
    97         if ( in_array( $action, array( 'trash', 'spam', 'delete' ) ) ) {
     97        if ( in_array( $action, array( 'trash', 'spam', 'delete' ), true ) ) {
    9898            $total = $_POST['_total'] - 1;
    9999
    100100            // unspam, untrash should make the total go up
    101         } elseif ( in_array( $action, array( 'untrash', 'unspam' ) ) ) {
     101        } elseif ( in_array( $action, array( 'untrash', 'unspam' ), true ) ) {
    102102            $total = $_POST['_total'] + 1;
    103103        }
    … …  
    109109        // Check for either possible total
    110110        $message = sprintf( 'returned value: %1$d $total: %2$d  $recalc_total: %3$d', (int) $xml->response[0]->comment[0]->supplemental[0]->total[0], $total, $recalc_total );
    111         $this->assertTrue( in_array( (int) $xml->response[0]->comment[0]->supplemental[0]->total[0], array( $total, $recalc_total ) ), $message );
     111        $this->assertTrue( in_array( (int) $xml->response[0]->comment[0]->supplemental[0]->total[0], array( $total, $recalc_total ), true ), $message );
    112112    }
    113113
    … …  
    244244
    245245        // Force delete the comment
    246         if ( 'delete' == $action ) {
     246        if ( 'delete' === $action ) {
    247247            wp_delete_comment( $comment->comment_ID, true );
    248248        }

  • trunk/tests/phpunit/tests/ajax/DimComment.php

    r42343 r45607  
    9090        // Check the status
    9191        $current = wp_get_comment_status( $comment->comment_ID );
    92         if ( in_array( $prev_status, array( 'unapproved', 'spam' ) ) ) {
     92        if ( in_array( $prev_status, array( 'unapproved', 'spam' ), true ) ) {
    9393            $this->assertEquals( 'approved', $current );
    9494        } else {
    … …  
    104104
    105105        // Check for either possible total
    106         $this->assertTrue( in_array( (int) $xml->response[0]->comment[0]->supplemental[0]->total[0], array( $total, $recalc_total ) ) );
     106        $this->assertTrue( in_array( (int) $xml->response[0]->comment[0]->supplemental[0]->total[0], array( $total, $recalc_total ), true ) );
    107107    }
    108108

  • trunk/tests/phpunit/tests/ajax/EditComment.php

    r42343 r45607  
    9797
    9898        // Manually update the comment_post_ID, because wp_update_comment() will prevent it.
    99         $wpdb->query( "UPDATE {$wpdb->comments} SET comment_post_ID=0 WHERE comment_ID={$comment->comment_ID}" );
     99        $wpdb->update( $wpdb->comments, array( 'comment_post_ID' => 0 ), array( 'comment_ID' => $comment->comment_ID ) );
    100100        clean_comment_cache( $comment->comment_ID );
    101101

  • trunk/tests/phpunit/tests/ajax/Response.php

    r42343 r45607  
    8686        ob_end_clean();
    8787
    88         $this->assertTrue( in_array( 'Content-Type: text/xml; charset=' . get_option( 'blog_charset' ), $headers ) );
     88        $this->assertTrue( in_array( 'Content-Type: text/xml; charset=' . get_option( 'blog_charset' ), $headers, true ) );
    8989    }
    9090

  • trunk/tests/phpunit/tests/basic.php

    r45424 r45607  
    2323        list( $version ) = explode( '-', $GLOBALS['wp_version'] );
    2424        // package.json uses x.y.z, so fill cleaned $wp_version for .0 releases
    25         if ( 1 == substr_count( $version, '.' ) ) {
     25        if ( 1 === substr_count( $version, '.' ) ) {
    2626            $version .= '.0';
    2727        }

  • trunk/tests/phpunit/tests/comment.php

    r43467 r45607  
    673673        if ( isset( $GLOBALS['phpmailer']->mock_sent )
    674674            && ! empty( $GLOBALS['phpmailer']->mock_sent )
    675             && WP_TESTS_EMAIL == $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0]
     675            && WP_TESTS_EMAIL === $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0]
    676676        ) {
    677677            $email_sent_when_comment_added = true;
    … …  
    701701        if ( isset( $GLOBALS['phpmailer']->mock_sent )
    702702            && ! empty( $GLOBALS['phpmailer']->mock_sent )
    703             && 'test@test.com' == $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0]
     703            && 'test@test.com' === $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0]
    704704        ) {
    705705            $email_sent_when_comment_approved = true;
    … …  
    723723        if ( isset( $GLOBALS['phpmailer']->mock_sent ) &&
    724724            ! empty( $GLOBALS['phpmailer']->mock_sent ) &&
    725             'test@test.com' == $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] ) {
     725            'test@test.com' === $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] ) {
    726726                $email_sent_when_comment_added = true;
    727727                reset_phpmailer_instance();

  • trunk/tests/phpunit/tests/comment/walker.php

    r43571 r45607  
    5555
    5656    public function comment( $comment, $args, $depth ) {
    57         if ( 1 == $depth ) {
     57        if ( 1 === $depth ) {
    5858            $this->test_walker->assertTrue( $this->walker->has_children );
    5959            $this->test_walker->assertTrue( $args['has_children'] ); // Back compat
    60         } elseif ( 2 == $depth ) {
     60        } elseif ( 2 === $depth ) {
    6161            $this->test_walker->assertFalse( $this->walker->has_children );
    6262            $this->test_walker->assertFalse( $args['has_children'] ); // Back compat

  • trunk/tests/phpunit/tests/compat.php

    r43571 r45607  
    126126            $heredoc,
    127127            // object data
    128             new classA(),
    129             // undefined data
     128            new ClassA(),
     129            // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged -- intentionally undefined data
    130130            @$undefined_var,
    131             // unset data
     131            // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged -- intentionally unset data
    132132            @$unset_var,
    133133        );
    … …  
    316316
    317317/* used in test_mb_substr_phpcore */
    318 class classA {
     318class ClassA {
    319319    public function __toString() {
    320320        return 'Class A object';

  • trunk/tests/phpunit/tests/customize/manager.php

    r45588 r45607  
    26692669    function filter_customize_dynamic_setting_args_for_test_dynamic_settings( $setting_args, $setting_id ) {
    26702670        $this->assertInternalType( 'string', $setting_id );
    2671         if ( in_array( $setting_id, array( 'foo', 'bar' ) ) ) {
     2671        if ( in_array( $setting_id, array( 'foo', 'bar' ), true ) ) {
    26722672            $setting_args = array( 'default' => "dynamic_{$setting_id}_default" );
    26732673        }

  • trunk/tests/phpunit/tests/customize/nav-menu-item-setting.php

    r43571 r45607  
    387387        $db_ids     = wp_list_pluck( $menu_items, 'db_id' );
    388388        $this->assertContains( $item_id, $db_ids );
    389         $i                         = array_search( $item_id, $db_ids );
     389        $i                         = array_search( $item_id, $db_ids, true );
    390390        $updated_item              = $menu_items[ $i ];
    391391        $post_value['post_status'] = $post_value['status'];
    … …  
    677677        $db_ids     = wp_list_pluck( $menu_items, 'db_id' );
    678678        $this->assertContains( $item_id, $db_ids );
    679         $i                         = array_search( $item_id, $db_ids );
     679        $i                         = array_search( $item_id, $db_ids, true );
    680680        $updated_item              = $menu_items[ $i ];
    681681        $post_value['post_status'] = $post_value['status'];

  • trunk/tests/phpunit/tests/customize/nav-menu-setting.php

    r43571 r45607  
    225225        $menus     = wp_get_nav_menus();
    226226        $menus_ids = wp_list_pluck( $menus, 'term_id' );
    227         $i         = array_search( $menu_id, $menus_ids );
     227        $i         = array_search( $menu_id, $menus_ids, true );
    228228        $this->assertInternalType( 'int', $i, 'Update-previewed menu does not appear in wp_get_nav_menus()' );
    229229        $filtered_menu = $menus[ $i ];
    … …  
    270270        $menus     = wp_get_nav_menus();
    271271        $menus_ids = wp_list_pluck( $menus, 'term_id' );
    272         $i         = array_search( $menu_id, $menus_ids );
     272        $i         = array_search( $menu_id, $menus_ids, true );
    273273        $this->assertInternalType( 'int', $i, 'Insert-previewed menu was not injected into wp_get_nav_menus()' );
    274274        $filtered_menu = $menus[ $i ];

  • trunk/tests/phpunit/tests/date/query.php

    r43571 r45607  
    10151015        $days_of_year = array( -1, 0, 367 );
    10161016        foreach ( $days_of_year as $day_of_year ) {
    1017             $this->assertFalse( @$this->q->validate_date_values( array( 'dayofyear' => $day_of_year ) ) );
     1017            $this->assertFalse( $this->q->validate_date_values( array( 'dayofyear' => $day_of_year ) ) );
    10181018        }
    10191019    }

  • trunk/tests/phpunit/tests/db.php

    r45603 r45607  
    2424    public static function setUpBeforeClass() {
    2525        parent::setUpBeforeClass();
    26         self::$_wpdb = new wpdb_exposed_methods_for_testing();
     26        self::$_wpdb = new WpdbExposedMethodsForTesting();
    2727    }
    2828
    … …  
    347347
    348348    public function filter_allowed_incompatible_sql_mode( $modes ) {
    349         $pos = array_search( 'ONLY_FULL_GROUP_BY', $modes );
     349        $pos = array_search( 'ONLY_FULL_GROUP_BY', $modes, true );
    350350        $this->assertGreaterThanOrEqual( 0, $pos );
    351351
    … …  
    366366        $id = 0;
    367367        // This, obviously, is an incorrect prepare.
     368        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    368369        $prepared = $wpdb->prepare( "SELECT * FROM $wpdb->users WHERE id = $id", $id );
    369370        $this->assertEquals( "SELECT * FROM $wpdb->users WHERE id = 0", $prepared );
    … …  
    383384        global $wpdb;
    384385
     386        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    385387        $prepared = @$wpdb->prepare( "SELECT * FROM $wpdb->users WHERE id = %d AND user_login = %s", 1, array( 'admin' ) );
    386388        $this->assertEquals( "SELECT * FROM $wpdb->users WHERE id = 1 AND user_login = ''", $prepared );
    387389
     390        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    388391        $prepared = @$wpdb->prepare( "SELECT * FROM $wpdb->users WHERE id = %d AND user_login = %s", array( 1 ), 'admin' );
    389392        $this->assertEquals( "SELECT * FROM $wpdb->users WHERE id = 0 AND user_login = 'admin'", $prepared );
    … …  
    403406        global $wpdb;
    404407
     408        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    405409        $prepared = @$wpdb->prepare( "SELECT * FROM $wpdb->users WHERE id = %d AND user_login = %s", array( 1, array( 'admin' ) ) );
    406410        $this->assertEquals( "SELECT * FROM $wpdb->users WHERE id = 1 AND user_login = ''", $prepared );
    407411
     412        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    408413        $prepared = @$wpdb->prepare( "SELECT * FROM $wpdb->users WHERE id = %d AND user_login = %s", array( array( 1 ), 'admin' ) );
    409414        $this->assertEquals( "SELECT * FROM $wpdb->users WHERE id = 0 AND user_login = 'admin'", $prepared );
    … …  
    421426        array_unshift( $args, $query );
    422427
     428        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    423429        $prepared = @call_user_func_array( array( $wpdb, 'prepare' ), $args );
    424430        $this->assertEquals( $expected, $prepared );
    … …  
    588594        $wpdb->last_result = $last_result;
    589595
     596        // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
    590597        $result = $wpdb->get_col( $query, $column );
    591598
    … …  
    10441051        }
    10451052
    1046         if ( ! in_array( $expected_charset, array( 'utf8', 'utf8mb4', 'latin1' ) ) ) {
     1053        if ( ! in_array( $expected_charset, array( 'utf8', 'utf8mb4', 'latin1' ), true ) ) {
    10471054            $this->markTestSkipped( 'This test only works with utf8, utf8mb4 or latin1 character sets' );
    10481055        }
    … …  
    11171124        );
    11181125
    1119         $row = $wpdb->get_row( "SELECT * FROM $wpdb->postmeta WHERE meta_key='$key'" );
     1126        $row = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->postmeta WHERE meta_key=%s", $key ) );
    11201127
    11211128        $this->assertNull( $row->meta_value );
    … …  
    11401147        );
    11411148
    1142         $row = $wpdb->get_row( "SELECT * FROM $wpdb->postmeta WHERE meta_key='$key'" );
     1149        $row = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->postmeta WHERE meta_key=%s", $key ) );
    11431150
    11441151        $this->assertSame( $value, $row->meta_value );
    … …  
    11551162        );
    11561163
    1157         $row = $wpdb->get_row( "SELECT * FROM $wpdb->postmeta WHERE meta_key='$key'" );
     1164        $row = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->postmeta WHERE meta_key=%s", $key ) );
    11581165
    11591166        $this->assertNull( $row->meta_value );
    … …  
    11781185        );
    11791186
    1180         $row = $wpdb->get_row( "SELECT * FROM $wpdb->postmeta WHERE meta_key='$key'" );
     1187        $row = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->postmeta WHERE meta_key=%s", $key ) );
    11811188
    11821189        $this->assertNull( $row->meta_value );
    … …  
    11931200        );
    11941201
    1195         $row = $wpdb->get_row( "SELECT * FROM $wpdb->postmeta WHERE meta_key='$key'" );
     1202        $row = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->postmeta WHERE meta_key=%s", $key ) );
    11961203
    11971204        $this->assertSame( $value, $row->meta_value );
    … …  
    12161223        );
    12171224
    1218         $row = $wpdb->get_row( "SELECT * FROM $wpdb->postmeta WHERE meta_key='$key'" );
     1225        $row = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->postmeta WHERE meta_key=%s", $key ) );
    12191226
    12201227        $this->assertNull( $row->meta_value );
    … …  
    12291236        );
    12301237
    1231         $row = $wpdb->get_row( "SELECT * FROM $wpdb->postmeta WHERE meta_key='$key'" );
     1238        $row = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->postmeta WHERE meta_key=%s", $key ) );
    12321239
    12331240        $this->assertNull( $row );
    … …  
    15731580        $sql = str_replace( '{ESCAPE}', $escape, $sql );
    15741581
     1582        // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
    15751583        $actual = $wpdb->prepare( $sql, $values );
    15761584
    … …  
    16541662        $wpdb->query( "CREATE TABLE {$wpdb->prefix}test_placeholder( a VARCHAR(100) );" );
    16551663        $sql = $wpdb->prepare( "INSERT INTO {$wpdb->prefix}test_placeholder VALUES(%s)", $value );
     1664
     1665        // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
    16561666        $wpdb->query( $sql );
    16571667
    … …  
    16681678
    16691679        $sql = $wpdb->prepare( ' %s %1$c ', 'foo' );
     1680        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    16701681        $sql = $wpdb->prepare( " $sql %s ", 'foo' );
    16711682

  • trunk/tests/phpunit/tests/db/charset.php

    r45588 r45607  
    2828        require_once( dirname( dirname( __FILE__ ) ) . '/db.php' );
    2929
    30         self::$_wpdb = new wpdb_exposed_methods_for_testing();
     30        self::$_wpdb = new WpdbExposedMethodsForTesting();
    3131
    3232        if ( self::$_wpdb->use_mysqli ) {

  • trunk/tests/phpunit/tests/dbdelta.php

    r43571 r45607  
    3434        // Forcing MyISAM, because InnoDB only started supporting FULLTEXT indexes in MySQL 5.7.
    3535        $wpdb->query(
    36             "
    37             CREATE TABLE {$wpdb->prefix}dbdelta_test (
    38                 id bigint(20) NOT NULL AUTO_INCREMENT,
    39                 column_1 varchar(255) NOT NULL,
    40                 column_2 text,
    41                 column_3 blob,
    42                 PRIMARY KEY  (id),
    43                 KEY key_1 (column_1($this->max_index_length)),
    44                 KEY compound_key (id,column_1($this->max_index_length)),
    45                 FULLTEXT KEY fulltext_key (column_1)
    46             ) ENGINE=MyISAM
    47             "
     36            $wpdb->prepare(
     37                "
     38                CREATE TABLE {$wpdb->prefix}dbdelta_test (
     39                    id bigint(20) NOT NULL AUTO_INCREMENT,
     40                    column_1 varchar(255) NOT NULL,
     41                    column_2 text,
     42                    column_3 blob,
     43                    PRIMARY KEY  (id),
     44                    KEY key_1 (column_1(%d)),
     45                    KEY compound_key (id,column_1(%d)),
     46                    FULLTEXT KEY fulltext_key (column_1)
     47                ) ENGINE=MyISAM
     48                ",
     49                $this->max_index_length,
     50                $this->max_index_length
     51            )
    4852        );
    4953
    … …  
    300304        global $wpdb;
    301305
     306        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    302307        $table_row = $wpdb->get_row( "select $column from {$table} where $column = '$value'" );
    303308
    … …  
    318323        global $wpdb;
    319324
    320         $table_fields = $wpdb->get_results( "DESCRIBE {$table}" );
     325        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
     326        $table_fields = $wpdb->get_results( "DESCRIBE $table" );
    321327
    322328        $this->assertCount( 1, wp_list_filter( $table_fields, array( 'Field' => $column ) ) );
    … …  
    334340        global $wpdb;
    335341
    336         $table_indices = $wpdb->get_results( "SHOW INDEX FROM {$table}" );
     342        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
     343        $table_indices = $wpdb->get_results( "SHOW INDEX FROM $table" );
    337344
    338345        $this->assertCount(
    … …  
    359366        global $wpdb;
    360367
    361         $table_fields = $wpdb->get_results( "DESCRIBE {$table}" );
     368        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
     369        $table_fields = $wpdb->get_results( "DESCRIBE $table" );
    362370
    363371        $this->assertCount( 0, wp_list_filter( $table_fields, array( 'Field' => $column ) ) );
    … …  
    386394            ) ENGINE=InnoDB ROW_FORMAT=DYNAMIC";
    387395
     396        // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
    388397        $wpdb->query( $create );
    389398
     399        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    390400        $index = $wpdb->get_row( "SHOW INDEXES FROM $table_name WHERE Key_name='a_key';" );
    391401
    392402        $actual = dbDelta( $create, false );
    393403
     404        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    394405        $wpdb->query( "DROP TABLE IF EXISTS $table_name;" );
    395406
    396         if ( 191 != $index->Sub_part ) {
     407        if ( 191 !== $index->Sub_part ) {
    397408            $this->markTestSkipped( 'This test requires the index to be truncated.' );
    398409        }
    … …  
    528539        ";
    529540
     541        // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
    530542        $wpdb->query( $schema );
    531543
    … …  
    557569            ";
    558570
     571        // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
    559572        $wpdb->query( $schema );
    560573
    … …  
    603616        ";
    604617
     618        // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
    605619        $wpdb->query( $schema );
    606620
    … …  
    9991013        ";
    10001014
     1015        // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
    10011016        $wpdb->query( $schema );
    10021017

  • trunk/tests/phpunit/tests/external-http/basic.php

    r40519 r45607  
    1414
    1515        $response = wp_remote_get( 'https://secure.php.net/supported-versions.php' );
    16         if ( 200 != wp_remote_retrieve_response_code( $response ) ) {
     16        if ( 200 !== wp_remote_retrieve_response_code( $response ) ) {
    1717            $this->fail( 'Could not contact PHP.net to check versions.' );
    1818        }
    … …  
    2626
    2727        $response = wp_remote_get( "https://dev.mysql.com/doc/relnotes/mysql/{$matches[1]}/en/" );
    28         if ( 200 != wp_remote_retrieve_response_code( $response ) ) {
     28        if ( 200 !== wp_remote_retrieve_response_code( $response ) ) {
    2929            $this->fail( 'Could not contact dev.MySQL.com to check versions.' );
    3030        }

  • trunk/tests/phpunit/tests/feed/atom.php

    r43571 r45607  
    7272        global $post;
    7373        try {
     74            // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    7475            @require( ABSPATH . 'wp-includes/feed-atom.php' );
    7576            $out = ob_get_clean();
    … …  
    163164            $link_alts = xml_find( $entries[ $key ]['child'], 'link' );
    164165            foreach ( $link_alts as $link_alt ) {
    165                 if ( 'alternate' == $link_alt['attributes']['rel'] ) {
     166                if ( 'alternate' === $link_alt['attributes']['rel'] ) {
    166167                    $this->assertEquals( get_permalink( $post ), $link_alt['attributes']['href'] );
    167168                }
    … …  
    186187            $categories = xml_find( $entries[ $key ]['child'], 'category' );
    187188            foreach ( $categories as $category ) {
    188                 $this->assertTrue( in_array( $category['attributes']['term'], $terms ) );
     189                $this->assertTrue( in_array( $category['attributes']['term'], $terms, true ) );
    189190            }
    190191            unset( $terms );
    … …  
    199200            $link_replies = xml_find( $entries[ $key ]['child'], 'link' );
    200201            foreach ( $link_replies as $link_reply ) {
    201                 if ( 'replies' == $link_reply['attributes']['rel'] && 'application/atom+xml' == $link_reply['attributes']['type'] ) {
     202                if ( 'replies' === $link_reply['attributes']['rel'] && 'application/atom+xml' === $link_reply['attributes']['type'] ) {
    202203                    $this->assertEquals( get_post_comments_feed_link( $post->ID, 'atom' ), $link_reply['attributes']['href'] );
    203204                }

  • trunk/tests/phpunit/tests/feed/rss2.php

    r45247 r45607  
    8484        global $post;
    8585        try {
     86            // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    8687            @require( ABSPATH . 'wp-includes/feed-rss2.php' );
    8788            $out = ob_get_clean();

  • trunk/tests/phpunit/tests/file.php

    r45345 r45607  
    209209
    210210        // Check to see if the system parameters prevent signature verifications.
    211         if ( is_wp_error( $verify ) && 'signature_verification_unsupported' == $verify->get_error_code() ) {
     211        if ( is_wp_error( $verify ) && 'signature_verification_unsupported' === $verify->get_error_code() ) {
    212212            $this->markTestSkipped( 'This system does not support Signature Verification.' );
    213213        }
    … …  
    229229        unlink( $file );
    230230
    231         if ( is_wp_error( $verify ) && 'signature_verification_unsupported' == $verify->get_error_code() ) {
     231        if ( is_wp_error( $verify ) && 'signature_verification_unsupported' === $verify->get_error_code() ) {
    232232            $this->markTestSkipped( 'This system does not support Signature Verification.' );
    233233        }

  • trunk/tests/phpunit/tests/formatting/WPSpecialchars.php

    r42343 r45607  
    1818        // Allowed entities should be unchanged
    1919        foreach ( $allowedentitynames as $ent ) {
    20             if ( 'apos' == $ent ) {
     20            if ( 'apos' === $ent ) {
    2121                // But for some reason, PHP doesn't allow &apos;
    2222                continue;

  • trunk/tests/phpunit/tests/formatting/WPTexturize.php

    r42343 r45607  
    15771577                return '!openq1!';
    15781578            case '&#8217;':
    1579                 if ( 'apostrophe' == $context ) {
     1579                if ( 'apostrophe' === $context ) {
    15801580                    return '!apos!';
    15811581                } else {
    … …  
    20022002                return '!q1!';
    20032003            case '&#8217;':
    2004                 if ( 'apostrophe' == $context ) {
     2004                if ( 'apostrophe' === $context ) {
    20052005                    return '!apos!';
    20062006                } else {

  • trunk/tests/phpunit/tests/functions.php

    r45602 r45607  
    845845        $charsets     = mb_detect_order();
    846846        $old_charsets = $charsets;
    847         if ( ! in_array( 'EUC-JP', $charsets ) ) {
     847        if ( ! in_array( 'EUC-JP', $charsets, true ) ) {
    848848            $charsets[] = 'EUC-JP';
    849849            mb_detect_order( $charsets );
    … …  
    870870        $charsets     = mb_detect_order();
    871871        $old_charsets = $charsets;
    872         if ( ! in_array( 'EUC-JP', $charsets ) ) {
     872        if ( ! in_array( 'EUC-JP', $charsets, true ) ) {
    873873            $charsets[] = 'EUC-JP';
    874874            mb_detect_order( $charsets );
    … …  
    966966        $extensions = wp_get_ext_types();
    967967
    968         foreach ( $extensions as $type => $extensionList ) {
    969             foreach ( $extensionList as $extension ) {
     968        foreach ( $extensions as $type => $extension_list ) {
     969            foreach ( $extension_list as $extension ) {
    970970                $this->assertEquals( $type, wp_ext2type( $extension ) );
    971971                $this->assertEquals( $type, wp_ext2type( strtoupper( $extension ) ) );

  • trunk/tests/phpunit/tests/functions/deprecated.php

    r42343 r45607  
    136136        }
    137137        foreach ( $search as $v ) {
    138             if ( $name == $v[ $key ] ) {
     138            if ( $name === $v[ $key ] ) {
    139139                return $v;
    140140            }

  • trunk/tests/phpunit/tests/http/base.php

    r45035 r45607  
    5555        foreach ( array( 'curl', 'streams', 'fsockopen' ) as $t ) {
    5656            remove_filter( "use_{$t}_transport", '__return_false' ); // Just strip them all
    57             if ( $t != $this->transport ) {
     57            if ( $t !== $this->transport ) {
    5858                add_filter( "use_{$t}_transport", '__return_false' ); // and add it back if need be..
    5959            }
    … …  
    248248        }
    249249
    250         $this->assertTrue( isset( $headers['test1'] ) && 'test' == $headers['test1'] );
     250        $this->assertTrue( isset( $headers['test1'] ) && 'test' === $headers['test1'] );
    251251        $this->assertTrue( isset( $headers['test2'] ) && '0' === $headers['test2'] );
    252252        // cURL/HTTP Extension Note: Will never pass, cURL does not pass headers with an empty value.

  • trunk/tests/phpunit/tests/image/base.php

    r42343 r45607  
    7676        $detected_width  = 0;
    7777        $detected_height = 0;
    78         $image_size      = @getimagesize( $filename );
     78        $image_size      = getimagesize( $filename );
    7979
    8080        if ( isset( $image_size[0] ) ) {

  • trunk/tests/phpunit/tests/image/siteIcon.php

    r44785 r45607  
    5656
    5757        // Remove custom size.
    58         unset( $this->wp_site_icon->site_icon_sizes[ array_search( 321, $this->wp_site_icon->site_icon_sizes ) ] );
     58        unset( $this->wp_site_icon->site_icon_sizes[ array_search( 321, $this->wp_site_icon->site_icon_sizes, true ) ] );
    5959        // Remove the filter we added
    6060        remove_filter( 'site_icon_image_sizes', array( $this, '_custom_test_sizes' ) );
    … …  
    9696
    9797        // Remove custom size.
    98         unset( $this->wp_site_icon->site_icon_sizes[ array_search( 321, $this->wp_site_icon->site_icon_sizes ) ] );
     98        unset( $this->wp_site_icon->site_icon_sizes[ array_search( 321, $this->wp_site_icon->site_icon_sizes, true ) ] );
    9999    }
    100100

  • trunk/tests/phpunit/tests/import/import.php

    r43571 r45607  
    2929        // crude but effective: make sure there's no residual data in the main tables
    3030        foreach ( array( 'posts', 'postmeta', 'comments', 'terms', 'term_taxonomy', 'term_relationships', 'users', 'usermeta' ) as $table ) {
     31            // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    3132            $wpdb->query( "DELETE FROM {$wpdb->$table}" );
    3233        }

  • trunk/tests/phpunit/tests/kses.php

    r45588 r45607  
    149149        foreach ( $bad as $k => $x ) {
    150150            $result = wp_kses_bad_protocol( wp_kses_normalize_entities( $x ), wp_allowed_protocols() );
    151             if ( ! empty( $result ) && $result != 'alert(1);' && $result != 'alert(1)' ) {
     151            if ( ! empty( $result ) && 'alert(1);' !== $result && 'alert(1)' !== $result ) {
    152152                switch ( $k ) {
    153153                    case 6:
    … …  
    184184        foreach ( $safe as $x ) {
    185185            $result = wp_kses_bad_protocol( wp_kses_normalize_entities( $x ), array( 'http', 'https', 'dummy' ) );
    186             if ( $result != $x && $result != 'http://example.org/' ) {
     186            if ( $result !== $x && 'http://example.org/' !== $result ) {
    187187                $this->fail( "wp_kses_bad_protocol incorrectly blocked $x" );
    188188            }
    … …  
    193193        $xss = simplexml_load_file( DIR_TESTDATA . '/formatting/xssAttacks.xml' );
    194194        foreach ( $xss->attack as $attack ) {
    195             if ( in_array( $attack->name, array( 'IMG Embedded commands 2', 'US-ASCII encoding', 'OBJECT w/Flash 2', 'Character Encoding Example' ) ) ) {
     195            if ( in_array( (string) $attack->name, array( 'IMG Embedded commands 2', 'US-ASCII encoding', 'OBJECT w/Flash 2', 'Character Encoding Example' ), true ) ) {
    196196                continue;
    197197            }
    … …  
    199199            $code = (string) $attack->code;
    200200
    201             if ( $code == 'See Below' ) {
     201            if ( 'See Below' === $code ) {
    202202                continue;
    203203            }
    204204
    205             if ( substr( $code, 0, 4 ) == 'perl' ) {
     205            if ( substr( $code, 0, 4 ) === 'perl' ) {
    206206                $pos  = strpos( $code, '"' ) + 1;
    207207                $code = substr( $code, $pos, strrpos( $code, '"' ) - $pos );
    … …  
    211211            $result = trim( wp_kses_data( $code ) );
    212212
    213             if ( $result == '' || $result == 'XSS' || $result == 'alert("XSS");' || $result == "alert('XSS');" ) {
     213            if ( in_array( $result, array( '', 'XSS', 'alert("XSS");', "alert('XSS');" ), true ) ) {
    214214                continue;
    215215            }
    … …  
    325325
    326326    function _wp_kses_allowed_html_filter( $html, $context ) {
    327         if ( 'post' == $context ) {
     327        if ( 'post' === $context ) {
    328328            return array( 'a' => array( 'href' => true ) );
    329329        } else {

  • trunk/tests/phpunit/tests/l10n.php

    r45505 r45607  
    7676
    7777        $this->assertNotEmpty( $installed_translations['default']['en_GB'] );
    78         $data_en_GB = $installed_translations['default']['en_GB'];
    79         $this->assertEquals( '2016-10-26 00:01+0200', $data_en_GB['PO-Revision-Date'] );
    80         $this->assertEquals( 'Development (4.4.x)', $data_en_GB['Project-Id-Version'] );
    81         $this->assertEquals( 'Poedit 1.8.10', $data_en_GB['X-Generator'] );
     78        $data_en_gb = $installed_translations['default']['en_GB'];
     79        $this->assertEquals( '2016-10-26 00:01+0200', $data_en_gb['PO-Revision-Date'] );
     80        $this->assertEquals( 'Development (4.4.x)', $data_en_gb['Project-Id-Version'] );
     81        $this->assertEquals( 'Poedit 1.8.10', $data_en_gb['X-Generator'] );
    8282
    8383        $this->assertNotEmpty( $installed_translations['admin']['es_ES'] );
    84         $data_es_ES = $installed_translations['admin']['es_ES'];
    85         $this->assertEquals( '2016-10-25 18:29+0200', $data_es_ES['PO-Revision-Date'] );
    86         $this->assertEquals( 'Administration', $data_es_ES['Project-Id-Version'] );
    87         $this->assertEquals( 'Poedit 1.8.10', $data_es_ES['X-Generator'] );
     84        $data_es_es = $installed_translations['admin']['es_ES'];
     85        $this->assertEquals( '2016-10-25 18:29+0200', $data_es_es['PO-Revision-Date'] );
     86        $this->assertEquals( 'Administration', $data_es_es['Project-Id-Version'] );
     87        $this->assertEquals( 'Poedit 1.8.10', $data_es_es['X-Generator'] );
    8888    }
    8989

  • trunk/tests/phpunit/tests/l10n/loadTextdomainJustInTime.php

    r42343 r45607  
    176176
    177177        switch_to_locale( 'de_DE' );
    178         $expected_de_DE = i18n_plugin_test();
     178        $expected_de_de = i18n_plugin_test();
    179179
    180180        switch_to_locale( 'es_ES' );
    181         $expected_es_ES = i18n_plugin_test();
     181        $expected_es_es = i18n_plugin_test();
    182182
    183183        restore_current_locale();
    184184
    185         $this->assertSame( 'Das ist ein Dummy Plugin', $expected_de_DE );
    186         $this->assertSame( 'This is a dummy plugin', $expected_es_ES );
     185        $this->assertSame( 'Das ist ein Dummy Plugin', $expected_de_de );
     186        $this->assertSame( 'This is a dummy plugin', $expected_es_es );
    187187    }
    188188

  • trunk/tests/phpunit/tests/l10n/localeSwitcher.php

    r42343 r45607  
    8787        switch_to_locale( 'de_DE' );
    8888
    89         $wp_locale_de_DE = clone $wp_locale;
    90 
    91         // Cleanup.
    92         restore_previous_locale();
    93 
    94         $this->assertEqualSetsWithIndex( $expected, $wp_locale_de_DE->number_format );
     89        $wp_locale_de_de = clone $wp_locale;
     90
     91        // Cleanup.
     92        restore_previous_locale();
     93
     94        $this->assertEqualSetsWithIndex( $expected, $wp_locale_de_de->number_format );
    9595    }
    9696
    9797    public function test_switch_to_locale_en_US() {
    9898        switch_to_locale( 'en_GB' );
    99         $locale_en_GB = get_locale();
     99        $locale_en_gb = get_locale();
    100100        switch_to_locale( 'en_US' );
    101         $locale_en_US = get_locale();
    102 
    103         // Cleanup.
    104         restore_current_locale();
    105 
    106         $this->assertSame( 'en_GB', $locale_en_GB );
    107         $this->assertSame( 'en_US', $locale_en_US );
     101        $locale_en_us = get_locale();
     102
     103        // Cleanup.
     104        restore_current_locale();
     105
     106        $this->assertSame( 'en_GB', $locale_en_gb );
     107        $this->assertSame( 'en_US', $locale_en_us );
    108108    }
    109109

  • trunk/tests/phpunit/tests/link/themeFile.php

    r42819 r45607  
    99            self::markTestSkipped( 'symlink() is not available.' );
    1010        }
     11        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    1112        if ( ! @symlink( DIR_TESTDATA . '/theme-file-parent', WP_CONTENT_DIR . '/themes/theme-file-parent' ) ) {
    1213            self::markTestSkipped( 'Could not create parent symlink.' );
    1314        }
     15        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    1416        if ( ! @symlink( DIR_TESTDATA . '/theme-file-child', WP_CONTENT_DIR . '/themes/theme-file-child' ) ) {
    1517            self::markTestSkipped( 'Could not create child symlink.' );

  • trunk/tests/phpunit/tests/media.php

    r45424 r45607  
    12571257     */
    12581258    function filter_wp_embed_shortcode_custom( $content, $url ) {
    1259         if ( 'https://www.example.com/?video=1' == $url ) {
     1259        if ( 'https://www.example.com/?video=1' === $url ) {
    12601260            $content = '@embed URL was replaced@';
    12611261        }
    … …  
    14471447        foreach ( $image_meta['sizes'] as $name => $size ) {
    14481448            // Whitelist the sizes that should be included so we pick up 'medium_large' in 4.4.
    1449             if ( in_array( $name, $intermediates ) ) {
     1449            if ( in_array( $name, $intermediates, true ) ) {
    14501450                $expected .= $uploads_dir_url . $year_month . '/' . $size['file'] . ' ' . $size['width'] . 'w, ';
    14511451            }
    … …  
    14921492        foreach ( $image_meta['sizes'] as $name => $size ) {
    14931493            // Whitelist the sizes that should be included so we pick up 'medium_large' in 4.4.
    1494             if ( in_array( $name, $intermediates ) ) {
     1494            if ( in_array( $name, $intermediates, true ) ) {
    14951495                $expected .= $uploads_dir_url . $size['file'] . ' ' . $size['width'] . 'w, ';
    14961496            }
    … …  
    15691569        foreach ( $image_meta['sizes'] as $name => $size ) {
    15701570            // Whitelist the sizes that should be included so we pick up 'medium_large' in 4.4.
    1571             if ( in_array( $name, $intermediates ) ) {
     1571            if ( in_array( $name, $intermediates, true ) ) {
    15721572                $expected .= $uploads_dir_url . $year_month . '/' . $size['file'] . ' ' . $size['width'] . 'w, ';
    15731573            }
    … …  
    18481848        foreach ( $image_meta['sizes'] as $name => $size ) {
    18491849            // Whitelist the sizes that should be included so we pick up 'medium_large' in 4.4.
    1850             if ( in_array( $name, $intermediates ) ) {
     1850            if ( in_array( $name, $intermediates, true ) ) {
    18511851                $expected .= $uploads_dir . $year_month . '/' . $size['file'] . ' ' . $size['width'] . 'w, ';
    18521852            }

  • trunk/tests/phpunit/tests/meta.php

    r43571 r45607  
    371371        $string_mid = "{$meta_id}.0";
    372372
     373        // phpcs:ignore WordPress.PHP.StrictComparisons.LooseComparison -- intentional implicit casting check
    373374        $this->assertTrue( floor( $string_mid ) == $string_mid );
    374375        $this->assertNotEquals( false, get_metadata_by_mid( 'user', $string_mid ) );

  • trunk/tests/phpunit/tests/meta/registerMeta.php

    r43571 r45607  
    506506
    507507    public function filter_get_object_subtype_for_customtype( $subtype, $object_id ) {
    508         if ( $object_id % 2 === 1 ) {
     508        if ( 1 === ( $object_id % 2 ) ) {
    509509            return 'odd';
    510510        }

  • trunk/tests/phpunit/tests/multisite.php

    r42343 r45607  
    3232
    3333            // currently there is no wrapper function for the registration_log
    34             $reg_blog = $wpdb->get_col( "SELECT email FROM {$wpdb->registration_log} WHERE {$wpdb->registration_log}.blog_id = 1 AND IP LIKE '" . $ip . "'" );
     34            $reg_blog = $wpdb->get_col( $wpdb->prepare( "SELECT email FROM {$wpdb->registration_log} WHERE {$wpdb->registration_log}.blog_id = 1 AND IP LIKE %s", $ip ) );
    3535            $this->assertEquals( $user->user_email, $reg_blog[ count( $reg_blog ) - 1 ] );
    3636        }

  • trunk/tests/phpunit/tests/multisite/getSpaceUsed.php

    r42343 r45607  
    2929            // src directory already contains a content directory with site content, then the initial expectation
    3030            // will be polluted. We create sites until an empty one is available.
    31             while ( 0 != get_space_used() ) {
     31            while ( 0 !== get_space_used() ) {
    3232                restore_current_blog();
    3333                $blog_id = self::factory()->blog->create();
    … …  
    6565            // so that we can remove any uploaded files and directories without concern of a conflict with
    6666            // existing content directories in src.
    67             while ( 0 != get_space_used() ) {
     67            while ( 0 !== get_space_used() ) {
    6868                restore_current_blog();
    6969                $blog_id = self::factory()->blog->create();

  • trunk/tests/phpunit/tests/multisite/site.php

    r44806 r45607  
    167167            // Check existence of each database table for the created site.
    168168            foreach ( $wpdb->tables( 'blog', false ) as $table ) {
    169                 $suppress     = $wpdb->suppress_errors();
     169                $suppress = $wpdb->suppress_errors();
     170
     171                // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    170172                $table_fields = $wpdb->get_results( "DESCRIBE $prefix$table;" );
     173
    171174                $wpdb->suppress_errors( $suppress );
    172175
    … …  
    175178
    176179                // And the table should not be empty, unless commentmeta, termmeta, or links.
     180                // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    177181                $result = $wpdb->get_results( "SELECT * FROM $prefix$table LIMIT 1" );
    178                 if ( 'commentmeta' == $table || 'termmeta' == $table || 'links' == $table ) {
     182
     183                if ( 'commentmeta' === $table || 'termmeta' === $table || 'links' === $table ) {
    179184                    $this->assertEmpty( $result );
    180185                } else {
    … …  
    245250            $prefix = $wpdb->get_blog_prefix( $blog_id );
    246251            foreach ( $wpdb->tables( 'blog', false ) as $table ) {
    247                 $suppress     = $wpdb->suppress_errors();
     252                $suppress = $wpdb->suppress_errors();
     253
     254                // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    248255                $table_fields = $wpdb->get_results( "DESCRIBE $prefix$table;" );
     256
    249257                $wpdb->suppress_errors( $suppress );
    250258                $this->assertNotEmpty( $table_fields, $prefix . $table );
    … …  
    283291            $prefix = $wpdb->get_blog_prefix( $blog_id );
    284292            foreach ( $wpdb->tables( 'blog', false ) as $table ) {
    285                 $suppress     = $wpdb->suppress_errors();
     293                $suppress = $wpdb->suppress_errors();
     294
     295                // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    286296                $table_fields = $wpdb->get_results( "DESCRIBE $prefix$table;" );
     297
    287298                $wpdb->suppress_errors( $suppress );
    288299                $this->assertEmpty( $table_fields );
    … …  
    321332            $prefix = $wpdb->get_blog_prefix( $blog_id );
    322333            foreach ( $wpdb->tables( 'blog', false ) as $table ) {
    323                 $suppress     = $wpdb->suppress_errors();
     334                $suppress = $wpdb->suppress_errors();
     335
     336                // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
    324337                $table_fields = $wpdb->get_results( "DESCRIBE $prefix$table;" );
     338
    325339                $wpdb->suppress_errors( $suppress );
    326340                $this->assertNotEmpty( $table_fields, $prefix . $table );
    … …  
    856870         */
    857871        function _domain_exists_cb( $exists, $domain, $path, $site_id ) {
    858             if ( 'foo' == $domain && 'bar/' == $path ) {
     872            if ( 'foo' === $domain && 'bar/' === $path ) {
    859873                return 1234;
    860874            } else {

  • trunk/tests/phpunit/tests/oembed/controller.php

    r44155 r45607  
    122122        }
    123123
    124         if ( $url === self::UNTRUSTED_PROVIDER_URL ) {
     124        if ( self::UNTRUSTED_PROVIDER_URL === $url ) {
    125125            return array(
    126126                'response' => array(

  • trunk/tests/phpunit/tests/oembed/headers.php

    r42343 r45607  
    3131        $headers = xdebug_get_headers();
    3232
    33         $this->assertTrue( in_array( 'Content-Type: text/xml; charset=' . get_option( 'blog_charset' ), $headers ) );
     33        $this->assertTrue( in_array( 'Content-Type: text/xml; charset=' . get_option( 'blog_charset' ), $headers, true ) );
    3434    }
    3535}

  • trunk/tests/phpunit/tests/pomo/mo.php

    r43571 r45607  
    179179
    180180    function test_overloaded_mb_functions() {
    181         if ( ( ini_get( 'mbstring.func_overload' ) & 2 ) == 0 ) {
     181        if ( ( ini_get( 'mbstring.func_overload' ) & 2 ) === 0 ) {
    182182            $this->markTestSkipped( __METHOD__ . ' only runs when mbstring.func_overload is enabled.' );
    183183        }

  • trunk/tests/phpunit/tests/pomo/pluralForms.php

    r42343 r45607  
    6161        foreach ( $locales as $slug => $locale ) {
    6262            $plural_expression = $locale->plural_expression;
    63             if ( $plural_expression !== 'n != 1' ) {
     63            if ( 'n != 1' !== $plural_expression ) {
    6464                $plural_expressions[] = array( $slug, $locale->nplurals, $plural_expression );
    6565            }
    … …  
    8383        $parenthesized = self::parenthesize_plural_expression( $expression );
    8484        $old_style     = tests_make_plural_form_function( $nplurals, $parenthesized );
    85         $pluralForms   = new Plural_Forms( $expression );
     85        $plural_forms  = new Plural_Forms( $expression );
    8686
    8787        $generated_old = array();
    … …  
    9090        foreach ( range( 0, 200 ) as $i ) {
    9191            $generated_old[] = $old_style( $i );
    92             $generated_new[] = $pluralForms->get( $i );
     92            $generated_new[] = $plural_forms->get( $i );
    9393        }
    9494
    … …  
    152152     */
    153153    public function test_simple( $expression, $expected ) {
    154         $pluralForms = new Plural_Forms( $expression );
    155         $actual      = array();
     154        $plural_forms = new Plural_Forms( $expression );
     155        $actual       = array();
    156156        foreach ( array_keys( $expected ) as $num ) {
    157             $actual[ $num ] = $pluralForms->get( $num );
     157            $actual[ $num ] = $plural_forms->get( $num );
    158158        }
    159159
    … …  
    213213    public function test_exceptions( $expression, $expected_exception, $call_get ) {
    214214        try {
    215             $pluralForms = new Plural_Forms( $expression );
     215            $plural_forms = new Plural_Forms( $expression );
    216216            if ( $call_get ) {
    217                 $pluralForms->get( 1 );
     217                $plural_forms->get( 1 );
    218218            }
    219219        } catch ( Exception $e ) {

  • trunk/tests/phpunit/tests/post.php

    r45588 r45607  
    9696
    9797            $tcache = wp_cache_get( $id, 'ctax_relationships' );
    98             if ( 'cpt' == $post_type ) {
     98            if ( 'cpt' === $post_type ) {
    9999                $this->assertInternalType( 'array', $tcache );
    100100                $this->assertEquals( 2, count( $tcache ) );
    … …  
    10091009    function test_utf8mb3_post_saves_with_emoji() {
    10101010        global $wpdb;
    1011         $_wpdb = new wpdb_exposed_methods_for_testing();
     1011        $_wpdb = new WpdbExposedMethodsForTesting();
    10121012
    10131013        if ( 'utf8' !== $_wpdb->get_col_charset( $wpdb->posts, 'post_title' ) ) {

  • trunk/tests/phpunit/tests/post/meta.php

    r43982 r45607  
    9696        );
    9797        sort( $expected );
    98         $this->assertTrue( in_array( get_post_meta( self::$post_id, 'nonunique', true ), $expected ) );
     98        $this->assertTrue( in_array( get_post_meta( self::$post_id, 'nonunique', true ), $expected, true ) );
    9999        $actual = get_post_meta( self::$post_id, 'nonunique', false );
    100100        sort( $actual );

  • trunk/tests/phpunit/tests/post/nav-menu.php

    r44966 r45607  
    652652        $tag_id  = self::factory()->tag->create();
    653653
    654         $wpdb->query( "UPDATE $wpdb->posts SET ID=$new_id WHERE ID=$page_id" );
    655         $wpdb->query( "UPDATE $wpdb->terms SET term_id=$new_id WHERE term_id=$tag_id" );
    656         $wpdb->query( "UPDATE $wpdb->term_taxonomy SET term_id=$new_id WHERE term_id=$tag_id" );
     654        $wpdb->update( $wpdb->posts, array( 'ID' => $new_id ), array( 'ID' => $page_id ) );
     655        $wpdb->update( $wpdb->terms, array( 'term_id' => $new_id ), array( 'term_id' => $tag_id ) );
     656        $wpdb->update( $wpdb->term_taxonomy, array( 'term_id' => $new_id ), array( 'term_id' => $tag_id ) );
    657657
    658658        update_option( 'page_on_front', $new_id );

  • trunk/tests/phpunit/tests/post/objects.php

    r42343 r45607  
    3737        $this->assertInternalType( 'array', $post );
    3838        $this->assertFalse( isset( $post['post_type'] ) );
    39         $this->assertTrue( in_array( 'post', $post ) );
     39        $this->assertTrue( in_array( 'post', $post, true ) );
    4040
    4141        $post = get_post( $id );

  • trunk/tests/phpunit/tests/post/thumbnails.php

    r43571 r45607  
    7272        set_post_thumbnail( self::$post, self::$attachment_id );
    7373
    74         $WP_Query = new WP_Query(
     74        $query = new WP_Query(
    7575            array(
    7676                'post_type' => 'any',
    … …  
    8080        );
    8181
    82         $this->assertFalse( $WP_Query->thumbnails_cached );
    83 
    84         update_post_thumbnail_cache( $WP_Query );
    85 
    86         $this->assertTrue( $WP_Query->thumbnails_cached );
     82        $this->assertFalse( $query->thumbnails_cached );
     83
     84        update_post_thumbnail_cache( $query );
     85
     86        $this->assertTrue( $query->thumbnails_cached );
    8787    }
    8888
    … …  
    393393        );
    394394
    395         $post = $which_post === 1 ? self::$different_post : self::$post;
     395        $post = 1 === $which_post ? self::$different_post : self::$post;
    396396
    397397        add_filter( 'post_thumbnail_size', array( $this, 'filter_post_thumbnail_size' ), 10, 2 );

  • trunk/tests/phpunit/tests/post/types.php

    r43571 r45607  
    306306        );
    307307
    308         $this->assertInternalType( 'int', array_search( 'bar', $wp->public_query_vars ) );
    309         $this->assertTrue( unregister_post_type( 'foo' ) );
    310         $this->assertFalse( array_search( 'bar', $wp->public_query_vars ) );
     308        $this->assertInternalType( 'int', array_search( 'bar', $wp->public_query_vars, true ) );
     309        $this->assertTrue( unregister_post_type( 'foo' ) );
     310        $this->assertFalse( array_search( 'bar', $wp->public_query_vars, true ) );
    311311    }
    312312

  • trunk/tests/phpunit/tests/post/wpPostType.php

    r43571 r45607  
    7979        $post_type_object->add_rewrite_rules();
    8080
    81         $this->assertFalse( in_array( 'foobar', $wp->public_query_vars ) );
     81        $this->assertFalse( in_array( 'foobar', $wp->public_query_vars, true ) );
    8282    }
    8383
    … …  
    9999
    100100        $post_type_object->add_rewrite_rules();
    101         $in_array = in_array( 'foobar', $wp->public_query_vars );
     101        $in_array = in_array( 'foobar', $wp->public_query_vars, true );
    102102
    103103        $post_type_object->remove_rewrite_rules();
    104         $in_array_after = in_array( 'foobar', $wp->public_query_vars );
     104        $in_array_after = in_array( 'foobar', $wp->public_query_vars, true );
    105105
    106106        $this->assertTrue( $in_array );
    … …  
    129129        $rewrite_tags_after = $wp_rewrite->rewritecode;
    130130
    131         $this->assertNotFalse( array_search( "%$post_type%", $rewrite_tags ) );
    132         $this->assertFalse( array_search( "%$post_type%", $rewrite_tags_after ) );
     131        $this->assertNotFalse( array_search( "%$post_type%", $rewrite_tags, true ) );
     132        $this->assertFalse( array_search( "%$post_type%", $rewrite_tags_after, true ) );
    133133    }
    134134

  • trunk/tests/phpunit/tests/rest-api.php

    r44568 r45607  
    256256    function test_rest_route_query_var() {
    257257        rest_api_init();
    258         $this->assertTrue( in_array( 'rest_route', $GLOBALS['wp']->public_query_vars ) );
     258        $this->assertTrue( in_array( 'rest_route', $GLOBALS['wp']->public_query_vars, true ) );
    259259    }
    260260

  • trunk/tests/phpunit/tests/rest-api/rest-posts-controller.php

    r45588 r45607  
    24922492
    24932493    public function revoke_assign_term( $caps, $cap, $user_id, $args ) {
    2494         if ( 'assign_term' === $cap && isset( $args[0] ) && $this->forbidden_cat == $args[0] ) {
     2494        if ( 'assign_term' === $cap && isset( $args[0] ) && $this->forbidden_cat === $args[0] ) {
    24952495            $caps = array( 'do_not_allow' );
    24962496        }

  • trunk/tests/phpunit/tests/rest-api/rest-request.php

    r43571 r45607  
    464464        $data = $valid->get_error_data( 'rest_missing_callback_param' );
    465465
    466         $this->assertTrue( in_array( 'someinteger', $data['params'] ) );
    467         $this->assertTrue( in_array( 'someotherinteger', $data['params'] ) );
     466        $this->assertTrue( in_array( 'someinteger', $data['params'], true ) );
     467        $this->assertTrue( in_array( 'someotherinteger', $data['params'], true ) );
    468468    }
    469469

  • trunk/tests/phpunit/tests/shortcode.php

    r44569 r45607  
    3939    // [footag foo="bar"]
    4040    function _shortcode_footag( $atts ) {
    41         return @"foo = {$atts['foo']}";
     41        $foo = isset( $atts['foo'] ) ? $atts['foo'] : '';
     42        return "foo = $foo";
    4243    }
    4344
    … …  
    445446    function _filter_atts2( $out, $pairs, $atts ) {
    446447        // If foo attribute equals "foo1", change it to be default value
    447         if ( isset( $out['foo'] ) && 'foo1' == $out['foo'] ) {
     448        if ( isset( $out['foo'] ) && 'foo1' === $out['foo'] ) {
    448449            $out['foo'] = $pairs['foo'];
    449450        }

  • trunk/tests/phpunit/tests/taxonomy.php

    r43571 r45607  
    812812        register_taxonomy( 'foo', 'post', array( 'query_var' => 'bar' ) );
    813813
    814         $this->assertInternalType( 'int', array_search( 'bar', $wp->public_query_vars ) );
     814        $this->assertInternalType( 'int', array_search( 'bar', $wp->public_query_vars, true ) );
    815815        $this->assertTrue( unregister_taxonomy( 'foo' ) );
    816         $this->assertFalse( array_search( 'bar', $wp->public_query_vars ) );
     816        $this->assertFalse( array_search( 'bar', $wp->public_query_vars, true ) );
    817817    }
    818818

  • trunk/tests/phpunit/tests/term/cache.php

    r43571 r45607  
    9191            }
    9292
    93             if ( $i % 3 === 0 ) {
     93            if ( 0 === ( $i % 3 ) ) {
    9494                $step = 1;
    9595            } else {

  • trunk/tests/phpunit/tests/term/getTerms.php

    r43571 r45607  
    24752475
    24762476        foreach ( $found as $f ) {
    2477             if ( $t1 == $f->term_id ) {
     2477            if ( $t1 === $f->term_id ) {
    24782478                $this->assertSame( 3, $f->count );
    2479             } elseif ( $t2 == $f->term_id ) {
     2479            } elseif ( $t2 === $f->term_id ) {
    24802480                $this->assertSame( 2, $f->count );
    24812481            } else {
    … …  
    25482548
    25492549        foreach ( $found as $f ) {
    2550             if ( $t1 == $f->term_id ) {
     2550            if ( $t1 === $f->term_id ) {
    25512551                $this->assertEquals( 1, $f->count );
    2552             } elseif ( $t2 == $f->term_id ) {
     2552            } elseif ( $t2 === $f->term_id ) {
    25532553                $this->assertEquals( 2, $f->count );
    25542554            } else {

  • trunk/tests/phpunit/tests/term/wpInsertTerm.php

    r43571 r45607  
    823823        $cached_children = get_option( 'wptests_tax_children' );
    824824        $this->assertNotEmpty( $cached_children[ $t ] );
    825         $this->assertTrue( in_array( $found['term_id'], $cached_children[ $t ] ) );
     825        $this->assertTrue( in_array( $found['term_id'], $cached_children[ $t ], true ) );
    826826    }
    827827

  • trunk/tests/phpunit/tests/term/wpTaxonomy.php

    r43571 r45607  
    2323
    2424        $taxonomy_object->add_rewrite_rules();
    25         $this->assertFalse( in_array( 'foobar', $wp->public_query_vars ) );
     25        $this->assertFalse( in_array( 'foobar', $wp->public_query_vars, true ) );
    2626    }
    2727
    … …  
    4444
    4545        $taxonomy_object->add_rewrite_rules();
    46         $in_array = in_array( 'foobar', $wp->public_query_vars );
     46        $in_array = in_array( 'foobar', $wp->public_query_vars, true );
    4747
    4848        $taxonomy_object->remove_rewrite_rules();
    49         $in_array_after = in_array( 'foobar', $wp->public_query_vars );
     49        $in_array_after = in_array( 'foobar', $wp->public_query_vars, true );
    5050
    5151        $this->assertTrue( $in_array );
    … …  
    7575        $rewrite_tags_after = $wp_rewrite->rewritecode;
    7676
    77         $this->assertNotFalse( array_search( "%$taxonomy%", $rewrite_tags ) );
    78         $this->assertFalse( array_search( "%$taxonomy%", $rewrite_tags_after ) );
     77        $this->assertNotFalse( array_search( "%$taxonomy%", $rewrite_tags, true ) );
     78        $this->assertFalse( array_search( "%$taxonomy%", $rewrite_tags_after, true ) );
    7979    }
    8080

  • trunk/tests/phpunit/tests/term/wpUpdateTerm.php

    r43571 r45607  
    686686        $cached_children = get_option( 'wptests_tax_children' );
    687687        $this->assertNotEmpty( $cached_children[ $t2 ] );
    688         $this->assertTrue( in_array( $found['term_id'], $cached_children[ $t2 ] ) );
     688        $this->assertTrue( in_array( $found['term_id'], $cached_children[ $t2 ], true ) );
    689689    }
    690690

  • trunk/tests/phpunit/tests/theme.php

    r44966 r45607  
    244244            foreach ( $themes as $name => $theme ) {
    245245                // switch to this theme
    246                 if ( $i === 2 ) {
     246                if ( 2 === $i ) {
    247247                    switch_theme( $theme['Template'], $theme['Stylesheet'] );
    248248                } else {

  • trunk/tests/phpunit/tests/theme/support.php

    r44138 r45607  
    143143
    144144    function supports_foobar( $yesno, $args, $feature ) {
    145         if ( $args[0] == $feature[0] ) {
     145        if ( $args[0] === $feature[0] ) {
    146146            return true;
    147147        }

  • trunk/tests/phpunit/tests/theme/themeDir.php

    r42816 r45607  
    146146        // Ignore themes in the default /themes directory.
    147147        foreach ( $themes as $theme_name => $theme ) {
    148             if ( $theme->get_theme_root() != $this->theme_root ) {
     148            if ( $theme->get_theme_root() !== $this->theme_root ) {
    149149                unset( $themes[ $theme_name ] );
    150150            }
    … …  
    210210
    211211        $templates = $theme['Template Files'];
    212         $this->assertTrue( in_array( $this->theme_root . '/page-templates/template-top-level.php', $templates ) );
     212        $this->assertTrue( in_array( $this->theme_root . '/page-templates/template-top-level.php', $templates, true ) );
    213213    }
    214214

  • trunk/tests/phpunit/tests/user.php

    r45588 r45607  
    8282        foreach ( $user_list as $user ) {
    8383            // only include the users we just created - there might be some others that existed previously
    84             if ( in_array( $user->ID, $nusers ) ) {
     84            if ( in_array( $user->ID, $nusers, true ) ) {
    8585                $found[] = $user->ID;
    8686            }
    … …  
    162162        // so we'll just check to make sure our values are included somewhere.
    163163        foreach ( $vals as $k => $v ) {
    164             $this->assertTrue( isset( $out[ $k ] ) && $out[ $k ][0] == $v );
     164            $this->assertTrue( isset( $out[ $k ] ) && $out[ $k ][0] === $v );
    165165        }
    166166        // delete one key and check again
    … …  
    171171        // make sure that key is excluded from the results
    172172        foreach ( $vals as $k => $v ) {
    173             if ( $k == $key_to_delete ) {
     173            if ( $k === $key_to_delete ) {
    174174                $this->assertFalse( isset( $out[ $k ] ) );
    175175            } else {
    176                 $this->assertTrue( isset( $out[ $k ] ) && $out[ $k ][0] == $v );
     176                $this->assertTrue( isset( $out[ $k ] ) && $out[ $k ][0] === $v );
    177177            }
    178178        }
    … …  
    600600        }
    601601
    602         @update_user_meta( $id2, 'key', 'value' );
     602        update_user_meta( $id2, 'key', 'value' );
    603603
    604604        $metas = array_keys( get_user_meta( 1 ) );
    … …  
    10231023        );
    10241024
    1025         $this->assertTrue( in_array( self::$contrib_id, $users ) );
     1025        $this->assertTrue( in_array( (string) self::$contrib_id, $users, true ) );
    10261026    }
    10271027
    … …  
    10341034        );
    10351035
    1036         $this->assertTrue( in_array( self::$contrib_id, $users ) );
     1036        $this->assertTrue( in_array( (string) self::$contrib_id, $users, true ) );
    10371037    }
    10381038
    … …  
    10451045        );
    10461046
    1047         $this->assertTrue( in_array( self::$contrib_id, $users ) );
     1047        $this->assertTrue( in_array( (string) self::$contrib_id, $users, true ) );
    10481048    }
    10491049
    … …  
    10561056        );
    10571057
    1058         $this->assertTrue( in_array( self::$contrib_id, $users ) );
     1058        $this->assertTrue( in_array( (string) self::$contrib_id, $users, true ) );
    10591059    }
    10601060
    … …  
    10671067        );
    10681068
    1069         $this->assertTrue( in_array( self::$contrib_id, $users ) );
     1069        $this->assertTrue( in_array( (string) self::$contrib_id, $users, true ) );
    10701070    }
    10711071
    … …  
    12021202         */
    12031203        if ( ! empty( $GLOBALS['phpmailer']->mock_sent ) ) {
    1204             $was_admin_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[0] ) && WP_TESTS_EMAIL == $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] );
    1205             $was_user_email_sent  = ( isset( $GLOBALS['phpmailer']->mock_sent[1] ) && 'blackburn@battlefield3.com' == $GLOBALS['phpmailer']->mock_sent[1]['to'][0][0] );
     1204            $was_admin_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[0] ) && WP_TESTS_EMAIL === $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] );
     1205            $was_user_email_sent  = ( isset( $GLOBALS['phpmailer']->mock_sent[1] ) && 'blackburn@battlefield3.com' === $GLOBALS['phpmailer']->mock_sent[1]['to'][0][0] );
    12061206        }
    12071207
    … …  
    12271227         */
    12281228        if ( ! empty( $GLOBALS['phpmailer']->mock_sent ) ) {
    1229             $was_admin_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[0] ) && WP_TESTS_EMAIL == $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] );
    1230             $was_user_email_sent  = ( isset( $GLOBALS['phpmailer']->mock_sent[1] ) && 'blackburn@battlefield3.com' == $GLOBALS['phpmailer']->mock_sent[1]['to'][0][0] );
     1229            $was_admin_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[0] ) && WP_TESTS_EMAIL === $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] );
     1230            $was_user_email_sent  = ( isset( $GLOBALS['phpmailer']->mock_sent[1] ) && 'blackburn@battlefield3.com' === $GLOBALS['phpmailer']->mock_sent[1]['to'][0][0] );
    12311231        }
    12321232
    … …  
    14661466
    14671467        if ( ! empty( $GLOBALS['phpmailer']->mock_sent ) ) {
    1468             $was_confirmation_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[0] ) && 'after@example.com' == $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] );
     1468            $was_confirmation_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[0] ) && 'after@example.com' === $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] );
    14691469        }
    14701470
    … …  
    15031503
    15041504        if ( ! empty( $GLOBALS['phpmailer']->mock_sent ) ) {
    1505             $was_confirmation_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[0] ) && 'after@example.com' == $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] );
     1505            $was_confirmation_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[0] ) && 'after@example.com' === $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] );
    15061506        }
    15071507

  • trunk/tests/phpunit/tests/user/capabilities.php

    r45507 r45607  
    15471547        // Prevents fatal errors in ::tearDown()'s and other uses of restore_current_blog()
    15481548        $function_stack = wp_debug_backtrace_summary( null, 0, false );
    1549         if ( in_array( 'restore_current_blog', $function_stack ) ) {
     1549        if ( in_array( 'restore_current_blog', $function_stack, true ) ) {
    15501550            return;
    15511551        }

  • trunk/tests/phpunit/tests/user/query.php

    r44373 r45607  
    5454
    5555        $this->assertEquals( '', $users->get( 'fields' ) );
    56         $this->assertEquals( '', @$users->query_vars['fields'] );
     56        if ( isset( $users->query_vars['fields'] ) ) {
     57            $this->assertEquals( '', $users->query_vars['fields'] );
     58        }
    5759
    5860        $users->set( 'fields', 'all' );
    … …  
    11371139        );
    11381140
    1139         $foundCount    = count( $q->get_results() );
    1140         $expectedCount = 10; // 13 total users minus 3 from query
     1141        $found_count    = count( $q->get_results() );
     1142        $expected_count = 10; // 13 total users minus 3 from query
    11411143
    11421144        $this->assertContains( "AND user_nicename NOT IN ( 'peter','paul','mary' )", $q->query_where );
    1143         $this->assertEquals( $expectedCount, $foundCount );
     1145        $this->assertEquals( $expected_count, $found_count );
    11441146    }
    11451147
    … …  
    12381240        );
    12391241
    1240         $foundCount    = count( $q->get_results() );
    1241         $expectedCount = 10; // 13 total users minus 3 from query
     1242        $found_count    = count( $q->get_results() );
     1243        $expected_count = 10; // 13 total users minus 3 from query
    12421244
    12431245        $this->assertContains( "AND user_login NOT IN ( '$user_login1','$user_login2','$user_login3' )", $q->query_where );
    1244         $this->assertEquals( $expectedCount, $foundCount );
     1246        $this->assertEquals( $expected_count, $found_count );
    12451247    }
    12461248

  • trunk/tests/phpunit/tests/widgets.php

    r43571 r45607  
    151151        $names = wp_list_pluck( $wp_registered_sidebars, 'name' );
    152152        for ( $i = 1; $i <= $num; $i++ ) {
    153             if ( in_array( "$id_base $i", $names ) ) {
     153            if ( in_array( "$id_base $i", $names, true ) ) {
    154154                $result[] = true;
    155155            }

  • trunk/tests/phpunit/tests/wp.php

    r44577 r45607  
    2323
    2424        $this->assertSame( $public_qv_count + 2, count( $this->wp->public_query_vars ) );
    25         $this->assertTrue( in_array( 'test', $this->wp->public_query_vars ) );
    26         $this->assertTrue( in_array( 'test2', $this->wp->public_query_vars ) );
     25        $this->assertTrue( in_array( 'test', $this->wp->public_query_vars, true ) );
     26        $this->assertTrue( in_array( 'test2', $this->wp->public_query_vars, true ) );
    2727    }
    2828
    … …  
    3131
    3232        $this->wp->add_query_var( 'test' );
    33         $this->assertTrue( in_array( 'test', $this->wp->public_query_vars ) );
     33        $this->assertTrue( in_array( 'test', $this->wp->public_query_vars, true ) );
    3434        $this->wp->remove_query_var( 'test' );
    3535

  • trunk/tests/phpunit/tests/xmlrpc/mw/getRecentPosts.php

    r42343 r45607  
    110110            $this->assertStringMatchesFormat( '%d', $result['wp_post_thumbnail'] );
    111111
    112             if ( ! empty( $result['wp_post_thumbnail'] ) || $result['postid'] == self::$post_id ) {
     112            if ( ! empty( $result['wp_post_thumbnail'] ) || $result['postid'] === self::$post_id ) {
    113113                $attachment_id = get_post_meta( $result['postid'], '_thumbnail_id', true );
    114114

  • trunk/tests/phpunit/tests/xmlrpc/wp/editPost.php

    r45424 r45607  
    446446
    447447        // For good measure, check that the expected value is in the array
    448         $this->assertTrue( in_array( $enclosure_string, get_post_meta( $post_id, 'enclosure' ) ) );
     448        $this->assertTrue( in_array( $enclosure_string, get_post_meta( $post_id, 'enclosure' ), true ) );
    449449
    450450        // Attempt to add a brand new enclosure via XML-RPC
    … …  
    456456        // Check that the new enclosure is in the enclosure meta
    457457        $new_enclosure_string = "{$new_enclosure['url']}\n{$new_enclosure['length']}\n{$new_enclosure['type']}\n";
    458         $this->assertTrue( in_array( $new_enclosure_string, get_post_meta( $post_id, 'enclosure' ) ) );
     458        $this->assertTrue( in_array( $new_enclosure_string, get_post_meta( $post_id, 'enclosure' ), true ) );
    459459
    460460        // Check that the old enclosure is in the enclosure meta
    461         $this->assertTrue( in_array( $enclosure_string, get_post_meta( $post_id, 'enclosure' ) ) );
     461        $this->assertTrue( in_array( $enclosure_string, get_post_meta( $post_id, 'enclosure' ), true ) );
    462462    }
    463463

  • trunk/tests/phpunit/tests/xmlrpc/wp/getPages.php

    r42343 r45607  
    5656
    5757    function remove_editor_edit_page_cap( $caps, $cap, $user_id, $args ) {
    58         if ( in_array( $cap, array( 'edit_page', 'edit_others_pages' ) ) ) {
    59             if ( $user_id == self::$editor_id && $args[0] == self::$post_id ) {
     58        if ( in_array( $cap, array( 'edit_page', 'edit_others_pages' ), true ) ) {
     59            if ( $user_id === self::$editor_id && $args[0] === self::$post_id ) {
    6060                return array( false );
    6161            }
    … …  
    7979            $this->assertNotIXRError( $result );
    8080
    81             if ( $result['page_id'] == self::$post_id ) {
     81            if ( $result['page_id'] === self::$post_id ) {
    8282                $found_incapable = true;
    8383                break;

  • trunk/tests/phpunit/tests/xmlrpc/wp/getTerms.php

    r42343 r45607  
    122122
    123123        foreach ( $results as $term ) {
    124             if ( $term['term_id'] == $cat1 ) {
     124            if ( $term['term_id'] === $cat1 ) {
    125125                break;  // found cat1 first as expected
    126             } elseif ( $term['term_id'] == $cat2 ) {
     126            } elseif ( $term['term_id'] === $cat2 ) {
    127127                $this->assertFalse( false, 'Incorrect category ordering.' );
    128128            }

  • trunk/tests/phpunit/wp-mail-real-test.php

    r44563 r45607  
    5454
    5555// make sure we're installed
    56 assert( true == is_blog_installed() );
     56assert( true === is_blog_installed() );
    5757
    5858// phpcs:ignore Generic.NamingConventions.UpperCaseConstantName.ConstantNotUpperCase
Note: See TracChangeset for help on using the changeset viewer.