review comments resolved

Author: mgovindhasamy

hyperwallet/models.py

@@ -822,16 +822,20 @@ def __init__(self, data):
         self.defaults = {
             'token': None,
             'status': None,
-            'clientRefundId': None,
+            'createdOn': None,
+            'clientTransferId': None,
             'sourceToken': None,
             'sourceAmount': None,
+            'sourceFeeAmount': None,
             'sourceCurrency': None,
             'destinationToken': None,
             'destinationAmount': None,
+            'destinationFeeAmount': None,
             'destinationCurrency': None,
-            'createdOn': None,
+            'foreignExchanges': None,
             'notes': None,
-            'memo': None
+            'memo': None,
+            'expiresOn': None
         }
 
         for (param, default) in self.defaults.items():

hyperwallet/tests/test_api.py

@@ -2219,20 +2219,20 @@ def test_uploadDocumentsForUser_success(self, mock_put):
     '''
 
     @mock.patch('hyperwallet.utils.ApiClient._makeRequest')
-    def test_transfer_refunds_success(self, mock_post):
+    def test_create_transfer_refunds_success(self, mock_post):
 
         mock_post.return_value = self.data
         response = self.api.createTransferRefund('token',self.data)
         self.assertTrue(response.token, self.data.get('token'))
 
-    def test_transfer_refunds_fail_need_payment_token(self):
+    def test_create_transfer_refunds_fail_need_payment_token(self):
 
         with self.assertRaises(HyperwalletException) as exc:
             self.api.createTransferRefund()
 
         self.assertEqual(exc.exception.message, 'transferToken is required')
 
-    def test_transfer_refunds_fail_need_data(self):
+    def test_create_transfer_refunds_fail_need_data(self):
 
         with self.assertRaises(HyperwalletException) as exc:
             self.api.createTransferRefund('token')
@@ -2245,22 +2245,22 @@ def test_transfer_refunds_fail_need_data(self):
 
     '''
 
-    def test_transfer_spend_back_refunds_user_token(self):
+    def test_create_transfer_spend_back_refunds_user_token(self):
 
         with self.assertRaises(HyperwalletException) as exc:
             self.api.createTransferSpendBackRefund()
 
         self.assertEqual(exc.exception.message, 'transferToken is required')
 
-    def test_transfer_spend_back_refunds_fail_need_payment_token(self):
+    def test_create_transfer_spend_back_refunds_fail_need_payment_token(self):
 
         with self.assertRaises(HyperwalletException) as exc:
             self.api.createTransferSpendBackRefund('token')
 
         self.assertEqual(exc.exception.message, 'sourceToken is required')
 
     @mock.patch('hyperwallet.utils.ApiClient._makeRequest')
-    def test_transfer_spend_back_refunds_success(self, mock_get):
+    def test_create_transfer_spend_back_refunds_success(self, mock_get):
 
         mock_get.return_value = self.data
         response = self.api.createTransferSpendBackRefund('token', 'token')