apache / bookkeeper / 186

72%

push

jenkins

Make LedgerHandle injectable

This patch removes the hard dependency of LedgerHandle on BookKeeper.
This allows us to create an instance of LedgerHandle independently,
with a fully mocked BookieClient or LedgerManager and be clear on
where the objects used by the LedgerHandle are coming from.

Author: Ivan Kelly <ivank@apache.org>

Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Jia Zhai <None>, Sijie Guo <sijie@apache.org>

This closes #1589 from ivankelly/injectable-lh

290 of 290 new or added lines in 22 files covered. (100.0%)

24829 of 34298 relevant lines covered (72.39%)

0.72 hits per line