CRM-16473 test fixes to support addition of User api & uf_match entry to createLogged...

diff --git a/tests/phpunit/CiviTest/CiviCaseTestCase.php b/tests/phpunit/CiviTest/CiviCaseTestCase.php
index 73fb4eef4b1ef66e3a466772fb9a97f38eeb67b6..af9a22624a79bd667f0747f5a0645e232293a750 100644 (file)
--- a/tests/phpunit/CiviTest/CiviCaseTestCase.php
+++ b/tests/phpunit/CiviTest/CiviCaseTestCase.php
@@ -99,6 +99,7 @@ class CiviCaseTestCase extends CiviUnitTestCase {
       'civicrm_managed',
       'civicrm_relationship',
       'civicrm_relationship_type',
+      'civicrm_uf_match',
     );
 
     $this->quickCleanup($this->tablesToTruncate);

diff --git a/tests/phpunit/CiviTest/CiviUnitTestCase.php b/tests/phpunit/CiviTest/CiviUnitTestCase.php
index eadb9f2436d08478319483e59cc7f1975d56f5b1..646cd763ea5f3b3d9a0130d400371729eb90c7a9 100755 (executable)
--- a/tests/phpunit/CiviTest/CiviUnitTestCase.php
+++ b/tests/phpunit/CiviTest/CiviUnitTestCase.php
@@ -534,7 +534,7 @@ class CiviUnitTestCase extends PHPUnit_Extensions_Database_TestCase {
       CRM_Core_Transaction::forceRollbackIfEnabled();
       \Civi\Core\Transaction\Manager::singleton(TRUE);
 
-      $tablesToTruncate = array('civicrm_contact');
+      $tablesToTruncate = array('civicrm_contact', 'civicrm_uf_match');
       $this->quickCleanup($tablesToTruncate);
       $this->createDomainContacts();
     }

diff --git a/tests/phpunit/api/v3/ACLPermissionTest.php b/tests/phpunit/api/v3/ACLPermissionTest.php
index 6d54ed3d433fb7145ae4ce7d2a8000023545ae34..0d81bbe6e661502c32d591a9bf8e4b6b7a2cc01e 100644 (file)
--- a/tests/phpunit/api/v3/ACLPermissionTest.php
+++ b/tests/phpunit/api/v3/ACLPermissionTest.php
@@ -63,6 +63,7 @@ class api_v3_ACLPermissionTest extends CiviUnitTestCase {
       'civicrm_acl_contact_cache',
       'civicrm_contribution',
       'civicrm_participant',
+      'civicrm_uf_match',
     );
     $this->quickCleanup($tablesToTruncate);
     $config = CRM_Core_Config::singleton();

diff --git a/tests/phpunit/api/v3/ActivityTest.php b/tests/phpunit/api/v3/ActivityTest.php
index 0236aed7c812ca3ad50e6422306b946541754cd7..454ceafb85f9b8711a9b00a519fd4ea54e604355 100644 (file)
--- a/tests/phpunit/api/v3/ActivityTest.php
+++ b/tests/phpunit/api/v3/ActivityTest.php
@@ -104,6 +104,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'civicrm_contact',
       'civicrm_activity',
       'civicrm_activity_contact',
+      'civicrm_uf_match',
     );
     $this->quickCleanup($tablesToTruncate, TRUE);
     $this->callAPISuccess('option_value', 'delete', array('id' => $this->test_activity_type_id));

diff --git a/tests/phpunit/api/v3/CaseTest.php b/tests/phpunit/api/v3/CaseTest.php
index 0d6d7ac0b196d67465aa749cb7849b90274c6c30..14f8744c6939702fd0cdf7e390fc2af0de607ec4 100644 (file)
--- a/tests/phpunit/api/v3/CaseTest.php
+++ b/tests/phpunit/api/v3/CaseTest.php
@@ -93,7 +93,7 @@ class api_v3_CaseTest extends CiviCaseTestCase {
       'case_type_id' => 1,
     );
 
