SammyOina commented 1 month ago

What type of PR is this?

What does this do?

New Features
- Introduced mock implementations for AgentService_AlgoClient and AgentService_DataClient to enhance testing capabilities.
- Added new test cases for various functionalities, improving overall test coverage and error handling.
Bug Fixes
- Improved error handling in the NewAgentClient and NewManagerClient functions.
Tests
- Added multiple unit tests across various packages to validate functionality and error scenarios, including progress bar interactions and client configurations.

Which issue(s) does this PR fix/relate to?

Related Issue #159

Have you included tests for your changes?

Did you document any new/modified feature?

Notes

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 1.96850% with 249 lines in your changes missing coverage. Please review.

Project coverage is 43.40%. Comparing base (184617d) to head (253ed38). Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
agent/mocks/agent_grpc_algo.go	0.00%	124 Missing :warning:
agent/mocks/agent_grpc_data.go	0.00%	124 Missing :warning:
pkg/clients/grpc/connect.go	83.33%	1 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #286 +/- ## ========================================== - Coverage 43.40% 43.40% -0.01% ========================================== Files 77 79 +2 Lines 5234 6513 +1279 ========================================== + Hits 2272 2827 +555 - Misses 2620 3330 +710 - Partials 342 356 +14 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

dborovcanin commented 3 weeks ago

@SammyOina Please rebase.