-
Notifications
You must be signed in to change notification settings - Fork 3
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 a test for log #11
Comments
Don't we have any tests for this? It seems that those lines are covered in Hera? |
We have tests for erroneous usage of
|
Yup will take a look asap! |
Currently added tests (via #114):
Two more cases we could add:
|
@jwasinger @hugo-dc can you add that single last tests, I expect it to be successful, but if it fails we may need to adjust Hera. |
Sure. |
Cf. ethereum#394: this is not currently possible with
testeth
but we should add something to test thelog
EEI method.The text was updated successfully, but these errors were encountered: