orbit-oss/flask
4
5
Fork 2
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Raise a runtime error if run_async is called without real ContextVars

Browse source 
Werkzeug offers a ContextVar replacement for Python < 3.7, however it
doesn't work across asyncio tasks, hence it makes sense to error out
rather than find there are odd bugs.

Note the docs build requires the latest (dev) Werkzeug due to this
change (to import ContextVar from werkzeug.local).
This commit is contained in:
pgjones 2021-02-10 21:14:58 +00:00 • committed by David Lord
parent c9f774d650
commit 47e0f2aab8
3 changed files with 19 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

9
tests/test_async.py
View file

@ -1,10 +1,12 @@
import asyncio
import sys
import pytest
from flask import abort
from flask import Flask
from flask import request
from flask.helpers import run_async
@pytest.fixture(name="async_app")
@ -23,6 +25,7 @@ def _async_app():
return app
@pytest.mark.skipif(sys.version_info < (3, 7), reason="requires Python >= 3.7")
def test_async_request_context(async_app):
test_client = async_app.test_client()
response = test_client.get("/")
@ -31,3 +34,9 @@ def test_async_request_context(async_app):
assert b"POST" in response.get_data()
response = test_client.get("/error")
assert response.status_code == 412
@pytest.mark.skipif(sys.version_info >= (3, 7), reason="should only raise Python < 3.7")
def test_async_runtime_error():
with pytest.raises(RuntimeError):
run_async(None)