Improved code coverage

[marcoSanti]

This pull request introduces unit tests for the StorageManager and ClientManager classes and refactors parts of their implementation to improve testability and coverage reporting.

• Fixed Docker CI/CD tests: previusly the test suite executind within the Docker image was executing twice the server unit tests, instead of executing the POSIX syscall unit tests. This commit now uniforms it to the standard unit tests.

• Refactor ClientManager for improved gcovr compatibility

  • Modify ClientManager::ClientDataBuffers to store objects instead of pointers. This allows gcovr to correctly detect destructor calls for member attributes.
  • Introduce a dedicated constructor for ClientDataBuffers to simplify object creation and ensure proper coverage reporting. Previously, move operations were interpreted by gcovr as partially covered code.

• Add unit tests

  • Add comprehensive unit tests for the StorageManager class.
  • Add comprehensive unit tests for the ClientManager class.

• Improve StorageManager testability

  • Remove the dependency on the global client_manager variable by passing a reference through the StorageManager constructor.
  • This enables unit tests to compile and run without requiring a globally defined client_manager.

• Bug fix

  • Fix a minor issue in StorageManager::getPaths() where empty paths were previously added to the returned vector.

• Build system update

  • Move the CAPIO-CL dependency from capio/server/CMakeLists.txt to the top-level project CMakeLists.txt, allowing both the server components and the unit tests to compile successfully.

[codecov]

Codecov Report

❌ Patch coverage is 71.90083% with 34 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.59%. Comparing base (79e5b06) to head (ba3c3c1).
⚠️ Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
capio/tests/unit/server/src/storage_manager.cpp	66.12%	0 Missing and 21 partials ⚠️
capio/tests/unit/server/src/client_manager.cpp	65.51%	1 Missing and 9 partials ⚠️
capio/tests/unit/server/src/main.cpp	82.35%	0 Missing and 3 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #208      +/-   ##
==========================================
+ Coverage   56.14%   58.59%   +2.45%     
==========================================
  Files          71       75       +4     
  Lines        3482     3635     +153     
  Branches     2165     2285     +120     
==========================================
+ Hits         1955     2130     +175     
+ Misses        575      536      -39     
- Partials      952      969      +17     

☔ 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.

[GlassOfWhiskey]

LGTM