-
Notifications
You must be signed in to change notification settings - Fork 198
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
En 10861 scheduled txs unit tests #3610
En 10861 scheduled txs unit tests #3610
Conversation
…us, add factoru for scheduled data syncer
consensus/spos/bls/export_test.go
Outdated
@@ -213,6 +213,11 @@ func (sr *subroundBlock) ReceivedBlockHeader(cnsDta *consensus.Message) bool { | |||
return sr.receivedBlockHeader(cnsDta) | |||
} | |||
|
|||
// ReceivedBlockBodyAndHeader is called when both a header and block body have been received t |
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.
Remove "t" from the end of the line
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.
done
epochStart/bootstrap/process.go
Outdated
@@ -201,6 +205,9 @@ func NewEpochStartBootstrap(args ArgsEpochStartBootstrap) (*epochStartBootstrap, | |||
maxHardCapForMissingNodes: args.GeneralConfig.TrieSync.MaxHardCapForMissingNodes, | |||
trieSyncerVersion: args.GeneralConfig.TrieSync.TrieSyncerVersion, | |||
enableEpochs: args.EnableEpochs, | |||
dataSyncerFactory: args.DataSyncerCreator, | |||
storerScheduledSCRs: genericMocks.NewStorerMock("path", 0), |
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 storer should be mock?
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.
Right, it shouldn't be a mock here in productive code, but it can be initialized with nil, as it will get updated with the correct storer once the epoch is known.
@@ -0,0 +1,20 @@ | |||
package bootstrap |
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.
Missing comments for exported methods
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.
done
@@ -0,0 +1,26 @@ | |||
package types |
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.
Missing comments for exported types
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.
done
@@ -51,7 +51,7 @@ func newStartInEpochShardHeaderDataSyncerWithScheduled( | |||
}, 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.
Add comments for exported methods in this file
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.
done
Add some unit tests and fixes