aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <>
Subject Re: Review Request 22792: Add test coverage for MesosLog.
Date Thu, 19 Jun 2014 22:06:09 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated June 19, 2014, 10:06 p.m.)

Review request for Aurora and Maxim Khutornenko.

Repository: aurora


This brings instruction coverage to 86% in MesosLog, 81% overall.  Unfortunately in this effort
i learned of a shortcoming in jacoco, described here:
> Code with exceptions shows no coverage. Why?

> JaCoCo determines code execution with so called probes. Probes are inserted into the
control flow at certain positions.
> Code is considered as executed when a subsequent probe has been executed. In case of
exceptions such a sequence of
> instructions is aborted somewhere in the middle and not marked as executed.

This issue prevents us from achieving 100% coverage in MesosLog, and will do so in other classes
that throw exceptions.

Diffs (updated)

  src/main/java/org/apache/aurora/scheduler/log/mesos/ b8a94ff90dfcb4d18a79ae3d60fc8384eebf590f

  src/test/java/org/apache/aurora/scheduler/log/mesos/ 23c1d7a4190c58912d99009ae75bdd87488cd688



./gradlew build -Pq


Bill Farner

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message