style ci

Author: recca0120

.github/workflows/tests.yml

@@ -9,30 +9,54 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        php: [ '7.1','7.2', '7.3', '7.4', '8.0', '8.1' ]
-        stability: [ prefer-stable ]
+        php: [ '7.1','7.2', '7.3', '7.4', '8.0', '8.1', '8.2' ]
 
-    name: PHP ${{ matrix.php }} - ${{ matrix.stability }}
+    name: PHP ${{ matrix.php }}
+
+    env:
+      extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, fileinfo
+      key: cache-v1 # can be any string, change to clear the extension cache.
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
+
+      - name: Setup cache environment
+        id: extcache
+        uses: shivammathur/cache-extensions@v1
+        with:
+          php-version: ${{ matrix.php }}
+          extensions: ${{ env.extensions }}
+          key: ${{ env.key }}
+
+      - name: Cache extensions
+        uses: actions/cache@v3
+        with:
+          path: ${{ steps.extcache.outputs.dir }}
+          key: ${{ steps.extcache.outputs.key }}
+          restore-keys: ${{ steps.extcache.outputs.key }}
 
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ matrix.php }}
-          extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, gd, redis, memcached
+          extensions: ${{ env.extensions }}
           tools: composer:v2
-          coverage: none
+          coverage: xdebug
 
-      - name: Install dependencies
-        uses: nick-invision/retry@v1
+      - name: Get Composer Cache Directory
+        id: composer-cache
+        run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
+
+      - name: Cache Composer dependencies
+        uses: actions/cache@v3
         with:
-          timeout_minutes: 5
-          max_attempts: 5
-          command: composer update --${{ matrix.stability }} --prefer-dist --no-interaction --no-progress ${{ matrix.flags }}
+          path: ${{ steps.composer-cache.outputs.dir }}
+          key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
+          restore-keys: ${{ runner.os }}-composer-
+
+      - name: Install Composer dependencies
+        run: composer install --prefer-dist --no-interaction --no-progress
 
       - name: Execute tests
-        continue-on-error: ${{ matrix.php > 8 }}
-        run: vendor/bin/phpunit --verbose
+        run: vendor/bin/phpunit --testdox --verbose

src/Gateway.php

@@ -68,6 +68,7 @@
  *     echo "Transaction reference = " . $sale_id . "\n";
  * }
  * </code>
+ *
  * @method RequestInterface authorize(array $options = [])
  * @method RequestInterface completeAuthorize(array $options = [])
  * @method RequestInterface capture(array $options = [])
@@ -106,7 +107,7 @@ public function getDefaultParameters()
     }
 
     /**
-     * @param array $options
+     * @param  array  $options
      * @return RequestInterface
      */
     public function purchase(array $options = [])
@@ -115,7 +116,7 @@ public function purchase(array $options = [])
     }
 
     /**
-     * @param array $options
+     * @param  array  $options
      * @return RequestInterface
      */
     public function completePurchase(array $options = [])
@@ -126,7 +127,7 @@ public function completePurchase(array $options = [])
     }
 
     /**
-     * @param array $options
+     * @param  array  $options
      * @return RequestInterface|NotificationInterface
      */
     public function acceptNotification(array $options = [])
@@ -135,7 +136,7 @@ public function acceptNotification(array $options = [])
     }
 
     /**
-     * @param array $options
+     * @param  array  $options
      * @return RequestInterface
      */
     public function receiveTransactionInfo(array $options = [])

src/Message/AcceptNotificationRequest.php

@@ -7,7 +7,7 @@
 class AcceptNotificationRequest extends CompletePurchaseRequest implements NotificationInterface
 {
     /**
-     * @param mixed $data
+     * @param  mixed  $data
      * @return AcceptNotificationResponse
      */
     public function sendData($data)

src/Message/CompletePurchaseRequest.php

@@ -18,7 +18,7 @@ class CompletePurchaseRequest extends AbstractRequest
     use HasCVS;
 
     /**
-     * @param int $authAmount
+     * @param  int  $authAmount
      * @return CompletePurchaseRequest
      */
     public function setAuthAmount($authAmount)
@@ -35,7 +35,7 @@ public function getAuthAmount()
     }
 
     /**
-     * @param string $rtnCode
+     * @param  string  $rtnCode
      * @return CompletePurchaseRequest
      */
     public function setRtnCode($rtnCode)
@@ -52,7 +52,7 @@ public function getRtnCode()
     }
 
     /**
-     * @param string $rtnMsg
+     * @param  string  $rtnMsg
      * @return CompletePurchaseRequest
      */
     public function setRtnMsg($rtnMsg)
@@ -69,7 +69,7 @@ public function getRtnMsg()
     }
 
     /**
-     * @param string $paymentDate
+     * @param  string  $paymentDate
      * @return CompletePurchaseRequest
      */
     public function setPaymentDate($paymentDate)
@@ -103,7 +103,7 @@ public function getTransferOutAccount()
     }
 
     /**
-     * @param string $checkMacValue
+     * @param  string  $checkMacValue
      * @return CompletePurchaseRequest
      */
     public function setCheckMacValue($checkMacValue)
@@ -145,7 +145,7 @@ public function getData()
     }
 
     /**
-     * @param mixed $data
+     * @param  mixed  $data
      * @return CompletePurchaseResponse
      */
     public function sendData($data)
@@ -155,6 +155,7 @@ public function sendData($data)
 
     /**
      * @return array
+     *
      * @throws InvalidResponseException
      */
     private function checkMacValue($data)

src/Message/PurchaseRequest.php

@@ -11,7 +11,7 @@ class PurchaseRequest extends AbstractRequest
     use HasMerchant;
 
     /**
-     * @param string $orderNo
+     * @param  string  $orderNo
      * @return PurchaseRequest
      */
     public function setOrderNo($orderNo)
@@ -28,7 +28,7 @@ public function getOrderNo()
     }
 
     /**
-     * @param string $receiveUrl
+     * @param  string  $receiveUrl
      * @return PurchaseRequest
      */
     public function setReceiveUrl($receiveUrl)
@@ -45,7 +45,7 @@ public function getReceiveUrl()
     }
 
     /**
-     * @param string $orderDesc
+     * @param  string  $orderDesc
      * @return PurchaseRequest
      */
     public function setOrderDesc($orderDesc)
@@ -62,7 +62,7 @@ public function getOrderDesc()
     }
 
     /**
-     * @param string $cardHolder
+     * @param  string  $cardHolder
      * @return PurchaseRequest
      */
     public function setCardHolder($cardHolder)
@@ -79,7 +79,7 @@ public function getCardHolder()
     }
 
     /**
-     * @param string $mobile
+     * @param  string  $mobile
      * @return PurchaseRequest
      */
     public function setMobile($mobile)
@@ -96,7 +96,7 @@ public function getMobile()
     }
 
     /**
-     * @param string $telNumber
+     * @param  string  $telNumber
      * @return PurchaseRequest
      */
     public function setTelNumber($telNumber)
@@ -113,7 +113,7 @@ public function getTelNumber()
     }
 
     /**
-     * @param string $email
+     * @param  string  $email
      * @return PurchaseRequest
      */
     public function setEmail($email)
@@ -130,7 +130,7 @@ public function getEmail()
     }
 
     /**
-     * @param string $address
+     * @param  string  $address
      * @return PurchaseRequest
      */
     public function setAddress($address)
@@ -147,7 +147,7 @@ public function getAddress()
     }
 
     /**
-     * @param string $memberId
+     * @param  string  $memberId
      * @return PurchaseRequest
      */
     public function setMemberId($memberId)
@@ -170,7 +170,7 @@ public function getMemberId()
      * Ibon、Famiport目前最大值為7天
      * ※僅限付款方式為2.3.4.6。
      *
-     * @param string $deadlineDate
+     * @param  string  $deadlineDate
      * @return PurchaseRequest
      */
     public function setDeadlineDate($deadlineDate)
@@ -189,7 +189,7 @@ public function getDeadlineDate()
     }
 
     /**
-     * @param string $deadlineTime
+     * @param  string  $deadlineTime
      * @return PurchaseRequest
      */
     public function setDeadlineTime($deadlineTime)
@@ -206,7 +206,7 @@ public function getDeadlineTime()
     }
 
     /**
-     * @param string $deadlineTime
+     * @param  string  $deadlineTime
      * @return PurchaseRequest
      */
     public function setUSN($deadlineTime)
@@ -224,6 +224,7 @@ public function getUSN()
 
     /**
      * @return array
+     *
      * @throws InvalidRequestException
      */
     public function getData()
@@ -254,7 +255,7 @@ public function getData()
     }
 
     /**
-     * @param array $data
+     * @param  array  $data
      * @return PurchaseResponse
      */
     public function sendData($data)

src/Message/PurchaseResponse.php

@@ -17,8 +17,8 @@ class PurchaseResponse extends AbstractResponse implements RedirectResponseInter
     /**
      * Constructor.
      *
-     * @param PurchaseRequest $request the initiating request.
-     * @param mixed $data
+     * @param  PurchaseRequest  $request the initiating request.
+     * @param  mixed  $data
      */
     public function __construct(PurchaseRequest $request, $data)
     {

src/Message/ReceiveTransactionInfoRequest.php

@@ -18,7 +18,7 @@ class ReceiveTransactionInfoRequest extends AbstractRequest
     use HasCVS;
 
     /**
-     * @param string $payEndDate
+     * @param  string  $payEndDate
      * @return ReceiveTransactionInfoRequest
      */
     public function setPayEndDate($payEndDate)
@@ -36,6 +36,7 @@ public function getPayEndDate()
 
     /**
      * @return array
+     *
      * @throws InvalidRequestException
      */
     public function getData()
@@ -57,7 +58,7 @@ public function getData()
     }
 
     /**
-     * @param mixed $data
+     * @param  mixed  $data
      * @return ResponseInterface
      */
     public function sendData($data)

src/Support/Helper.php

@@ -5,8 +5,8 @@
 class Helper
 {
     /**
-     * @param string $date
-     * @param string $format
+     * @param  string  $date
+     * @param  string  $format
      * @return string
      */
     public static function parseDate($date, $format = null)

src/Traits/HasBooking.php

@@ -5,7 +5,7 @@
 trait HasBooking
 {
     /**
-     * @param string $bookingId
+     * @param  string  $bookingId
      * @return $this
      */
     public function setBookingId($bookingId)
@@ -22,7 +22,7 @@ public function getBookingId()
     }
 
     /**
-     * @param string $custOrderNo
+     * @param  string  $custOrderNo
      * @return $this
      */
     public function setCustOrderNo($custOrderNo)
@@ -39,7 +39,7 @@ public function getCustOrderNo()
     }
 
     /**
-     * @param string $sendType
+     * @param  string  $sendType
      * @return $this
      */
     public function setSendType($sendType)