Skip to content

fix(e2e): wait for next challenge render#41888

Merged
raisedadead merged 4 commits intofreeCodeCamp:mainfrom
ojeytonwilliams:fix/flaky-cypress-test
Apr 22, 2021
Merged

fix(e2e): wait for next challenge render#41888
raisedadead merged 4 commits intofreeCodeCamp:mainfrom
ojeytonwilliams:fix/flaky-cypress-test

Conversation

@ojeytonwilliams
Copy link
Copy Markdown
Contributor

The issue was caused by cy.contains('Go to next challenge').click(); and then not waiting for the result before visiting the next page.

@ojeytonwilliams ojeytonwilliams requested a review from a team April 22, 2021 11:33
@github-actions github-actions bot added the scope: tools/scripts Scripts for supporting dev work, generating config and build artifacts, etc. label Apr 22, 2021
Copy link
Copy Markdown
Member

@raisedadead raisedadead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the test run was cancelled. Not sure if this is ready or WIP?

@ojeytonwilliams ojeytonwilliams marked this pull request as draft April 22, 2021 13:38
@ojeytonwilliams
Copy link
Copy Markdown
Contributor Author

ojeytonwilliams commented Apr 22, 2021

I'm still in the dark about what's going on here. I'll give an update when I have more of a clue.

@raisedadead raisedadead added the status: PR in works Work in Progress (WIP) Issues. label Apr 22, 2021
@raisedadead
Copy link
Copy Markdown
Member

Yeah, I figured that you were stuck.

@ojeytonwilliams
Copy link
Copy Markdown
Contributor Author

I still don't have much of a clue what the root cause of this is. Possibly something strange with Gatsby (since it's got its own webpack config that's creating these chunks) or vercel/serve.

@ojeytonwilliams ojeytonwilliams marked this pull request as ready for review April 22, 2021 16:21
@ojeytonwilliams
Copy link
Copy Markdown
Contributor Author

@raisedadead I'm not 100% confident it's fixed, but it has passed the tests.

@raisedadead raisedadead removed the status: PR in works Work in Progress (WIP) Issues. label Apr 22, 2021
@raisedadead raisedadead changed the title test: wait for next challenge render fix(e2e): wait for next challenge render Apr 22, 2021
@raisedadead raisedadead enabled auto-merge (squash) April 22, 2021 16:23
@raisedadead raisedadead disabled auto-merge April 22, 2021 16:23
@raisedadead raisedadead merged commit 71faba1 into freeCodeCamp:main Apr 22, 2021
@ojeytonwilliams ojeytonwilliams deleted the fix/flaky-cypress-test branch April 22, 2021 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: tools/scripts Scripts for supporting dev work, generating config and build artifacts, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants