Mark reversed payments failed

Payum · Dec 29, 2016 · 10e56fc · 10e56fc
1 parent 9f53df6
commit 10e56fc
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/Action/PaymentDetailsStatusAction.php b/Action/PaymentDetailsStatusAction.php
@@ -136,6 +136,7 @@ public function execute($request)
                     Api::PAYMENTSTATUS_FAILED,
                     Api::PAYMENTSTATUS_EXPIRED,
                     Api::PAYMENTSTATUS_DENIED,
+                    Api::PAYMENTSTATUS_REVERSED,
                     Api::PAYMENTSTATUS_CANCELED_REVERSAL,
                 );
                 if (in_array($paymentStatus, $failedStatuses)) {

diff --git a/Tests/Action/PaymentDetailsStatusActionTest.php b/Tests/Action/PaymentDetailsStatusActionTest.php
@@ -294,6 +294,25 @@ public function shouldMarkFailedIfAtLeastOnePaymentStatusFailed()
         $this->assertTrue($request->isFailed());
     }
 
+    /**
+     * @test
+     */
+    public function shouldMarkFailedIfAtLeastOnePaymentStatusReversed()
+    {
+        $action = new PaymentDetailsStatusAction();
+
+        $request = new GetHumanStatus(array(
+            'PAYMENTREQUEST_0_AMT' => 12,
+            'CHECKOUTSTATUS' => Api::CHECKOUTSTATUS_PAYMENT_COMPLETED,
+            'PAYMENTREQUEST_0_PAYMENTSTATUS' => Api::PAYMENTSTATUS_COMPLETED,
+            'PAYMENTREQUEST_9_PAYMENTSTATUS' => Api::PAYMENTSTATUS_REVERSED,
+        ));
+
+        $action->execute($request);
+
+        $this->assertTrue($request->isFailed());
+    }
+
     /**
      * @test
      */