[go: nahoru, domu]

Skip to content
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

Fix Safari Sauce Labs test runner #2372

Open
fatso83 opened this issue May 12, 2021 · 4 comments
Open

Fix Safari Sauce Labs test runner #2372

fatso83 opened this issue May 12, 2021 · 4 comments
Labels

Comments

@fatso83
Copy link
Contributor
fatso83 commented May 12, 2021

Our integration-tests pipeline in CircleCI has been failing for two months. This happened when introducing "safari" without explicit version (was version 9). Logs show that we have been trying to do this earlier as well. Basically, our npm run test-cloud script fails for any Safari version > 9.

Example: https://app.circleci.com/pipelines/github/sinonjs/sinon/572/workflows/79f646b4-553b-4ed7-a622-b585860850ef/jobs/4557/parallel-runs/0/steps/0-102

Overview of builds: https://app.circleci.com/pipelines/github/sinonjs/sinon?branch=master

To reproduce, checkout master, set the sauce labs username and key and run the script. It will fail with a general HTTP 500. These things are generally near impossible to debug, so I have filed a support request. No too hopeful.

Addendum
When we have a config that actually can run our tests we need to re-add Safari. Was removed in ecad8a8

@fatso83
Copy link
Contributor Author
fatso83 commented May 13, 2021

Houston, we have contact! The Sauce Labs team responded in a few hours.

Screenshot_20210513_073421_com.google.android.gm.jpg

So it seems this is a mochify or min-wd thing. @mantoni is probably the only one to know about these things. The whole webdriver thing and configuration is a bit opaque to me.

@fatso83
Copy link
Contributor Author
fatso83 commented May 20, 2021

Created mantoni/min-webdriver#31. This seems quite doable.

@fatso83
Copy link
Contributor Author
fatso83 commented May 25, 2021

I fiddled around with this for almost two hours without getting anywhere ...

mroderick added a commit to sinonjs/referee-sinon that referenced this issue Feb 21, 2022
This is not currently possible due to missing W3C spec feautre in
[min-webdriver][0]. This has been reported in [`sinon#2372`][1].

For now, let's remove Safari from the list, so we at least can test in
the other runtimes.

[0]: mantoni/min-webdriver#31
[1]: sinonjs/sinon#2372
mroderick added a commit to sinonjs/referee-sinon that referenced this issue Feb 21, 2022
This is not currently possible due to missing W3C spec feautre in
[min-webdriver][0]. This has been reported in [`sinon#2372`][1].

For now, let's remove Safari from the list, so we at least can test in
the other runtimes.

[0]: mantoni/min-webdriver#31
[1]: sinonjs/sinon#2372
Copy link
stale bot commented Dec 27, 2023

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the wontfix label Dec 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant