Changeset 47122 for trunk/tests/phpunit/tests/user/query.php

Timestamp:

01/29/2020 12:43:23 AM (7 years ago)

Author:

SergeyBiryukov

Message:

Docs: Improve inline comments per the documentation standards.

Includes minor code layout fixes for better readability.

See #48303.

File:

• trunk/tests/phpunit/tests/user/query.php (modified) (5 diffs)

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

@@ -555 +555 @@
                 $this->assertNotEquals( $_query_vars, $query->query_vars );
 
-                // All values get reset
+                // All values get reset.
                 $query->prepare_query( array( 'number' => 8 ) );
                 $this->assertNotEmpty( $query->query_limit );
                 $this->assertEquals( 'LIMIT 0, 8', $query->query_limit );
 
-                // All values get reset
+                // All values get reset.
                 $query->prepare_query( array( 'fields' => 'all' ) );
                 $this->assertEmpty( $query->query_limit );
@@ -1140 +1140 @@
 
                 $found_count    = count( $q->get_results() );
-                $expected_count = 10; // 13 total users minus 3 from query
+                $expected_count = 10; // 13 total users minus 3 from query.
 
                 $this->assertContains( "AND user_nicename NOT IN ( 'peter','paul','mary' )", $q->query_where );
@@ -1241 +1241 @@
 
                 $found_count    = count( $q->get_results() );
-                $expected_count = 10; // 13 total users minus 3 from query
+                $expected_count = 10; // 13 total users minus 3 from query.
 
                 $this->assertContains( "AND user_login NOT IN ( '$user_login1','$user_login2','$user_login3' )", $q->query_where );
@@ -1444 +1444 @@
          */
         public function test_get_multiple_roles_with_meta() {
-                // Create administrator user + meta
+                // Create administrator user + meta.
                 update_user_meta( self::$admin_ids[0], 'mk1', 1 );
                 update_user_meta( self::$admin_ids[0], 'mk2', 1 );
 
-                // Create editor user + meta
+                // Create editor user + meta.
                 update_user_meta( self::$editor_ids[0], 'mk1', 1 );
                 update_user_meta( self::$editor_ids[0], 'mk2', 2 );
 
-                // Create subscriber user + meta
+                // Create subscriber user + meta.
                 update_user_meta( self::$sub_ids[0], 'mk1', 1 );
                 update_user_meta( self::$sub_ids[0], 'mk2', 1 );
 
-                // Create contributor user + meta
+                // Create contributor user + meta.
                 update_user_meta( self::$contrib_id, 'mk1', 1 );
                 update_user_meta( self::$contrib_id, 'mk2', 2 );
 
-                // Fetch users
+                // Fetch users.
                 $users = get_users(
                         array(
@@ -1482 +1482 @@
                 );
 
-                // Check results
+                // Check results.
                 $this->assertEquals( 1, count( $users ) );
                 $this->assertSame( self::$editor_ids[0], (int) $users[0]->ID );