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

Consensus client tests #3306

Open
nazar-pc opened this issue Dec 10, 2024 · 0 comments
Open

Consensus client tests #3306

nazar-pc opened this issue Dec 10, 2024 · 0 comments
Labels
help wanted Extra attention is needed improvement it is already working, but can be better refactor

Comments

@nazar-pc
Copy link
Member

We basically don't have client-side consensus tests for the whole protocol (sc-consensus-subspace), only for some of the underlying components/primitives.

The biggest challenge is the large API surface of Substrate that requires mocking of a lot of stuff. Without this infrastructure it is very hard to write tests and creating such infrastructure without running the whole node is a major undertaking.

One idea I had to address this was to introduce additional Subspace-specific abstractions that will be implemented for Substrate initially, but could be mocked much more easily for testing purposes.

@nazar-pc nazar-pc added help wanted Extra attention is needed refactor improvement it is already working, but can be better labels Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed improvement it is already working, but can be better refactor
Projects
Status: No status
Development

No branches or pull requests

1 participant