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

Timestamp:

01/29/2020 12:43:23 AM (5 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 );