Skip to content
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

HTML report doesn't include the total number of scenarios passed and failed. #159

Closed
sswaroopgupta opened this issue Sep 11, 2017 · 13 comments
Assignees

Comments

@sswaroopgupta
Copy link

From @osandadeshan on August 18, 2017 11:13

Expected behavior

HTML report should include the total number of scenarios passed and failed. Also when a user clicks on a specification name, it should show the number of scenarios passed and failed for that specification.

Actual behavior

HTML report doesn't include the total number of scenarios passed and failed. Also when a user clicks on a specification name, it doesn't show the number of scenarios passed and failed for that specification.

Steps to reproduce

  1. Create two or more specifications
  2. Run all those specifications
  3. Open the html-report
  4. There will only show the number of failed, pass, skipped in specifications wise

Gauge version

Gauge version: 0.9.1

Plugins

html-report (4.0.2)
java (0.6.4)
spectacle (0.1.2)
xml-report (0.2.0)

capture

Copied from original issue: getgauge/gauge#777

@j-morsum
Copy link

That's an issue that would be great to have solved. It has been bugging me since I started using Gauge almost a year and a half ago.

@osandadeshan
Copy link

osandadeshan commented Oct 25, 2017 via email

@nehashri nehashri added the ready label Oct 26, 2017
@sriv
Copy link
Member

sriv commented Oct 27, 2017

@jorgecunhamorsum @osandadeshan - yes, this seems like a good thing to do.

We are working on adding theme support for html-report and as part of this, we hope to allow customizations that suits custom requirements.

While this may take a while, I think I may have an idea that might work for you. Please try the below steps:

  • Ensure that you have the latest html-report installed (currently, the released version is 4.0.2).

  • On windows, the html report is installed at %APPDATA%\gauge\plugins\html-report\4.0.2, on *nix it will be in $HOME/.gauge/plugins/html-report/4.0.2.

  • Edit the file %APPDATA%\gauge\plugins\html-report\4.0.2\themes\default\views\partials.tmpl. Line 190 should look like this:

  • Change it to this:

  • Save and run gauge specs.

  • You should see something like this:

image

@j-morsum
Copy link

Thanks @sriv for the information. By the way you forgot to include the change to line 190 but I assume it is to remove the "hidden" from the class of that div. I've tried that and works as described.

It's an improvement, but what I was mentioning, and what I'm looking for, is the statistics for the scenarios of the whole suite and not only for each specification.

@zabil zabil removed the ready label Jan 15, 2018
@ajoecker
Copy link

any news on this ?

especially, as the Change it to this:" line is empty, so what needs to be changed ?

@kgilmore88
Copy link

I would really like to see this as well. Also, if anyone has the information that was removed from the earlier comment on what to change, that would be great.

@gaugebot
Copy link

gaugebot bot commented Oct 10, 2018

The fix should be available in nightly >= 10-10-2018

@sswaroopgupta
Copy link
Author

The current UI is
screen shot 2018-10-11 at 10 09 55 am

The expected UI is
expected_html

@sswaroopgupta
Copy link
Author

This seems to have something with resolution. With my default resolution on Mac the issue is there 1440x900. However when I change the resolution to 1680x1050 it is working as expected.

@osandadeshan
Copy link

osandadeshan commented Oct 11, 2018 via email

@sswaroopgupta
Copy link
Author

Thanks @osandadeshan for your input. The responsiveness of the report will be tracked as a part of #200

@j-morsum
Copy link

j-morsum commented Oct 25, 2018

Thank you for (finally) adding this information. I've just installed the new version and tested and all works fine.

I've made a few changes, like the success rate and pie chart to be based on the scenarios numbers instead of specs and it is perfect for me now. Once again thanks.

new html report

@sswaroopgupta
Copy link
Author

@j-morsum Thank you for your kind words. If you think the success rate and pie chart makes more sense with scenario numbers - kindly raise an issue. This helps us discuss and see if this helps a wider audience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

9 participants