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

uber / phabricator-jenkins-plugin / 213
50%
master: 90%

Build:
Build:
LAST BUILD BRANCH: print2
DEFAULT BRANCH: master
Ran 20 Jul 2015 02:07AM UTC
Jobs 3
Files 19
Run time 5min
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

pending completion
213

push

travis-ci

sectioneight
Refactor PhabricatorNotifier comment builder

Instead of having one heaping class that's responsible for a bunch of
business logic, breaking out the comment building and code coverage
comparison into a self-contained class that is easier to test and understand.

Future refactors will break down the Notifier even further, but this is
one atomic piece of work that can go in.

Additionally, fixed a bug in the TestUtils default coverage data, which
was off by 100x for cobertura results (since cobertura the plugin
returns them as floats from 0-1)

69 of 69 new or added lines in 2 files covered. (100.0%)

327 of 651 relevant lines covered (50.23%)

8.57 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
14
100.0
src/main/java/com/uber/jenkins/phabricator/PhabricatorNotifier.java

Uncovered Existing Lines

Lines Coverage ∆ File
44
100.0
src/main/java/com/uber/jenkins/phabricator/PhabricatorNotifier.java
Jobs
ID Job ID Ran Files Coverage
1 213.1 20 Jul 2015 02:07AM UTC 0
51.8
Travis Job 213.1
2 213.2 20 Jul 2015 02:09AM UTC 0
51.8
Travis Job 213.2
3 213.3 20 Jul 2015 02:13AM UTC 0
50.38
Travis Job 213.3
Source Files on build 213
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #213
  • 345ef8ba on github
  • Prev Build on refactor-commenter (#209)
  • Next Build on refactor-commenter (#216)
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