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

test: Disable resource caching when running e2e tests (no-changelog) #5336

Closed
wants to merge 2 commits into from

Conversation

OlegIvaniv
Copy link
Contributor

@OlegIvaniv OlegIvaniv commented Feb 2, 2023

The node-creator spec is failing because the intercept can't hook types/nodes.json because it's being cached by the browser. This PR sets cacheDisabled to true via Chromium remote:debugger:protocol effectively preventing the browser from caching assets.
I also notice that the timeout for the waitForLoad command is passed in the wrong way so I've fixed that, this should generally help with tests flakiness as it often takes more than 4s(default CY timeout) to load page in GH runner.
Github issue / Community forum post (link here to close automatically):

@OlegIvaniv OlegIvaniv self-assigned this Feb 2, 2023
@OlegIvaniv OlegIvaniv changed the title test(editor): Disable resource caching when running e2e tests (no-changelog) test: Disable resource caching when running e2e tests (no-changelog) Feb 2, 2023
netroy
netroy previously approved these changes Feb 2, 2023
@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label Feb 2, 2023
netroy
netroy previously approved these changes Feb 2, 2023
@netroy netroy dismissed their stale review February 2, 2023 16:32

e2e tests failing

@OlegIvaniv OlegIvaniv closed this Feb 3, 2023
@OlegIvaniv
Copy link
Contributor Author

Closing this one since we've implemented better version of this fix in #5347

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

Successfully merging this pull request may close these issues.

2 participants