Skip to content

chore: Re-merge 1009 files historic plugin unit tests pt 3 #1029

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

Merged

Conversation

jsync-swirlds
Copy link
Contributor

Force push erased the result, so re-running the same PR.

ata-nas added 12 commits April 16, 2025 08:43
…the compress convenience method for the zstd compression type for consistency + tests

Signed-off-by: Atanas Atanasov <[email protected]>
Signed-off-by: Atanas Atanasov <[email protected]>
Signed-off-by: Atanas Atanasov <[email protected]>
…the end of zip file names

Signed-off-by: Atanas Atanasov <[email protected]>
@jsync-swirlds jsync-swirlds added the pull request label to get past the "label required" check when no label is needed or appropriate. label Apr 16, 2025
@jsync-swirlds jsync-swirlds added this to the 0.9.0 milestone Apr 16, 2025
@jsync-swirlds jsync-swirlds self-assigned this Apr 16, 2025
@jsync-swirlds jsync-swirlds requested review from a team as code owners April 16, 2025 15:44
@jsync-swirlds jsync-swirlds merged commit f50e1df into 505-plugins-services Apr 16, 2025
12 of 14 checks passed
@jsync-swirlds jsync-swirlds deleted the 1009-files-historic-plugin-unit-tests-pt-3 branch April 16, 2025 15:45
Copy link

codecov bot commented Apr 16, 2025

Codecov Report

Attention: Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ava/org/hiero/block/node/base/CompressionType.java 85.71% 1 Missing ⚠️
...ck/node/blocks/files/historic/ZipBlockArchive.java 83.33% 0 Missing and 1 partial ⚠️
@@                    Coverage Diff                     @@
##             505-plugins-services    #1029      +/-   ##
==========================================================
+ Coverage                   77.30%   79.56%   +2.26%     
- Complexity                    737      748      +11     
==========================================================
  Files                          99       98       -1     
  Lines                        3256     3245      -11     
  Branches                      316      312       -4     
==========================================================
+ Hits                         2517     2582      +65     
+ Misses                        638      550      -88     
- Partials                      101      113      +12     
Files with missing lines Coverage Δ Complexity Δ
...ro/block/node/blocks/files/historic/BlockPath.java 100.00% <ø> (+45.00%) 9.00 <0.00> (+6.00)
...ava/org/hiero/block/node/base/CompressionType.java 96.55% <85.71%> (-3.45%) 11.00 <0.00> (ø)
...ck/node/blocks/files/historic/ZipBlockArchive.java 26.95% <83.33%> (+26.95%) 3.00 <0.00> (+3.00)

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

jsync-swirlds added a commit that referenced this pull request Apr 17, 2025
Signed-off-by: Atanas Atanasov <[email protected]>
Co-authored-by: Atanas Atanasov <[email protected]>
jsync-swirlds added a commit that referenced this pull request Apr 17, 2025
Signed-off-by: Atanas Atanasov <[email protected]>
Co-authored-by: Atanas Atanasov <[email protected]>
Signed-off-by: Joseph Sinclair <[email protected]>
jsync-swirlds added a commit that referenced this pull request Apr 17, 2025
Signed-off-by: Atanas Atanasov <[email protected]>
Co-authored-by: Atanas Atanasov <[email protected]>
Signed-off-by: Joseph Sinclair <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pull request label to get past the "label required" check when no label is needed or appropriate.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants