Set error detail only for WebApplicationExceptions

Mercateo / rest-jersey-utils

Apache License 2.0

2 stars 4 forks source link

Set error detail only for WebApplicationExceptions #3

Closed BernhardBln closed 6 years ago

BernhardBln commented 6 years ago

Do not set the error detail with the exception message for ordinary exceptions.

The exception message is now only set to the error detail field if the exception is a WebApplicationException.

Also fixed:

Ignore IntelliJ project file in git
Added @ToString annotation to SimpleExceptionJson in order to see a better error message when tests are failing

coveralls commented 6 years ago

Coverage increased (+0.4%) to 96.104% when pulling b4113bf7794d17e38c88bab261150523b86e4537 on BernhardBln:master into 887ccc84c32df2ceb10f5861e712554fb392449d on Mercateo:master.

BernhardBln commented 6 years ago

Yes, I admit, I didn't write a test for the toString method of the JSON pojo... but how about tweaking the coverage settings instead? :D

JoergAdler commented 6 years ago

thx a lot :-)