[action] add evm london test #3402
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3402 +/- ##
=======================================
Coverage 75.28% 75.28%
=======================================
Files 241 241
Lines 22258 22258
=======================================
Hits 16758 16758
Misses 4597 4597
Partials 903 903 Continue to review full report at Codecov.
|
|
|
action/execution.go
Outdated
| @@ -109,6 +109,11 @@ func (ex *Execution) Payload() []byte { return ex.data } | |||
| // AccessList returns the access list | |||
| func (ex *Execution) AccessList() types.AccessList { return ex.accessList } | |||
|
|
|||
| // SetAccessList set the access list | |||
| func (ex *Execution) SetAccessList(list types.AccessList) { | |||
| ex.accessList = list | |||
There was a problem hiding this comment.
latest master added NewExecutionWithAccessList, pls rebase and use it
|
I don't think a new testdata folder is needed for dulplicated data |
evm version is changed and the consumed gas of every contract is different from other version like Istanbul. So add a new dir just for London . |
they are the same test as in \iceland folder, but running with London EVM with adjusted test result (b/c consumed/returned gas amount changed) |
| @@ -63,6 +65,7 @@ type ( | |||
|
|
|||
| // GenesisBlockHeight defines an genesis blockHeight | |||
| GenesisBlockHeight struct { | |||
| IsLondon bool `json:"isLondon"` | |||
| @@ -358,6 +386,9 @@ func (sct *SmartContractTest) prepareBlockchain( | |||
|
|
|||
| cfg.Chain.TrieDBPath = testTriePath | |||
| cfg.ActPool.MinGasPriceStr = "0" | |||
| if sct.InitGenesis.IsLondon { | |||
| cfg.Genesis.Blockchain.ToBeEnabledBlockHeight = 0 | |||
| } | |||
| @@ -1018,6 +1049,86 @@ func TestIstanbulEVM(t *testing.T) { | |||
| }) | |||
| } | |||
|
|
|||
| func TestLondonEVM(t *testing.T) { | |||
| cfg := config.Default | |||
| config.SetEVMNetworkID(cfg.Chain.EVMNetworkID) | |||
There was a problem hiding this comment.
config.SetEVMNetworkID is no longer needed after this commit: 7d66698
| @@ -0,0 +1,36 @@ | |||
| { | |||
| "initGenesis": { | |||
| "isLondon" : true, | |||
| @@ -0,0 +1,45 @@ | |||
| { | |||
| "initGenesis": { | |||
| "isLondon" : true, | |||
The rawbytecode of the contract is different from the one in another folder. So its ok to leave the solidity source code here |
* upstream/master: (24 commits) account type with zero init nonce (iotexproject#3387) [api] Separate Server and Server Handler (iotexproject#3485) [ioctl] Build hdwallet derive command line into new ioctl (iotexproject#3418) [ioctl] Build hdwallet create command line into new ioctl (iotexproject#3470) [makefile] add go mod tidy (iotexproject#3471) [api] update chain metrics (iotexproject#3484) remove config.EVMNetworkID() (iotexproject#3460) [filedao] remove checkMasterChainDBFile() (iotexproject#3463) [api] add crashlog (iotexproject#3456) [api] Move generateBlockMeta to grpcserver.go (iotexproject#3303) [ioctl] Build action hash command line into new ioctl (iotexproject#3425) [ioctl] Build hdwallet export command line into new ioctl (iotexproject#3423) [ioctl] Refactor nodereward command in new ioctl (iotexproject#3416) [ioctl] Cleanup TestNewNodeDelegateCmd (iotexproject#3421) [blockchain] Remove BoltDBDaoOption (iotexproject#3465) remove InMemDaoOption (iotexproject#3464) [action] add evm london test (iotexproject#3402) [ioctl] create main for ioctl/newcmd (iotexproject#3296) [ioctl] Build block bucket command line into new ioctl (iotexproject#3386) [ioctl] Build hdwallet import command line into new ioctl (iotexproject#3419) ...
Description
add testdata-london for evm London version. see #3081
Fixes # (issue)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist: