a few phpcs fixups
authorEileen McNaughton <eileen@fuzion.co.nz>
Tue, 3 Feb 2015 06:40:20 +0000 (19:40 +1300)
committerEileen McNaughton <eileen@fuzion.co.nz>
Tue, 3 Feb 2015 06:40:20 +0000 (19:40 +1300)
24 files changed:
tests/phpunit/api/v3/APIStandardsTest.php [deleted file]
tests/phpunit/api/v3/APITest.php
tests/phpunit/api/v3/ActivityTest.php
tests/phpunit/api/v3/ActivityTypeTest.php
tests/phpunit/api/v3/AllTests.php
tests/phpunit/api/v3/BatchTest.php
tests/phpunit/api/v3/CRM11793Test.php
tests/phpunit/api/v3/CaseTest.php
tests/phpunit/api/v3/ConstantTest.php
tests/phpunit/api/v3/ContactTest.php
tests/phpunit/api/v3/ContributionSoftTest.php
tests/phpunit/api/v3/ContributionTest.php
tests/phpunit/api/v3/CustomFieldTest.php
tests/phpunit/api/v3/DashboardContactTest.php
tests/phpunit/api/v3/DashboardTest.php
tests/phpunit/api/v3/EventTest.php
tests/phpunit/api/v3/GroupOrganizationTest.php
tests/phpunit/api/v3/MembershipStatusTest.php
tests/phpunit/api/v3/MembershipTest.php
tests/phpunit/api/v3/OptionValueTest.php
tests/phpunit/api/v3/PaymentProcessorTypeTest.php
tests/phpunit/api/v3/RelationshipTypeTest.php
tests/phpunit/api/v3/UFJoinTest.php
tests/phpunit/api/v3/UFMatchTest.php

