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

Dragging the zip doesn't work, I need to open it and drag the files #7

Open
mattip opened this issue Jul 17, 2022 · 7 comments
Open

Comments

@mattip
Copy link

mattip commented Jul 17, 2022

This looks really neat.

I am trying to add the reporting/upload to pytest-html. When I download the artifact zip file, and then drag-and-drop it on https://carreau.github.io/pytest-json-report-viewer/, nothing happens. Do I need to repack the results somehow?

@Carreau
Copy link
Owner

Carreau commented Jul 17, 2022

Oh, I don't know how to unpack the zil in JS yet, you need to drag the 5 json files at once.
Then it seem to work.

@Carreau
Copy link
Owner

Carreau commented Jul 17, 2022

Also I'll try to fix it but I have a server that can directly take a url : https://pytest-viewer.fly.dev/gh/pytest-dev/pytest-html/pull/518

Right now I think it fails as the token is expired.

@mattip mattip changed the title What am I doing wrong? Dragging the zip doesn't work, I need to open it and drag the files Jul 17, 2022
@mattip
Copy link
Author

mattip commented Jul 17, 2022

OK, so this is a documentation issue. I changed the title.

@mattip
Copy link
Author

mattip commented Jul 18, 2022

Getting somewhere, now I can see the different test runs using this archive. But I don't see individual test results. I am using --durations 0, do I need to add more arguments to the pytest run ?
Screenshot from 2022-07-18 07-36-08

@Carreau
Copy link
Owner

Carreau commented Jul 18, 2022

so it looks like with your repo the nodeid are of the form:

testing/test_pytest_html.py::TestHTML::test_durations, and I did not take into account that they could have :: twice, so currently this behave as if all the tests were part of TestHTML. I'll try to fix that.

@Carreau
Copy link
Owner

Carreau commented Jul 18, 2022

If you use this URL it should work now. https://pytest-viewer.fly.dev/

@mattip
Copy link
Author

mattip commented Jul 18, 2022

Cool, it works!

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

No branches or pull requests

2 participants