Support pytest_aiohttp

[ikhlestov]

At the present pytest_aiohttp doesn't work correct with pytest_asyncio (pytest_aiohttp issue and pytest_asyncio issue ).
On the other hand aresponses doesn't require pytest_asyncio directly and can be executed without it. But aresponses depends on event_loop fixture here. So the main questions - can we use existing fixture event_loop from asyncio or loop from aiohttp on the fly for the aresponses fixture initialization?

"""Test script to show aresponses work without pytest_asyncio
Steps:
- install all required packages as `pip install pytest aiohttp pytest_aiohttp aresponses`
- run script as `pytest test_example.py -p no:asyncio
"""
import pytest
import aiohttp


# Simple server definition
async def handle(request):
    return aiohttp.web.Response(text="test response")


def create_app():
    app = aiohttp.web.Application()
    app.add_routes([aiohttp.web.get('/', handle)])
    return app


# Tests
@pytest.fixture
async def client(aiohttp_client):
    app = create_app()
    return await aiohttp_client(app)


async def test_root_url(client):
    resp = await client.get('/')
    assert resp.status == 200
    assert await resp.text() == "test response"


# this additional fixture required by aresponses
# by default aresponses depends on `event_loop` fixture defined at the pytest_asyncio
@pytest.fixture
def event_loop(loop):
    return loop


async def test_aresponses_without_aiohttp(aresponses):
    aresponses.add('foo.com', '/', 'get', 'hi there!!')
    url = 'http://foo.com'
    async with aiohttp.ClientSession() as session:
        async with session.get(url) as response:
            text = await response.text()
            assert text == 'hi there!!'

[brycedrennan]

If you just define your own fixture in conftest.py will that work?

from aresponses import ResponsesMockServer

@pytest.fixture
async def aresponses(loop):
    async with ResponsesMockServer(loop=loop) as server:
        yield server

[ikhlestov]

Yes, it works, thank you. I think it's a nice resolution. Should I create a PR with readme update with additional example for aresponses+aiohttp_client or there is no reason for it?

[brycedrennan]

That would be great thank you!

[XayOn]

Yet pytest-asyncio is included in the setup.py as installs_requires.
Thus provoking the aformentioned bug ( aio-libs/pytest-aiohttp#8 ).

[XayOn]

Also, latest pytest releases will await coroutines just fine without pytest-asyncio.

[brycedrennan]

@XayOn do you have some documentation that shows pytest now supports asyncio? I found this which implies it doesn’t.
pytest-dev/pytest#2175