-
Notifications
You must be signed in to change notification settings - Fork 476
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 comprehensive RPC API test suite #230
Comments
We want it very much, been wanting it forever. |
I'm working on fixing the existing RPC test suite right now. |
The 'running' part is basically taken care of by Hive, the work here is mostly fixing up the RPC simulation to work with the newer Hive simulation API. Quite a few things were broken in the last round of heavy Hive development, I'm basically trying to get it back together right now. It would be very nice to also fix the other RPC test suite. This one is written in python and was also broken by the Hive API changes made during the big refactoring last year. I don't have time to fix that one though. If @matilote is willing and able, he could pick up that one or rewrite it. |
A warning though: at this time, Hive is still a bit of a mess. It really takes time to figure out how it works, and most documentation is outdated. That's not to say it's impossible to work on, and we would really appreciate help, just need to coordinate. |
The 'ethclient' RPC simulation now works again (as of #418). @tkstanczak The tests found two issues in Nethermind:
I think the first one should be easy to fix. |
Check if this is still valid #119
The text was updated successfully, but these errors were encountered: