Changeset 38 for trunk

Timestamp:

04/08/08 23:50:22 (4 years ago)

Author:

tim

Message:

Implements and tests reserve. Some ugly bits are
starting to show: repeated code in parameter handling,
and hackery in the response parsing.

I'm going to keep working on making the API complete,
then we can start to refactor out the ugly bits. Perhaps
more patterns will emerge in the meantime.

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

@@ -98 +98 @@
                                       document.find("//" + bal).find("CurrencyCode").text)
 
-        if document.getroot().tag.find("PayResponse") >= 0:
+        # Hackish at best... 
+        root_tag = document.getroot().tag
+        if root_tag.find("PayResponse") >= 0 or root_tag.find("ReserveResponse") >= 0:
             self.transaction = TransactionResponse()
             self.transaction.id = document.find("//TransactionId").text
@@ -343 +345 @@
         pass
 
-    def reserve(self):
-        pass
+    def reserve(self,
+                caller_token,
+                sender_token,
+                recipient_token,
+                amount,
+                caller_reference,
+                date = None,
+                charge_fee_to='Recipient'):
+        params = {'Action': 'Reserve',
+                  'CallerTokenId': caller_token,
+                  'SenderTokenId': sender_token,
+                  'RecipientTokenId': recipient_token,
+                  'TransactionAmount.Amount': amount,
+                  'TransactionAmount.CurrencyCode': 'USD',
+                  'CallerReference': caller_reference,
+                  'ChargeFeeTo': charge_fee_to,
+                  }
+        if date is not None:
+            params['TransactionDate'] = date
+
+        return self.execute(params)
 
     def retryTransaction(self):

trunk/fpys/tests/client_test.py

@@ -161 +161 @@
     assert response.transaction.id == "133I77HJS56JVM7M54OZIRITRVLUT5F227U"
     assert response.transaction.status == "Initiated"
+
+def test_reserve():
+    response = fps_client.reserve("Z34XMGF4GCILGV7EV2D45DDO4Q6WXEJZ9175UNR5I9LFEC1H8MMX3R6NBJUJH8MQ",
+                                  "2146KQCZ13JKRP8BHCMI7JRHPTBAPZGU2VB9FTJ84UN7UF7LXAE33YJHSDB8XCG2",
+                                  "Z44XQGE4GAI1GV4E92DU5KDOTQPWXKJC91V5MNRII7LFICFH8HMX3RNNPJU4HCMN",
+                                  "19.95",
+                                  "unit_test_ref_1")
+    assert response.success == True
+    assert response.transaction.status == "Initiated"
+    assert response.transaction.id == "134OLF7MHB2L4V9T54RHADQ9FCK5NLVZHDC"
+
+    

trunk/fpys/tests/wsgi_responder.py

@@ -71 +71 @@
         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>"""
+        return [response]
+
 flexible_payment_service = FlexiblePaymentService()