-
Notifications
You must be signed in to change notification settings - Fork 2
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
Tmma 461 expand cypress tests #223
Conversation
…ed unused test file and old visualiations test script
|
||
// now we are gonna wait 30 seconds... | ||
|
||
cy.wait(30000) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Rodsy Did you get the chance can you time this? If not can you try 15 seconds, then 7 seconds, and half agin lower if possible. To find a consistent lowest value we can wait. Alternatively rewrite the test to use the behaviour an end user would need to use, press reload and check in a loop.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, np boss.
@Rodsy At present the user journey and large dataset tests need to be excluded running on py-web-t0.epi.bris.ac.uk which is why the Cypress tests failed, see email. Can you exclude them from running as with the previous visualisation test please? |
No description provided.