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
Add option for filtering out full covered files from HTML report #268
Conversation
220aa91
to
e5b5d52
Compare
Thank you for the suggestion. The filtering option might be beneficial if it can be switched (as optional). But I'm not confident enough yet, and may need to check the behavior a little more. |
This would be a very welcome feature. I have a project with a lot of modules, and it can be hard to spot the ones with missing coverage in either the HTML or terminal output. |
Hi @parroty, did you have time to think about this PR? |
Thank you for the follow-up, and sorry for not being responsive. I think I previously couldn't make the tests pass. Is it possible for you to re-push the changes to run the actions workflow? (I cannot see a re-run option from my end. It could be the previous run was too old). |
Thank you for the PR. One request would be, is it possible to add a brief description for the option ( |
e5b5d52
to
dc8c4dd
Compare
@parroty done |
Thanks for the quick update. The test failed on the size check part as the HTML layout has changed. Could you update the corresponding test part too? stacktrace:
diff --git a/test/html_test.exs b/test/html_test.exs
index 73c8dda..bb29e3f 100644
--- a/test/html_test.exs
+++ b/test/html_test.exs
@@ -5,7 +5,7 @@ defmodule ExCoveralls.HtmlTest do
alias ExCoveralls.Html
@file_name "excoveralls.html"
- @file_size 20191
+ @file_size 20375
@test_output_dir "cover_test/"
@test_template_path "lib/templates/html/htmlcov/" |
dc8c4dd
to
6482d8e
Compare
Oh sorry, I didn't see that error, now it should be ok |
Thanks! |
I find it useful to focus on files not fully covered so I added an option for HTML reports that allows hiding all fully covered files.
This is only a proposal and some open points/missing things remain to be done but I would like to know your opinion before actually implement all of them.
I know that I can copy the
eex
template on my repo and add the needed if clause to obtain this behavior but I think this can be useful to other people.Open points