chore(cypress): extract cypress into top-level package and upgrade to v15#2257
Open
wreality wants to merge 11 commits into
Open
chore(cypress): extract cypress into top-level package and upgrade to v15#2257wreality wants to merge 11 commits into
wreality wants to merge 11 commits into
Conversation
✅ Deploy Preview for pilcrow-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Move cypress tests from client/test/cypress to a top-level cypress/ directory with its own package.json so cypress can be installed and run independently of the Quasar client build. - New cypress/ package with its own eslint config and cypress.config.cjs - Remove cypress deps and eslint overrides from client/ - Drop test/cypress from client/tsconfig.json exclude - Update root package.json, .lando.extras.yml, e2e workflow and docs to reference the new cypress/ path
- Bump cypress to v15 in cypress/package.json and lockfile - Fix browser launch config in cypress.config.cjs for v15 - Resolve SSL and node engine errors in the cypress container - Update do-test-e2e workflow for the new cypress version
Add a lint-cypress job that runs yarn lint in the new cypress/ package on PRs touching cypress files, and wire a cypress path filter into the changes job.
wreality
force-pushed
the
chore/cypress-reorg
branch
from
7467695 to
4ac72b7
Compare
Collaborative Community Review Upstream
 |
||||||||||||||||||||||||||||
| Project |
Collaborative Community Review Upstream
|
| Branch Review |
chore/cypress-reorg
|
| Run status |
|
| Run duration | 02m 58s |
| Commit |
|
| Committer | Brian Adams |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
 |
0
|
 |
0
|
 |
0
|
 |
0
|
 |
115
|
| View all changes introduced in this branch ↗︎ | |
…handling The build-image action's version output is empty because pilcrow-toolkit's parse-build-output never sets it. Work around this by extracting the version from the tags output instead. Also fix the resetDb cypress task which crashed the config file on GraphQL errors due to a missing return before reject().
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
The fpm image doesn't include the SSL cert verification fix that was added to the unit-test Dockerfile stage. Add it at runtime in the start-stack action before running migrate:fresh.
Sanctum needs the port in stateful domains for non-standard ports. Without this, session cookies aren't sent on localhost:8888 requests, causing all authenticated tests to fail.
gmeben
reviewed
Apr 22, 2026
Contributor
gmeben
left a comment
gmeben
left a comment
There was a problem hiding this comment.
I can't get cypress to install or run. cd cypress && lando cypress run results in the error:
error Command "cypress" not found.
cypress/base:18.16.1 is Node 18, but Cypress 15 requires Node 20+. The --ignore-engines flag silenced the install-time check but left a broken install (missing node_modules/.bin/cypress), which surfaced as `error Command "cypress" not found.` from `lando cypress run`. Bump the service to node:22 / cypress/base:22.20.0 to match the CI workflow's Node 22, and drop the no-longer-needed --ignore-engines. Update the matching snippet in docs/developers/testing.md.
Contributor
Author
|
Bumped the service to Node 22 / lando extras disable cypress
lando extras enable cypress
lando rebuild -y -s cypress
lando cypress run |
# Conflicts: # client/package.json # client/yarn.lock
Carries forward the axe-core bump from master (originally in client/package.json) into cypress/, where the dep now lives after the cypress extraction.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Reorganizes cypress tests into their own top-level
cypress/package so they can be installed and run independently of the Quasar client build, then upgrades cypress to v15.client/test/cypress→cypress/with its ownpackage.json,yarn.lock, eslint config, andcypress.config.cjs. Removes cypress deps and eslint overrides fromclient/. Updates rootpackage.json,.lando.extras.yml, the e2e workflow, and docs to reference the new path.lint-cypressjob and acypresspath filter so cypress-only changes trigger the right jobs.Test plan
lando rebuildbrings up the cypress service cleanlycd cypress && lando cypress runpassescd cypress && yarn lintpasseslint-cypressjob runs on cypress-touching PRsdo-test-e2e) runs cypress fromcypress/