Skip to content

Commit

Permalink
Merge pull request #49 from zakiarsyad/CC-6526/add-metadata
Browse files Browse the repository at this point in the history
[CC-6526] Add metadata field for credit card function
  • Loading branch information
mesmur authored May 19, 2022
2 parents 2b17c78 + 99a6b19 commit ae0c9b4
Show file tree
Hide file tree
Showing 6 changed files with 42 additions and 3 deletions.
16 changes: 14 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -257,6 +257,9 @@ charge = CreditCard.create_authorization(
external_id="card_preAuth-1594106356",
amount=75000,
card_cvn="123",
metadata={
"meta": "data",
},
)
print(charge)
```
Expand All @@ -282,7 +285,10 @@ Will return
"bank_reconciliation_id": "5941063625146828103011",
"approval_code": "831000",
"created": "2020-07-07T07:19:22.921Z",
"id": "5f0421fa8cc1e8001973a1d6"
"id": "5f0421fa8cc1e8001973a1d6",
"metadata": {
"meta": "data"
}
}
```

Expand Down Expand Up @@ -323,6 +329,9 @@ charge = CreditCard.create_charge(
external_id="card_charge-1594106478",
amount=75000,
card_cvn="123",
metadata={
"meta": "data",
},
)
print(charge)
```
Expand All @@ -348,7 +357,10 @@ Will return
"bank_reconciliation_id": "5941064846646923303008",
"approval_code": "831000",
"created": "2020-07-07T07:21:25.027Z",
"id": "5f0422752bbbe50019a368b5"
"id": "5f0422752bbbe50019a368b5",
"metadata": {
"meta": "data"
}
}
```

Expand Down
9 changes: 9 additions & 0 deletions tests/integration/test_credit_card.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,9 @@ def test_create_authorization_return_correct_keys(self, CreditCard):
external_id=f"card_preAuth-{int(time.time())}",
amount=75000,
card_cvn="123",
metadata={
"meta": "data",
},
)

self.assert_returned_object_has_same_key_as_sample_response(
Expand All @@ -33,6 +36,9 @@ def test_reverse_authorization_return_correct_keys(self, CreditCard):
external_id=f"card_preAuth-{int(time.time())}",
amount=75000,
card_cvn="123",
metadata={
"meta": "data",
},
)
reverse_auth = CreditCard.reverse_authorization(
credit_card_charge_id=charge.id,
Expand All @@ -49,6 +55,9 @@ def test_create_charge_return_correct_keys(self, CreditCard):
external_id=f"card_preAuth-{int(time.time())}",
amount=75000,
card_cvn="123",
metadata={
"meta": "data",
},
)

self.assert_returned_object_has_same_key_as_sample_response(
Expand Down
4 changes: 3 additions & 1 deletion tests/sampleresponse/credit_card.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
def reverse_auth_response():
return {
"status": "SUCCEEDED",
"currency": "IDR",
"credit_card_charge_id": "5f0421fa8cc1e8001973a1d6",
"business_id": "5ed75086a883856178afc12e",
"external_id": "card_preAuth-1594106356",
Expand Down Expand Up @@ -31,6 +30,9 @@ def charge_response():
"approval_code": "831000",
"created": "2020-07-07T07:19:22.921Z",
"id": "5f0421fa8cc1e8001973a1d6",
"metadata": {
"meta": "data",
},
}


Expand Down
6 changes: 6 additions & 0 deletions tests/unit/models/creditcard/test_create_authorization.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,9 @@ def default_credit_card_data(self):
"amount": 75000,
"card_cvn": "123",
"x_idempotency_key": "test-idemp_123",
"metadata": {
"meta": "data",
},
}
params = (args, kwargs)
url = "/credit_card_charges"
Expand All @@ -35,6 +38,9 @@ def api_requestor_request_data(self, default_credit_card_data):
"external_id": "mock_card_preAuth-123",
"amount": 75000,
"card_cvn": "123",
"metadata": {
"meta": "data",
},
}
return (tested_class, class_name, method_name, http_method_name, url, params, headers, body)

Expand Down
6 changes: 6 additions & 0 deletions tests/unit/models/creditcard/test_create_charge.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,9 @@ def default_credit_card_data(self):
"billing_details": billing_details,
"installment": installment,
"promotion": promotion,
"metadata": {
"meta": "data",
},
}
params = (args, kwargs)
url = "/credit_card_charges"
Expand Down Expand Up @@ -64,6 +67,9 @@ def api_requestor_request_data(self, default_credit_card_data):
"reference_id": "Xendit-123",
"original_amount": 75000
},
"metadata": {
"meta": "data",
},
}
return (tested_class, class_name, method_name, http_method_name, url, params, headers, body)

Expand Down
4 changes: 4 additions & 0 deletions xendit/models/creditcard/credit_card.py
Original file line number Diff line number Diff line change
Expand Up @@ -160,6 +160,7 @@ def create_authorization(
x_idempotency_key=None,
for_user_id=None,
x_api_version=None,
metadata=None,
**kwargs,
):
"""Send POST Request to create Credit Card Authorization (API Reference: Credit Card/Create Authorization)
Expand All @@ -180,6 +181,7 @@ def create_authorization(
- **x_idempotency_key (str)
- **for_user_id (str)
- **x_api_version (str)
- **metadata (dict)
Returns:
Expand Down Expand Up @@ -255,6 +257,7 @@ def create_charge(
x_idempotency_key=None,
for_user_id=None,
x_api_version=None,
metadata=None,
**kwargs,
):
"""Send POST Request to create Credit Card Charge (API Reference: Credit Card/Create Charge)
Expand All @@ -275,6 +278,7 @@ def create_charge(
- **x_idempotency_key (str)
- **for_user_id (str)
- **x_api_version (str)
- **metadata (dict)
Returns:
CreditCardCharge
Expand Down

0 comments on commit ae0c9b4

Please sign in to comment.