Make WordPress Core

Changeset 47013


Ignore:
Timestamp:
12/25/2019 09:39:46 PM (5 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.