I admit I don't understand the async story in Python. I assumed directly returning a coroutine in an async function would be equivalent.

Could you write a test for this? You should be able to do this:

init(integrations=[AioHttpIntegration()])
init()

and copy the rest from an existing test.

Codecov Report

Merging #261 into master will decrease coverage by 0.72%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #261      +/-   ##
==========================================
- Coverage   81.74%   81.01%   -0.73%     
==========================================
  Files          32       32              
  Lines        2750     2750              
  Branches      444      444              
==========================================
- Hits         2248     2228      -20     
- Misses        340      362      +22     
+ Partials      162      160       -2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3f9d748...268c7bb. Read the comment docs.

The problem is, that if you call a Coroutine (something with async def) you get a coroutine object back. To get the actual return (or exception or whatever is the result of the coroutine) you need to await it. So returning a coroutine from a coroutine (without the await inside) means, that you need to write await twice.

I'll check the test situation.

nvm, yes, brainfart. I think I was too much in Rust's async support where using a return means you would have to return an actual future.

Btw, urllib3 and certifi seems to be missing in the test requirements

I assume you did this:

pip install -r test-requirements.txt

while I usually do this in addition:

pip install -e .

try this: sh scripts/runtox.sh aiohttp

There are a lot of "optional test requirements" because some things don't work in certain python versions.

There is now a unittest that fails without the fix and works with it :)

Awesome, thanks!