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

CypressError: Timed out after waiting 60000ms for your remote page to load #1131

Closed
asnaith opened this issue Jun 11, 2021 · 2 comments
Closed
Labels
Issue: Bug 🐛 Added to issues relating to a critical severity bugs. Product: Files Testing Added to issues to signal that it is part of an epic

Comments

@asnaith
Copy link
Contributor

asnaith commented Jun 11, 2021

We have been experiencing sporadic page load timeouts whilst running ui tests in CI. It doesn't seem to be an issue with a specific test.

Below is an example:

https://github.com/ChainSafe/files-ui/pull/1127/checks?check_run_id=2801773985
Settings -- mobile -- can navigate to the settings security page on a phone -- before all hook (failed)

I have looked into this but haven't found a fix just yet.

Suggestions:

Improvement A

I think cypress is sometimes trying to perform an action (page visit) too quickly, before the browser has had chance to respond to the previous commands (window object creation for ethereum wallet) and this is seen more in CI because it's not as powerful as our development machines.

Perhaps we could add a guard statement, with logging, that checks for the created window object before proceeding to the page visit (not too sure how to write that yet) or at least we could a slight wait statement between the two commands but the former would be better.

I think it could be a similar situation with #1070 too. Cypress commands running too quickly before the browser can respond to a previous one.

Improvement B

After reviewing test.yml / package.json it seems that we are only using "cypress run" to execute the tests. According to this doc that will mean we are using Electron by default in CI.

We can specify it to use chrome using the command cypress run --browser chrome instead. I read online that there are other teams experiencing similar issues with a electron / cypress / ci combo that were resolved by doing this.

@Tbaut
Copy link
Collaborator

Tbaut commented Jun 14, 2021

I think it's the same as #1070 indeed. I'll close it in favor of this one.

@Tbaut
Copy link
Collaborator

Tbaut commented Aug 10, 2021

I think we can close this one, we got rid of the readFiles tasks thanks to cy.session in #1421

@Tbaut Tbaut closed this as completed Aug 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Bug 🐛 Added to issues relating to a critical severity bugs. Product: Files Testing Added to issues to signal that it is part of an epic
Projects
None yet
Development

No branches or pull requests

2 participants