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

Broken ethereum tests in 4.0.0 #1064

Closed
ufoscout opened this issue Feb 8, 2024 · 1 comment · Fixed by #1066
Closed

Broken ethereum tests in 4.0.0 #1064

ufoscout opened this issue Feb 8, 2024 · 1 comment · Fixed by #1066

Comments

@ufoscout
Copy link
Contributor

ufoscout commented Feb 8, 2024

Your CI job failed to execute the ethereum tests but the build is reported as successful.
For example check here and here, the builds are green but the tests have failed with this error:

     Running `/native-linux-runner /target/x86_64-unknown-linux-gnu/release/revme statetest ethtests/GeneralStateTests/ ethtests/LegacyTests/Constantinople/GeneralStateTests/ ethtests/EIPTests/StateTests/stEIP1153-transientStorage/ ethtests/EIPTests/StateTests/stEIP4844-blobtransactions/`

Running tests in ethtests/GeneralStateTests/...
Finished execution. Total CPU time: 0.000000s
3 threads returned an error, out of 4 total:
Test ethtests/GeneralStateTests/stSLoadTest/sloadGasCost.json failed: unknown field `currentExcessBlobGas`, expected one of `currentCoinbase`, `currentDifficulty`, `currentGasLimit`, `currentNumber`, `currentTimestamp`, `currentBaseFee`, `previousHash`, `currentRandom`, `currentBeaconRoot`, `currentWithdrawalsRoot`, `parentBlobGasUsed`, `parentExcessBlobGas` at line 17 column 35
Test ethtests/GeneralStateTests/stStaticCall/static_callcodecallcall_100_2.json failed: unknown field `currentExcessBlobGas`, expected one of `currentCoinbase`, `currentDifficulty`, `currentGasLimit`, `currentNumber`, `currentTimestamp`, `currentBaseFee`, `previousHash`, `currentRandom`, `currentBeaconRoot`, `currentWithdrawalsRoot`, `parentBlobGasUsed`, `parentExcessBlobGas` at line 17 column 35
Test ethtests/GeneralStateTests/stStaticCall/static_log0_emptyMem.json failed: unknown field `currentExcessBlobGas`, expected one of `currentCoinbase`, `currentDifficulty`, `currentGasLimit`, `currentNumber`, `currentTimestamp`, `currentBaseFee`, `previousHash`, `currentRandom`, `currentBeaconRoot`, `currentWithdrawalsRoot`, `parentBlobGasUsed`, `parentExcessBlobGas` at line 17 column 35
Test ethtests/GeneralStateTests/stSLoadTest/sloadGasCost.json failed: unknown field `currentExcessBlobGas`, expected one of `currentCoinbase`, `currentDifficulty`, `currentGasLimit`, `currentNumber`, `currentTimestamp`, `currentBaseFee`, `previousHash`, `currentRandom`, `currentBeaconRoot`, `currentWithdrawalsRoot`, `parentBlobGasUsed`, `parentExcessBlobGas` at line 17 column 35
@rakita
Copy link
Member

rakita commented Feb 8, 2024

I have run them locally but didn't update to newest test suit. Thanks for flagging.

Tests are failing bcs of newly added fields.

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

Successfully merging a pull request may close this issue.

2 participants