-
Notifications
You must be signed in to change notification settings - Fork 117
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
test: Corner cases for ethereum transactions #13229
test: Corner cases for ethereum transactions #13229
Conversation
Signed-off-by: Zhivko Kelchev <zhivko.kelchev@limechain.tech>
Add PayReceivable contract for ethereum tests Add gasUsed assertion Signed-off-by: Zhivko Kelchev <zhivko.kelchev@limechain.tech>
# Conflicts: # hedera-node/hedera-app/src/xtest/java/common/AbstractXTest.java
Node: HAPI Test (Restart) Results2 tests 2 ✅ 8m 6s ⏱️ Results for commit bf9b420. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Node Death Reconnect) Results2 tests 2 ✅ 8m 47s ⏱️ Results for commit bf9b420. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Token) Results237 tests 235 ✅ 20m 29s ⏱️ Results for commit bf9b420. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Misc) Results468 tests 458 ✅ 42m 50s ⏱️ Results for commit bf9b420. ♻️ This comment has been updated with latest results. |
Signed-off-by: Zhivko Kelchev <zhivko.kelchev@limechain.tech>
Node: HAPI Test (Crypto) Results335 tests 335 ✅ 41m 49s ⏱️ Results for commit bf9b420. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Time Consuming) Results21 tests 21 ✅ 54m 36s ⏱️ Results for commit bf9b420. ♻️ This comment has been updated with latest results. |
Node: Unit Test Results 2 318 files + 1 2 318 suites +1 3h 14m 50s ⏱️ + 2m 58s Results for commit bf9b420. ± Comparison against base commit b86dab5. This pull request removes 4016 and adds 3789 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Node: HAPI Test (Smart Contract) Results605 tests 605 ✅ 1h 12m 49s ⏱️ Results for commit bf9b420. ♻️ This comment has been updated with latest results. |
# Conflicts: # hedera-node/hedera-app/src/xtest/java/common/AbstractXTest.java
Signed-off-by: Zhivko Kelchev <zhivko.kelchev@limechain.tech>
Signed-off-by: Zhivko Kelchev <zhivko.kelchev@limechain.tech>
Closing this PR because we will deprecate the x-tests: #13425 |
Description:
This PR adds ethereum transactions to the Xtests.
Related issue(s):
Fixes #12627
Notes for reviewer:
Throttle definitions were added, because we need to validate childThrottleIsAvailable on commit here.
I tried to add tests regarding the optional access list of EIP-2039 transactions, but I didn't find any access list implementations. Furthermore, when I try to encode ethereum data with the access list, it fails here.
I also tried to modify encode
encodeTx()
, but the consumed gas was the same as before.Checklist