Merge pull request #5536 from totten/4.5-httpclient
[civicrm-core.git] / tests / phpunit / api / v3 / ActivityTest.php
index 3be599cc91437fb2509e1728528e26cbba1a74c7..0236aed7c812ca3ad50e6422306b946541754cd7 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,13 +50,13 @@ 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
+    // Connect to the database
     parent::setUp();
 
     $this->_contactID = $this->individualCreate();
@@ -76,7 +77,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'status_id' => 2,
       'priority_id' => 1,
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
     );
     $this->_params2 = array(
@@ -95,6 +96,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
 
   /**
    * Tears down the fixture, for example, closes a network connection.
+   *
    * This method is called after a test is executed.
    */
   public function tearDown() {
@@ -108,14 +110,14 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   * Check with empty array
+   * Check fails with empty array.
    */
   public function testActivityCreateEmpty() {
-    $result = $this->callAPIFailure('activity', 'create', array());
+    $this->callAPIFailure('activity', 'create', array());
   }
 
   /**
-   * Check if required fields are not passed
+   * Check if required fields are not passed.
    */
   public function testActivityCreateWithoutRequired() {
     $params = array(
@@ -126,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(
@@ -135,7 +137,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Test activity',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'activity_name' => 'Fubar activity type',
@@ -147,24 +149,24 @@ 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(
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       '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(
@@ -172,18 +174,19 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'activity_name' => 'Test activity type',
     );
 
-    $result = $this->callAPIFailure('activity', 'create', $params);
+    $this->callAPIFailure('activity', 'create', $params);
   }
 
   /**
-   * Ensure that an invalid activity type causes failure
-   * oddly enough this test was failing because the creation of the invalid type
+   * Ensure that an invalid activity type causes failure.
+   *
+   * Oddly enough this test was failing because the creation of the invalid type
    * got added to the set up routine. Probably a mis-fix on a test
    */
   public function testActivityCreateWithNonNumericActivityTypeId() {
@@ -192,7 +195,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'activity_type_id' => 'Invalid Test activity type',
@@ -202,7 +205,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   * Check with incorrect required fields
+   * Check with incorrect required fields.
    */
   public function testActivityCreateWithUnknownActivityTypeId() {
     $params = array(
@@ -210,7 +213,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'activity_type_id' => 699,
@@ -225,7 +228,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'priority_id' => 44,
@@ -238,13 +241,16 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
 
+  /**
+   * Test create succeeds with valid string for priority.
+   */
   public function testActivityCreateWithValidStringPriority() {
     $params = array(
       'source_contact_id' => $this->_contactID,
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'priority_id' => 'Urgent',
@@ -255,53 +261,54 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $this->assertEquals(1, $result['values'][$result['id']]['priority_id']);
   }
 
+  /**
+   * Test create fails with invalid priority string.
+   */
   public function testActivityCreateWithInValidStringPriority() {
     $params = array(
       'source_contact_id' => $this->_contactID,
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'priority_id' => 'ergUrgent',
       'activity_type_id' => 1,
     );
 
-    $result = $this->callAPIFailure('activity', 'create', $params,
+    $this->callAPIFailure('activity', 'create', $params,
       "'ergUrgent' is not a valid option for field priority_id");
   }
 
   /**
-   *  Test civicrm_activity_create() with valid parameters
+   * Test civicrm_activity_create() with valid parameters.
    */
   public function testActivityCreate() {
 
-    $result = $this->callAPISuccess('activity', 'create', $this->_params);
+    $this->callAPISuccess('activity', 'create', $this->_params);
     $result = $this->callAPISuccess('activity', 'get', $this->_params);
-    $this->assertEquals($result['values'][$result['id']]['duration'], 120, 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['subject'], 'test activity type id', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['activity_date_time'], '2011-06-02 14:36:13', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['location'], 'Pensulvania', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['details'], 'a test activity', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['status_id'], 2, 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['id'], $result['id'], 'in line ' . __LINE__);
+    $this->assertEquals($result['values'][$result['id']]['duration'], 120);
+    $this->assertEquals($result['values'][$result['id']]['subject'], 'test activity type id');
+    $this->assertEquals($result['values'][$result['id']]['activity_date_time'], '2011-06-02 14:36:13');
+    $this->assertEquals($result['values'][$result['id']]['location'], 'Pennsylvania');
+    $this->assertEquals($result['values'][$result['id']]['details'], 'a test activity');
+    $this->assertEquals($result['values'][$result['id']]['status_id'], 2);
+    $this->assertEquals($result['values'][$result['id']]['id'], $result['id']);
   }
 
   /**
-   *  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();
 
-    $defaults = array();
-
     $params = array(
       'source_contact_id' => $this->_contactID,
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => '20110316',
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'activity_type_id' => 29,
@@ -309,28 +316,29 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     );
 
     $result = $this->callAPISuccess('activity', 'create', $params);
-    //todo test target & assignee are set
 
-    //$this->assertEquals($result['values'][$result['id']]['source_contact_id'], $this->_contactID, 'in line ' . __LINE__);
     $result = $this->callAPISuccess('activity', 'get', array('id' => $result['id']));
-    $this->assertEquals($result['values'][$result['id']]['duration'], 120, 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['subject'], 'Make-it-Happen Meeting', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['activity_date_time'], '2011-03-16 00:00:00', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['location'], 'Pensulvania', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['details'], 'a test activity', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['status_id'], 1, 'in line ' . __LINE__);
+    $this->assertEquals($result['values'][$result['id']]['duration'], 120);
+    $this->assertEquals($result['values'][$result['id']]['subject'], 'Make-it-Happen Meeting');
+    $this->assertEquals($result['values'][$result['id']]['activity_date_time'], '2011-03-16 00:00:00');
+    $this->assertEquals($result['values'][$result['id']]['location'], 'Pennsylvania');
+    $this->assertEquals($result['values'][$result['id']]['details'], 'a test activity');
+    $this->assertEquals($result['values'][$result['id']]['status_id'], 1);
   }
 
+  /**
+   * Test get returns target and assignee contacts.
+   */
   public function testActivityReturnTargetAssignee() {
 
-    $description = "Example demonstrates setting & retrieving the target & source";
+    $description = "Demonstrates setting & retrieving activity target & source.";
     $subfile = "GetTargetandAssignee";
     $params = array(
       'source_contact_id' => $this->_contactID,
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => '20110316',
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'activity_type_id' => 1,
@@ -341,20 +349,20 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
 
     $result = $this->callAPIAndDocument('activity', 'create', $params, __FUNCTION__, __FILE__, $description, $subfile);
     $result = $this->callAPISuccess('activity', 'get', array(
-        'id' => $result['id'],
-        'version' => $this->_apiversion,
-        'return.assignee_contact_id' => 1,
-        'return.target_contact_id' => 1
-      ));
+      'id' => $result['id'],
+      'version' => $this->_apiversion,
+      'return.assignee_contact_id' => 1,
+      'return.target_contact_id' => 1,
+    ));
 
-    $this->assertEquals($this->_contactID, $result['values'][$result['id']]['assignee_contact_id'][0], 'in line ' . __LINE__);
-    $this->assertEquals($this->_contactID, $result['values'][$result['id']]['target_contact_id'][0], 'in line ' . __LINE__);
+    $this->assertEquals($this->_contactID, $result['values'][$result['id']]['assignee_contact_id'][0]);
+    $this->assertEquals($this->_contactID, $result['values'][$result['id']]['target_contact_id'][0]);
   }
 
+  /**
+   * Test civicrm_activity_create() using example code.
+   */
   public function testActivityCreateExample() {
-    /**
-     *  Test civicrm_activity_create() using example code
-     */
     require_once 'api/v3/examples/Activity/Create.php';
     $result = activity_create_example();
     $expectedResult = activity_create_expectedresult();
@@ -362,8 +370,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_create() with valid parameters
-   *  and some custom data
+   * Test civicrm_activity_create() with valid parameters and custom data.
    */
   public function testActivityCreateCustom() {
     $ids = $this->entityCustomGroupWithSingleFieldCreate(__FUNCTION__, __FILE__);
@@ -371,10 +378,9 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $params['custom_' . $ids['custom_field_id']] = "custom string";
     $result = $this->callAPIAndDocument($this->_entity, 'create', $params, __FUNCTION__, __FILE__);
     $result = $this->callAPISuccess($this->_entity, 'get', array(
-        'return.custom_' . $ids['custom_field_id'] => 1,
-        'version' => 3,
-        'id' => $result['id']
-      ));
+      'return.custom_' . $ids['custom_field_id'] => 1,
+      'id' => $result['id'],
+    ));
     $this->assertEquals("custom string", $result['values'][$result['id']]['custom_' . $ids['custom_field_id']], ' in line ' . __LINE__);
 
     $this->customFieldDelete($ids['custom_field_id']);
@@ -382,14 +388,13 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_create() with valid parameters
-   *  and some custom data
+   * Test civicrm_activity_create() with valid parameters and custom data.
    */
   public function testActivityCreateCustomContactRefField() {
 
     $this->callAPISuccess('contact', 'create', array('id' => $this->_contactID, 'sort_name' => 'Contact, Test'));
     $subfile = 'ContactRefCustomField';
-    $description = "demonstrates create with Contact Reference Custom Field";
+    $description = "Demonstrates create with Contact Reference Custom Field.";
     $ids = $this->entityCustomGroupWithSingleFieldCreate(__FUNCTION__, __FILE__);
     $params = array(
       'custom_group_id' => $ids['custom_group_id'],
@@ -406,13 +411,13 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $params = $this->_params;
     $params['custom_' . $customField['id']] = "$this->_contactID";
 
-    $result = $this->callAPIAndDocument($this->_entity, 'create', $params, __FUNCTION__, __FILE__, $description, $subfile, 'Create');
+    $result = $this->callAPIAndDocument($this->_entity, 'create', $params, __FUNCTION__, __FILE__, $description, $subfile);
     $result = $this->callAPIAndDocument($this->_entity, 'get', array(
-        'return.custom_' . $customField['id'] => 1,
-        'id' => $result['id']
-      ), __FUNCTION__, __FILE__, 'Get with Contact Ref Custom Field', 'ContactRefCustomFieldGet', 'get');
+      'return.custom_' . $customField['id'] => 1,
+      'id' => $result['id'],
+    ), __FUNCTION__, __FILE__, 'Get with Contact Ref Custom Field', 'ContactRefCustomFieldGet');
 
-    $this->assertEquals('Anderson, Anthony', $result['values'][$result['id']]['custom_' . $customField['id']], ' in line ' . __LINE__);
+    $this->assertEquals('Anderson, Anthony', $result['values'][$result['id']]['custom_' . $customField['id']]);
     $this->assertEquals($this->_contactID, $result['values'][$result['id']]['custom_' . $customField['id'] . "_id"], ' in line ' . __LINE__);
     $this->assertEquals('Anderson, Anthony', $result['values'][$result['id']]['custom_' . $customField['id'] . '_1'], ' in line ' . __LINE__);
     $this->assertEquals($this->_contactID, $result['values'][$result['id']]['custom_' . $customField['id'] . "_1_id"], ' in line ' . __LINE__);
@@ -421,7 +426,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_create() with an invalid text status_id
+   * Test civicrm_activity_create() with an invalid text status_id.
    */
   public function testActivityCreateBadTextStatus() {
 
@@ -430,17 +435,17 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Discussion on Apis for v3',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 'Invalid',
       'activity_name' => 'Test activity type',
     );
 
-    $result = $this->callAPIFailure('activity', 'create', $params);
+    $this->callAPIFailure('activity', 'create', $params);
   }
 
   /**
-   *  Test civicrm_activity_create() with an invalid text status_id
+   * Test civicrm_activity_create() with an invalid text status_id.
    */
   public function testActivityCreateSupportActivityStatus() {
 
@@ -449,7 +454,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Discussion on Apis for v3',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'activity_status_id' => 'Invalid',
       'activity_name' => 'Test activity type',
@@ -461,8 +466,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
 
 
   /**
-   *  Test civicrm_activity_create() with valid parameters,
-   *  using a text status_id
+   * Test civicrm_activity_create() with using a text status_id.
    */
   public function testActivityCreateTextStatus() {
 
@@ -471,34 +475,33 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => date('Ymd'),
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 'Scheduled',
       'activity_name' => 'Test activity type',
     );
 
     $result = $this->callAPISuccess('activity', 'create', $params);
-    $this->assertEquals($result['values'][$result['id']]['duration'], 120, 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['subject'], 'Make-it-Happen Meeting', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['activity_date_time'], date('Ymd') . '000000', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['location'], 'Pensulvania', 'in line ' . __LINE__);
-    $this->assertEquals($result['values'][$result['id']]['details'], 'a test activity', 'in line ' . __LINE__);
+    $this->assertEquals($result['values'][$result['id']]['duration'], 120);
+    $this->assertEquals($result['values'][$result['id']]['subject'], 'Make-it-Happen Meeting');
+    $this->assertEquals($result['values'][$result['id']]['activity_date_time'], date('Ymd') . '000000');
+    $this->assertEquals($result['values'][$result['id']]['location'], 'Pennsylvania');
+    $this->assertEquals($result['values'][$result['id']]['details'], 'a test activity');
   }
 
   /**
-   *  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
-    //  13
-    $description = "Function demonstrates getting asignee_contact_id & using it to get the contact";
+    // Insert rows in civicrm_activity creating activities 4 and 13
+    $description = "Demonstrates getting assignee_contact_id & using it to get the contact.";
     $subfile = 'ReturnAssigneeContact';
     $activity = $this->callAPISuccess('activity', 'create', $this->_params);
 
@@ -534,8 +537,8 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $this->assertEquals("test activity type id", $result['values'][0]['subject'], 'In line ' . __LINE__);
   }
 
-  /*
-   * test that get functioning does filtering
+  /**
+   * test that get functioning does filtering.
    */
   public function testGetFilter() {
     $params = array(
@@ -543,7 +546,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => '20110316',
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'activity_name' => 'Test activity type',
@@ -558,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;
@@ -590,25 +593,25 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $activity = $this->callAPISuccess('Activity', 'Create', $params);
 
     $activityget = $this->callAPISuccess('Activity', 'get', array(
-        'id' => $activity['id'],
-        'target_contact_id' => $contact2,
-        'return.target_contact_id' => 1
-      ));
+      'id' => $activity['id'],
+      'target_contact_id' => $contact2,
+      'return.target_contact_id' => 1,
+    ));
     $this->assertEquals($activity['id'], $activityget['id'], 'In line ' . __LINE__);
     $this->assertEquals($contact2, $activityget['values'][$activityget['id']]['target_contact_id'][0], 'In line ' . __LINE__);
 
     $activityget = $this->callAPISuccess('activity', 'get', array(
-        'target_contact_id' => $this->_contactID,
-        'return.target_contact_id' => 1,
-        'id' => $activity['id']
-      ));
+      'target_contact_id' => $this->_contactID,
+      'return.target_contact_id' => 1,
+      'id' => $activity['id'],
+    ));
     if ($activityget['count'] > 0) {
       $this->assertNotEquals($contact2, $activityget['values'][$activityget['id']]['target_contact_id'][0], 'In line ' . __LINE__);
     }
   }
 
-  /*
-   * test that get functioning does filtering
+  /**
+   * test that get functioning does filtering.
    */
   public function testGetStatusID() {
     $params = array(
@@ -616,7 +619,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => '20110316',
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'activity_name' => 'Test activity type',
@@ -633,14 +636,14 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $this->assertEquals(0, $result['count'], 'No activities of status 1 should exist');
     $result = $this->callAPISuccess('Activity', 'Get', array(
       'version' => $this->_apiversion,
-      'status_id' => '2'
+      'status_id' => '2',
     ));
     $this->assertEquals(0, $result['count'], 'No activities of status 1 should exist');
 
   }
 
-  /*
-   * test that get functioning does filtering
+  /**
+   * test that get functioning does filtering.
    */
   public function testGetFilterMaxDate() {
     $params = array(
@@ -648,7 +651,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => '20110101',
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity',
       'status_id' => 1,
       'activity_name' => 'Test activity type',
@@ -661,7 +664,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $result = $this->callAPISuccess('Activity', 'Get', array(
       'version' => 3,
     ));
-    $description = "demonstrates _low filter (at time of writing doesn't work if contact_id is set";
+    $description = "Demonstrates _low filter (at time of writing doesn't work if contact_id is set.";
     $subfile = "DateTimeLow";
     $this->assertEquals(2, $result['count']);
     $params = array(
@@ -670,10 +673,10 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'sequential' => 1,
     );
     $result = $this->callAPIAndDocument('Activity', 'Get', $params, __FUNCTION__, __FILE__, $description, $subfile);
-    $this->assertEquals(1, $result['count'], 'in line ' . __LINE__);
-    $description = "demonstrates _high filter (at time of writing doesn't work if contact_id is set";
+    $this->assertEquals(1, $result['count']);
+    $description = "Demonstrates _high filter (at time of writing doesn't work if contact_id is set.";
     $subfile = "DateTimeHigh";
-    $this->assertEquals('2012-02-16 00:00:00', $result['values'][0]['activity_date_time'], 'in line ' . __LINE__);
+    $this->assertEquals('2012-02-16 00:00:00', $result['values'][0]['activity_date_time']);
     $params = array(
       'source_contact_id' => $this->_contactID,
       'version' => 3,
@@ -683,15 +686,15 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $result = $this->callAPIAndDocument('Activity', 'Get', $params, __FUNCTION__, __FILE__, $description, $subfile);
 
     $this->assertEquals(1, $result['count']);
-    $this->assertEquals('2011-01-01 00:00:00', $result['values'][0]['activity_date_time'], 'in line ' . __LINE__);
+    $this->assertEquals('2011-01-01 00:00:00', $result['values'][0]['activity_date_time']);
 
     $this->callAPISuccess('Activity', 'Delete', array('version' => 3, 'id' => $activityOne['id']));
     $this->callAPISuccess('Activity', 'Delete', array('version' => 3, 'id' => $activityTwo['id']));
   }
 
   /**
-   *  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__);
@@ -699,9 +702,9 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $params = $this->_params;
     $params['custom_' . $ids['custom_field_id']] = "custom string";
 
-    $result = $this->callAPISuccess($this->_entity, 'create', $params);
+    $this->callAPISuccess($this->_entity, 'create', $params);
 
-    //  Retrieve the test value
+    // Retrieve the test value.
     $params = array(
       'activity_type_id' => $this->test_activity_type_value,
       'sequential' => 1,
@@ -717,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__);
@@ -727,7 +730,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $params['custom_' . $ids['custom_field_id']] = "custom string";
 
     $result = $this->callAPISuccess($this->_entity, 'create', $params);
-    //  Retrieve the test value
+    // Retrieve the test value
     $params = array(
       'contact_id' => $this->_params['source_contact_id'],
       'activity_type_id' => $this->test_activity_type_value,
@@ -739,19 +742,19 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
 
     $this->assertEquals($this->test_activity_type_value, $result['values'][0]['activity_type_id'], 'In line ' . __LINE__);
     $this->assertEquals('test activity type id', $result['values'][0]['subject'], 'In line ' . __LINE__);
-    $this->assertEquals($result['values'][0]['id'], $result['id'], 'in line ' . __LINE__);
+    $this->assertEquals($result['values'][0]['id'], $result['id']);
   }
 
   /**
-   * Check activity deletion with empty params
+   * Check activity deletion with empty params.
    */
   public function testDeleteActivityForEmptyParams() {
     $params = array('version' => $this->_apiversion);
-    $result = $this->callAPIFailure('activity', 'delete', $params);
+    $this->callAPIFailure('activity', 'delete', $params);
   }
 
   /**
-   * Check activity deletion without activity id
+   * Check activity deletion without activity id.
    */
   public function testDeleteActivityWithoutId() {
     $params = array(
@@ -762,7 +765,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   * Check activity deletion without activity type
+   * Check activity deletion without activity type.
    */
   public function testDeleteActivityWithoutActivityType() {
     $params = array('id' => 1);
@@ -770,7 +773,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   * Check activity deletion with incorrect data
+   * Check activity deletion with incorrect data.
    */
   public function testDeleteActivityWithIncorrectActivityType() {
     $params = array(
@@ -782,7 +785,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   * Check activity deletion with correct data
+   * Check activity deletion with correct data.
    */
   public function testDeleteActivity() {
     $result = $this->callAPISuccess('activity', 'create', $this->_params);
@@ -791,12 +794,11 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'version' => $this->_apiversion,
     );
 
-    $result = $this->callAPISuccess('activity', 'delete', $params);
-    $this->documentMe($params, $result, __FUNCTION__, __FILE__);
+    $this->callAPIAndDocument('activity', 'delete', $params, __FUNCTION__, __FILE__);
   }
 
   /**
-   * Check if required fields are not passed
+   * Check if required fields are not passed.
    */
   public function testActivityUpdateWithoutRequired() {
     $params = array(
@@ -822,7 +824,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   * Check with incorrect required fields
+   * Check with incorrect required fields.
    */
   public function testActivityUpdateWithIncorrectContactActivityType() {
     $params = array(
@@ -838,7 +840,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);
@@ -864,22 +866,22 @@ 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__);
 
     $params = $this->_params;
 
-    //  Create an activity with custom data
+    // Create an activity with custom data
     //this has been updated from the previous 'old format' function - need to make it work
     $params = array(
       'source_contact_id' => $this->_contactID,
       'subject' => 'Make-it-Happen Meeting',
       'activity_date_time' => '2009-10-18',
       'duration' => 120,
-      'location' => 'Pensulvania',
+      'location' => 'Pennsylvania',
       'details' => 'a test activity to check the update api',
       'status_id' => 1,
       'activity_name' => 'Test activity type',
@@ -890,16 +892,16 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
 
     $activityId = $result['id'];
     $result = $this->callAPISuccess($this->_entity, 'get', array(
-        'return.custom_' . $ids['custom_field_id'] => 1,
-        'version' => 3,
-        'id' => $result['id']
-      ));
-    $this->assertEquals("custom string", $result['values'][$result['id']]['custom_' . $ids['custom_field_id']], ' in line ' . __LINE__);
-    $this->assertEquals("2009-10-18 00:00:00", $result['values'][$result['id']]['activity_date_time'], ' in line ' . __LINE__);
+      'return.custom_' . $ids['custom_field_id'] => 1,
+      'version' => 3,
+      'id' => $result['id'],
+    ));
+    $this->assertEquals("custom string", $result['values'][$result['id']]['custom_' . $ids['custom_field_id']]);
+    $this->assertEquals("2009-10-18 00:00:00", $result['values'][$result['id']]['activity_date_time']);
     $fields = $this->callAPISuccess('activity', 'getfields', array('version' => $this->_apiversion));
     $this->assertTrue(is_array($fields['values']['custom_' . $ids['custom_field_id']]));
 
-    //  Update the activity with custom data
+    // Update the activity with custom data.
     $params = array(
       'id' => $activityId,
       'source_contact_id' => $this->_contactID,
@@ -914,16 +916,16 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
     $result = $this->callAPISuccess('Activity', 'Create', $params);
 
     $result = $this->callAPISuccess($this->_entity, 'get', array(
-        'return.custom_' . $ids['custom_field_id'] => 1,
-        'version' => 3,
-        'id' => $result['id']
-      ));
-    $this->assertEquals("Updated my test data", $result['values'][$result['id']]['custom_' . $ids['custom_field_id']], ' in line ' . __LINE__);
+      'return.custom_' . $ids['custom_field_id'] => 1,
+      'version' => 3,
+      'id' => $result['id'],
+    ));
+    $this->assertEquals("Updated my test data", $result['values'][$result['id']]['custom_' . $ids['custom_field_id']]);
   }
 
   /**
-   *  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;
@@ -1026,8 +1028,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);
@@ -1052,7 +1054,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   * Check activity update with status
+   * Check activity update with status.
    */
   public function testActivityUpdateWithStatus() {
     $activity = $this->callAPISuccess('activity', 'create', $this->_params);
@@ -1072,13 +1074,13 @@ 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);
-    //  Updating the activity but not providing anything for the source contact
-    //  (It was set as $this->_contactID earlier.)
+    // Updating the activity but not providing anything for the source contact
+    // (It was set as $this->_contactID earlier.)
     $params = array(
       'id' => $activity['id'],
       'subject' => 'Updated Make-it-Happen Meeting',
@@ -1095,27 +1097,27 @@ 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);
     $activity2 = $this->callAPISuccess('activity', 'create', $this->_params2);
-    //  Get activities associated with contact $this->_contactID
+    // Get activities associated with contact $this->_contactID.
     $params = array(
       'contact_id' => $this->_contactID,
     );
     $result = $this->callAPISuccess('activity', 'get', $params);
 
     $this->assertEquals(2, $result['count'], 'In line ' . __LINE__);
-    $this->assertEquals($this->test_activity_type_value, $result['values'][$activity['id']]['activity_type_id'], 'In line ' . __LINE__);
-    $this->assertEquals('Test activity type', $result['values'][$activity['id']]['activity_name'], 'In line ' . __LINE__);
-    $this->assertEquals('Test activity type', $result['values'][$activity2['id']]['activity_name'], 'In line ' . __LINE__);
+    $this->assertEquals($this->test_activity_type_value, $result['values'][$activity['id']]['activity_type_id']);
+    $this->assertEquals('Test activity type', $result['values'][$activity['id']]['activity_name']);
+    $this->assertEquals('Test activity type', $result['values'][$activity2['id']]['activity_name']);
   }
 
-  /*
-   * test chained Activity format
+  /**
+   * Test chained Activity format.
    */
-  public function testchainedActivityGet() {
+  public function testChainedActivityGet() {
 
     $activity = $this->callAPISuccess('Contact', 'Create', array(
       'display_name' => "bob brown",
@@ -1130,7 +1132,7 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       ),
       'api.activity.create' => array(
         'subject' => 'send letter',
-        'activity_type_id' => '$value.api.activity_type.create.values.0.value'
+        'activity_type_id' => '$value.api.activity_type.create.values.0.value',
       ),
     ));
 
@@ -1142,15 +1144,15 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
   }
 
   /**
-   *  Test civicrm_activity_contact_get() with invalid Contact Id
+   * Test civicrm_activity_contact_get() with invalid Contact ID.
    */
   public function testActivitiesContactGetWithInvalidContactId() {
     $params = array('contact_id' => 'contact');
-    $result = $this->callAPIFailure('activity', 'get', $params);
+    $this->callAPIFailure('activity', 'get', $params);
   }
 
   /**
-   *  Test civicrm_activity_contact_get() with contact having no Activity
+   * Test civicrm_activity_contact_get() with contact having no Activity.
    */
   public function testActivitiesContactGetHavingNoActivity() {
     $params = array(
@@ -1165,16 +1167,19 @@ class api_v3_ActivityTest extends CiviUnitTestCase {
       'contact_id' => $contact['id'],
     );
     $result = $this->callAPISuccess('activity', 'get', $params);
-    $this->assertEquals($result['count'], 0, 'in line ' . __LINE__);
+    $this->assertEquals($result['count'], 0);
   }
 
+  /**
+   * Test getfields function.
+   */
   public function testGetFields() {
     $params = array('action' => 'create');
-    $result = $this->callAPIAndDocument('activity', 'getfields', $params, __FUNCTION__, __FILE__, NULL, NULL, 'getfields');
-    $this->assertTrue(is_array($result['values']), 'get fields doesnt return values array in line ' . __LINE__);
-    // $this->assertTrue(is_array($result['values']['priority_id']['options']));
+    $result = $this->callAPIAndDocument('activity', 'getfields', $params, __FUNCTION__, __FILE__, NULL, NULL);
+    $this->assertTrue(is_array($result['values']), 'get fields doesn\'t return values array');
     foreach ($result['values'] as $key => $value) {
-      $this->assertTrue(is_array($value), $key . " is not an array in line " . __LINE__);
+      $this->assertTrue(is_array($value), $key . " is not an array");
     }
   }
+
 }