New connector for working with the Catalist Match API

[austinweisgrau]

Connector for working with the Catalist Match API.

The Catalist Match tool requires OAuth2.0 client credentials for the API as well as
credentials for accessing the Catalist sftp bucket. Each Catalist client is given
their own bucket alias named after a tree species, used for constructing the
filepath within the sftp bucket.

Accessing the Catalist sftp bucket and Match API both require the source IP address
to be explicitly white-listed by Catalist.

Example usage:

tbl = Table.from_csv(...)
client = CatalistMatch(...)
match_result = client.match(tbl)

Tests

Tests are included in this PR.

Changes to parsons

Two changes are made to the parsons package outside of this new connector.

28949a9 makes a small change to the api_connector error handler to look for error messages in a response text attribute, where the error message is found in responses from the Catalist API.

1ffb046 adds a development dependency pytest-mock to support the ergonomic use of mocking in pytest. This dependency is then used in the tests on the catalist match connector in 31f3a18.

Expected future changes

Following a community development session, JSON responses from the API will be parsed using python dataclasses. Currently JSON responses are returned raw, which matches the normal pattern across parsons connectors.

[austinweisgrau]

Working on updates to type hints to be backwards compatible with Python versions back to 3.8

[austinweisgrau]

@IanRFerguson or @shaunagm Do you know what's up with that docker failure? When I hit details I just get {"error": "Our service is temporarily unavailable. We'll be back soon"}

[austinweisgrau]

@IanRFerguson or @shaunagm Do you know what's up with that docker failure? When I hit details I just get {"error": "Our service is temporarily unavailable. We'll be back soon"}

NVM I got it with 001b527

[IanRFerguson]

Looks excellent overall - you crushed this! I have a few comments throughout but nothing structural or functional, the code is super legible and easy to navigate.

One note - I think adding a docs/catalist.rst file is required before merging. I left a comment on setup / obtaining client creds and I reckon the docs would be a good place to outline that

[austinweisgrau]

Looks excellent overall - you crushed this! I have a few comments throughout but nothing structural or functional, the code is super legible and easy to navigate.

One note - I think adding a docs/catalist.rst file is required before merging. I left a comment on setup / obtaining client creds and I reckon the docs would be a good place to outline that

Added the docs/catalist.rst file with 9b73523, that was new for me though so not totally sure I did that correctly.

[IanRFerguson]

Thanks for addressing the comments so thoroughly! Looks great

[shaunagm]

Is this used in the tests? I see putest and requests-mock being imported but not pytest-mock

[austinweisgrau]

It creates a pytest fixture called "mocker", which is used in the pytest fixtures in this file