clean up fixes for batch 5

[civicrm-core.git] / tests / phpunit / CRM / Member / BAO / MembershipTypeTest.php

diff --git a/tests/phpunit/CRM/Member/BAO/MembershipTypeTest.php b/tests/phpunit/CRM/Member/BAO/MembershipTypeTest.php
index 320f73ddfe8445b3ccc5269e73db4199927e1b27..a9c20abb56a1d682497a6eba3d5854648886b429 100644 (file)
--- a/tests/phpunit/CRM/Member/BAO/MembershipTypeTest.php
+++ b/tests/phpunit/CRM/Member/BAO/MembershipTypeTest.php
@@ -32,18 +32,8 @@ require_once 'CiviTest/CiviUnitTestCase.php';
  * Class CRM_Member_BAO_MembershipTypeTest
  */
 class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
-  /**
-   * @return array
-   */
-  function get_info() {
-    return array(
-      'name' => 'MembershipType BAOs',
-      'description' => 'Test all Member_BAO_MembershipType methods.',
-      'group' => 'CiviCRM BAO Tests',
-    );
-  }
 
-  function setUp() {
+  public function setUp() {
     parent::setUp();
 
     //create relationship
@@ -65,19 +55,19 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
   /**
    * Tears down the fixture, for example, closes a network connection.
    * This method is called after a test is executed.
-   *
    */
-  function tearDown() {
+  public function tearDown() {
     $this->relationshipTypeDelete($this->_relationshipTypeId);
     $this->membershipStatusDelete($this->_membershipStatusID);
     $this->contactDelete($this->_orgContactID);
     $this->contactDelete($this->_indiviContactID);
   }
 
-  /* check function add()
-     *
-     */
-  function testAdd() {
+  /**
+   * check function add()
+   *
+   */
+  public function testAdd() {
     $ids = array();
     $params = array(
       'name' => 'test type',
@@ -104,10 +94,11 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
     $this->membershipTypeDelete(array('id' => $membershipType->id));
   }
 
-  /* check function retrive()
-     *
-     */
-  function testRetrieve() {
+  /**
+   * check function retrive()
+   *
+   */
+  public function testRetrieve() {
     $ids = array();
     $params = array(
       'name' => 'General',
@@ -124,17 +115,18 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
     );
     $membershipType = CRM_Member_BAO_MembershipType::add($params, $ids);
 
-    $params  = array('name' => 'General');
+    $params = array('name' => 'General');
     $default = array();
-    $result  = CRM_Member_BAO_MembershipType::retrieve($params, $default);
+    $result = CRM_Member_BAO_MembershipType::retrieve($params, $default);
     $this->assertEquals($result->name, 'General', 'Verify membership type name.');
     $this->membershipTypeDelete(array('id' => $membershipType->id));
   }
 
-  /* check function isActive()
-     *
-     */
-  function testSetIsActive() {
+  /**
+   * check function isActive()
+   *
+   */
+  public function testSetIsActive() {
     $ids = array();
     $params = array(
       'name' => 'General',
@@ -163,10 +155,11 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
     $this->membershipTypeDelete(array('id' => $membership->id));
   }
 
-  /* check function del()
-     *
-     */
-  function testdel() {
+  /**
+   * check function del()
+   *
+   */
+  public function testdel() {
     $ids = array();
     $params = array(
       'name' => 'General',
@@ -189,10 +182,11 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
     $this->assertEquals($result, TRUE, 'Verify membership deleted.');
   }
 
-  /* check function convertDayFormat( )
-     *
-     */
-  function testConvertDayFormat() {
+  /**
+   * check function convertDayFormat( )
+   *
+   */
+  public function testConvertDayFormat() {
     $ids = array();
     $params = array(
       'name' => 'General',
@@ -219,10 +213,11 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
     $this->membershipTypeDelete(array('id' => $membership->id));
   }
 
-  /* check function getMembershipTypes( )
-     *
-     */
-  function testGetMembershipTypes() {
+  /**
+   * check function getMembershipTypes( )
+   *
+   */
+  public function testGetMembershipTypes() {
     $ids = array();
     $params = array(
       'name' => 'General',
@@ -244,10 +239,11 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
     $this->membershipTypeDelete(array('id' => $membership->id));
   }
 
-  /* check function getMembershipTypeDetails( )
-     *
-     */
-  function testGetMembershipTypeDetails() {
+  /**
+   * check function getMembershipTypeDetails( )
+   *
+   */
+  public function testGetMembershipTypeDetails() {
     $ids = array();
     $params = array(
       'name' => 'General',
@@ -271,10 +267,11 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
     $this->membershipTypeDelete(array('id' => $membership->id));
   }
 
-  /* check function getDatesForMembershipType( )
-     *
-     */
-  function testGetDatesForMembershipType() {
+  /**
+   * check function getDatesForMembershipType( )
+   *
+   */
+  public function testGetDatesForMembershipType() {
     $ids = array();
     $params = array(
       'name' => 'General',
@@ -297,10 +294,11 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
     $this->membershipTypeDelete(array('id' => $membership->id));
   }
 
-  /* check function getRenewalDatesForMembershipType( )
-     *
-     */
-  function testGetRenewalDatesForMembershipType() {
+  /**
+   * check function getRenewalDatesForMembershipType( )
+   *
+   */
+  public function testGetRenewalDatesForMembershipType() {
     $ids = array();
     $params = array(
       'name' => 'General',
@@ -340,10 +338,11 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
     $this->membershipTypeDelete(array('id' => $membershipType->id));
   }
 
-  /* check function getMembershipTypesByOrg( )
-     *
-     */
-  function testGetMembershipTypesByOrg() {
+  /**
+   * check function getMembershipTypesByOrg( )
+   *
+   */
+  public function testGetMembershipTypesByOrg() {
     $ids = array();
     $params = array(
       'name' => 'General',
@@ -370,4 +369,3 @@ class CRM_Member_BAO_MembershipTypeTest extends CiviUnitTestCase {
     $this->membershipTypeDelete(array('id' => $membershipType->id));
   }
 }
-