test: issues with multiple api keys - Githubissues

ApeWorX / ape-infura

Infura network provider plugin for the Ape Framework

https://www.apeworx.io/

Apache License 2.0

8 stars 19 forks source link

test: issues with multiple api keys #89

Closed antazoey closed 2 months ago

antazoey commented 2 months ago

What I did

tests were failing from the merging of this PR: https://github.com/ApeWorX/ape-infura/pull/88 The issues were from API keys not resetting.

How I did it

instead of "load_api_keys", make api keys a cached property and have it get deleted on disconnect also in the tests, i had to reset the env patch

How to verify it

Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)