chore: switch requests_testadapter for requests_mock#8595
chore: switch requests_testadapter for requests_mock#859506kellyjac wants to merge 1 commit intoinvoke-ai:mainfrom
Conversation
requests_testadapter is an archived repository Carries on from invoke-ai#4252 (comment)
|
Thanks! Unfortunately, If we want to migrate, https://github.com/getsentry/responses appears to be the de-facto choice and is actively maintained. Looks like there is a bit more to it than the other libraries but it's still pretty small. |
|
ah, darn. I'll rework this 🙂 Also I'm not sure why my imports are in the wrong order. I thought I ran ruff. Maybe it was affected when I rebased. |
|
Maybe the venv wasn't activated when running As for |
|
Ah ok. Yeah, it's just it downloads a tonne of stuff on to a drive I don't have much space on. 😅 I'll work something out |
|
Author has gone offline. Closing request. |
Summary
requests_testadapter is an archived repository
Carries on from #4252 (comment)
Implementation using
requests_mockis broadly similar and I wouldn't say it's much more complex thanrequests_testadapterTODO still need to update uv lock, help updating this would be good. Also wanted to raise the PR to get feedback now.
Related Issues / Discussions
relates to #4252
QA Instructions
Rerunning the tests is sufficient.
I've not ran the longer/more extensive suite but the default tests all run.
Merge Plan
Non user facing code so shouldn't require a merge plan.
Checklist
What's Newcopy (if doing a release after this PR)