Changeset 13358 for trunk/tests/phpunit/testcases/xprofile/class-bp-xprofile-group.php

Timestamp:

11/08/2022 06:27:01 AM (4 years ago)

Author:

imath

Message:

xProfile: allow the loop to fetch fields from multiple fields group

You can now pass an array of fields group ID into the $profile_group_id
parameter.

Props boonebgorges, espellcaste

Closes ​https://github.com/buddypress/buddypress/pull/30
Fixes #4075

File:

• trunk/tests/phpunit/testcases/xprofile/class-bp-xprofile-group.php (modified) (3 diffs)

trunk/tests/phpunit/testcases/xprofile/class-bp-xprofile-group.php

@@ -132 +132 @@
 
         /**
+         * @group get_xprofile_groups
+         * @group BP4075
+         */
+        public function test_get_specific_xprofile_groups() {
+                $g1 = self::factory()->xprofile_group->create();
+                $g2 = self::factory()->xprofile_group->create();
+                $g3 = self::factory()->xprofile_group->create();
+                $e1 = [ $g1, $g2 ];
+
+                // Comma-separated list of profile group ids.
+                $groups1 = BP_XProfile_Group::get( [ 'profile_group_id' => join( ',', $e1 ) ] );
+
+                $this->assertSame( $e1, array_map( 'absint', wp_list_pluck( $groups1, 'id' ) ) );
+
+                // Array of profile group ids.
+                $groups2 = BP_XProfile_Group::get( [ 'profile_group_id' => $e1 ] );
+
+                $this->assertSame( $e1, array_map( 'absint', wp_list_pluck( $groups2, 'id' ) ) );
+        }
+
+        /**
          * @group member_types
          * @ticket BP5192
@@ -444 +465 @@
 
         /**
-         * @group BP7435
-         * @group cache
-         */
-        public function test_group_ids_query_should_be_cached() {
-                global $wpdb;
-
-                $group_ids   = array( 1 ); // Default group.
+         * @group BP4075
+         */
+        public function test_group_ids_query() {
+                $g1 = self::factory()->xprofile_group->create();
+                $g2 = self::factory()->xprofile_group->create();
+
+                $group_ids   = [ 1 ]; // Default group.
                 $group_ids[] = self::factory()->xprofile_group->create();
                 $group_ids[] = self::factory()->xprofile_group->create();
                 $group_ids[] = self::factory()->xprofile_group->create();
-
-                $params_1 = array(
-                        'exclude_groups' => false,
-                );
-
-                $params_2 = array(
-                        'exclude_groups' => array( 0 ),
-                );
+                $group_ids[] = $g1;
+                $group_ids[] = $g2;
+
+                $found_1 = BP_XProfile_Group::get_group_ids();
+                $this->assertEqualSets( $group_ids, $found_1 );
+
+                $found_2 = BP_XProfile_Group::get_group_ids( [ 'profile_group_id' => $g1 ] );
+                $this->assertCount( 1, $found_2 );
+                $this->assertSame( [ $g1 ], $found_2 );
+
+                $found_3 = BP_XProfile_Group::get_group_ids( [ 'profile_group_id' => [ $g1 ] ] );
+                $this->assertCount( 1, $found_3 );
+                $this->assertSame( [ $g1 ], $found_3 );
+
+                $found_4 = BP_XProfile_Group::get_group_ids( [ 'profile_group_id' => [ $g2 ] ] );
+                $this->assertCount( 1, $found_4 );
+                $this->assertSame( [ $g2 ], $found_4 );
+
+                $found_5 = BP_XProfile_Group::get_group_ids( [ 'profile_group_id' => [ $g1, $g2 ] ] );
+                $this->assertCount( 2, $found_5 );
+                $this->assertSame( [ $g1, $g2 ], $found_5 );
+
+                $found_6 = BP_XProfile_Group::get_group_ids( [ 'profile_group_id' => join( ',', [ $g1, $g2 ] ) ] );
+                $this->assertCount( 2, $found_6 );
+                $this->assertSame( [ $g1, $g2 ], $found_6 );
+
+                $found_7 = BP_XProfile_Group::get_group_ids( [ 'profile_group_id' => true ] );
+                $this->assertEqualSets( $group_ids, $found_7 );
+        }
+
+        /**
+         * @group BP7435
+         * @group cache
+         */
+        public function test_group_ids_query_should_be_cached() {
+                global $wpdb;
+
+                $group_ids   = array( 1 ); // Default group.
+                $g1          = self::factory()->xprofile_group->create();
+                $g2          = self::factory()->xprofile_group->create();
+                $group_ids[] = self::factory()->xprofile_group->create();
+                $group_ids[] = self::factory()->xprofile_group->create();
+                $group_ids[] = self::factory()->xprofile_group->create();
+                $group_ids[] = $g1;
+                $group_ids[] = $g2;
+
+                $params_1 = [ 'exclude_groups' => false ];
 
                 // Prime cache.
@@ -469 +529 @@
                 $num_queries = $wpdb->num_queries;
 
-                $found_1 = BP_XProfile_Group::get_group_ids( $params_1 );
-                $this->assertEqualSets( $group_ids, $found_1 );
+                $found_2 = BP_XProfile_Group::get_group_ids( $params_1 );
+                $this->assertEqualSets( $group_ids, $found_2 );
                 $this->assertSame( $num_queries, $wpdb->num_queries );
 
                 // Different parameters should trigger a cache miss.
-                $found_2 = BP_XProfile_Group::get_group_ids( $params_2 );
-                $this->assertEqualSets( $group_ids, $found_2 );
+                $found_3 = BP_XProfile_Group::get_group_ids( [ 'exclude_groups' => [ 0 ] ] );
+                $this->assertEqualSets( $group_ids, $found_3 );
+                $this->assertNotSame( $num_queries, $wpdb->num_queries );
+
+                // Again, different parameters should trigger a cache miss.
+                $found_4 = BP_XProfile_Group::get_group_ids( [ 'profile_group_id' => [ $g1, $g2 ] ] );
+                $this->assertEqualSets( [ $g1, $g2 ], $found_4 );
                 $this->assertNotSame( $num_queries, $wpdb->num_queries );
         }