Skip to content

[v10] Add async tests #3039

Merged
basfroman merged 102 commits intoSDKv10from
test/roman/add-async-e2e-tests
Sep 2, 2025
Merged

[v10] Add async tests #3039
basfroman merged 102 commits intoSDKv10from
test/roman/add-async-e2e-tests

Conversation

@basfroman
Copy link
Collaborator

@basfroman basfroman commented Aug 30, 2025

Context

Before I start making a massive change to the SDK, I will add asynchronous e2e tests to make sure that the subsequent changes (which will affect the asynchronous subtensor and asynchronous extrinsics) do not break the correct logic. It also helps to identify errors and bugs in the existing asynchronous logic of the SDK and the async-substrate-interface.

Includes:

  • all e2e tests has async version
  • python 3.9 support dropped

Adds:

  • AsyncSubtensor comes to tests as async fixture already from context menedger
  • Now we have more that 300 e2e test. Since GitHab Actions supports a maximum of 256 elements in a matrix, we now use a reusable workflow to support more tests.
  • still have some async tests issue (it needs investigation)
    • functools.partial + asyncio.gather breaks the loop for py3.9-11, but works for 3.12-13
    • while we support python 3.9 then asyncio.gather (without functools.partial) will break group repeating requests and return RuntimeError
    • some tests are passed but with warning Event loop is closed

basfroman and others added 30 commits July 7, 2025 17:45
…devnet-ready-update-fee"

This reverts commit 9665972, reversing
changes made to 81e0a44.
@basfroman basfroman changed the title Add async tests [v10] Add async tests Aug 30, 2025
@basfroman basfroman requested a review from a team September 2, 2025 20:57
@basfroman basfroman merged commit db473d4 into SDKv10 Sep 2, 2025
336 checks passed
@basfroman basfroman deleted the test/roman/add-async-e2e-tests branch September 2, 2025 21:41
This was referenced Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

E2E tests for async SDK

3 participants