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

Consider using RuntimeTester to test runtime #4854

Open
matklad opened this issue Sep 21, 2021 · 3 comments
Open

Consider using RuntimeTester to test runtime #4854

matklad opened this issue Sep 21, 2021 · 3 comments
Assignees
Labels
C-housekeeping Category: Refactoring, cleanups, code quality Groomed Node Node team P-low Priority: low T-node Team: issues relevant to the node experience team

Comments

@matklad
Copy link
Contributor

matklad commented Sep 21, 2021

In #4497 and #4812 we added runtime-tester infrastructure – a high-level driver for nearcore which can be used to simulate various block processing scenarios. We successfully used it for fuzzing (#4524) and for providing minimal reproductions for issues (#4771).

As an actionable next step, I suggest trying to use this same infra for some of our tests. The goals would be:

A good candidate is

Arc::new(nearcore::NightshadeRuntime::new(
file – it contains 6 instances of nearcore::NightshadeRuntime::new and it feel like they all should be replaceable with Scenario APIs.

@matklad matklad added C-housekeeping Category: Refactoring, cleanups, code quality T-node Team: issues relevant to the node experience team labels Sep 21, 2021
@matklad
Copy link
Contributor Author

matklad commented Sep 21, 2021

cc @posvyatokum (as the author of runtime tester), @Longarithm (as both the creator of and the sufferer from some tests which do nearcore::NightshadeRuntime::new)

@stale
Copy link

stale bot commented Dec 20, 2021

This issue has been automatically marked as stale because it has not had recent activity in the last 2 months.
It will be closed in 7 days if no further activity occurs.
Thank you for your contributions.

@stale
Copy link

stale bot commented Apr 24, 2022

This issue has been automatically marked as stale because it has not had recent activity in the last 2 months.
It will be closed in 7 days if no further activity occurs.
Thank you for your contributions.

@stale stale bot added the S-stale label Apr 24, 2022
@exalate-issue-sync exalate-issue-sync bot added T-nodeX and removed T-node Team: issues relevant to the node experience team labels Jun 28, 2022
@akhi3030 akhi3030 removed the S-stale label Jul 8, 2022
@matklad matklad added T-node Team: issues relevant to the node experience team and removed T-nodeX labels Aug 4, 2022
@exalate-issue-sync exalate-issue-sync bot added the P-low Priority: low label Aug 11, 2022
@gmilescu gmilescu added the Node Node team label Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-housekeeping Category: Refactoring, cleanups, code quality Groomed Node Node team P-low Priority: low T-node Team: issues relevant to the node experience team
Projects
None yet
Development

No branches or pull requests

5 participants