-    $result = $this->callAPIFailure('case', 'create', $params);
+    $this->callAPIFailure('case', 'create', $params);
   }
 
   /**

diff --git a/tests/phpunit/api/v3/CaseTypeTest.php b/tests/phpunit/api/v3/CaseTypeTest.php
index 48a5af5ca63bbfe59a92dd69eec9562351328eea..ef9a5a5088f59385dec376f6befacae5e73364a3 100644 (file)
--- a/tests/phpunit/api/v3/CaseTypeTest.php
+++ b/tests/phpunit/api/v3/CaseTypeTest.php
@@ -69,7 +69,7 @@ class api_v3_CaseTypeTest extends CiviCaseTestCase {
    */
   public function tearDown() {
     parent::tearDown();
-    $this->quickCleanup(array('civicrm_case_type'));
+    $this->quickCleanup(array('civicrm_case_type', 'civicrm_uf_match'));
   }
 
   /**

diff --git a/tests/phpunit/api/v3/ContributionRecurTest.php b/tests/phpunit/api/v3/ContributionRecurTest.php
index 07445a715cc110c8c900b1f3760fc93fe5bf9a1f..d7f77b5aa148beafc8c27c16182b0a852471ae7d 100644 (file)
--- a/tests/phpunit/api/v3/ContributionRecurTest.php
+++ b/tests/phpunit/api/v3/ContributionRecurTest.php
@@ -66,7 +66,7 @@ class api_v3_ContributionRecurTest extends CiviUnitTestCase {
   }
 
   public function testGetContributionRecur() {
-    $result = $this->callAPISuccess($this->_entity, 'create', $this->params);
+    $this->callAPISuccess($this->_entity, 'create', $this->params);
     $getParams = array(
       'amount' => '500',
     );
@@ -93,7 +93,7 @@ class api_v3_ContributionRecurTest extends CiviUnitTestCase {
   public function testDeleteContributionRecur() {
     $result = $this->callAPISuccess($this->_entity, 'create', $this->params);
     $deleteParams = array('id' => $result['id']);
-    $result = $this->callAPIAndDocument($this->_entity, 'delete', $deleteParams, __FUNCTION__, __FILE__);
+    $this->callAPIAndDocument($this->_entity, 'delete', $deleteParams, __FUNCTION__, __FILE__);
     $checkDeleted = $this->callAPISuccess($this->_entity, 'get', array());
     $this->assertEquals(0, $checkDeleted['count']);
   }

diff --git a/tests/phpunit/api/v3/ContributionTest.php b/tests/phpunit/api/v3/ContributionTest.php
index 5c5fc60f39c5de0360a061b1ade78aab055d35ce..134d8128e845388ec782d7e62f48af12a7afcc69 100644 (file)
--- a/tests/phpunit/api/v3/ContributionTest.php
+++ b/tests/phpunit/api/v3/ContributionTest.php
@@ -113,6 +113,7 @@ class api_v3_ContributionTest extends CiviUnitTestCase {
    */
   public function tearDown() {
     $this->quickCleanUpFinancialEntities();
+    $this->quickCleanup(array('civicrm_uf_match'));
   }
 
   /**

diff --git a/tests/phpunit/api/v3/ProfileTest.php b/tests/phpunit/api/v3/ProfileTest.php
index 3815b22bcc70163f9f0576cee5d9afe0d2c80b41..6b6dd6b18923f22a8cbb16fcee1863ed6f938e6b 100644 (file)
--- a/tests/phpunit/api/v3/ProfileTest.php
+++ b/tests/phpunit/api/v3/ProfileTest.php
@@ -59,6 +59,7 @@ class api_v3_ProfileTest extends CiviUnitTestCase {
       'civicrm_address',
       'civicrm_membership',
       'civicrm_contribution',
+      'civicrm_uf_match',
     ), TRUE);
     $this->callAPISuccess('membership_type', 'delete', array('id' => $this->_membershipTypeID));
     // ok can't be bothered wring an api to do this & truncating is crazy

diff --git a/tests/phpunit/api/v3/SyntaxConformanceTest.php b/tests/phpunit/api/v3/SyntaxConformanceTest.php
index 3dc5663db9a51328b02df645857b0a9264aef5b6..1c0a6e264c38036b029cab320ca536d9cfb90202 100644 (file)
--- a/tests/phpunit/api/v3/SyntaxConformanceTest.php
+++ b/tests/phpunit/api/v3/SyntaxConformanceTest.php
@@ -102,6 +102,7 @@ class api_v3_SyntaxConformanceTest extends CiviUnitTestCase {
       'Extension',
       'ReportTemplate',
       'System',
+      'User',
     );
     $this->toBeImplemented['delete'] = array(
       'MembershipPayment',
@@ -118,6 +119,7 @@ class api_v3_SyntaxConformanceTest extends CiviUnitTestCase {
       'Entity',
       'Domain',
       'Setting',
+      'User',
     );
     $this->deprecatedAPI = array('Location', 'ActivityType', 'SurveyRespondant');
     $this->deletableTestObjects = array();
@@ -278,8 +280,10 @@ class api_v3_SyntaxConformanceTest extends CiviUnitTestCase {
    *
    * Mailing Contact Just doesn't support id. We have always insisted on finding a way to
    * support id in API but in this case the underlying tables are crying out for a restructure
-   * & it just doesn't make sense, on the otherhand Event need id to be existant as pseudo property
-   * is been associated with it, so we need to bypass for get api otherwise it will through pseudo_match validation
+   * & it just doesn't make sense.
+   *
+   * User doesn't support get By ID because the user id is actually the CMS user ID & is not part of
+   *   CiviCRM - so can only be tested through UserTest - not SyntaxConformanceTest.
    *
    * @param bool $sequential
    *
@@ -287,7 +291,7 @@ class api_v3_SyntaxConformanceTest extends CiviUnitTestCase {
    *   Entities that cannot be retrieved by ID
    */
   public static function toBeSkipped_getByID($sequential = FALSE) {
-    return array('MailingContact', 'Event');
+    return array('MailingContact', 'User');
   }
 
   /**
@@ -327,6 +331,7 @@ class api_v3_SyntaxConformanceTest extends CiviUnitTestCase {
       'Profile',
       'CustomValue',
       'Setting',
+      'User',
     );
     if ($sequential === TRUE) {
       return $entitiesWithout;