Merge pull request #4939 from colemanw/CRM-15789

[civicrm-core.git] / tests / phpunit / CiviTest / PaypalPro.php

diff --git a/tests/phpunit/CiviTest/PaypalPro.php b/tests/phpunit/CiviTest/PaypalPro.php
index eeb2fd71f8503426f18c12eba0e4e2aac8b9ead8..1a9ce0a411299c0fd136bc620dbfb16b4c23f84d 100644 (file)
--- a/tests/phpunit/CiviTest/PaypalPro.php
+++ b/tests/phpunit/CiviTest/PaypalPro.php
@@ -1,16 +1,20 @@
 <?php
 require_once "CRM/Financial/DAO/PaymentProcessor.php";
+
+/**
+ * Class PaypalPro
+ */
 class PaypalPro extends PHPUnit_Framework_Testcase {
   /**
    * Helper function to create
    * a payment processor of type Paypal Pro
    *
-   * @return $paymentProcessor id of created payment processor
-   * @todo this appears not to be working but because it doesn't extend the test class
+   * @return int
+   *   $paymentProcessor id of created payment processor@todo this appears not to be working but because it doesn't extend the test class
    * callAPISuccess won't work
    * I have duplicated this on the main test class as a work around
    */
-  static function create() {
+  public static function create() {
 
     $paymentProcessor = new CRM_Financial_DAO_PaymentProcessor();
     $paymentParams = array(
@@ -38,11 +42,13 @@ class PaypalPro extends PHPUnit_Framework_Testcase {
   /**
    * Helper function to delete a PayPal Pro
    * payment processor
-   * @param  int $id - id of the PayPal Pro payment processor
-   * to be deleted
-   * @return boolean true if payment processor deleted, false otherwise
+   * @param int $id
+   *   Id of the PayPal Pro payment processor.
+   *   to be deleted
+   * @return boolean
+   *   true if payment processor deleted, false otherwise
    */
-  static function delete($id) {
+  public static function delete($id) {
     $pp = new CRM_Financial_DAO_PaymentProcessor();
     $pp->id = $id;
     if ($pp->find(TRUE)) {