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

Spec glob seems to have broken in 9.3.1 #19783

Closed
nickbrook72 opened this issue Jan 20, 2022 · 4 comments · Fixed by #19789
Closed

Spec glob seems to have broken in 9.3.1 #19783

nickbrook72 opened this issue Jan 20, 2022 · 4 comments · Fixed by #19789
Assignees
Labels
type: bug type: regression A bug that didn't appear until a specific Cy version release v9.3.1 🐛 Issue present since 9.3.1

Comments

@nickbrook72
Copy link

Current behavior

We use Bitbucket Pipelines for tests, and in order to run in parallel we split up the specs using simple globs.
e.g:

cypress run -s 'cypress/integration/[a-c]*/**

This worked fine up till 9.3.0, but broke in 9.3.1 with the following error:

`Cypress encountered an error while parsing the argument spec

You passed: cypress/integration/[a-c]*/**

The error was: spec must be a string or comma-separated list`

Desired behavior

It should pull in all the specs referenced by the glob i.e all specs beginning with a, b or c

Test code to reproduce

cypress run -s 'cypress/integration/[a-c]*/**

Cypress Version

9.3.1

Other

You fixed a glob-related bug in 9.3.1: regression due to that?

@nickbrook72
Copy link
Author

Note that it works if I append a comma
i.e:
cypress run -s 'cypress/integration/[a-c]*/**,'

works

@tbiethman
Copy link
Contributor

@nickbrook72 Thank you for reporting. I am able to reproduce with the provided command and will look into the 9.3 changes around this: 273b703

@tbiethman tbiethman self-assigned this Jan 20, 2022
@tbiethman tbiethman added type: regression A bug that didn't appear until a specific Cy version release type: bug and removed stage: work in progress labels Jan 20, 2022
@cypress-bot cypress-bot bot added the stage: needs review The PR code is done & tested, needs review label Jan 20, 2022
@jennifer-shehane jennifer-shehane added the v9.3.1 🐛 Issue present since 9.3.1 label Jan 20, 2022
@cypress-bot cypress-bot bot added stage: pending release and removed stage: needs review The PR code is done & tested, needs review labels Jan 21, 2022
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jan 21, 2022

The code for this is done in cypress-io/cypress#19789, but has yet to be released.
We'll update this issue and reference the changelog when it's released.

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jan 31, 2022

Released in 9.4.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v9.4.0, please open a new issue.

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Jan 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug type: regression A bug that didn't appear until a specific Cy version release v9.3.1 🐛 Issue present since 9.3.1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants