Replace delegate based server impl with call handler

Open marcoferrer opened this issue 6 years ago • 1 comments

Dec 31 '19 04:12 marcoferrer

Codecov Report

Merging #102 into master will decrease coverage by 22.5%. The diff coverage is 89.21%.

@@              Coverage Diff              @@
##             master     #102       +/-   ##
=============================================
- Coverage      87.8%   65.29%   -22.51%     
- Complexity       21       22        +1     
=============================================
  Files            16       19        +3     
  Lines           369      608      +239     
  Branches         50       75       +25     
=============================================
+ Hits            324      397       +73     
- Misses           26      183      +157     
- Partials         19       28        +9

Impacted Files	Coverage Δ	Complexity Δ
...us/coroutines/client/ClientStreamingCallChannel.kt	`100% <ø> (+18.18%)`	`0 <0> (ø)`	:arrow_down:
...oferrer/krotoplus/coroutines/server/ServerCalls.kt	`5.81% <ø> (-82.56%)`	`0 <0> (ø)`
.../marcoferrer/krotoplus/coroutines/call/CallExts.kt	`89.74% <100%> (+4.89%)`	`0 <0> (ø)`	:arrow_down:
...s/coroutines/server/DeferredCancellationHandler.kt	`78.57% <78.57%> (ø)`	`5 <5> (?)`
...oplus/coroutines/client/ClientResponseObservers.kt	`80.55% <80.55%> (ø)`	`0 <0> (?)`
...rer/krotoplus/coroutines/call/CallReadyObserver.kt	`93.33% <93.33%> (ø)`	`7 <7> (?)`
.../krotoplus/coroutines/server/ServerCallHandlers.kt	`95.08% <95.08%> (ø)`	`0 <0> (?)`
...oferrer/krotoplus/coroutines/client/ClientCalls.kt	`89.85% <95.12%> (-1.06%)`	`0 <0> (ø)`
...otoplus/coroutines/client/ClientBidiCallChannel.kt	`6.66% <0%> (-90%)`	`0% <0%> (ø)`
... and 9 more

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 aad7b71...b232741. Read the comment docs.

Dec 31 '19 04:12 codecov[bot]