diff --git a/tests/phpunit/api/v3/APIStandardsTest.php b/tests/phpunit/api/v3/APIStandardsTest.php
deleted file mode 100644 (file)
index e8b5ff0..0000000
+++ /dev/null
@@ -1,178 +0,0 @@
-<?php
-// vim: set si ai expandtab tabstop=4 shiftwidth=4 softtabstop=4:
-
-/**
- *  File to v3 APIs for Standards compliance
- *
- *  (PHP 5)
- *
- * @copyright Copyright CiviCRM LLC (C) 2009
- * @license   http://www.fsf.org/licensing/licenses/agpl-3.0.html
- *              GNU Affero General Public License version 3
- * @version   $Id:
- * @package   CiviCRM
- *
- *   This file is part of CiviCRM
- *
- *   CiviCRM is free software; you can redistribute it and/or
- *   modify it under the terms of the GNU Affero General Public License
- *   as published by the Free Software Foundation; either version 3 of
- *   the License, or (at your option) any later version.
- *
- *   CiviCRM is distributed in the hope that it will be useful,
- *   but WITHOUT ANY WARRANTY; without even the implied warranty of
- *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *   GNU Affero General Public License for more details.
- *
- *   You should have received a copy of the GNU Affero General Public
- *   License along with this program.  If not, see
- *   <http://www.gnu.org/licenses/>.
- */
-
-/**
- *  Include class definitions
- */
-require_once 'CiviTest/CiviUnitTestCase.php';
-
-/**
- *  Test APIv3 civicrm_activity_* functions
- *
- * @package CiviCRM_APIv3
- * @todo determine where help functions should sit (here or 'up the tree'), & best way to define API dir
- */
-class api_v3_APIStandardsTest extends CiviUnitTestCase {
-
-  protected $_apiversion;
-  protected $_apiDir;
-  protected $_functionFiles;
-  protected $_regexForGettingAPIStdFunctions;
-  /* This test case doesn't require DB reset */
-
-
-  public $DBResetRequired = FALSE;
-
-  /**
-   *  Test setup for every test
-   *
-   *  Connect to the database, truncate the tables that will be used
-   *  and redirect stdin to a temporary file
-   */
-  public function setUp() {
-    //  Connect to the database
-    parent::setUp();
-    $this->useTransaction(TRUE);
-    $this->_apiversion = 3;
-    $this->_apiDir = "../api/v3/";
-    $this->_functionFiles = array('Entity.php', 'utils.php');
-
-    //should possibly insert variable rather than '3' in below
-    $this->_regexForGettingAPIStdFunctions = '/^civicrm_api3_  #starts with civicrm_api_3 (ignore internal functions)
-                                                  .*_              #any number of characters up to the last _
-    # negative look ahead on string getfields
-                                                  (?:(?!getfields))/x';
-    // functions to skip from utils.php mainly since they get sucked in via
-    // a require chain in the include files
-    $this->_skipFunctionList = array(
-      // location api is deprecated
-      'civicrm_api3_location_create',
-      'civicrm_api3_location_get',
-      'civicrm_api3_location_delete',
-      // most of these seem to be internal functions
-      'civicrm_api3_entity_create',
-      'civicrm_api3_entity_delete',
-      'civicrm_api3_survey_respondant_count',
-      // functions from utils.php
-      'civicrm_api3_verify_one_mandatory',
-      'civicrm_api3_verify_mandatory',
-      'civicrm_api3_get_dao',
-      'civicrm_api3_create_success',
-      'civicrm_api3_create_error',
-      'civicrm_api3_error',
-      'civicrm_api3_check_contact_dedupe',
-      'civicrm_api3_api_check_permission',
-    );
-  }
-
-  /**
-   * Test checks that all v3 API return a standardised error message when
-   * the $params passed in is not an array.
-   */
-  public function testParamsNotArray() {
-    /* @codingStandardsIgnoreStart
-     * I have commented this out as the check for is_array has been moved to civicrm_api. But keeping in place as
-     * this test, in contrast to the standards test, tests all existing API rather than just CRUD ones
-     * so want to keep code for re-use
-        $files = $this->getAllFilesinAPIDir();
-        $this->assertGreaterThan(1, count($files),"something has gone wrong listing the files in line " . __LINE__);
-        $this->requireOnceFilesArray($files);
-        $apiStdFunctions = $this->getAllAPIStdFunctions();
-        $this->assertGreaterThan(1, count($apiStdFunctions),"something has gone wrong getting the std functions in line " . __LINE__);
-        $params = 'string';
-        foreach ($apiStdFunctions as $key => $function) {
-            if ( in_array( $function, $this->_skipFunctionList ) ) {
-                continue;
-            }
-            try {
-                $result = $function($params);
-            }
-            catch ( Exception $e ) {
-              continue;
-            }
-
-        }
-    @codingStandardsIgnoreEnd */
-  }
-
-  /**
-   * Get all the files in the API directory for the relevant version which contain API functions
-   * @return array
-   *   array of php files in the directory excluding helper files
-   */
-  public function getAllFilesinAPIDir() {
-    $files = array();
-    $handle = opendir($this->_apiDir);
-
-    while (($file = readdir($handle)) !== FALSE) {
-      if (strstr($file, ".php") &&
-        $file != 'Entity.php' &&
-        $file != 'Generic.php' &&
-        $file != 'utils.php'
-      ) {
-        $files[] = $file;
-      }
-    }
-
-    closedir($handle);
-    return $files;
-  }
-
-  /**
-   * Require once  Files
-   * @param array $files
-   *   list of files to load.
-   */
-  public function requireOnceFilesArray($files) {
-    foreach ($files as $key => $file) {
-      require_once $this->_apiDir . $file;
-    }
-  }
-
-  /**
-   * Get all api exposed functions that are expected to conform to standards
-   * @return array
-   */
-  public function getAllAPIStdFunctions() {
-    $functionlist = get_defined_functions();
-    $functions = preg_grep($this->_regexForGettingAPIStdFunctions, $functionlist['user']);
-    foreach ($functions as $key => $function) {
-      if (stristr($function, 'getfields')) {
-        unset($functions[$key]);
-      }
-      if (stristr($function, '_generic_')) {
-        unset($functions[$key]);
-      }
-    }
-    return $functions;
-  }
-
-}
index 665f8d588d04961eccb371ea8f91d31f0534ddc2..5c21a9d55af3272b6ab4972a19bd38736ff6b187 100644 (file)
@@ -69,11 +69,15 @@ class api_v3_APITest extends CiviUnitTestCase {
   }
 
   /**
-   * test that error doesn't occur for non-existant file
+   * Test that error doesn't occur for non-existent file.
    */
   public function testAPIWrapperIncludeNoFile() {
-
-    $result = $this->callAPIFailure('RandomFile', 'get', array(), 'API (RandomFile,get) does not exist (join the API team and implement it!)');
+    $this->callAPIFailure(
+      'RandomFile',
+      'get',
+      array(),
+      'API (RandomFile,get) does not exist (join the API team and  implement it!)'
+    );
   }
 
   public function testAPIWrapperCamelCaseFunction() {
@@ -133,7 +137,7 @@ class api_v3_APITest extends CiviUnitTestCase {
   }
 
   /**
-   * Test that calling via wrapper works
+   * Test that calling via wrapper works.
    */
   public function testv3Wrapper() {
     try {
@@ -147,11 +151,11 @@ class api_v3_APITest extends CiviUnitTestCase {
   }
 
   /**
-   * Test exception is thrown
+   * Test exception is thrown.
    */
-  public function testv3WrapperException() {
+  public function testV3WrapperException() {
     try {
-      $result = civicrm_api3('contact', 'create', array('debug' => 1));
+      civicrm_api3('contact', 'create', array('debug' => 1));
     }
     catch (CiviCRM_API3_Exception $e) {
       $this->assertEquals('mandatory_missing', $e->getErrorCode());
@@ -163,7 +167,10 @@ class api_v3_APITest extends CiviUnitTestCase {
     $this->fail('Exception was expected');
   }
 
-  public function testCreate_NoStringNullResult() {
+  /**
+   * Test result parsing for null.
+   */
+  public function testCreateNoStringNullResult() {
     // create an example contact
     // $contact = CRM_Core_DAO::createTestObject('CRM_Contribute_DAO_ContributionPage')->toArray();
     $result = $this->callAPISuccess('ContributionPage', 'create', array(
@@ -187,11 +194,12 @@ class api_v3_APITest extends CiviUnitTestCase {
       'currency' => '',
     ));
 
-    // check return format
+    // Check return format.
     $this->assertEquals(1, $result['count']);
     foreach ($result['values'] as $resultValue) {
       $this->assertEquals('New title', $resultValue['title']);
-      $this->assertEquals('', $resultValue['currency']); // BUG: $resultValue['location'] === 'null'
+      // BUG: $resultValue['location'] === 'null'.
+      $this->assertEquals('', $resultValue['currency']);
     }
   }
 
index 7ffdf55fcfc279b02f095fab1d4f54dc7d738d79..d1c75590e249d2a230c1ba35765a8c1be46b29a3 100644 (file)
@@ -1,5 +1,6 @@
 <?php
 /**
+ * @file
  *  File for the TestActivity class
  *
  *  (PHP 5)
@@ -35,7 +36,7 @@ require_once 'CiviTest/CiviUnitTestCase.php';
 
 
 /**
- *  Test APIv3 civicrm_activity_* functions
+ * Test APIv3 civicrm_activity_* functions
  *
  * @package CiviCRM_APIv3
  * @subpackage API_Activity
@@ -49,10 +50,10 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   protected $_contactID;
 
   /**
-   *  Test setup for every test.
+   * Test setup for every test.
    *
-   *  Connect to the database, truncate the tables that will be used
-   *  and redirect stdin to a temporary file
+   * Connect to the database, truncate the tables that will be used
+   * and redirect stdin to a temporary file
    */
   public function setUp() {
     // Connect to the database
@@ -127,8 +128,8 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_create() with mismatched activity_type_id
-   *  and activity_name
+   * Test civicrm_activity_create() with mismatched activity_type_id
+   * and activity_name.
    */
   public function testActivityCreateMismatchNameType() {
     $params = array(
@@ -148,7 +149,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_id() with missing source_contact_id is put with the current user.
+   * Test civicrm_activity_id() with missing source_contact_id is put with the current user.
    */
   public function testActivityCreateWithMissingContactId() {
     $params = array(
@@ -161,11 +162,11 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'activity_name' => 'Test activity type',
     );
 
-    $result = $this->callAPISuccess('activity', 'create', $params);
+    $this->callAPISuccess('activity', 'create', $params);
   }
 
   /**
-   *  Test civicrm_activity_id() with non-numeric source_contact_id.
+   * Test civicrm_activity_id() with non-numeric source_contact_id.
    */
   public function testActivityCreateWithNonNumericContactId() {
     $params = array(
@@ -296,7 +297,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_create() with valid parameters - use type_id.
+   * Test civicrm_activity_create() with valid parameters - use type_id.
    */
   public function testActivityCreateCampaignTypeID() {
     $this->enableCiviCampaign();
@@ -368,7 +369,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_create() with valid parameters and custom data.
+   * Test civicrm_activity_create() with valid parameters and custom data.
    */
   public function testActivityCreateCustom() {
     $ids = $this->entityCustomGroupWithSingleFieldCreate(__FUNCTION__, __FILE__);
@@ -386,7 +387,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_create() with valid parameters and custom data.
+   * Test civicrm_activity_create() with valid parameters and custom data.
    */
   public function testActivityCreateCustomContactRefField() {
 
@@ -488,14 +489,14 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_get() with no params
+   * Test civicrm_activity_get() with no params
    */
   public function testActivityGetEmpty() {
     $result = $this->callAPISuccess('activity', 'get', array());
   }
 
   /**
-   *  Test civicrm_activity_get() with a good activity ID
+   * Test civicrm_activity_get() with a good activity ID
    */
   public function testActivityGetGoodID1() {
     // Insert rows in civicrm_activity creating activities 4 and
@@ -560,7 +561,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
 
 
   /**
-   *  Test civicrm_activity_get() with filter target_contact_id
+   * Test civicrm_activity_get() with filter target_contact_id
    */
   public function testActivityGetTargetFilter() {
     $params = $this->_params;
@@ -692,8 +693,8 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_get() with a good activity ID which
-   *  has associated custom data
+   * Test civicrm_activity_get() with a good activity ID which
+   * has associated custom data
    */
   public function testActivityGetGoodIDCustom() {
     $ids = $this->entityCustomGroupWithSingleFieldCreate(__FUNCTION__, __FILE__);
@@ -719,8 +720,8 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_get() with a good activity ID which
-   *  has associated custom data
+   * Test civicrm_activity_get() with a good activity ID which
+   * has associated custom data
    */
   public function testActivityGetContact_idCustom() {
     $ids = $this->entityCustomGroupWithSingleFieldCreate(__FUNCTION__, __FILE__);
@@ -840,7 +841,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_update() to update an existing activity
+   * Test civicrm_activity_update() to update an existing activity
    */
   public function testActivityUpdate() {
     $result = $this->callAPISuccess('activity', 'create', $this->_params);
@@ -866,8 +867,8 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_update() with valid parameters
-   *  and some custom data
+   * Test civicrm_activity_update() with valid parameters
+   * and some custom data
    */
   public function testActivityUpdateCustom() {
     $ids = $this->entityCustomGroupWithSingleFieldCreate(__FUNCTION__, __FILE__);
@@ -924,8 +925,8 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_update() for core activity fields
-   *  and some custom data
+   * Test civicrm_activity_update() for core activity fields
+   * and some custom data
    */
   public function testActivityUpdateCheckCoreFields() {
     $params = $this->_params;
@@ -1028,8 +1029,8 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_update() where the DB has a date_time
-   *  value and there is none in the update params.
+   * Test civicrm_activity_update() where the DB has a date_time
+   * value and there is none in the update params.
    */
   public function testActivityUpdateNotDate() {
     $result = $this->callAPISuccess('activity', 'create', $this->_params);
@@ -1074,8 +1075,8 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_update() where the source_contact_id
-   *  is not in the update params.
+   * Test civicrm_activity_update() where the source_contact_id
+   * is not in the update params.
    */
   public function testActivityUpdateKeepSource() {
     $activity = $this->callAPISuccess('activity', 'create', $this->_params);
@@ -1097,7 +1098,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activities_contact_get()
+   * Test civicrm_activities_contact_get()
    */
   public function testActivitiesContactGet() {
     $activity = $this->callAPISuccess('activity', 'create', $this->_params);
index 890e03f32b1740404e9def8cda9a63ad22fc116c..7664ec4466cb82fa8fcf5bb669fcb45c0ddaa4a6 100644 (file)
@@ -43,7 +43,7 @@ class api_v3_ActivityTypeTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_type_get()
+   * Test civicrm_activity_type_get().
    */
   public function testActivityTypeGet() {
     $params = array();
@@ -53,7 +53,7 @@ class api_v3_ActivityTypeTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_type_create()
+   * Test civicrm_activity_type_create().
    */
   public function testActivityTypeCreate() {
     $params = array(
@@ -68,7 +68,7 @@ class api_v3_ActivityTypeTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_type_create - check id
+   * Test civicrm_activity_type_create - check id
    */
   public function testActivityTypecreatecheckId() {
     $params = array(
@@ -81,7 +81,7 @@ class api_v3_ActivityTypeTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_type_delete()
+   * Test civicrm_activity_type_delete()
    */
   public function testActivityTypeDelete() {
     $params = array(
index 8ddcd46251742f350416ad5186a83a27376f64ff..bc567a31cc601392ba2c99321ea9d66dacc6acc1 100644 (file)
@@ -62,7 +62,7 @@ class api_v3_AllTests extends CiviTestSuite {
   }
 
   /**
-   *  Build test suite dynamically
+   * Build test suite dynamically
    */
   public static function suite() {
     $inst = self::getInstance();
index 5ca467eae675171c450b4f26ff25545667ffea62..1b0cb06b441d92ea33fa5a5a1cf4f14c6e250915 100644 (file)
@@ -47,7 +47,7 @@ class api_v3_BatchTest extends CiviUnitTestCase {
   }
 
   /**
-   * Create a sample batch
+   * Create a sample batch.
    */
   public function batchCreate() {
     $params = $this->_params;
@@ -116,7 +116,7 @@ class api_v3_BatchTest extends CiviUnitTestCase {
   }
 
   /**
-   * Test civicrm_batch_delete using the new $params['id'] syntax
+   * Test civicrm_batch_delete using the new $params['id'] syntax.
    */
   public function testBatchDeleteCorrectSyntax() {
     $batchID = $this->batchCreate();
index 7df9c9cdcc5e5ee64b1968959d1241b6047ef5e5..1fd5571c7db7bf002752bcbf57cabdb28dedd693 100644 (file)
@@ -14,10 +14,10 @@ require_once 'CiviTest/CiviUnitTestCase.php';
 class api_v3_CRM11793Test extends CiviUnitTestCase {
 
   /**
-   *  Test setup for every test
+   * Test setup for every test
    *
-   *  Connect to the database, truncate the tables that will be used
-   *  and redirect stdin to a temporary file
+   * Connect to the database, truncate the tables that will be used
+   * and redirect stdin to a temporary file
    */
   public function setUp() {
     //  Connect to the database
@@ -35,10 +35,10 @@ class api_v3_CRM11793Test extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_contact_create
+   * Test civicrm_contact_create
    *
-   *  Verify that attempt to create individual contact with only
-   *  first and last names succeeds
+   * Verify that attempt to create individual contact with only
+   * first and last names succeeds
    */
   public function testCRM11793Organization() {
     $this->_testCRM11793ContactType('Organization');
index 76780b16eb07c8992813263c0e72ef7cadc3ec3e..9e2725eb57e07cf5166357ce0bf60e8f237e5287 100644 (file)
@@ -45,10 +45,10 @@ class api_v3_CaseTest extends CiviCaseTestCase {
   protected $followup_activity_type_value;
 
   /**
-   *  Test setup for every test
+   * Test setup for every test
    *
-   *  Connect to the database, truncate the tables that will be used
-   *  and redirect stdin to a temporary file
+   * Connect to the database, truncate the tables that will be used
+   * and redirect stdin to a temporary file
    */
   public function setUp() {
     $this->_entity = 'case';
@@ -247,7 +247,7 @@ class api_v3_CaseTest extends CiviCaseTestCase {
   }
 
   /**
-   *  Test activity api create for case activities
+   * Test activity api create for case activities
    */
   public function testCaseActivityCreate() {
     // Create a case first
@@ -285,7 +285,7 @@ class api_v3_CaseTest extends CiviCaseTestCase {
   }
 
   /**
-   *  Test activity api update for case activities
+   * Test activity api update for case activities
    */
   public function testCaseActivityUpdate() {
     // Need to create the case and activity before we can update it
index ece88d4c7ac7d92299caafbcee477c7484fbbd9f..cc6cc223a74f993daafd705ea03e0fafdd6c8189 100644 (file)
@@ -44,10 +44,10 @@ class api_v3_ConstantTest extends CiviUnitTestCase {
   protected $_apiversion = 3;
 
   /**
-   *  Test setup for every test
+   * Test setup for every test
    *
-   *  Connect to the database, truncate the tables that will be used
-   *  and redirect stdin to a temporary file
+   * Connect to the database, truncate the tables that will be used
+   * and redirect stdin to a temporary file
    */
   public function setUp() {
     //  Connect to the database
@@ -56,7 +56,7 @@ class api_v3_ConstantTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_constant_get( ) for unknown constant
+   * Test civicrm_constant_get( ) for unknown constant
    */
   public function testUnknownConstant() {
     $result = $this->callAPIFailure('constant', 'get', array(
@@ -65,7 +65,7 @@ class api_v3_ConstantTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_constant_get( 'activityStatus' )
+   * Test civicrm_constant_get( 'activityStatus' )
    */
   public function testActivityStatus() {
 
@@ -81,7 +81,7 @@ class api_v3_ConstantTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_constant_get( 'activityType' )
+   * Test civicrm_constant_get( 'activityType' )
    */
   public function testActivityType() {
 
@@ -95,7 +95,7 @@ class api_v3_ConstantTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_address_getoptions( 'location_type_id' )
+   * Test civicrm_address_getoptions( 'location_type_id' )
    */
   public function testLocationTypeGet() {
     // needed to get rid of cached values from previous tests
@@ -113,7 +113,7 @@ class api_v3_ConstantTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_phone_getoptions( 'phone_type_id' )
+   * Test civicrm_phone_getoptions( 'phone_type_id' )
    */
   public function testPhoneType() {
     $params = array(
@@ -130,7 +130,7 @@ class api_v3_ConstantTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_constant_get( 'mailProtocol' )
+   * Test civicrm_constant_get( 'mailProtocol' )
    */
   public function testmailProtocol() {
     $params = array(
index cb76059f4b2183467204e8d47d717523d74c9544..35bc19a9fe67065f8a4468882cf4a8ffe51a25b2 100644 (file)
@@ -53,8 +53,8 @@ class api_v3_ContactTest extends CiviUnitTestCase {
   /**
    * Test setup for every test.
    *
-   *  Connect to the database, truncate the tables that will be used
-   *  and redirect stdin to a temporary file
+   * Connect to the database, truncate the tables that will be used
+   * and redirect stdin to a temporary file
    */
   public function setUp() {
     // Connect to the database.
@@ -232,7 +232,7 @@ class api_v3_ContactTest extends CiviUnitTestCase {
    * Test old keys still work.
    *
    * Verify that attempt to create individual contact with
-   *  first and last names and old key values works
+   * first and last names and old key values works
    */
   public function testCreateNameIndividualOldKeys() {
     $params = array(
@@ -257,7 +257,7 @@ class api_v3_ContactTest extends CiviUnitTestCase {
    * Test preferred keys work.
    *
    * Verify that attempt to create individual contact with
-   *  first and last names and old key values works
+   * first and last names and old key values works
    */
   public function testCreateNameIndividualRecommendedKeys2() {
     $params = array(
@@ -283,7 +283,7 @@ class api_v3_ContactTest extends CiviUnitTestCase {
    * Test household name is sufficient for create.
    *
    * Verify that attempt to create household contact with only
-   *  household name succeeds
+   * household name succeeds
    */
   public function testCreateNameHousehold() {
     $params = array(
@@ -298,7 +298,7 @@ class api_v3_ContactTest extends CiviUnitTestCase {
    * Test organization name is sufficient for create.
    *
    * Verify that attempt to create organization contact with only
-   *  organization name succeeds.
+   * organization name succeeds.
    */
   public function testCreateNameOrganization() {
     $params = array(
@@ -797,7 +797,7 @@ class api_v3_ContactTest extends CiviUnitTestCase {
 
   /**
    * Verify that attempt to create individual contact with first
-   *  and last names and email succeeds
+   * and last names and email succeeds
    */
   public function testCreateIndividualWithNameEmail() {
     $params = array(
@@ -826,7 +826,7 @@ class api_v3_ContactTest extends CiviUnitTestCase {
 
   /**
    * Verify that attempt to create individual contact with first
-   *  and last names, email and location type succeeds
+   * and last names, email and location type succeeds
    */
   public function testCreateIndividualWithNameEmailLocationType() {
     $params = array(
@@ -892,7 +892,7 @@ class api_v3_ContactTest extends CiviUnitTestCase {
 
   /**
    * Verify that attempt to create household contact with details
-   *  succeeds
+   * succeeds
    */
   public function testCreateHouseholdDetails() {
     $params = array(
@@ -1132,7 +1132,7 @@ class api_v3_ContactTest extends CiviUnitTestCase {
   /**
    * Test civicrm_contact_get() return only first name & last name.
    *
-   *  Use comma separated string return without a space
+   * Use comma separated string return without a space
    */
   public function testContactGetReturnFirstLastNoComma() {
     $contact = $this->callAPISuccess('contact', 'create', $this->_params);
@@ -1257,7 +1257,7 @@ class api_v3_ContactTest extends CiviUnitTestCase {
 
   /**
    * Test birth date params incl value, array & birth_date_high, birth_date_low
-   *  && deceased
+   * && deceased
    */
   public function testContactGetBirthDate() {
     $contact1 = $this->callAPISuccess('contact', 'create', array_merge($this->_params, array('birth_date' => 'first day of next month - 2 years')));
@@ -1288,7 +1288,7 @@ class api_v3_ContactTest extends CiviUnitTestCase {
 
   /**
    * Test Deceased date params incl value, array & Deceased_date_high, Deceased date_low
-   *  && deceased.
+   * && deceased.
    */
   public function testContactGetDeceasedDate() {
     $contact1 = $this->callAPISuccess('contact', 'create', array_merge($this->_params, array('deceased_date' => 'first day of next month - 2 years')));
index 01c07100ee08a8a429431c45c1c8446bbfa8db37..4c0355666a4b8f04b47c0235ab3be5d5e185074d 100644 (file)
@@ -281,7 +281,7 @@ class api_v3_ContributionSoftTest extends CiviUnitTestCase {
   }
 
   /**
-   *  civicrm_contribution_soft_delete methods
+   * civicrm_contribution_soft_delete methods
    *
    */
   public function testDeleteEmptyParamsContributionSoft() {
@@ -316,8 +316,8 @@ class api_v3_ContributionSoftTest extends CiviUnitTestCase {
   ///////////////// civicrm_contribution_search methods
 
   /**
-   *  Test civicrm_contribution_search with empty params.
-   *  All available contributions expected.
+   * Test civicrm_contribution_search with empty params.
+   * All available contributions expected.
    */
   public function testSearchEmptyParams() {
     $p = array(
@@ -339,7 +339,7 @@ class api_v3_ContributionSoftTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_contribution_soft_search. Success expected.
+   * Test civicrm_contribution_soft_search. Success expected.
    */
   public function testSearch() {
     $p1 = array(
index edaa93a211db6d94b210642a0a9984e202aa45cc..dd15e6ce85e19bfdf266fb1dc3a9b98381fec8ec 100644 (file)
@@ -1208,8 +1208,8 @@ class api_v3_ContributionTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_contribution_search with empty params.
-   *  All available contributions expected.
+   * Test civicrm_contribution_search with empty params.
+   * All available contributions expected.
    */
   public function testSearchEmptyParams() {
     $params = array();
@@ -1250,7 +1250,7 @@ class api_v3_ContributionTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_contribution_search. Success expected.
+   * Test civicrm_contribution_search. Success expected.
    */
   public function testSearch() {
     $p1 = array(
index de09416c29e238561a692150e7198595448dcad5..6ae4ecfd86afb8fc6b2b79eb388db642f3179def 100644 (file)
@@ -204,7 +204,7 @@ class api_v3_CustomFieldTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test  using example code
+   * Test  using example code
    */
   /*function testCustomFieldCreateExample( )
   {
index 15cfc18e5f2f509a24ed7bdde142fdb2ddac3a49..1e448121f3d64d7e8ee93be8a7285f775902c955 100644 (file)
@@ -37,10 +37,10 @@ class api_v3_DashboardContactTest extends CiviUnitTestCase {
   protected $_apiversion = 3;
 
   /**
-   *  Test setup for every test
+   * Test setup for every test
    *
-   *  Connect to the database, truncate the tables that will be used
-   *  and redirect stdin to a temporary file
+   * Connect to the database, truncate the tables that will be used
+   * and redirect stdin to a temporary file
    */
   public function setUp() {
     //  Connect to the database
index 41c7988ac1ad5bbeef8a5fac15ba6363dd194857..eb496ddd9e4eac2ce3b2d1cc463f464156d211c6 100644 (file)
@@ -37,10 +37,10 @@ class api_v3_DashboardTest extends CiviUnitTestCase {
   protected $_apiversion = 3;
 
   /**
-   *  Test setup for every test
+   * Test setup for every test
    *
-   *  Connect to the database, truncate the tables that will be used
-   *  and redirect stdin to a temporary file
+   * Connect to the database, truncate the tables that will be used
+   * and redirect stdin to a temporary file
    */
   public function setUp() {
     //  Connect to the database
index 6b5f19dd90bf575db99c8ac6c3bc9f99f367d99c..a627b22c89016718de30c5354a7293014d655dc3 100644 (file)
@@ -436,7 +436,7 @@ class api_v3_EventTest extends CiviUnitTestCase {
   ///////////////// civicrm_event_search methods
 
   /**
-   *  Test civicrm_event_search with wrong params type
+   * Test civicrm_event_search with wrong params type
    */
   public function testSearchWrongParamsType() {
     $params = 'a string';
@@ -444,23 +444,23 @@ class api_v3_EventTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_event_search with empty params
+   * Test civicrm_event_search with empty params.
    */
   public function testSearchEmptyParams() {
-    $event = $this->callAPISuccess('event', 'create', $this->_params[1]);
+    $this->callAPISuccess('event', 'create', $this->_params[1]);
 
-    $getparams = array(
+    $getParams = array(
       'sequential' => 1,
     );
-    $result = $this->callAPISuccess('event', 'get', $getparams);
-    $this->assertEquals($result['count'], 3, 'In line ' . __LINE__);
+    $result = $this->callAPISuccess('event', 'get', $getParams);
+    $this->assertEquals($result['count'], 3);
     $res = $result['values'][0];
-    $this->assertArrayKeyExists('title', $res, 'In line ' . __LINE__);
-    $this->assertEquals($res['event_type_id'], $this->_params[1]['event_type_id'], 'In line ' . __LINE__);
+    $this->assertArrayKeyExists('title', $res);
+    $this->assertEquals($res['event_type_id'], $this->_params[1]['event_type_id']);
   }
 
   /**
-   *  Test civicrm_event_search. Success expected.
+   * Test civicrm_event_search. Success expected.
    */
   public function testSearch() {
     $params = array(
@@ -476,8 +476,8 @@ class api_v3_EventTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_event_search. Success expected.
-   *  return.offset and return.max_results test (CRM-5266)
+   * Test civicrm_event_search. Success expected.
+   * return.offset and return.max_results test (CRM-5266)
    */
   public function testSearchWithOffsetAndMaxResults() {
     $maxEvents = 5;
index a44a7508afaedd19b4806d13fdc31cc88fa5b85f..d152d380837e72dc8460b9dae88dce4f9c73e8b8 100644 (file)
@@ -174,7 +174,7 @@ class api_v3_GroupOrganizationTest extends CiviUnitTestCase {
   ///////////////// civicrm_group_organization_remove methods
 
   /**
-   *  Test civicrm_group_organization_remove with params not an array.
+   * Test civicrm_group_organization_remove with params not an array.
    */
   public function testGroupOrganizationDeleteParamsNotArray() {
     $params = 'delete';
@@ -192,7 +192,7 @@ class api_v3_GroupOrganizationTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_group_organization_remove with valid params.
+   * Test civicrm_group_organization_remove with valid params.
    */
   public function testGroupOrganizationDelete() {
     $paramsC = array(
@@ -208,7 +208,7 @@ class api_v3_GroupOrganizationTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_group_organization_remove with invalid params key.
+   * Test civicrm_group_organization_remove with invalid params key.
    */
   public function testGroupOrganizationDeleteWithInvalidKey() {
     $paramsDelete = array(
index 3149530ef98228afdcf23ee68e797058e0bf6de4..0662f58c6bd150bc278d7124af08165e88a717a2 100644 (file)
@@ -59,7 +59,7 @@ class api_v3_MembershipStatusTest extends CiviUnitTestCase {
 
 
   /**
-   *  Test civicrm_membership_status_get with empty params
+   * Test civicrm_membership_status_get with empty params
    */
   public function testGetEmptyParams() {
     $result = $this->callAPISuccess('membership_status', 'get', array());
@@ -69,7 +69,7 @@ class api_v3_MembershipStatusTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_membership_status_get. Success expected.
+   * Test civicrm_membership_status_get. Success expected.
    */
   public function testGet() {
     $params = array(
@@ -80,7 +80,7 @@ class api_v3_MembershipStatusTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_membership_status_get. Success expected.
+   * Test civicrm_membership_status_get. Success expected.
    */
   public function testGetLimit() {
     $result = $this->callAPISuccess('membership_status', 'get', array());
index 369b6bc855366a591a83ae0b093b84c967b86987..1d2c007fb14c83aaf5ae56ed5c994503503cd4a8 100644 (file)
@@ -1170,7 +1170,7 @@ class api_v3_MembershipTest extends CiviUnitTestCase {
   /**
    * Test correct end and start dates are calculated for fixed multi year memberships.
    *
-   *  The empty start date is calculated to be the start_date (1 Nov prior to the join_date - so 1 Nov 14)
+   * The empty start date is calculated to be the start_date (1 Nov prior to the join_date - so 1 Nov 14)
    *
    * In this set our start date is after the start day and after the rollover day so we do get an extra year
    * and we end one day before the rollover day. Start day is 1 Nov so we end on 31 Oct
@@ -1231,7 +1231,7 @@ class api_v3_MembershipTest extends CiviUnitTestCase {
   /**
    * Test correct end and start dates are calculated for fixed multi year memberships.
    *
-   *  The empty start date is calculated to be the start_date (1 Nov prior to the join_date - so 1 Nov 14)
+   * The empty start date is calculated to be the start_date (1 Nov prior to the join_date - so 1 Nov 14)
    *
    * The empty start date is calculated to be the start_date (1 Nov prior to the join_date - so 1 Nov 14)
    * In this set our join date is after the start day and after the rollover day so we do get an extra year
index c767a2318ab0396c41405e1801bf427a9a963efb..207e8434e07bba50bb533e862b6f73849ab34c1a 100644 (file)
@@ -56,7 +56,7 @@ class api_v3_OptionValueTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test limit param
+   * Test limit param
    */
   public function testGetOptionValueLimit() {
     $params = array();
@@ -68,7 +68,7 @@ class api_v3_OptionValueTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test offset param
+   * Test offset param
    */
   public function testGetOptionValueOffSet() {
 
@@ -85,7 +85,7 @@ class api_v3_OptionValueTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test offset param
+   * Test offset param
    */
   public function testGetSingleValueOptionValueSort() {
     $description = "demonstrates use of Sort param (available in many api functions). Also, getsingle";
index 23a29ab83cec8251853386c1bff586993bfcd5b3..6c65c3c4ef7e902799c080e0378f582db1839077 100644 (file)
@@ -88,7 +88,7 @@ class api_v3_PaymentProcessorTypeTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test  using example code
+   * Test  using example code
    */
   public function testPaymentProcessorTypeCreateExample() {
     require_once 'api/v3/examples/PaymentProcessorType/Create.php';
index 55a8bed18214268bab761f6768a782979697f624..7de2155044b978fd1571c54900c4a95f93c17a80 100644 (file)
@@ -128,7 +128,7 @@ class api_v3_RelationshipTypeTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test  using example code
+   * Test  using example code
    */
   public function testRelationshipTypeCreateExample() {
     require_once 'api/v3/examples/RelationshipType/Create.php';
index b217000e0f3081b2ac35712048d410f3a25cf908..60d97874a0e9aa04a40bc9f4aa4698a53ca14ef8 100644 (file)
@@ -209,7 +209,7 @@ class api_v3_UFJoinTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_create() using example code
+   * Test civicrm_activity_create() using example code.
    */
   public function testUFJoinCreateExample() {
     require_once 'api/v3/examples/UFJoin/Create.php';
index 5c084778ebebcc110b56530fe9f0e5a0b2ccf0c5..4f8778a113cc1d7d7461e64670b1bc9973ec8742 100644 (file)
@@ -116,7 +116,7 @@ class api_v3_UFMatchTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_create() using example code
+   * Test civicrm_activity_create() using example code
    */
   public function testUFMatchGetExample() {
     require_once 'api/v3/examples/UFMatch/Get.php';