Skip to content

Add geckodriver to factory process #1351

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

Open
MikeMcC399 opened this issue May 16, 2025 · 2 comments · May be fixed by #1353
Open

Add geckodriver to factory process #1351

MikeMcC399 opened this issue May 16, 2025 · 2 comments · May be fixed by #1353

Comments

@MikeMcC399
Copy link
Collaborator

What would you like

Add the ability to specify a version of the driver package mozilla/geckodriver from mozilla/geckodriver in the Cypress Docker factory build process.

Add install scripts for mozilla/geckodriver to the cypress/factory image so that the installation is compatible with use by the npm wrapper package geckodriver bundled in the Cypress binary (see https://github.com/cypress-io/cypress/blob/develop/packages/server/package.json) since [email protected].

Why is this needed

  1. In support of Support testing with Firefox in air-gapped environment (Docker) #1348 , otherwise when attempting to test Firefox in an air-gapped environment, the test fails with:

    FetchError: request to https://raw.githubusercontent.com/mozilla/geckodriver/release/Cargo.toml failed, reason: getaddrinfo ENOTFOUND raw.githubusercontent.com

  2. To ensure that Firefox testing uses a deterministic set of versions. Without the version specification of the mozilla/geckodriver, Firefox testing dynamically downloads the latest driver version, which may change over time and could therefore cause different test results.

Non-goal

This request would change only the cypress/factory image.

Changes to cypress/browsers and cypress/included to include geckodriver would be a separate request, with this one as prerequisite.

@MikeMcC399

This comment has been minimized.

@MikeMcC399
Copy link
Collaborator Author

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

Successfully merging a pull request may close this issue.

1 participant