Skip to content

Conversation

@stof
Copy link
Contributor

@stof stof commented Jan 6, 2026

Older selenium versions support a non-standard feature to upload a file to the runner, which is used by the Mink driver when available. Newer Selenium versions implementing the W3C protocol don't support this anymore. Attaching a file to a file input requires using a file path that already exists on the runner. this configures the Mink testsuite to properly share the fixtures with the docker image of the runner.

Older selenium versions support a non-standard feature to upload a file to the runner, which is used by the Mink driver when available.
Newer Selenium versions implementing the W3C protocol don't support this anymore. Attaching a file to a file input requires using a file path that already exists on the runner. this configures the Mink testsuite to properly share the fixtures with the docker image of the runner.
@stof
Copy link
Contributor Author

stof commented Jan 6, 2026

Please wait before merging this PR in case the CI is not green, to avoid having to do even more PR to reach such state (previous PRs attempting to reach a working state were merged even if they did not succeed yet)

@stof
Copy link
Contributor Author

stof commented Jan 6, 2026

OK, apparently, this fixes the setup entirely. This PR is ready.

@robocoder robocoder merged commit b440f54 into instaclick:master Jan 6, 2026
10 checks passed
@stof stof deleted the patch-1 branch January 7, 2026 08:51
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.

3 participants