Skip to content

Conversation

@tanmaykm
Copy link
Member

@tanmaykm tanmaykm commented May 3, 2021

gRPCClient now reports back timeouts server errors via gRPC error codes. The gRPCServiceCallException struct has additional properties to hold the error code.

When an error is encountered during a call that returns a single result (not stream), the call method now throws an exception instead of returning nothing as the result.

Test suite to test error conditions added.

gRPCClient now reports back timeouts server errors via gRPC error codes. The `gRPCServiceCallException` struct has additional properties to hold the error code.

When an error is encountered during a call that returns a single result (not stream), the call method now throws an exception instead of returning `nothing` as the result.

Test suite to test error conditions added.
@codecov-commenter
Copy link

codecov-commenter commented May 3, 2021

Codecov Report

Merging #10 (8574b40) into main (305b4d8) will decrease coverage by 1.38%.
The diff coverage is 90.19%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
- Coverage   96.95%   95.57%   -1.39%     
==========================================
  Files           5        5              
  Lines         230      271      +41     
==========================================
+ Hits          223      259      +36     
- Misses          7       12       +5     
Impacted Files Coverage Δ
src/gRPCClient.jl 100.00% <ø> (ø)
src/grpc.jl 94.73% <66.66%> (-5.27%) ⬇️
src/curl.jl 95.77% <95.23%> (-0.42%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 305b4d8...8574b40. Read the comment docs.

@tanmaykm tanmaykm merged commit f8f2f68 into main May 3, 2021
@delete-merged-branch delete-merged-branch bot deleted the tan/ci branch May 3, 2021 04:59
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.

3 participants