-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add StateDB IAVL consistency tests #39
Conversation
0626c2c
to
78aefce
Compare
missing cosmos store changes still
for k := range s { | ||
keys[i] = k | ||
i++ | ||
} |
Check failure
Code scanning / gosec
the value in the range statement should be _ unless copying a map: want: for key := range m Error
for k := range j.dirties { | ||
keys[i] = k | ||
i++ | ||
} |
Check failure
Code scanning / gosec
the value in the range statement should be _ unless copying a map: want: for key := range m Error
} | ||
|
||
// set account number | ||
ethAcct.SetAccountNumber(accountNumberStart + uint64(i)) |
Check failure
Code scanning / gosec
Potential integer overflow by integer type conversion Error
if acct.GetAccountNumber() != accountNumberStart+uint64(i) { | ||
return fmt.Errorf( | ||
"account number mismatch: expected %d, got %d", | ||
accountNumberStart+uint64(i), acct.GetAccountNumber(), |
Check failure
Code scanning / gosec
Potential integer overflow by integer type conversion Error
|
||
// Ensure there are no number gaps | ||
for i, acct := range accounts { | ||
if acct.GetAccountNumber() != accountNumberStart+uint64(i) { |
Check failure
Code scanning / gosec
Potential integer overflow by integer type conversion Error
} | ||
|
||
// set account number | ||
ethAcct.SetAccountNumber(accountNumberStart + uint64(i)) |
Check warning
Code scanning / gosec
Errors unhandled. Warning
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days-before-close if no further activity occurs. |
SetState()
that should not modify IAVL node versions