Merge pull request #15546 from eileenmcnaughton/group_type

dev/core#1321 [test] add unit test to demonstrate behaviour
civicrm · Oct 24, 2019 · 97a7308 · 97a7308
2 parents 2f08eb6 + 4f92030
commit 97a7308
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/tests/phpunit/api/v3/GroupTest.php b/tests/phpunit/api/v3/GroupTest.php
@@ -273,6 +273,22 @@ public function testGroupTypeWithPseudoconstantArray() {
     $this->assertAPIArrayComparison([2, 1], $groupType);
   }
 
+  /**
+   * Test / demonstrate behaviour when attempting to filter by group_type.
+   *
+   * Per https://lab.civicrm.org/dev/core/issues/1321 the group_type filter is deceptive
+   * - it only filters on exact match not 'is one of'.
+   *
+   * @throws \CRM_Core_Exception
+   */
+  public function testGroupWithGroupTypeFilter() {
+    $this->groupCreate(['group_type' => ['Access Control'], 'name' => 'access_list', 'title' => 'access list']);
+    $this->groupCreate(['group_type' => ['Mailing List'], 'name' => 'mailing_list', 'title' => 'mailing list']);
+    $this->groupCreate(['group_type' => ['Access Control', 'Mailing List'], 'name' => 'group', 'title' => 'group']);
+    $group = $this->callAPISuccessGetSingle('Group', ['return' => 'id,title,group_type', 'group_type' => 'Mailing List']);
+    $this->assertEquals('mailing list', $group['title']);
+  }
+
   /**
    * @param int $version
    *