Skip to content
Merged
Prev Previous commit
Next Next commit
Merge branch 'master' into PY-benOff-fields
  • Loading branch information
stymoshchuk authored Jul 30, 2024
commit f755847464be39f10dd94bfc1fe917db8bcc8daf
5 changes: 1 addition & 4 deletions e2e_tests/account_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -148,12 +148,9 @@ def test_update_credit_account():
account_id = "3344334"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have a way to not have this id hardcoded, but create an account for the test or fetch an existing one?

_credit_limit = 4000
request = PatchCreditAccountRequest(account_id, tags={
"purpose": "tax",
"trackUserId": "userId_fe6885b5815463b26f65e71095832bdd916890f7"})
"purpose": "tax"})
response = client.accounts.update(request)
assert response.data.type == "creditAccount"
assert response.data.attributes.get("tags").get("purpose") == "tax"


def test_get_deposit_products():
response = create_deposit_account()
Expand Down
1 change: 1 addition & 0 deletions e2e_tests/payment_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -156,6 +156,7 @@ def test_ach_received_payment_dto():
"status": "Completed",
"wasAdvanced": True,
"isAdvanceable": True,
"direction": "Credit",
"amount": 100000,
"completionDate": "2022-01-23",
"companyName": "Uber",
Expand Down
4 changes: 3 additions & 1 deletion e2e_tests/received_payment_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,8 @@ def test_ach_received_payment_dto():
"createdAt": "2022-02-01T12:03:14.406Z",
"status": "Pending",
"wasAdvanced": False,
"isAdvanceable": True,
"isAdvanceable": False,
"direction": "Credit",
"amount": 500000,
"completionDate": "2020-07-30",
"companyName": "UBER LTD",
Expand Down Expand Up @@ -105,6 +106,7 @@ def test_ach_received_payment_dto():
"status": "Completed",
"wasAdvanced": True,
"isAdvanceable": True,
"direction": "Credit",
"amount": 100000,
"completionDate": "2022-01-23",
"companyName": "Uber",
Expand Down
2 changes: 1 addition & 1 deletion unit/models/payment.py
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ def __init__(self, _id: str, created_at: datetime, status: AchReceivedPaymentSta
@staticmethod
def from_json_api(_id, _type, attributes, relationships):
return AchReceivedPaymentDTO(_id, date_utils.to_datetime(attributes["createdAt"]), attributes["status"],
attributes["wasAdvanced"], attributes.get("isAdvanceable", None), date_utils.to_date(attributes["completionDate"]),
attributes["wasAdvanced"], attributes.get("isAdvanceable"), attributes["direction"], date_utils.to_date(attributes["completionDate"]),
attributes.get("returnReason"), attributes["amount"], attributes["description"],
attributes.get("addenda"), attributes.get("companyName"),
attributes.get("counterpartyRoutingNumber"), attributes.get("traceNumber"),
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.