INFRA-132 - Drupal.Classes.ClassCreateInstance.ParenthesisMissing

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

diff --git a/tests/phpunit/CiviTest/PaypalPro.php b/tests/phpunit/CiviTest/PaypalPro.php
index 399d4cca36822ec685a47ff4f89569987b4b5ce0..e9fd1c8adab34cec442a9524ae9d7de222d4d078 100644 (file)
--- a/tests/phpunit/CiviTest/PaypalPro.php
+++ b/tests/phpunit/CiviTest/PaypalPro.php
@@ -9,11 +9,12 @@ class PaypalPro extends PHPUnit_Framework_Testcase {
    * Helper function to create
    * a payment processor of type Paypal Pro
    *
-   * @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
+   * @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(
@@ -41,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 bool
+   *   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)) {
@@ -53,4 +56,5 @@ class PaypalPro extends PHPUnit_Framework_Testcase {
     }
     return $result;
   }
+
 }