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.

Sign up for GitHub

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

Jump to bottom

Use teapot.fly.dev over httpstat.us in tests #148

Merged
merged 2 commits into from
Oct 25, 2023
Merged

Use teapot.fly.dev over httpstat.us in tests #148

merged 2 commits into from
Oct 25, 2023

Conversation

lukasschwab
Copy link
Owner

@lukasschwab lukasschwab commented Oct 25, 2023 •
edited
Loading

Description

Theory: simple fly.io-hosted service might be more universally available, and easier for me to run locally. I observed some httpstat.us downtime.

Breaking changes

List any changes that break the API usage supported on master.

None; only tests.

Relevant issues

List GitHub issues relevant to this change.

Checklist

  • (If appropriate) README.md example usage has been updated.

@lukasschwab
Use my teapot service over httpstat.us in tests
b1aaa0c 
Theory: simple fly.io-hosted service might be more universally
available, and easier for me to run locally. I observed some httpstat.us
downtime.
@lukasschwab
Remove lingering httpstat.us reference
9e9b6d8
@lukasschwab lukasschwab changed the title Use my teapot service over httpstat.us in tests Use teapot.fly.dev over httpstat.us in tests Oct 25, 2023
@lukasschwab lukasschwab merged commit 53d59a2 into master Oct 25, 2023
@lukasschwab lukasschwab deleted the teapot-over-httpstatus branch October 25, 2023 19:45
@lukasschwab lukasschwab mentioned this pull request Oct 25, 2023
Closed
lukasschwab added a commit that referenced this pull request Oct 25, 2023
@lukasschwab
Mock requests.Session.get in TestClient
a21281b 
+ Eliminates dependency on teapot.fly.dev; see #148.

+ Fixes #149.

In the future we should store some and mock some arXiv API responses. At
the very least, this offers some backup for when the API is down; of
course, it introduces the risk of divergence from the live API.
@lukasschwab lukasschwab mentioned this pull request Oct 25, 2023
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lukasschwab