deep-stack/laconicd-deprecated
6
0
Fork 0
forked from cerc-io/laconicd-deprecated
Code Pull Requests Activity

fix: make gas estimation test reproducible (#890)

Browse Source 
Closes: #536

Solution: Use a fixed test account.

The source of non-deterministic is tx's intrinsic gas is different according to different number of zeros in input data,
so to make sure the gas number is reproducible, and the randomly generated test address is included in that.
This commit is contained in:
yihuang 2022-01-07 21:04:16 +08:00 committed by GitHub
parent e6c9b7723b
commit 5e15c64d02
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
x/evm/keeper/keeper_test.go
View File

@ -72,9 +72,12 @@ type KeeperTestSuite struct {
func (suite *KeeperTestSuite) DoSetupTest(t require.TestingT) { func (suite *KeeperTestSuite) DoSetupTest(t require.TestingT) {
checkTx := false checkTx := false
// account key // account key, use a constant account to keep unit test deterministic.
priv, err := ethsecp256k1.GenerateKey() ecdsaPriv, err := crypto.HexToECDSA("b71c71a67e1177ad4e901695e1b4b9ee17ae16c6668d313eac2f96dbcda3f291")
require.NoError(t, err) require.NoError(t, err)
priv := &ethsecp256k1.PrivKey{
Key: crypto.FromECDSA(ecdsaPriv),
}
suite.address = common.BytesToAddress(priv.PubKey().Address().Bytes()) suite.address = common.BytesToAddress(priv.PubKey().Address().Bytes())
suite.signer = tests.NewSigner(priv) suite.signer = tests.NewSigner(priv)