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
Fixes for chain simulator #5697
Conversation
- added & called Close on all inner components
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## feat/test-only-processor-node #5697 +/- ##
=================================================================
- Coverage 79.19% 79.05% -0.15%
=================================================================
Files 725 728 +3
Lines 95231 95452 +221
=================================================================
+ Hits 75420 75460 +40
- Misses 14456 14634 +178
- Partials 5355 5358 +3
☔ View full report in Codecov by Sentry. |
@@ -23,6 +23,9 @@ import ( | |||
"github.com/multiversx/mx-chain-go/testscommon" | |||
) | |||
|
|||
var oneEgld = big.NewInt(1000000000000000000) | |||
var initialStakedEgldPerNode = big.NewInt(0).Mul(oneEgld, big.NewInt(2500)) | |||
var initialSupply = big.NewInt(0).Mul(oneEgld, big.NewInt(20000000)) // 20 million EGLD |
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.
👍
// AggregateErrors can aggregate all provided error strings into a single error variable | ||
func AggregateErrors(errorStrings []string) error { |
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.
AggregateErrors is using global errClose
which makes it specific to close errors; rename function? or set errClose as parameter?
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.
good point. Will refactor 👍
Reasoning behind the pull request
Proposed changes
Testing procedure
Pre-requisites
Based on the Contributing Guidelines the PR author and the reviewers must check the following requirements are met:
feat
branch created?feat
branch merging, do all satellite projects have a proper tag insidego.mod
?