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

fix: drop asynctest #2566

Merged
merged 33 commits into from
Oct 28, 2023
Merged
Changes from 1 commit
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
e81fc79
fix: drop asynctest
dbluhm Oct 25, 2023
fd41c62
fix: mt admin test routes mock issues
dbluhm Oct 25, 2023
8b77dfa
fix: conn v1 manager mock issues
dbluhm Oct 25, 2023
6d102c0
chore: normalize references to mock, replace async_mock
dbluhm Oct 27, 2023
818ee7f
chore: split unittest mock imports
dbluhm Oct 27, 2023
cfc2b8c
fix: provide CoroutineMock adapter
dbluhm Oct 27, 2023
2b8cf85
fix: xfail 3 tests that never worked
dbluhm Oct 27, 2023
ce2bb16
fix: never awaited error in PPv1 test routes
dbluhm Oct 27, 2023
d920e02
fix: never awaited warnings in wallet test routes
dbluhm Oct 27, 2023
0447250
fix: never awaited error in default verkey strat tests
dbluhm Oct 27, 2023
0a6746e
fix: test validation result async issue
dbluhm Oct 27, 2023
5005c84
fix: unawaited task in test basic message q
dbluhm Oct 27, 2023
0b581a5
fix: bad assertion in inbound test session
dbluhm Oct 27, 2023
b43f174
fix: mock logger.exception isn't a coroutine
dbluhm Oct 27, 2023
956a129
fix: introductiont tests bad assertion
dbluhm Oct 27, 2023
eee5e67
fix: bad assertions for called once with
dbluhm Oct 27, 2023
e77e8fe
fix: the "fixed" tests
dbluhm Oct 27, 2023
f257101
fix: test trie should be asyncio test case
dbluhm Oct 27, 2023
698808b
fix: inbound test message should be async test case
dbluhm Oct 27, 2023
9ca6b5e
fix: warnings on vc holder tests
dbluhm Oct 27, 2023
1998cab
fix: inbound ws transport test warning
dbluhm Oct 27, 2023
b69ff45
fix: in mem storage test warning
dbluhm Oct 27, 2023
13a65cc
fix: bad called once assertions on mocks
dbluhm Oct 27, 2023
1b5f923
fix: storage tests missing fixtures
dbluhm Oct 27, 2023
f15bdd9
fix: bad log assertions
dbluhm Oct 27, 2023
f754f95
fix: inbound message router not coroutine mock
dbluhm Oct 27, 2023
eff84bc
fix: bad mock in discovery tests
dbluhm Oct 27, 2023
3e61029
fix: bad mock in multiledger tests
dbluhm Oct 27, 2023
5b64fc1
fix: bad mocks in test dispatcher
dbluhm Oct 27, 2023
89e87c1
fix: unawaited coroutines in test conductor
dbluhm Oct 27, 2023
6ecd912
fix: unawaited coroutines in test start
dbluhm Oct 27, 2023
baa72c0
fix: unawaited coroutines in test_upgrade
dbluhm Oct 27, 2023
0da450c
chore: drop asynctest dep
dbluhm Oct 27, 2023
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
Prev Previous commit
Next Next commit
fix: never awaited error in PPv1 test routes
Signed-off-by: Daniel Bluhm <[email protected]>
dbluhm committed Oct 27, 2023
commit ce2bb16fbcb3ab382f3656c9f780355848c9d0fa
Original file line number Diff line number Diff line change
@@ -189,9 +189,7 @@ async def test_presentation_exchange_credentials_list_single_referent(self):
# Since we are mocking import
importlib.reload(test_module)

mock_presentation_exchange.return_value.retrieve_by_id.return_value = (
mock_presentation_exchange
)
mock_presentation_exchange.retrieve_by_id.return_value = mock.MagicMock()

with mock.patch.object(test_module.web, "json_response") as mock_response:
await test_module.presentation_exchange_credentials_list(self.request)
@@ -224,8 +222,8 @@ async def test_presentation_exchange_credentials_list_multiple_referents(self):
# Since we are mocking import
importlib.reload(test_module)

mock_presentation_exchange.return_value.retrieve_by_id.return_value = (
mock_presentation_exchange
mock_presentation_exchange.retrieve_by_id = mock.CoroutineMock(
return_value=mock.MagicMock()
)

with mock.patch.object(test_module.web, "json_response") as mock_response: