dropwizard / dropwizard / 3871

86%

push

travis-ci

Exception Mapper Logging Enhancements (#2129)

* Made master POM Eclipse friendly.

Eclipse does not support the errorprone compiler; moved errorprone setup
from build-plugins to profiles. There are now two profiles, one for
Eclipse and one for everything else.

* Exception mapper logging enhancements

1) Implemented debug logging for JerseyViolationExceptionMapper.
2) Fixed compiler warning regarding serialization for
JerseyViolationException.
3) Wrapped debug statement in JsonPropcessingExeptionMapper with
"isDebugEnabled"; this will ensure the JVM does not bother loading and
caching the log string unless debug is actually enabled. This will also
allow more aggressive code optimization when running the server JVM.

* New unit test.

* Changes as per PR code review

Removed if (LOG.isDebugEnabled()) checks from log statements with
simplistic messages to match requested style.

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

6325 of 7425 relevant lines covered (85.19%)

78.02 hits per line