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
Initial auth for endpoints #8
Conversation
d275aa7
to
25374db
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, bare et par spørsmål. Og så trenger den en liten rebasing nå. ;)
tox.ini
Outdated
@@ -4,13 +4,19 @@ envlist = py39, flake8, black | |||
[testenv] | |||
deps = | |||
pytest | |||
responses |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hva er forskjellen på denne og requests-mock
som vi har brukt andre steder?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lurte på om starlette/fastapi TestClient
kanskje lekte litt bedre med responses
enn requests-mock
, uten å måtte aktivere real_http
. Skal dog sies jeg ikke undersøkte så mye nærmere, og litt for å prøve noe annet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah! Satt akkurat med det problemet tidligere i uka faktisk... Ser ut som responses
er mer populært også, 3.3k stjerner på GitHub vs 260.
7d8d27b
to
7db4b92
Compare
Men denne må kanskje settes litt på pause nå som vi ikke skal ha det selvbetjent i første omgang? |
No description provided.