CI: Unit Tests for gRPC Base Module

[d4v1d03]

Add unit tests for sdk/shared/grpc_base.py covering:

• GrpcClientBase.init() with various timeout configurations
• _handle_grpc_error() with different gRPC status codes (DEADLINE_EXCEEDED, UNAVAILABLE, NOT_FOUND, etc.)
• Mock gRPC errors to test error handling without actual server

[bomanaps]

Can I work on this

[d4v1d03]

Sure, i'll get you assigned

[bomanaps]

Sure, i'll get you assigned

The grpc_base.py file currently only has 2 features (initialization + error handling), but the CI requirements ask for tests covering 6 features. To match the requirements, grpc_base.py needs to be extended to include:

1. Connection management - creating/managing gRPC connections
2. Channel lifecycle - opening, closing, and reusing channels
3. Retry logic - automatic retries with backoff on failures

These features currently exist in connection.py and sdk.py separately. Either move them into grpc_base.py as a unified base class, or update the CI task description to reference the correct files where these features already live and are tested.

[bomanaps]

Sure, i'll get you assigned

The grpc_base.py file currently only has 2 features (initialization + error handling), but the CI requirements ask for tests covering 6 features. To match the requirements, grpc_base.py needs to be extended to include:

1. Connection management - creating/managing gRPC connections

2. Channel lifecycle - opening, closing, and reusing channels

3. Retry logic - automatic retries with backoff on failures

These features currently exist in connection.py and sdk.py separately. Either move them into grpc_base.py as a unified base class, or update the CI task description to reference the correct files where these features already live and are tested.

what about ?

[d4v1d03]

Connection management, channel lifecycle, and retry logic are handled elsewhere in the codebase and don't need to be moved.
I'll update the scope of this issue.

[bomanaps]

Connection management, channel lifecycle, and retry logic are handled elsewhere in the codebase and don't need to be moved. I'll update the scope of this issue.

Please any update on this?

[d4v1d03]

Yes I updated the scope of the issues on the specific unit tests that are required, pls have a look at the Issue description.