Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/rework test cassettes recording #1004

Merged
merged 4 commits into from Mar 6, 2022

Conversation

lavigne958
Copy link
Collaborator

Updated the test suite setup so a single cassette is used
for each test.

As usual skip the first commit.

This cassette will hold

  • the spreadsheet creation
  • any interaction during the test
  • the spreadsheet deletion

Use the new exponential backoff client.
Skip recording requests that return an error due to usage limit
exceeded.

Add a new client that provides exponential backoff retries.
This is useful for the test suite to retry the requests
if they encounter an API usage rate limit.

This Client instance is not production ready yet.
To use with caution.

Hopefully this is the last big update of all test cassettes.

closes #1003

Updatesd the test cassette to have a single cassette per test
Add a new client that provides exponential backoff retries.
This is usefull for the test suite to retry the requests
if they encounter an API usage rate limit.

This `Client` instance is not producation ready yet.
To use with caution.

I can make the request retry up to a couple of minutes for requests
that should fail on the first try.
The current list of matching codes when the client should retry
is not exhaustive enough and will be improved in the future.
Updated the test suite setup so a single cassette is used
for each test. This cassette will hold

- the spreadsheet creation
- any interaction during the test
- the spreadsheet deletion

Use the new exponential backoff client.
Skip recording requests that return an error due to usage limit
exceeded.
Add extra documentation and warnings about the new implementation
of the gspread client `BackoffClient` as it is not production ready yet.
@lavigne958 lavigne958 merged commit d9ffc30 into master Mar 6, 2022
@lavigne958 lavigne958 deleted the feature/rework_test_cassettes_recording branch March 6, 2022 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update tests cassettes (again)
1 participant