-
Notifications
You must be signed in to change notification settings - Fork 322
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
Update Blockchain Transition tests for BLOCKHASH refactoring EIP #167
Conversation
Oh I forgot that BLOCKHASH price is not updated for Metro yet in cpp implementation, so I'll need to update this PR after I implement the price change. |
@winsvega Why did Travis build fail here? |
@winsvega After changing the price of BLOCKHASH blockchain tests are fine (no Metro blockchain test uses BLOCKHASH), but the following state general tests fail:
And this one:
|
This needs to be updated after fixing incrementing the nonce of SYSTEM_ADDRESS (it should not be incremented in each block with blockhash contract updating, as it does now in cpp implementation) |
Updated with a fix for non-incrementing nonce. |
Replaced by #176 |
According to EIP state after Metropolis has a special contract that gets updated on each block ethereum/EIPs#210
cpp-ethereum implementation ethereum/aleth#4066