-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: Simulator gRPC Client implementation and initial configuration #182
Conversation
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.
Nice work...I left a few comments and questions
simulator/src/main/java/com/hedera/block/simulator/BlockStreamSimulatorApp.java
Outdated
Show resolved
Hide resolved
simulator/src/main/java/com/hedera/block/simulator/BlockStreamSimulatorApp.java
Outdated
Show resolved
Hide resolved
simulator/src/main/java/com/hedera/block/simulator/generator/GeneratorInjectionModule.java
Show resolved
Hide resolved
simulator/src/test/java/com/hedera/block/simulator/config/TestConfigBuilder.java
Outdated
Show resolved
Hide resolved
simulator/src/test/java/com/hedera/block/simulator/config/TestConfigBuilder.java
Show resolved
Hide resolved
simulator/src/test/java/com/hedera/block/simulator/BlockStreamSimulatorTest.java
Outdated
Show resolved
Hide resolved
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.
LG, one nit. After resolving Matt comments I think will be okay for merging 👍
simulator/src/main/java/com/hedera/block/simulator/generator/BlockAsDirBlockStreamManager.java
Outdated
Show resolved
Hide resolved
7145024
to
7750e4e
Compare
de156f8
to
f7d7fa0
Compare
- BlockAsDir Implementation for BlockStreamManager - Added Example files/blocks for streaming and testing (UT) - Unit Tests for all above - Changes to Smoke test to fix it. - Updated Helidon from 4.1.0 to 4.1.1 Signed-off-by: Alfredo Gutierrez <[email protected]>
Signed-off-by: Alfredo Gutierrez <[email protected]>
Signed-off-by: Alfredo Gutierrez <[email protected]>
Signed-off-by: Alfredo Gutierrez <[email protected]>
Signed-off-by: Alfredo Gutierrez <[email protected]>
f7d7fa0
to
f4ce0c4
Compare
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.
Looks good!
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.
review covers codecov.yml; looks good to me.
Signed-off-by: Alfredo Gutierrez <[email protected]>
…nclude with the app in main Signed-off-by: Alfredo Gutierrez <[email protected]>
d1b0cfb
Signed-off-by: Alfredo Gutierrez <[email protected]>
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.
Looks good
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #182 +/- ##
============================================
+ Coverage 99.73% 99.77% +0.03%
- Complexity 208 238 +30
============================================
Files 44 48 +4
Lines 763 872 +109
Branches 48 58 +10
============================================
+ Hits 761 870 +109
Misses 2 2
|
Description:
Related issue(s):
Fixes #159
Notes for reviewer:
Checklist