Skip to content

Conversation

luka-nextcloud
Copy link
Contributor

@luka-nextcloud luka-nextcloud commented Sep 29, 2025

  • Resolves: cypress memory issue in the workflow
  • Target version: main

Summary

  • added experimentalMemoryManagement: true into cypress.config.js
  • It looks like the memory issue is within cypress itself. So updated the workflow file to make it run parallel on multiple runners (same as the Text app)
image

Copy link
Contributor

@enjeck enjeck left a comment

Choose a reason for hiding this comment

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

seems like the test still fails 🤔

@luka-nextcloud luka-nextcloud changed the title chore: enable experimentalMemoryManagement for cypress test ci(cypress): Use cypress-split for running tests in parallel Oct 1, 2025
Copy link
Member

@grnd-alt grnd-alt left a comment

Choose a reason for hiding this comment

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

greatly needed to improve CI 🔥
we will have to adapt branch protection rules for the changed check name

@juliusknorr
Copy link
Member

I think we run into the same issue as in text here where we have not found any feasible solution other then sticking to a rather old cypress version

cypress-io/cypress#27415

Since the number of cypress tests is rather small we could think about moving to playwright instead. This happened on some other repos and so far my impression is that it is more stable to use.

@luka-nextcloud
Copy link
Contributor Author

Ok, I will look into playwright integration soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants