Skip to content

Update cypress to configure default browser#1283

Merged
zetter-rpf merged 4 commits into
mainfrom
update-cypress
Jan 12, 2026
Merged

Update cypress to configure default browser#1283
zetter-rpf merged 4 commits into
mainfrom
update-cypress

Conversation

@zetter-rpf

@zetter-rpf zetter-rpf commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

Related to https://github.com/RaspberryPiFoundation/digital-editor-issues/issues/878

It's currently not easy to run the cypress specs locally because they default to running in Electron (where they don't run correctly).

Update Cypress so we can configure a default browser.

Note that there's still a couple more things we will need to do to make it possible to run the cypress specs in the docker container to install Cypress and Chrome which is something I'm looking at.

 I'd like to set the default browser which was introduced in a 13.x version. Update it to the latest version that we can use. We can't use version 15 as it requires node 20+
Cypress chose to use the 'electron' browser by default on my machine. Some of the specs (the ones that stop code execution) don't work within electron. I think it's better to use a browser that our users are more likely to be using by default.

You can still choose another browser with the -b/--browser flag when running specs.
@zetter-rpf
zetter-rpf temporarily deployed to previews/1283/merge January 9, 2026 14:45 — with GitHub Actions Inactive
@zetter-rpf
zetter-rpf temporarily deployed to previews/1283/merge January 9, 2026 14:48 — with GitHub Actions Inactive
@zetter-rpf zetter-rpf changed the title Update cypress Update cypress & configure default browser Jan 12, 2026
@zetter-rpf zetter-rpf changed the title Update cypress & configure default browser Update cypress to configure default browser Jan 12, 2026

@danhalson danhalson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great

@zetter-rpf
zetter-rpf merged commit b99cd5c into main Jan 12, 2026
8 checks passed
@zetter-rpf
zetter-rpf deleted the update-cypress branch January 12, 2026 11:32
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.

2 participants