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

Add validation to test harness that every endpoint is being tested #151

Closed
njgheorghita opened this issue Oct 21, 2021 · 1 comment
Closed

Comments

@njgheorghita
Copy link
Collaborator

njgheorghita commented Oct 21, 2021

In one way or another, we want the test harness run to fail if it hasn't run against every currently implemented TrinEndpoint

re: #148 (comment)

@njgheorghita
Copy link
Collaborator Author

Closing since this is really hard to do ince all of the endpoints are stored as a large enum, which you cannot iterate through easily in rust. Further complications come from the fact that we might want one test for some endpoints and multiple for others. General guideline is that as you add a jsonrpc endpoint, you should also add tests to peertest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant