WordPress.org
Get WordPress

Make WordPress Core

Changeset 47184


Ignore:
Timestamp:
02/05/2020 04:28:13 AM (5 years ago)
Author:
SergeyBiryukov
Message:

Tests: Allow dbDelta() tests to (mostly) run on MySQL 8.0.11+.

  • MySQL 8.0.11 changed the GeometryCollection data type name to GeomCollection, with the latter being the preferred name.
  • MySQL 8.0.17 removed support for the display width attribute for integer data types. Previously, default display width of 20 digits was used: BIGINT(20).

The affected tests now check the MySQL server version and use the appropriate data types.

This leaves one unresolved failure on MySQL 8.0.17+ to be addressed in the future, caused by the same BIGINT display width discrepancy coming from wp_get_db_schema().

Props kaggdesign, ottok, jeremyfelt, SergeyBiryukov.
Fixes #44384, #49344. See #49364.

File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r47122 r47184  
    1616
    1717    /**
     18     * Display width for BIGINT data type.
     19     *
     20     * Prior to MySQL 8.0.17, default width of 20 digits was used: BIGINT(20).
     21     * Since MySQL 8.0.17, display width for integer data types is no longer supported.
     22     */
     23    protected $bigint_display_width = '';
     24
     25    /**
    1826     * Make sure the upgrade code is loaded before the tests are run.
    1927     */
    … …  
    3139
    3240        global $wpdb;
     41
     42        $db_version = $wpdb->db_version();
     43
     44        if ( version_compare( $db_version, '8.0.17', '<' ) ) {
     45            // Prior to MySQL 8.0.17, default width of 20 digits was used: BIGINT(20).
     46            $this->bigint_display_width = '(20)';
     47        }
    3348
    3449        // Forcing MyISAM, because InnoDB only started supporting FULLTEXT indexes in MySQL 5.7.
    … …  
    3752                "
    3853                CREATE TABLE {$wpdb->prefix}dbdelta_test (
    39                     id bigint(20) NOT NULL AUTO_INCREMENT,
     54                    id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    4055                    column_1 varchar(255) NOT NULL,
    4156                    column_2 text,
    … …  
    7994        $updates = dbDelta(
    8095            "CREATE TABLE {$wpdb->prefix}dbdelta_create_test (
    81                 id bigint(20) NOT NULL AUTO_INCREMENT,
     96                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    8297                column_1 varchar(255) NOT NULL,
    8398                PRIMARY KEY  (id)
    … …  
    114129            "
    115130            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    116                 id bigint(20) NOT NULL AUTO_INCREMENT,
     131                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    117132                column_1 varchar(255) NOT NULL,
    118133                PRIMARY KEY  (id),
    … …  
    133148        global $wpdb;
    134149
    135         // id: bigint(20) => int(11)
     150        // id: bigint => int(11)
    136151        $updates = dbDelta(
    137152            "
    … …  
    149164            array(
    150165                "{$wpdb->prefix}dbdelta_test.id"
    151                     => "Changed type of {$wpdb->prefix}dbdelta_test.id from bigint(20) to int(11)",
     166                    => "Changed type of {$wpdb->prefix}dbdelta_test.id from bigint{$this->bigint_display_width} to int(11)",
    152167            ),
    153168            $updates
    … …  
    165180            "
    166181            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    167                 id bigint(20) NOT NULL AUTO_INCREMENT,
     182                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    168183                column_1 varchar(255) NOT NULL,
    169184                extra_col longtext,
    … …  
    200215            "
    201216            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    202                 id bigint(20) NOT NULL AUTO_INCREMENT,
     217                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    203218                PRIMARY KEY  (id),
    204219                KEY key_1 (column_1($this->max_index_length)),
    … …  
    224239            "
    225240            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    226                 id bigint(20) NOT NULL AUTO_INCREMENT,
     241                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    227242                column_1 varchar(255) NOT NULL,
    228243                extra_col longtext,
    … …  
    276291            "
    277292            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    278                 id bigint(20) NOT NULL AUTO_INCREMENT,
     293                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    279294                column_1 varchar(255) NOT NULL,
    280295                PRIMARY KEY  (id),
    … …  
    421436            "
    422437            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    423                 id bigint(20) NOT NULL AUTO_INCREMENT,
     438                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    424439                column_1 varchar(255) NOT NULL,
    425440                column_2 tinytext,
    … …  
    446461            "
    447462            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    448                 id bigint(20) NOT NULL AUTO_INCREMENT,
     463                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    449464                column_1 varchar(255) NOT NULL,
    450465                column_2 text,
    … …  
    471486            "
    472487            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    473                 id bigint(20) NOT NULL AUTO_INCREMENT,
     488                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    474489                column_1 varchar(255) NOT NULL,
    475490                column_2 bigtext,
    … …  
    502517            "
    503518            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    504                 id bigint(20) NOT NULL AUTO_INCREMENT,
     519                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    505520                column_1 varchar(255) NOT NULL,
    506521                column_2 text,
    … …  
    532547        $schema = "
    533548            CREATE TABLE {$wpdb->prefix}dbdelta_test2 (
    534                 `id` bigint(20) NOT NULL AUTO_INCREMENT,
     549                `id` bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    535550                `column_1` varchar(255) NOT NULL,
    536551                PRIMARY KEY  (id),
    … …  
    555570        global $wpdb;
    556571
    557         if ( version_compare( $wpdb->db_version(), '5.4', '<' ) ) {
     572        $db_version = $wpdb->db_version();
     573
     574        if ( version_compare( $db_version, '5.4', '<' ) ) {
    558575            $this->markTestSkipped( 'Spatial indices require MySQL 5.4 and above.' );
    559576        }
    560577
     578        $geomcollection_name = 'geomcollection';
     579
     580        if ( version_compare( $db_version, '8.0.11', '<' ) ) {
     581            // Prior to MySQL 8.0.11, GeometryCollection data type name was used.
     582            $geomcollection_name = 'geometrycollection';
     583        }
     584
    561585        $schema =
    562586            "
    563587            CREATE TABLE {$wpdb->prefix}spatial_index_test (
    564                 non_spatial bigint(20) unsigned NOT NULL,
    565                 spatial_value geometrycollection NOT NULL,
     588                non_spatial bigint{$this->bigint_display_width} unsigned NOT NULL,
     589                spatial_value {$geomcollection_name} NOT NULL,
    566590                KEY non_spatial (non_spatial),
    567591                SPATIAL KEY spatial_key (spatial_value)
    … …  
    579603            "
    580604            CREATE TABLE {$wpdb->prefix}spatial_index_test (
    581                 non_spatial bigint(20) unsigned NOT NULL,
    582                 spatial_value geometrycollection NOT NULL,
    583                 spatial_value2 geometrycollection NOT NULL,
     605                non_spatial bigint{$this->bigint_display_width} unsigned NOT NULL,
     606                spatial_value {$geomcollection_name} NOT NULL,
     607                spatial_value2 {$geomcollection_name} NOT NULL,
    584608                KEY non_spatial (non_spatial),
    585609                SPATIAL KEY spatial_key (spatial_value)
    … …  
    609633        $schema = "
    610634            CREATE TABLE {$wpdb->prefix}dbdelta_test2 (
    611                 `id` bigint(20) NOT NULL AUTO_INCREMENT,
     635                `id` bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    612636                `references` varchar(255) NOT NULL,
    613637                PRIMARY KEY  (`id`),
    … …  
    639663            "
    640664            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    641                 id bigint(20) NOT NULL AUTO_INCREMENT,
     665                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    642666                column_1 varchar(255) NOT NULL,
    643667                column_2 text,
    … …  
    683707        $schema = "
    684708            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    685                 id bigint(20) NOT NULL AUTO_INCREMENT,
     709                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    686710                column_1 varchar(255) NOT NULL,
    687711                column_2 text,
    … …  
    722746            "
    723747            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    724                 id bigint(20) NOT NULL AUTO_INCREMENT,
     748                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    725749                column_1 varchar(255) NOT NULL,
    726750                column_2 text,
    … …  
    745769        $schema = "
    746770            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    747                 id bigint(20) NOT NULL AUTO_INCREMENT,
     771                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    748772                column_1 varchar(255) NOT NULL,
    749773                column_2 text,
    … …  
    780804            "
    781805            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    782                 id bigint(20) NOT NULL AUTO_INCREMENT,
     806                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    783807                column_1 varchar(255) NOT NULL,
    784808                column_2 text,
    … …  
    804828            "
    805829            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    806                 id bigint(20) NOT NULL AUTO_INCREMENT,
     830                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    807831                column_1 varchar(255) NOT NULL,
    808832                column_2 text,
    … …  
    828852            "
    829853            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    830                 id bigint(20) NOT NULL AUTO_INCREMENT,
     854                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    831855                column_1 varchar(255) NOT NULL,
    832856                column_2 text,
    … …  
    852876            "
    853877            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    854                 id bigint(20) NOT NULL AUTO_INCREMENT,
     878                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    855879                column_1 varchar(255) NOT NULL,
    856880                column_2 text,
    … …  
    876900            "
    877901            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    878                 id bigint(20) NOT NULL AUTO_INCREMENT,
     902                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    879903                column_1 varchar(255) NOT NULL,
    880904                column_2 text,
    … …  
    901925            "
    902926            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    903                 id bigint(20) NOT NULL AUTO_INCREMENT,
     927                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    904928                column_1 varchar(255) NOT NULL,
    905929                column_2 text,
    … …  
    926950            "
    927951            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    928                 id bigint(20) NOT NULL AUTO_INCREMENT,
     952                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    929953                column_1 varchar(255) NOT NULL,
    930954                column_2 text,
    … …  
    949973            "
    950974            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    951                 id bigint(20) NOT NULL AUTO_INCREMENT,
     975                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    952976                column_1 varchar(255) NOT NULL,
    953977                column_2 text,
    … …  
    967991            "
    968992            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    969                 id bigint(20) NOT NULL AUTO_INCREMENT,
     993                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    970994                column_1 varchar(255) NOT NULL,
    971995                column_2 text,
    … …  
    9851009            "
    9861010            CREATE TABLE {$wpdb->prefix}dbdelta_test (
    987                 id bigint(20) NOT NULL AUTO_INCREMENT,
     1011                id bigint{$this->bigint_display_width} NOT NULL AUTO_INCREMENT,
    9881012                column_1 varchar(255) NOT NULL,
    9891013                column_2 text,
Note: See TracChangeset for help on using the changeset viewer.