pytest-html icon indicating copy to clipboard operation
pytest-html copied to clipboard

warnings summary not reflected in html-summary (feature request)

Open nicain opened this issue 7 years ago • 10 comments
trafficstars

In addition to test failure, pytest captures a warnings summary and reports this at the end of the test summary. It would be great if this could be reflected in the html summary as well.

nicain avatar May 10 '18 05:05 nicain

Hi @davehunt, is this still something that is relevant? If so I'd like to take a stab at implementing it

gnikonorov avatar May 23 '20 01:05 gnikonorov

Sure, feel free to submit a PR! 👍 @gnikonorov

BeyondEvil avatar May 25 '20 06:05 BeyondEvil

Thank you @BeyondEvil!

gnikonorov avatar May 25 '20 15:05 gnikonorov

Still planning on working on this. I can start once https://github.com/pytest-dev/pytest/pull/7255 is merged

gnikonorov avatar May 30 '20 15:05 gnikonorov

Hey @gnikonorov this feature would be great. Are you still planning to do this?

nepumuk123 avatar Jun 16 '20 09:06 nepumuk123

Hi @nepumuk123, I am yes. We just need to wait for a version of pytest that contains https://github.com/pytest-dev/pytest/pull/7255 to be released. That should be coming soon. I'll update this issue when it happens

gnikonorov avatar Jun 16 '20 12:06 gnikonorov

Cool, thank you.

nepumuk123 avatar Jun 16 '20 13:06 nepumuk123

I think this is merged and released now @gnikonorov :)

BeyondEvil avatar Nov 10 '20 17:11 BeyondEvil

I'm going to pick this up again, @BeyondEvil

The solution would involve making a separate 'warnings summary' table similar to the results table but with only 1 data point, the warnings for a test. It would only show tests that emit warnings and would be hidden from view if there are no warnings.

This would involve a merge into your next-gen branch. Do you have any opposition?

gnikonorov avatar Dec 21 '20 01:12 gnikonorov

This would involve a merge into your next-gen branch. Do you have any opposition?

I have no idea what the implications will be, but we'll worry about that later. :)

BeyondEvil avatar Dec 21 '20 22:12 BeyondEvil