search

XRPLF / clio

An XRP Ledger API Server
https://xrpl.org
ISC License
56 stars 48 forks source link

Cover LoadBalancer with tests #1394

Closed kuznetsss closed 1 month ago

kuznetsss commented 2 months ago

Fixes #680. Fixes #1222.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 74.32763% with 105 lines in your changes are missing coverage. Please review.

Project coverage is 67.80%. Comparing base (f74b89c) to head (70fab34).

Files Patch % Lines
src/etl/impl/LedgerLoader.hpp 0.00% 31 Missing :warning:
src/etl/NetworkValidatedLedgers.cpp 0.00% 18 Missing :warning:
src/rpc/handlers/Subscribe.cpp 88.80% 4 Missing and 11 partials :warning:
src/etl/impl/SourceImpl.hpp 61.11% 3 Missing and 11 partials :warning:
src/etl/LoadBalancer.cpp 68.42% 6 Missing and 6 partials :warning:
src/rpc/handlers/Unsubscribe.cpp 91.86% 4 Missing and 3 partials :warning:
src/data/BackendInterface.cpp 0.00% 2 Missing :warning:
src/etl/Source.cpp 0.00% 2 Missing :warning:
src/etl/impl/AsyncData.hpp 0.00% 1 Missing and 1 partial :warning:
src/etl/ETLState.cpp 85.71% 0 Missing and 1 partial :warning:
... and 1 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #1394 +/- ## =========================================== + Coverage 65.60% 67.80% +2.20% =========================================== Files 227 232 +5 Lines 9317 9309 -8 Branches 5203 5203 =========================================== + Hits 6112 6312 +200 + Misses 1747 1638 -109 + Partials 1458 1359 -99 ```

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