1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
|
From af9b471eba92f1f353fec57f60e48702e79bcb80 Mon Sep 17 00:00:00 2001
From: Martin Weinelt <hexa@darmstadt.ccc.de>
Date: Thu, 21 Aug 2025 15:24:37 +0200
Subject: [PATCH] Fix tests with pytest 8.4
Pytest 8.4 will fail on async functions, if they are not handled by a
plugin. And since pytest-aiohttp relies on pytest-asyncio, the obvious
fix is to mark them as asyncio.
---
tests.py | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/tests.py b/tests.py
index e11c4d5..547d636 100644
--- a/tests.py
+++ b/tests.py
@@ -11,6 +11,7 @@
from jsonrpc_async import Server, ProtocolError, TransportError
+@pytest.mark.asyncio
async def test_send_message_timeout(aiohttp_client):
"""Test the catching of the timeout responses."""
@@ -37,6 +38,7 @@ def create_app():
assert isinstance(transport_error.value.args[1], asyncio.TimeoutError)
+@pytest.mark.asyncio
async def test_send_message(aiohttp_client):
"""Test the sending of messages."""
# catch non-json responses
@@ -100,6 +102,7 @@ def create_app():
"Error calling method 'my_method': Transport Error")
+@pytest.mark.asyncio
async def test_exception_passthrough(aiohttp_client):
async def callback(*args, **kwargs):
raise aiohttp.ClientOSError('aiohttp exception')
@@ -120,6 +123,7 @@ def create_app():
assert isinstance(transport_error.value.args[1], aiohttp.ClientOSError)
+@pytest.mark.asyncio
async def test_forbid_private_methods(aiohttp_client):
"""Test that we can't call private methods (those starting with '_')."""
def create_app():
@@ -137,6 +141,7 @@ def create_app():
await server.foo.bar._baz()
+@pytest.mark.asyncio
async def test_headers_passthrough(aiohttp_client):
"""Test that we correctly send RFC headers and merge them with users."""
async def handler(request):
@@ -170,6 +175,7 @@ async def callback(*args, **kwargs):
await server.foo()
+@pytest.mark.asyncio
async def test_method_call(aiohttp_client):
"""Mixing *args and **kwargs is forbidden by the spec."""
def create_app():
@@ -185,6 +191,7 @@ def create_app():
"JSON-RPC spec forbids mixing arguments and keyword arguments")
+@pytest.mark.asyncio
async def test_method_nesting(aiohttp_client):
"""Test that we correctly nest namespaces."""
async def handler(request):
@@ -211,6 +218,7 @@ def create_app():
"nest.testmethod.some.other.method") is True
+@pytest.mark.asyncio
async def test_calls(aiohttp_client):
"""Test RPC call with positional parameters."""
async def handler1(request):
@@ -265,6 +273,7 @@ def create_app():
await server.foobar({'foo': 'bar'})
+@pytest.mark.asyncio
async def test_notification(aiohttp_client):
"""Verify that we ignore the server response."""
async def handler(request):
@@ -283,6 +292,7 @@ def create_app():
assert await server.subtract(42, 23, _notification=True) is None
+@pytest.mark.asyncio
async def test_custom_loads(aiohttp_client):
"""Test RPC call with custom load."""
loads_mock = mock.Mock(wraps=json.loads)
@@ -306,6 +316,7 @@ def create_app():
assert loads_mock.call_count == 1
+@pytest.mark.asyncio
async def test_context_manager(aiohttp_client):
# catch non-json responses
async def handler1(request):
|
