Aiohttp: Tester l'erreur de boucle client avec l'application réelle

Longue histoire courte

Le modèle de test suggéré nécessite une duplication standard du code de l'application et ne montre pas le test de l'application directement.

https://docs.aiohttp.org/en/stable/testing.html#pytest

Comportement attendu

Je devrais pouvoir passer mon module d'application directement à test_client pour tester mon application.

Comportement réel

Les documents montrent la construction de l'application à partir de gestionnaires à la volée avant chaque test. Passer le vrai app directement à test_client fonctionne bien la première fois, puis échoue avec "RuntimeError: web.Application instance initialized with different loop".

Étapes à reproduire

# handler.py

async def foo():
    return 'foo'

# app.py

from aiohttp import web
from .handler import foo

app = web.Application()
app.add_routes([web.get('/foo/', foo)])

# test.py

from .app import app

async def test_foo_status(test_client):
    response = await client.get('/foo/')
    assert response.status == 200

async def test_foo_body(test_client):
    response = await client.get('/foo/')
    body = await response.text()
    assert body == 'foo'

Votre environnement

• Système d'exploitation : Ubuntu 18.04
• Python : 3.6.5
• aiohttp : 3.3.2

Solution de contournement

J'ai piraté le problème pour l'instant en jetant un coup d'œil au code source et en déterminant la cause de l'erreur.

        if self._loop is not None and self._loop is not loop:
            raise RuntimeError(
                "web.Application instance initialized with different loop")

https://github.com/aio-libs/aiohttp/blob/e561eaa/aiohttp/web_app.py#L134

Je ne comprends pas immédiatement pourquoi ce code est nécessaire. Je n'ai trouvé aucun contexte dans l'historique de git. La définition de app._loop = None dans ma méthode de nettoyage de test semble éviter le problème sans aucun effet secondaire négatif. Ce code est-il vraiment nécessaire ? Ce correctif pourrait-il être intégré dans test_client ?