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

Port transaction validation tests to new function #2695

Open
arnetheduck opened this issue Oct 4, 2024 · 0 comments
Open

Port transaction validation tests to new function #2695

arnetheduck opened this issue Oct 4, 2024 · 0 comments

Comments

@arnetheduck
Copy link
Member

proc validate*(tx: Transaction, fork: EVMFork) =
contains a transaction validation function that has fallen out of use.

The old function however has a set of checks and validations that the new function does not have - it also has test coverage which is missing from the new function - both of these issues need to be address such that the old tests and validations are checked.

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

No branches or pull requests

2 participants