-
Notifications
You must be signed in to change notification settings - Fork 266
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
#1346: Add static factory methods for BlockProcessResult #1361
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs 100.0% Coverage The version of Java (1.8.0_272) you have used to run this analysis is deprecated and we will stop accepting it accepting it soon.Please update to at least Java 11. |
@andreoss nice! thanks for the contribution, I'll take a look on it |
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.
approved 👍
pipeline:run |
@@ -39,18 +40,16 @@ | |||
*/ | |||
public class SimpleBlockProcessor implements BlockProcessor { | |||
public long lastKnownBlockNumber = 0; | |||
private List<Block> blocks = new ArrayList<Block>(); | |||
private final List<Block> blocks = new ArrayList<Block>(); | |||
private long requestId; | |||
private byte[] hash; | |||
private int count; |
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.
I think you could remove this field too, its never used.
Per #1346:
Add static factory methods for two types of
BlockProcessResultTest
and make ctor protected.Motivation and Context
Refactoring.
How Has This Been Tested?
All unit tests pass
Types of changes
Checklist: