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

When taking a screenshot at a fail also add a .txt file with the page HTML for debugging #39

Open
sjieg opened this issue Sep 14, 2016 · 1 comment

Comments

@sjieg
Copy link
Contributor

sjieg commented Sep 14, 2016

In many cases this can help making debugging easier and adding this feature shouldn't be too much work.

I was thinking to give the HTML file the same name as the screenshot. Possibly this could also be linked in Calliope.

@osteenbergen
Copy link
Contributor

You can solve this issue in two easy ways, add module similar to screenshot that only stores HTML output or extend the screenshot module. Both of them should require a configuration setting to enable this feature.

However I would prefer a third option: creating a new browser module called state which includes the screenshot/html features (screenshot module can be removed)

This state module would be responsible for writing all types of information:

  • HTML
  • Screenshot
  • Other information like OS/Browser

Depending on configuration flags it would write the requested output.

The easy options would either require some ugly code duplication (filename) or combine features that could be useful by itself (maybe someone doesn't want screenshots only html).

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

No branches or pull requests

2 participants