Feat: update funding source expiry from concession group

[thekaveman]

Closes #29

Implements a generic PUT handler in the client, returning a ListResponse per the known API specs.

Implements update_concession_group_funding_source_expiry method to update the expiration of an existing funding source in a concession group. Parses the ListResponse and returns the first .list item. This should be the only list item according to the known API specs.

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
littlepay/api
__init__.py					120
client.py
groups.py
Project Total

This report was generated by python-coverage-comment-action

[thekaveman]

Question @angela-tran:

Do you think it makes sense to create a dataclass for the structure emitted by this PUT response? I.e.

{
  "id": "9002b586-9c53-43c6-b274-5615adb856ee",
  "participant_id": "test_participant",
  "concession_expiry": "2024-07-03T00:00:00Z",
  "concession_created_at": "2024-07-03T00:00:00Z",
  "concession_updated_at": "2024-07-03T00:00:00Z"
}

We could do some parsing of the datetime to make that easier to consume, and it would also be reused for #28. Or just leave it as a dict?

Thoughts?

[angela-tran]

Question @angela-tran:

Do you think it makes sense to create a dataclass for the structure emitted by this PUT response? I.e.
...
We could do some parsing of the datetime to make that easier to consume, and it would also be reused for #28. Or just leave it as a dict?

Thoughts?

I think as a consumer, I would want to have the datetime objects rather than having to do the conversion myself, so the dataclass sounds good to me.

[thekaveman]

Thanks @angela-tran, PR is updated with a new dataclass and now ready for review.

Seeing a test failure in Python 3.10 related to parsing a datetime from ISO 8601 format 😠

Yep, it seems like before Python 3.11, datetime.fromisoformat() only supported the exact format output from datetime.isoformat() i.e. without the trailing Z character and with a time zone offset from UTC like +00:00 https://docs.python.org/3.11/library/datetime.html#datetime.datetime.fromisoformat

Return a datetime corresponding to a date_string in any valid ISO 8601 format...

Changed in version 3.11: Previously, this method only supported formats that could be emitted by date.isoformat() or datetime.isoformat().

I could update the code to handle this case, or we could move up to Python 3.11+ ... thoughts?

It felt easier to update the code than worry about changing compatibility with Python versions. Although benefits is using Python 3.11, it seems easier to support 3.10 for now.

[angela-tran]

This looks good to me 👍