-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace empty table in HTML report by messages #833
Conversation
@Godin I'm not so sure about this message:
Strictly speaking abstract classes will at least have a constructor (which might only be filtered if it is empty and private) also we have other filters also for non generated code. So I'm not sure whether this is correct right now or might become wrong when adding more filters in future. Therefore I would provide a less specific message, like
|
@marchof I was looking for neutral message and your proposal looks great! |
@Godin Just to make sure: same applies for change log entry. |
@marchof sure. And done. |
Using Ant Task and therefore Gradle Plugin, that is based on it, possible to generate report without classes.
For example in case of following
build.xml
and
build.gradle
execution of
will produce
Note the absence of messages in the second and third cases.
Also unfortunately many users don't provide screenshots and the way they describe such report is ambiguous: "empty report" and "zero(s)" - in both cases some indeed refer to such report, some other refer to report with classes where covered counters are zero.
Could also be noted that produced
index.html
claims to be XHTML 1.0 Strict whereas this is not the case because of<tbody/>
.So I propose to replace empty table by messages for two cases thanks to #817