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

rpc: Include more variations on fixtures for testing #611

Open
thanethomson opened this issue Oct 6, 2020 · 5 comments · Fixed by #653
Open

rpc: Include more variations on fixtures for testing #611

thanethomson opened this issue Oct 6, 2020 · 5 comments · Fixed by #653
Assignees

Comments

@thanethomson
Copy link
Contributor

Follows from discussion here: #604 (comment)

We need to evaluate the fixtures that we are testing for the RPC and see if we should add more variants for each case to ensure our serialization/deserialization is working as expected.

@thanethomson
Copy link
Contributor Author

Reopening until we've used the RPC probe's output to update our tests.

@thanethomson thanethomson reopened this Nov 18, 2020
@thanethomson thanethomson added this to the v0.17.0 milestone Nov 19, 2020
@thanethomson thanethomson self-assigned this Nov 19, 2020
@ebuchman
Copy link
Member

Probe looks awesome. One note is that KVStore txs never error, so we won't be able to tests txs that fail with it. We could use the built in counter-app for that though.

Not sure what the best way is to exhaustively test all the possible responses. I guess some set of plans that flex many of the edges we can think of, and maybe additionally some fuzzing?

@ebuchman
Copy link
Member

Oh as per #612 (comment) , probably a good idea to do this against Gaia

@thanethomson
Copy link
Contributor Author

Not sure what the best way is to exhaustively test all the possible responses. I guess some set of plans that flex many of the edges we can think of, and maybe additionally some fuzzing?

Fuzzing may help a little, but I'm not sure it'd help too much when testing against the kvstore.

Oh as per #612 (comment) , probably a good idea to do this against Gaia

Definitely! How can I get hold of some raw JSON-RPC requests that I can send to a fresh Gaia node?

@ebuchman
Copy link
Member

I guess it's just a matter of generating txs for the broadcast_tx endpoints, since everything else should be the same. We could use gaiacli for this, but not sure if there's a way to get it to dump the binary tx. Maybe @andynog has some ideas?

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