-
Notifications
You must be signed in to change notification settings - Fork 30
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: Add Shanghai tests by ethereum/execution-spec-tests
#129
Merged
+667
−66
Merged
Changes from 53 commits
Commits
Show all changes
57 commits
Select commit
Hold shift + click to select a range
6d4b549
Merge pull request #40 from kaiachain/release/v1.0.1
blukat29 9f03292
Merge pull request #64 from kaiachain/release/v1.0.2
blukat29 f329a68
Merge pull request #99 from kaiachain/release/v1.0.3
blukat29 8cc4f45
Add downloading test files
ulbqb 48250fe
Add state test
ulbqb c8c70d8
Add testing actual data
ulbqb 1e6fbff
Add relax precomplied range for tests
ulbqb 234ce9d
Keep legacy tests
ulbqb a410ec2
Simulate payload gas for Ethereum
ulbqb 38f2f88
Simulate staking reward for Ethereum
ulbqb f9f70ee
Simulate root hash for Ethereum
ulbqb cb809eb
Make BaseFee optional, Add switching test, Simulate reward only after…
ulbqb 1f22b56
Remove unused data
ulbqb 629c81c
Add Merge fork
ulbqb dee1d14
Fix Ethereum simulation for gas price and staking reward
ulbqb 04dd540
Revert statedb commit
ulbqb bf011f7
Enable tests for only Shanghai
ulbqb 99b56c2
Fix CancunInstructionSet
ulbqb f137c00
Update fork config
ulbqb 35d7f9c
Merge branch 'dev' into eip-7702-exec-tests
ulbqb e62559e
Fix params variables
ulbqb 7385421
Revert unused code
ulbqb a1a5678
Simulate only when TestExecutionSpecState
ulbqb de10698
Revert test name
ulbqb 3798751
Add skip forks
ulbqb bef1594
Remove an unused variable
ulbqb a41de19
Add initialization of flag for tests
ulbqb 68b567d
Test TestExecutionSpecState alone
ulbqb 04a7424
Fix for lint
ulbqb 4d64e25
Stop using slices
ulbqb 9046215
Fix downloading
ulbqb 0edb0b1
Fix for lint
ulbqb 8e95bd2
Fix for lint
ulbqb 18fb8b0
Skip a test
ulbqb fe667e1
Fix for lint
ulbqb f52aea4
Fix for review
ulbqb 0baf377
Fix func name
ulbqb 5dd825a
Reuse IntrinsicGasPayload
ulbqb 456452b
Remove Paris
ulbqb 733f541
Fix commnets
ulbqb 814a205
Add forgotten code
ulbqb e768968
Update tests/state_test.go
ulbqb 487bd28
Update tests/state_test.go
ulbqb 842a28d
Merge branch 'dev' into eip-7702-exec-tests
ulbqb b085c4e
Refactor simulateEthIntrinsicGas
ulbqb a5472a9
Fix isTestExecutionSpecState
ulbqb a72a27c
Update name
ulbqb 04126d2
Directly set gas price
ulbqb 9c311bd
Remove meaningless comments
ulbqb 92ea60b
Fix MaxPriorityFeePerGas
ulbqb 831c67d
Use Ethereum op code gas
ulbqb 0de49b0
Stop skipping forks
ulbqb bc2f181
Fix chain config
ulbqb 7e8a94b
Reset relaxPrecompileRangeForTest after TestExecutionSpecState
ulbqb 0afb644
Update comment
ulbqb e98fb5f
Revert ci
ulbqb 11899f7
Fix error handling
ulbqb File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
# This file contains sha256 checksums of optional build dependencies. | ||
|
||
# version:spec-tests 2.1.0 | ||
# https://github.com/ethereum/execution-spec-tests/releases | ||
# https://github.com/ethereum/execution-spec-tests/releases/download/v2.1.0/ | ||
ca89c76851b0900bfcc3cbb9a26cbece1f3d7c64a3bed38723e914713290df6c fixtures_develop.tar.gz |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this for testing? If so, you can simply run
cd tests; go test -run=TestExecutionSpecState -v
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is for a CI test. Separating tests is needed because
relaxPrecompileRangeForTest
andisTestExecutionSpecState
affect overall.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we are not running in parallel, you can run them altogether,
ofif you defer SetRelax...(false)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#129 (comment)
I couldn't use defer because of this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we use mutex, can this be reverted?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hyunsooda
The problem is running
TestKaiaSpecState
withrelaxPrecompileRangeForTest = true
. This is caused by startingTestExecutionSpecState
before finishingTestKaiaSpecState
due tot.Parallel()
.I think dropping
t.Parallel()
in all tests and resettingrelaxPrecompileRangeForTest
after finishingTestExecutionSpecState
are needed so thatTestKaiaSpecState
(common.RelaxPrecompileRangeForTest(true)
) doesn't affect other tests.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understood the spec of
t.Parallel()
.The test pauses when
t.Parallel()
is run and then next test starts. All tests which pause byt.Parallel()
resume in parallel after all tests are done.Current problem:
common.RelaxPrecompileRangeForTest(false)
inTestKaiaSpecState
t.Parallel()
andTestKaiaSpecState
pausesTestExecutionSpecState
andcommon.RelaxPrecompileRangeForTest(true)
TestKaiaSpecState
withrelaxPrecompileRangeForTest = true
Simple solution is running
common.RelaxPrecompileRangeForTest(false)
aftert.Parallel()
inTestKaiaSpecState
. But the root cause is not runningcommon.RelaxPrecompileRangeForTest(false)
at the end ofTestExecutionSpecState
. So I'll implement this using Test Suite.fixed at 7e8a94b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it, thank you. The other tests seem to have been still omitted in the
Makefile
, don't they?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed it at e98fb5f.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@blukat29 @ian0371 This thread can be closed as
Makefile
is reverted and remains unchanged.