Merge pull request #33 from mikealmond/patch-1

[AIMRefund] Pass credit card expiration date only when one is present

Author: delatbabel

src/Message/AIMRefundRequest.php

@@ -17,7 +17,12 @@ public function getData()
 
         $data['x_trans_id'] = $this->getTransactionReference();
         $data['x_card_num'] = $this->getCard()->getNumber();
-        $data['x_exp_date'] = $this->getCard()->getExpiryDate('my');
+
+        $expiryMonth = $this->getCard()->getExpiryMonth();
+        if (!empty($expiryMonth)) {
+            $data['x_exp_date'] = $this->getCard()->getExpiryDate('my');
+        }
+
         $data['x_amount'] = $this->getAmount();
 
         return $data;

tests/Message/AIMRefundRequestTest.php

@@ -29,5 +29,24 @@ public function testGetData()
         $this->assertSame('60O2UZ', $data['x_trans_id']);
         $this->assertSame($card['number'], $data['x_card_num']);
         $this->assertSame('12.00', $data['x_amount']);
+
+        $this->assertArrayHasKey('x_exp_date', $data);
+    }
+
+    public function testRefundWithSimplifiedCard()
+    {
+        $simplifiedCard = array(
+            'firstName' => 'Example',
+            'lastName' => 'User',
+            'number' => '1111',
+        );
+
+        $this->request->setCard($simplifiedCard);
+
+        $data = $this->request->getData();
+
+        $this->assertSame($simplifiedCard['number'], $data['x_card_num']);
+
+        $this->assertArrayNotHasKey('x_exp_date', $data);
     }
 }