• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

apache / bookkeeper / 186 / 1
72%
master: 72%

Build:
DEFAULT BRANCH: master
Ran 21 Aug 2018 03:20PM UTC
Files 436
Run time 58s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

21 Aug 2018 03:20PM UTC coverage: 72.392% (+0.03%) from 72.366%
186.1

push

jenkins

Ivan Kelly
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

24829 of 34298 relevant lines covered (72.39%)

0.72 hits per line

Source Files on job 186.1
  • Tree
  • List 0
  • Changed 40
  • Source Changed 21
  • Coverage Changed 40
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 186
  • 8e103282 on github
  • Prev Job for on master (#185.1)
  • Next Job for on master (#187.1)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc