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

Smoke test other tabs. #61

Closed
wants to merge 2 commits into from
Closed

Smoke test other tabs. #61

wants to merge 2 commits into from

Conversation

samcunliffe
Copy link
Member

@samcunliffe samcunliffe commented Mar 17, 2023

Now that #38 is merged, I'm reminded that I wanted to check all pages load without error.
🥁 ...... they don't.

Not sure if this behaviour is expected/intended (in which case we shouldn't test for error-free loading). But perhaps it should be possible to startwazp → not load anything → and just click around the (empty) tabs. What do you think?

To run locally

  • Same instructions as Integration testing mechanics. #36 (need chromedriver in your PATH).
  • Screenshots of the pages where there were errors in loading are saved in .test_fail_screenshots.
    • They look OK to me, so tentatively: do we just silence these loading errors?

Currently failing for genuine errors.
As i is before s in the alphabet this will fail first and fastest.
@samcunliffe samcunliffe added help wanted Extra attention is needed question Further information is requested labels Mar 17, 2023
@samcunliffe
Copy link
Member Author

Closed in favour of #82

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant