Changeset 40

Timestamp:

04/09/08 02:54:20 (4 years ago)

Author:

tim

Message:

Implemented refund. Learned that a new token is required to initiate
a refund. The doc says "The token of the original recipient who is now
the sender in the refund" which I took to mean the same token id was
required. This is not true.

Location:

trunk/fpys

Files:

• client.py (modified) (2 diffs)
• tests/client_test.py (modified) (1 diff)
• tests/wsgi_responder.py (modified) (1 diff)

trunk/fpys/client.py

@@ -100 +100 @@
         # Hackish at best... 
         root_tag = document.getroot().tag
-        for tag_name in ["PayResponse", "ReserveResponse", "SettleResponse"]:
+        for tag_name in ["PayResponse", "RefundResponse", "ReserveResponse", "SettleResponse"]:
             if self.success and root_tag.find(tag_name) >= 0:
                 self.transaction = TransactionResponse()
@@ -343 +343 @@
         return self.execute(params)
 
-    def refund(self):
-        pass
+    def refund(self,
+               caller_token,
+               refund_sender_token,
+               transaction_id,
+               caller_reference,
+               refund_amount=None,
+               date = None,
+               charge_fee_to='Recipient',
+               ):
+        params = {'Action': 'Refund',
+                  'CallerTokenId': caller_token,
+                  'RefundSenderTokenId': refund_sender_token,
+                  'TransactionId': transaction_id,
+                  'CallerReference': caller_reference,
+                  'ChargeFeeTo': charge_fee_to,
+                  }
+        if date is not None:
+            params['TransactionDate'] = date
+        if refund_amount is not None:
+            params['RefundAmount.Amount'] = refund_amount
+            params['RefundAmount.CurrencyCode'] = "USD"
+        return self.execute(params)
 
     def reserve(self,

trunk/fpys/tests/client_test.py

@@ -162 +162 @@
     assert response.transaction.status == "Initiated"
 
+def test_refund():
+    # the following was used to create a new token to authorize the refund
+    #     fps_client.installPaymentInstruction("MyRole == 'Sender';\nOperationType == 'Refund';",
+    #                                          "unit_test_refund_01",
+    #                                          "SingleUse",
+    #                                          "Refund of Test Transaction",
+    #                                          "unit_test_refund_01")
+    response = fps_client.refund(caller_token="Z34XMGF4GCILGV7EV2D45DDO4Q6WXEJZ9175UNR5I9LFEC1H8MMX3R6NBJUJH8MQ",
+                                 refund_sender_token="Z74XVGZ4G2IKGVIE52D453DOEQGWXMJ491V58NR6I3LFQC2H8BM73R8NMJUJHCMN",
+                                 transaction_id="134OLF7MHB2L4V9T54RHADQ9FCK5NLVZHDC",
+                                 caller_reference="Unit Test Refund",
+                                 refund_amount="19.95")
+    assert response.success == True
+    assert response.transaction.status == "Initiated"
+    assert response.transaction.id == "134P2CRSN5JFN3KDV3RKPKVQ3OG4H67PPR8"
+
 def test_reserve():
     response = fps_client.reserve("Z34XMGF4GCILGV7EV2D45DDO4Q6WXEJZ9175UNR5I9LFEC1H8MMX3R6NBJUJH8MQ",

trunk/fpys/tests/wsgi_responder.py

@@ -71 +71 @@
         return [response]
 
+    def Refund(self, environ):
+        response = """<ns0:RefundResponse xmlns:ns0="http://fps.amazonaws.com/doc/2007-01-08/"><ns0:TransactionResponse><TransactionId>134P2CRSN5JFN3KDV3RKPKVQ3OG4H67PPR8</TransactionId><Status>Initiated</Status></ns0:TransactionResponse><Status>Success</Status><RequestId>b1c83ba0-17d3-45c4-b188-b116d2a17c12:0</RequestId></ns0:RefundResponse>"""
+        return [response]
+
     def Reserve(self, environ):
         response = """<ns0:ReserveResponse xmlns:ns0="http://fps.amazonaws.com/doc/2007-01-08/"><ns0:TransactionResponse><TransactionId>134OLF7MHB2L4V9T54RHADQ9FCK5NLVZHDC</TransactionId><Status>Initiated</Status></ns0:TransactionResponse><Status>Success</Status><RequestId>cedef0ad-76f0-4604-82bb-ad28020a4ddc:0</RequestId></ns0:ReserveResponse>"""