Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix tests with aiohttp >= 3.10.0 #974

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

cjwatson
Copy link
Contributor

@cjwatson cjwatson commented Aug 5, 2024

Description:

aiohttp 3.10.0 removed the ability to create aiohttp.ClientSession instances without a running event loop (see
aio-libs/aiohttp#8583), but a few of blinkpy's tests relied on that.

Checklist:

  • Local tests with tox run successfully PR cannot be merged unless tests pass
  • Changes tested locally to ensure platform still works as intended (not done because I don't actually use this package, but this is a test-only change)
  • Tests added to verify new code works

aiohttp 3.10.0 removed the ability to create `aiohttp.ClientSession`
instances without a running event loop (see
aio-libs/aiohttp#8583), but a few of blinkpy's
tests relied on that.
@fronzbot
Copy link
Owner

fronzbot commented Aug 8, 2024

Awesome thanks for the PR!

@fronzbot fronzbot merged commit e2c747b into fronzbot:dev Aug 8, 2024
10 of 11 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants