Make WordPress Core

Changeset 47013


Ignore:
Timestamp:
12/25/2019 09:39:46 PM (7 years ago)
Author:
SergeyBiryukov
Message:

Tests: Update legacy arguments passed to WP_UnitTest_Factory_For_Blog::create_object().

This converts the arguments originally meant for wpmu_create_blog() to the ones used by wp_insert_site().

Follow-up to [47011].

Fixes #47195.

Location:
trunk/tests/phpunit
Files:
13 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/includes/factory/class-wp-unittest-factory-for-blog.php

    r47011 r47013  
    3434                global $wpdb;
    3535
    36                 // Map some arguments for backward compatibility with wpmu_create_blog() previously used here.
    37                 if ( ! isset( $args['public'] ) ) {
    38                         // Default to public, unless an options array was provided.
    39                         $args['public'] = ! isset( $args['meta'] ) ? 1 : 0;
    40                 }
    41 
    42                 if ( ! isset( $args['user_id'] ) ) {
    43                         $args['user_id'] = get_current_user_id();
    44                 }
    45 
    46                 if ( isset( $args['site_id'] ) ) {
     36                // Map some arguments for backward compatibility with `wpmu_create_blog()` previously used here.
     37                if ( ! isset( $args['network_id'] ) && isset( $args['site_id'] ) ) {
    4738                        $args['network_id'] = $args['site_id'];
    4839                        unset( $args['site_id'] );
  • trunk/tests/phpunit/tests/multisite.php

    r46586 r47013  
    4444                                'public'   => self::factory()->blog->create_many(
    4545                                        random_int( 0, 5 ),
    46                                         array( 'meta' => array( 'public' => 1 ) )
     46                                        array(
     47                                                'public' => 1,
     48                                        )
    4749                                ),
    4850                                'archived' => self::factory()->blog->create_many(
    4951                                        random_int( 0, 5 ),
    50                                         array( 'meta' => array( 'archived' => 1 ) )
     52                                        array(
     53                                                'public'   => 0,
     54                                                'archived' => 1,
     55                                        )
    5156                                ),
    5257                                'mature'   => self::factory()->blog->create_many(
    5358                                        random_int( 0, 5 ),
    54                                         array( 'meta' => array( 'mature' => 1 ) )
     59                                        array(
     60                                                'public' => 0,
     61                                                'mature' => 1,
     62                                        )
    5563                                ),
    5664                                'spam'     => self::factory()->blog->create_many(
    5765                                        random_int( 0, 5 ),
    58                                         array( 'meta' => array( 'spam' => 1 ) )
     66                                        array(
     67                                                'public' => 0,
     68                                                'spam'   => 1,
     69                                        )
    5970                                ),
    6071                                'deleted'  => self::factory()->blog->create_many(
    6172                                        random_int( 0, 5 ),
    62                                         array( 'meta' => array( 'deleted' => 1 ) )
     73                                        array(
     74                                                'public'  => 0,
     75                                                'deleted' => 1,
     76                                        )
    6377                                ),
    6478                        );
  • trunk/tests/phpunit/tests/multisite/bootstrap.php

    r47012 r47013  
    5656                        self::$site_ids = array(
    5757                                'wordpress.org/'          => array(
    58                                         'domain'  => 'wordpress.org',
    59                                         'path'    => '/',
    60                                         'site_id' => self::$network_ids['wordpress.org/'],
     58                                        'domain'     => 'wordpress.org',
     59                                        'path'       => '/',
     60                                        'network_id' => self::$network_ids['wordpress.org/'],
    6161                                ),
    6262                                'wordpress.org/foo/'      => array(
    63                                         'domain'  => 'wordpress.org',
    64                                         'path'    => '/foo/',
    65                                         'site_id' => self::$network_ids['wordpress.org/'],
     63                                        'domain'     => 'wordpress.org',
     64                                        'path'       => '/foo/',
     65                                        'network_id' => self::$network_ids['wordpress.org/'],
    6666                                ),
    6767                                'wordpress.org/foo/bar/'  => array(
    68                                         'domain'  => 'wordpress.org',
    69                                         'path'    => '/foo/bar/',
    70                                         'site_id' => self::$network_ids['wordpress.org/'],
     68                                        'domain'     => 'wordpress.org',
     69                                        'path'       => '/foo/bar/',
     70                                        'network_id' => self::$network_ids['wordpress.org/'],
    7171                                ),
    7272                                'make.wordpress.org/'     => array(
    73                                         'domain'  => 'make.wordpress.org',
    74                                         'path'    => '/',
    75                                         'site_id' => self::$network_ids['make.wordpress.org/'],
     73                                        'domain'     => 'make.wordpress.org',
     74                                        'path'       => '/',
     75                                        'network_id' => self::$network_ids['make.wordpress.org/'],
    7676                                ),
    7777                                'make.wordpress.org/foo/' => array(
    78                                         'domain'  => 'make.wordpress.org',
    79                                         'path'    => '/foo/',
    80                                         'site_id' => self::$network_ids['make.wordpress.org/'],
     78                                        'domain'     => 'make.wordpress.org',
     79                                        'path'       => '/foo/',
     80                                        'network_id' => self::$network_ids['make.wordpress.org/'],
    8181                                ),
    8282                                'www.w.org/'              => array(
  • trunk/tests/phpunit/tests/multisite/getIdFromBlogname.php

    r47012 r47013  
    3232                        self::$site_ids = array(
    3333                                'wordpress.org/'         => array(
    34                                         'domain'  => 'wordpress.org',
    35                                         'path'    => '/',
    36                                         'site_id' => self::$network_ids['wordpress.org/'],
     34                                        'domain'     => 'wordpress.org',
     35                                        'path'       => '/',
     36                                        'network_id' => self::$network_ids['wordpress.org/'],
    3737                                ),
    3838                                'foo.wordpress.org/'     => array(
    39                                         'domain'  => 'foo.wordpress.org',
    40                                         'path'    => '/',
    41                                         'site_id' => self::$network_ids['wordpress.org/'],
     39                                        'domain'     => 'foo.wordpress.org',
     40                                        'path'       => '/',
     41                                        'network_id' => self::$network_ids['wordpress.org/'],
    4242                                ),
    4343                                'wordpress.org/foo/'     => array(
    44                                         'domain'  => 'wordpress.org',
    45                                         'path'    => '/foo/',
    46                                         'site_id' => self::$network_ids['wordpress.org/'],
     44                                        'domain'     => 'wordpress.org',
     45                                        'path'       => '/foo/',
     46                                        'network_id' => self::$network_ids['wordpress.org/'],
    4747                                ),
    4848                                'www.wordpress.net/'     => array(
    49                                         'domain'  => 'www.wordpress.net',
    50                                         'path'    => '/',
    51                                         'site_id' => self::$network_ids['www.wordpress.net/'],
     49                                        'domain'     => 'www.wordpress.net',
     50                                        'path'       => '/',
     51                                        'network_id' => self::$network_ids['www.wordpress.net/'],
    5252                                ),
    5353                                'foo.wordpress.net/'     => array(
    54                                         'domain'  => 'foo.wordpress.net',
    55                                         'path'    => '/',
    56                                         'site_id' => self::$network_ids['www.wordpress.net/'],
     54                                        'domain'     => 'foo.wordpress.net',
     55                                        'path'       => '/',
     56                                        'network_id' => self::$network_ids['www.wordpress.net/'],
    5757                                ),
    5858                                'www.wordpress.net/foo/' => array(
    59                                         'domain'  => 'www.wordpress.net',
    60                                         'path'    => '/foo/',
    61                                         'site_id' => self::$network_ids['www.wordpress.net/'],
     59                                        'domain'     => 'www.wordpress.net',
     60                                        'path'       => '/foo/',
     61                                        'network_id' => self::$network_ids['www.wordpress.net/'],
    6262                                ),
    6363                        );
  • trunk/tests/phpunit/tests/multisite/getMainSiteId.php

    r47012 r47013  
    3636                                ),
    3737                                'wordpress.org/'     => array(
    38                                         'domain'  => 'wordpress.org',
    39                                         'path'    => '/',
    40                                         'site_id' => self::$network_ids['wordpress.org/'],
     38                                        'domain'     => 'wordpress.org',
     39                                        'path'       => '/',
     40                                        'network_id' => self::$network_ids['wordpress.org/'],
    4141                                ),
    4242                                'wordpress.org/foo/' => array(
    43                                         'domain'  => 'wordpress.org',
    44                                         'path'    => '/foo/',
    45                                         'site_id' => self::$network_ids['wordpress.org/'],
     43                                        'domain'     => 'wordpress.org',
     44                                        'path'       => '/foo/',
     45                                        'network_id' => self::$network_ids['wordpress.org/'],
    4646                                ),
    4747                        );
  • trunk/tests/phpunit/tests/multisite/network.php

    r47012 r47013  
    3939                        $sites = array(
    4040                                array(
    41                                         'domain'  => 'wordpress.org',
    42                                         'path'    => '/',
    43                                         'site_id' => self::$different_network_id,
     41                                        'domain'     => 'wordpress.org',
     42                                        'path'       => '/',
     43                                        'network_id' => self::$different_network_id,
    4444                                ),
    4545                                array(
    46                                         'domain'  => 'wordpress.org',
    47                                         'path'    => '/foo/',
    48                                         'site_id' => self::$different_network_id,
     46                                        'domain'     => 'wordpress.org',
     47                                        'path'       => '/foo/',
     48                                        'network_id' => self::$different_network_id,
    4949                                ),
    5050                                array(
    51                                         'domain'  => 'wordpress.org',
    52                                         'path'    => '/bar/',
    53                                         'site_id' => self::$different_network_id,
     51                                        'domain'     => 'wordpress.org',
     52                                        'path'       => '/bar/',
     53                                        'network_id' => self::$different_network_id,
    5454                                ),
    5555                        );
  • trunk/tests/phpunit/tests/multisite/site.php

    r47012 r47013  
    4545                        self::$site_ids = array(
    4646                                'make.wordpress.org/'     => array(
    47                                         'domain'  => 'make.wordpress.org',
    48                                         'path'    => '/',
    49                                         'site_id' => self::$network_ids['make.wordpress.org/'],
     47                                        'domain'     => 'make.wordpress.org',
     48                                        'path'       => '/',
     49                                        'network_id' => self::$network_ids['make.wordpress.org/'],
    5050                                ),
    5151                                'make.wordpress.org/foo/' => array(
    52                                         'domain'  => 'make.wordpress.org',
    53                                         'path'    => '/foo/',
    54                                         'site_id' => self::$network_ids['make.wordpress.org/'],
     52                                        'domain'     => 'make.wordpress.org',
     53                                        'path'       => '/foo/',
     54                                        'network_id' => self::$network_ids['make.wordpress.org/'],
    5555                                ),
    5656                        );
    … …  
    6464                        self::$uninitialized_site_id = wp_insert_site(
    6565                                array(
    66                                         'domain'  => 'uninitialized.org',
    67                                         'path'    => '/',
    68                                         'site_id' => self::$network_ids['make.wordpress.org/'],
     66                                        'domain'     => 'uninitialized.org',
     67                                        'path'       => '/',
     68                                        'network_id' => self::$network_ids['make.wordpress.org/'],
    6969                                )
    7070                        );
  • trunk/tests/phpunit/tests/multisite/siteQuery.php

    r47012 r47013  
    5050                        self::$site_ids = array(
    5151                                'wordpress.org/'          => array(
    52                                         'domain'  => 'wordpress.org',
    53                                         'path'    => '/',
    54                                         'site_id' => self::$network_ids['wordpress.org/'],
     52                                        'domain'     => 'wordpress.org',
     53                                        'path'       => '/',
     54                                        'network_id' => self::$network_ids['wordpress.org/'],
    5555                                ),
    5656                                'wordpress.org/foo/'      => array(
    57                                         'domain'  => 'wordpress.org',
    58                                         'path'    => '/foo/',
    59                                         'site_id' => self::$network_ids['wordpress.org/'],
     57                                        'domain'     => 'wordpress.org',
     58                                        'path'       => '/foo/',
     59                                        'network_id' => self::$network_ids['wordpress.org/'],
    6060                                ),
    6161                                'wordpress.org/foo/bar/'  => array(
    62                                         'domain'  => 'wordpress.org',
    63                                         'path'    => '/foo/bar/',
    64                                         'site_id' => self::$network_ids['wordpress.org/'],
     62                                        'domain'     => 'wordpress.org',
     63                                        'path'       => '/foo/bar/',
     64                                        'network_id' => self::$network_ids['wordpress.org/'],
    6565                                ),
    6666                                'make.wordpress.org/'     => array(
    67                                         'domain'  => 'make.wordpress.org',
    68                                         'path'    => '/',
    69                                         'site_id' => self::$network_ids['make.wordpress.org/'],
     67                                        'domain'     => 'make.wordpress.org',
     68                                        'path'       => '/',
     69                                        'network_id' => self::$network_ids['make.wordpress.org/'],
    7070                                ),
    7171                                'make.wordpress.org/foo/' => array(
    72                                         'domain'  => 'make.wordpress.org',
    73                                         'path'    => '/foo/',
    74                                         'site_id' => self::$network_ids['make.wordpress.org/'],
     72                                        'domain'     => 'make.wordpress.org',
     73                                        'path'       => '/foo/',
     74                                        'network_id' => self::$network_ids['make.wordpress.org/'],
    7575                                ),
    7676                                'www.w.org/'              => array(
    … …  
    8787                                ),
    8888                                'www.w.org/make/'         => array(
    89                                         'domain' => 'www.w.org',
    90                                         'path'   => '/make/',
    91                                         'meta'   => array(
    92                                                 'public'  => 1,
    93                                                 'lang_id' => 1,
    94                                         ),
     89                                        'domain'  => 'www.w.org',
     90                                        'path'    => '/make/',
     91                                        'public'  => 1,
     92                                        'lang_id' => 1,
    9593                                ),
    9694                        );
  • trunk/tests/phpunit/tests/multisite/wpGetSites.php

    r47012 r47013  
    1414                        self::$site_ids = array(
    1515                                'w.org/'      => array(
    16                                         'domain'  => 'w.org',
    17                                         'path'    => '/',
    18                                         'site_id' => 2,
     16                                        'domain'     => 'w.org',
     17                                        'path'       => '/',
     18                                        'network_id' => 2,
    1919                                ),
    2020                                'wp.org/'     => array(
    21                                         'domain'  => 'wp.org',
    22                                         'path'    => '/',
    23                                         'site_id' => 2,
    24                                         'meta'    => array( 'public' => 0 ),
     21                                        'domain'     => 'wp.org',
     22                                        'path'       => '/',
     23                                        'network_id' => 2,
     24                                        'public'     => 0,
    2525                                ),
    2626                                'wp.org/foo/' => array(
    27                                         'domain'  => 'wp.org',
    28                                         'path'    => '/foo/',
    29                                         'site_id' => 1,
    30                                         'meta'    => array( 'public' => 0 ),
     27                                        'domain'     => 'wp.org',
     28                                        'path'       => '/foo/',
     29                                        'network_id' => 1,
     30                                        'public'     => 0,
    3131                                ),
    3232                                'wp.org/oof/' => array(
  • trunk/tests/phpunit/tests/multisite/wpmuValidateBlogSignup.php

    r47012 r47013  
    3535                        self::$existing_blog_id = $factory->blog->create(
    3636                                array(
    37                                         'domain'  => $domain,
    38                                         'path'    => $path,
    39                                         'site_id' => $network->id,
     37                                        'domain'     => $domain,
     38                                        'path'       => $path,
     39                                        'network_id' => $network->id,
    4040                                )
    4141                        );
  • trunk/tests/phpunit/tests/option/multisite.php

    r46586 r47013  
    105105                                array(
    106106                                        'user_id' => $user_id,
    107                                         'meta'    => array(
    108                                                 'public' => 1,
    109                                         ),
     107                                        'public'  => 1,
    110108                                )
    111109                        );
  • trunk/tests/phpunit/tests/user/getActiveBlogForUser.php

    r47012 r47013  
    8484                                array(
    8585                                        'user_id' => self::$user_id,
    86                                         'meta'    => array( 'spam' => 1 ),
     86                                        'spam'    => 1,
    8787                                )
    8888                        );
  • trunk/tests/phpunit/tests/user/multisite.php

    r47012 r47013  
    123123                        $this->assertTrue( is_blog_user( get_current_blog_id() ) );
    124124
    125                         $blog_ids = array();
    126 
    127                         $blog_ids = self::factory()->blog->create_many( 1 );
    128                         foreach ( $blog_ids as $blog_id ) {
    129                                 $this->assertInternalType( 'int', $blog_id );
    130                                 $this->assertTrue( is_blog_user( $blog_id ) );
    131                                 $this->assertTrue( remove_user_from_blog( $user1_id, $blog_id ) );
    132                                 $this->assertFalse( is_blog_user( $blog_id ) );
    133                         }
     125                        $blog_id = self::factory()->blog->create( array( 'user_id' => get_current_user_id() ) );
     126
     127                        $this->assertInternalType( 'int', $blog_id );
     128                        $this->assertTrue( is_blog_user( $blog_id ) );
     129                        $this->assertTrue( remove_user_from_blog( $user1_id, $blog_id ) );
     130                        $this->assertFalse( is_blog_user( $blog_id ) );
    134131
    135132                        wp_set_current_user( $old_current );
    … …  
    158155                        $this->assertTrue( is_user_member_of_blog( $user1_id, $site_id ) );
    159156
    160                         $blog_ids = self::factory()->blog->create_many( 1 );
    161                         foreach ( $blog_ids as $blog_id ) {
    162                                 $this->assertInternalType( 'int', $blog_id );
    163 
    164                                 // Current user gets added to new blogs
    165                                 $this->assertTrue( is_user_member_of_blog( $user1_id, $blog_id ) );
    166                                 // Other users should not
    167                                 $this->assertFalse( is_user_member_of_blog( $user2_id, $blog_id ) );
    168 
    169                                 switch_to_blog( $blog_id );
    170 
    171                                 $this->assertTrue( is_user_member_of_blog( $user1_id ) );
    172                                 $this->assertFalse( is_user_member_of_blog( $user2_id ) );
    173 
    174                                 // Remove user 1 from blog
    175                                 $this->assertTrue( remove_user_from_blog( $user1_id, $blog_id ) );
    176 
    177                                 // Add user 2 to blog
    178                                 $this->assertTrue( add_user_to_blog( $blog_id, $user2_id, 'subscriber' ) );
    179 
    180                                 $this->assertFalse( is_user_member_of_blog( $user1_id ) );
    181                                 $this->assertTrue( is_user_member_of_blog( $user2_id ) );
    182 
    183                                 restore_current_blog();
    184 
    185                                 $this->assertFalse( is_user_member_of_blog( $user1_id, $blog_id ) );
    186                                 $this->assertTrue( is_user_member_of_blog( $user2_id, $blog_id ) );
    187                         }
     157                        $blog_id = self::factory()->blog->create( array( 'user_id' => get_current_user_id() ) );
     158
     159                        $this->assertInternalType( 'int', $blog_id );
     160
     161                        // Current user gets added to new blogs
     162                        $this->assertTrue( is_user_member_of_blog( $user1_id, $blog_id ) );
     163                        // Other users should not
     164                        $this->assertFalse( is_user_member_of_blog( $user2_id, $blog_id ) );
     165
     166                        switch_to_blog( $blog_id );
     167
     168                        $this->assertTrue( is_user_member_of_blog( $user1_id ) );
     169                        $this->assertFalse( is_user_member_of_blog( $user2_id ) );
     170
     171                        // Remove user 1 from blog
     172                        $this->assertTrue( remove_user_from_blog( $user1_id, $blog_id ) );
     173
     174                        // Add user 2 to blog
     175                        $this->assertTrue( add_user_to_blog( $blog_id, $user2_id, 'subscriber' ) );
     176
     177                        $this->assertFalse( is_user_member_of_blog( $user1_id ) );
     178                        $this->assertTrue( is_user_member_of_blog( $user2_id ) );
     179
     180                        restore_current_blog();
     181
     182                        $this->assertFalse( is_user_member_of_blog( $user1_id, $blog_id ) );
     183                        $this->assertTrue( is_user_member_of_blog( $user2_id, $blog_id ) );
    188184
    189185                        wpmu_delete_user( $user1_id );
Note: See TracChangeset for help on using the changeset viewer.