aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <wfar...@apache.org>
Subject Review Request 26574: Handle anonymous inner classes better in per-class coverage check.
Date Fri, 10 Oct 2014 20:59:23 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26574/
-----------------------------------------------------------

Review request for Aurora, Joshua Cohen and Maxim Khutornenko.


Bugs: AURORA-822
    https://issues.apache.org/jira/browse/AURORA-822


Repository: aurora


Description
-------

`javac` inserts synthetic zero-arg constructors into anonymous classes, which jacoco sometimes
detects as covered by tests.  This was throwing off the "each class must have _some_ coverage"
by considering classes without coverage to actually have some coverage.

I've attempted to fix this by ignoring the default constructor in anonymoous classes, which
should be more accurate.  One sign that this is working — lots of modules came onto the
radar as lacking coverage, which is legitimate.


Diffs
-----

  build.gradle 8f7eed0b58a00f8e5e0521512975087d5788b5b6 

Diff: https://reviews.apache.org/r/26574/diff/


Testing
-------

./gradlew build -Pq


Thanks,

Bill Farner


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