Added test coverage reporting

[smithdc1]

🤞 CodeCov will respond.

[codecov-io]

Codecov Report

❗ No coverage uploaded for pull request base (master@b9be797). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   56.95%           
=========================================
  Files             ?        7           
  Lines             ?      223           
  Branches          ?       31           
=========================================
  Hits              ?      127           
  Misses            ?       86           
  Partials          ?       10           

Flag	Coverage Δ
#unittests	56.95% <0.00%> (?)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b9be797...4ebe349. Read the comment docs.

[smithdc1]

I've tried to add coverage to the templates, as that is mostly what this package will be.

This isn't currently working as expected 😕

• Coverage Report gives coverage for all the files (incl. templates) (detail in GH Action logs)
• They are in the current working directory (detail in GH Action logs)
• But are not appearing in codecov
• The Upload file shows coverage data for templates... but files are missing from the network section at the top. 🤷

May need to ask codecov support, unless anyone has any ideas?

[carltongibson]

...unless anyone has any ideas?

None immediately, sorry. 🙂 — I'd need to look into it.

[smithdc1]

None immediately, sorry. 🙂 — I'd need to look into it.

I've asked codecov support, let's see what they say.

https://community.codecov.io/t/missing-coverage-data/1357

[thomasrockhu]

Hi, Tom from Codecov here, sorry that it's been taking a while for us to get back to you. I'll get back to you on the community thread today.

[thomasrockhu]

Hi @smithdc1, just confirming that we patched the bash uploader with a new flag -X html. Please add this when calling Codecov, and it should parse the coverage on those files. Let me know if that doesn't work as expected.

[codecov-commenter]

Codecov Report

❗ No coverage uploaded for pull request base (master@bee47a9). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   29.12%           
=========================================
  Files             ?       35           
  Lines             ?      570           
  Branches          ?       25           
=========================================
  Hits              ?      166           
  Misses            ?      395           
  Partials          ?        9           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bee47a9...89ee83f. Read the comment docs.

[smithdc1]

Hi @thomasrockhu,

Super thanks for your help. 👏

This has worked now (https://codecov.io/gh/django-crispy-forms/crispy-tailwind/pull/5/tree?path=crispy_tailwind%2Ftemplates%2Ftailwind%2Flayout)

Although there is now a bit of work to do to with the coverage % 🙈

p.s. using the bash uploader was easier than the github action 😄

[thomasrockhu]

@smithdc1 Amazing! Glad it worked out for you. We currently maintain the bash uploader more, but we'd love to get feedback on the GitHub Action. What would have made using the Action easier for you?

[smithdc1]

Hi @thomasrockhu
There may be a bit of comparing different times. GH have only recently released the API to enable uploads from public repos without a token.

This time I needed to enable this additional setting for coverage to work on HTML. I had a quick look at the docs but couldn't see how to pass settings onto the bash uploader?

[thomasrockhu]

Yes, it's something that we are aware of and working around for. Thanks for bringing it up, I'll try to prod about passing along other arguments down from the action