-
Notifications
You must be signed in to change notification settings - Fork 103
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
cypress/run has no executor param #429
Comments
Did this used to work in v2? Looking at the previous implementation, seems like that aspect didn't change. In any case, I think this is a valid feature request. I will forward this ticket to the appropriate team. They will evaluate the priority of this and consider their capacity to pick it up. Please note that this does not guarantee that this issue will be resolved. The ticket will indicate status changes during evaluation, so we ask that you please refrain from asking for updates. Thanks! |
It did work with 2.2.0, we were using it. But it now fails to compile in CircleCI using v3. As mentioned, it does work with cypress/run-tests though. But it should be possible for both job types. |
@Phonesis You can just specify your own executor for your job using something like https://github.com/cypress-io/circleci-orb/blob/master/.circleci/test-deploy.yml#L22. |
@jordanpowell88 As mentioned, the problem resides with cypress/run Your example is for cypress/run-tests |
Yes, |
Personally i still think cypress/run should have an executor option because many may want to use it but with a specific browser version in mind. Like we did. |
Just a follow up to this, and why i think it is important to add to cypress/run. install-browsers seems to fail a lot when you are running tests in parallel on multiple machines. We are constantly seeing it fail at the firefox install stage. installing them all each time adds 1-2 minutes to each run as well. So simply using a defined executor is very important. I have attempted to convert our pipeline to use cypress/run-tests. The problem I have now hit is that cypress/run-tests doesn't seem to include the following param:
cypress/run does include this and we use it. What is the equivalent for cypress/run-tests? |
Did you ever solve this @Phonesis? Also using a custom executor in version 2 but failing in version 3. |
@borkanee yes, solved it by using the start-command, which appears to do same thing / let you install stuff
|
It appears that with v3 of the orb you can no longer specify a specific executor for browsers when using the cypress/run command/job.
It appears to only be available for cypress/run-tests.
Is this intended? If so, the documentation is not clear or useful. For the install-browser param it says:
How do you do this for cypress/run?
I would expect this to be valid:
The text was updated successfully, but these errors were encountered: