From ade6bae47271c15a32397c33f8ecb022d0fe4c58 Mon Sep 17 00:00:00 2001 From: Simon Willison Date: Fri, 3 May 2019 21:25:49 -0400 Subject: [PATCH] Ensure test_black.py executes first under pytest Otherwise it throws an exception because it gets disrupted by the asyncio activity in the other tests. --- tests/conftest.py | 6 ++++++ tests/test_black.py | 4 ---- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/tests/conftest.py b/tests/conftest.py index e7a3da6b..89db0281 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -8,3 +8,9 @@ def pytest_unconfigure(config): import sys del sys._called_from_test + + +def pytest_collection_modifyitems(items): + # Ensure test_black.py runs first before any asyncio code kicks in + test_black = [fn for fn in items if fn.name == "test_black"][0] + items.insert(0, items.pop(items.index(test_black))) diff --git a/tests/test_black.py b/tests/test_black.py index 37fca5a6..b5bfcfd0 100644 --- a/tests/test_black.py +++ b/tests/test_black.py @@ -7,10 +7,6 @@ import sys code_root = Path(__file__).parent.parent -# @pytest.mark.skipif( -# sys.version_info[:2] > (3, 6), -# reason="Breaks on 3.7 at the moment, but it only needs to run under one Python version", -# ) def test_black(): runner = CliRunner() result = runner.invoke(