Skip to content

Refactor#162

Merged
marcoSanti merged 10 commits into
capio-v2from
refactor
Aug 30, 2025
Merged

Refactor#162
marcoSanti merged 10 commits into
capio-v2from
refactor

Conversation

@marcoSanti
Copy link
Copy Markdown
Member

No description provided.

Split definition of classes and implementation to avoid recursive
include issues that were rising when defining the capio_server backend
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 30, 2025

Codecov Report

❌ Patch coverage is 26.99309% with 1163 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.25%. Comparing base (dd9269f) to head (5902919).
⚠️ Report is 1 commits behind head on capio-v2.

Files with missing lines Patch % Lines
capio-server/src/capio-cl-engine/json_parser.cpp 2.14% 172 Missing and 148 partials ⚠️
.../communication-service/data-plane/mtcl_backend.cpp 3.57% 110 Missing and 106 partials ⚠️
...pio-server/src/capio-cl-engine/capio_cl_engine.cpp 41.11% 106 Missing and 73 partials ⚠️
...o-server/src/storage-service/capio_memory_file.cpp 4.59% 79 Missing and 4 partials ⚠️
...ication-service/control-plane/fs_control_plane.cpp 1.38% 32 Missing and 39 partials ⚠️
...rver/src/storage-service/capio_storage_service.cpp 41.17% 41 Missing and 19 partials ⚠️
...-service/control-plane/multicast_control_plane.cpp 62.03% 10 Missing and 31 partials ⚠️
...rver/src/client-manager/request_handler_engine.cpp 54.87% 15 Missing and 22 partials ⚠️
capio-server/src/file-manager/fs_monitor.cpp 37.50% 15 Missing and 20 partials ⚠️
capio-server/src/utils/distributed_semaphore.cpp 0.00% 26 Missing and 9 partials ⚠️
... and 14 more
Additional details and impacted files
@@             Coverage Diff              @@
##           capio-v2     #162      +/-   ##
============================================
- Coverage     43.55%   43.25%   -0.30%     
============================================
  Files            81       83       +2     
  Lines          4459     4513      +54     
  Branches       2654     2674      +20     
============================================
+ Hits           1942     1952      +10     
- Misses         1126     1289     +163     
+ Partials       1391     1272     -119     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@marcoSanti marcoSanti force-pushed the refactor branch 2 times, most recently from 85cfacb to 09e8a8c Compare August 30, 2025 09:00
@marcoSanti marcoSanti force-pushed the refactor branch 3 times, most recently from c527c79 to 189f4e7 Compare August 30, 2025 10:46
@marcoSanti marcoSanti merged commit 9af295c into capio-v2 Aug 30, 2025
28 checks passed
@marcoSanti marcoSanti deleted the refactor branch August 30, 2025 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant