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
Coverage for: factory/bootstrap #5139
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## rc/v1.6.0 #5139 +/- ##
=============================================
+ Coverage 70.90% 77.78% +6.87%
=============================================
Files 675 654 -21
Lines 87793 85979 -1814
=============================================
+ Hits 62251 66880 +4629
+ Misses 20864 13911 -6953
- Partials 4678 5188 +510
... and 159 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
…into unittests_factory_bootstrap # Conflicts: # factory/bootstrap/bootstrapComponents_test.go
@@ -82,9 +82,6 @@ func NewBootstrapComponentsFactory(args BootstrapComponentsFactoryArgs) (*bootst | |||
if args.WorkingDir == "" { | |||
return nil, errors.ErrInvalidWorkingDir | |||
} | |||
if check.IfNil(args.StatusCoreComponents) { |
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.
👍
@@ -29,6 +30,15 @@ func CreateShardCoordinator( | |||
prefsConfig config.PreferencesConfig, | |||
log logger.Logger, | |||
) (sharding.Coordinator, core.NodeType, error) { | |||
if check.IfNil(nodesConfig) { |
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.
👍
factory/bootstrap/shardingFactory.go
Outdated
if check.IfNil(nodesConfig) { | ||
return nil, "", errErd.ErrNilGenesisNodesSetupHandler | ||
} | ||
if pubKey == nil { |
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.
please use check.IfNil
…into unittests_factory_bootstrap # Conflicts: # storage/factory/openStorage.go # storage/factory/openStorage_test.go # storage/latestData/latestDataProvider_test.go
errors/errors.go
Outdated
var ErrNilLogger = errors.New("nil logger") | ||
|
||
// ErrNilShuffleOutCloser signals that a nil shuffler out closer has been provided | ||
var ErrNilShuffleOutCloser = errors.New("nil shuffler out closer") |
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.
should we go with shuffler out
or with shuffle out
?
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.
updated, typo
t.Run("NewIndexHashedNodesCoordinator fails should error", func(t *testing.T) { | ||
t.Parallel() |
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 test name and the previous one have the same name, rename here?
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.
fixed
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.
System test passed
@@ Log scanner @@
unittests_factory_bootstrap
================================================================================
- Known Warnings 11
- New Warnings 2
- Known Errors 0
- New Errors 0
- Panics 0
================================================================================ - block hash does not match 2754
- wrong nonce in block 912
- miniblocks does not match 0
- num miniblocks does not match 0
- miniblock hash does not match 0
- block bodies does not match 0
- receipts hash missmatch 0
================================================================================ - No jailed nodes on the testnet
================================================================================
The base branch was changed.
…into unittests_factory_bootstrap # Conflicts: # errors/errors.go # storage/errors.go
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
?