Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update aiohttp dependency #1606

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,6 @@ def receive_message(
def get_application(self):
return self.transport.make_application()

@unittest_run_loop
async def test_start_x(self):
with async_mock.patch.object(
test_module.web, "TCPSite", async_mock.MagicMock()
Expand All @@ -82,7 +81,6 @@ async def test_start_x(self):
with pytest.raises(test_module.InboundTransportSetupError):
await self.transport.start()

@unittest_run_loop
async def test_send_message(self):
await self.transport.start()

Expand All @@ -97,7 +95,6 @@ async def test_send_message(self):

await self.transport.stop()

@unittest_run_loop
async def test_send_receive_message(self):
await self.transport.start()

Expand All @@ -112,7 +109,6 @@ async def test_send_receive_message(self):

await self.transport.stop()

@unittest_run_loop
async def test_send_message_outliers(self):
await self.transport.start()

Expand Down Expand Up @@ -149,7 +145,6 @@ async def test_send_message_outliers(self):

await self.transport.stop()

@unittest_run_loop
async def test_invite_message_handler(self):
await self.transport.start()

Expand Down
2 changes: 0 additions & 2 deletions aries_cloudagent/transport/inbound/tests/test_ws_transport.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@ def receive_message(
if self.result_event:
self.result_event.set()

@unittest_run_loop
async def test_start_x(self):
with async_mock.patch.object(
test_module.web, "TCPSite", async_mock.MagicMock()
Expand All @@ -78,7 +77,6 @@ async def test_start_x(self):
with pytest.raises(test_module.InboundTransportSetupError):
await self.transport.start()

@unittest_run_loop
async def test_message_and_response(self):
await self.transport.start()

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ async def setUpAsync(self):
self.profile = InMemoryProfile.test_profile()
self.message_results = []
self.headers = {}
await super().setUpAsync()

async def receive_message(self, request):
payload = await request.json()
Expand All @@ -34,7 +35,6 @@ async def get_application(self):
app.add_routes([web.post("/", self.receive_message)])
return app

@unittest_run_loop
async def test_handle_message_no_api_key(self):
server_addr = f"http://localhost:{self.server.port}"

Expand All @@ -49,7 +49,6 @@ async def send_message(transport, payload, endpoint):
assert self.headers.get("x-api-key") is None
assert self.headers.get("content-type") == "application/json"

@unittest_run_loop
async def test_handle_message_api_key(self):
server_addr = f"http://localhost:{self.server.port}"
api_key = "test1234"
Expand All @@ -68,7 +67,6 @@ async def send_message(transport, payload, endpoint, api_key):
assert self.message_results == [{}]
assert self.headers.get("x-api-key") == api_key

@unittest_run_loop
async def test_handle_message_packed_compat_mime_type(self):
server_addr = f"http://localhost:{self.server.port}"

Expand All @@ -84,7 +82,6 @@ async def send_message(transport, payload, endpoint):
assert self.message_results == [{}]
assert self.headers.get("content-type") == "application/ssi-agent-wire"

@unittest_run_loop
async def test_handle_message_packed_standard_mime_type(self):
server_addr = f"http://localhost:{self.server.port}"

Expand All @@ -101,7 +98,6 @@ async def send_message(transport, payload, endpoint):
assert self.message_results == [{}]
assert self.headers.get("content-type") == "application/didcomm-envelope-enc"

@unittest_run_loop
async def test_stats(self):
server_addr = f"http://localhost:{self.server.port}"

Expand All @@ -122,7 +118,6 @@ async def send_message(transport, payload, endpoint):
"outbound-http:POST": 1,
}

@unittest_run_loop
async def test_transport_coverage(self):
transport = HttpTransport()
assert transport.wire_format is None
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ class TestWsTransport(AioHTTPTestCase):
async def setUpAsync(self):
self.profile = InMemoryProfile.test_profile()
self.message_results = []
await super().setUpAsync()

async def receive_message(self, request):
ws = web.WebSocketResponse()
Expand All @@ -35,7 +36,6 @@ async def get_application(self):
app.add_routes([web.get("/", self.receive_message)])
return app

@unittest_run_loop
async def test_handle_message(self):
server_addr = f"ws://localhost:{self.server.port}"

Expand Down
10 changes: 1 addition & 9 deletions aries_cloudagent/utils/tests/test_http.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ class TestTransportUtils(AioHTTPTestCase):
async def setUpAsync(self):
self.fail_calls = 0
self.succeed_calls = 0
await super().setUpAsync()

async def get_application(self):
app = web.Application()
Expand All @@ -31,7 +32,6 @@ async def succeed_route(self, request):
ret = web.json_response([True])
return ret

@unittest_run_loop
async def test_fetch_stream(self):
server_addr = f"http://localhost:{self.server.port}"
stream = await fetch_stream(
Expand All @@ -41,15 +41,13 @@ async def test_fetch_stream(self):
assert result == b"[true]"
assert self.succeed_calls == 1

@unittest_run_loop
async def test_fetch_stream_default_client(self):
server_addr = f"http://localhost:{self.server.port}"
stream = await fetch_stream(f"{server_addr}/succeed")
result = await stream.read()
assert result == b"[true]"
assert self.succeed_calls == 1

@unittest_run_loop
async def test_fetch_stream_fail(self):
server_addr = f"http://localhost:{self.server.port}"
with self.assertRaises(FetchError):
Expand All @@ -61,7 +59,6 @@ async def test_fetch_stream_fail(self):
)
assert self.fail_calls == 2

@unittest_run_loop
async def test_fetch(self):
server_addr = f"http://localhost:{self.server.port}"
result = await fetch(
Expand All @@ -70,14 +67,12 @@ async def test_fetch(self):
assert result == [1]
assert self.succeed_calls == 1

@unittest_run_loop
async def test_fetch_default_client(self):
server_addr = f"http://localhost:{self.server.port}"
result = await fetch(f"{server_addr}/succeed", json=True)
assert result == [1]
assert self.succeed_calls == 1

@unittest_run_loop
async def test_fetch_fail(self):
server_addr = f"http://localhost:{self.server.port}"
with self.assertRaises(FetchError):
Expand All @@ -89,7 +84,6 @@ async def test_fetch_fail(self):
)
assert self.fail_calls == 2

@unittest_run_loop
async def test_put_file(self):
server_addr = f"http://localhost:{self.server.port}"
with async_mock.patch("builtins.open", mock_open(read_data="data")):
Expand All @@ -103,7 +97,6 @@ async def test_put_file(self):
assert result == [1]
assert self.succeed_calls == 1

@unittest_run_loop
async def test_put_file_default_client(self):
server_addr = f"http://localhost:{self.server.port}"
with async_mock.patch("builtins.open", mock_open(read_data="data")):
Expand All @@ -116,7 +109,6 @@ async def test_put_file_default_client(self):
assert result == [1]
assert self.succeed_calls == 1

@unittest_run_loop
async def test_put_file_fail(self):
server_addr = f"http://localhost:{self.server.port}"
with async_mock.patch("builtins.open", mock_open(read_data="data")):
Expand Down
8 changes: 4 additions & 4 deletions aries_cloudagent/utils/tests/test_repeat.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,17 +4,17 @@


class TestRepeat(TestCase):
def test_iter(self):
async def test_iter(self):
expect = [5, 7, 11, 17, 25]
seq = test_module.RepeatSequence(5, interval=5.0, backoff=0.25)
assert [round(attempt.next_interval) for attempt in seq] == expect

seq = test_module.RepeatSequence(2, interval=5.0, backoff=0.25)
attempt = seq.start()
attempt = attempt.next()
attempt.timeout(interval=0.01)
with self.assertRaises(StopIteration):
attempt.next()
async with attempt.timeout(interval=0.01):
with self.assertRaises(StopIteration):
attempt.next()

async def test_aiter(self):
seq = test_module.RepeatSequence(5, interval=5.0, backoff=0.25)
Expand Down
4 changes: 2 additions & 2 deletions requirements.txt
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
aiohttp~=3.7.4
aiohttp~=3.8.1
aiohttp-apispec~=2.2.1
aiohttp-cors~=0.7.0
apispec~=3.3.0
async-timeout~=3.0.1
async-timeout~=4.0.2
aioredis~=2.0.0
base58~=2.1.0
deepmerge~=0.3.0
Expand Down