Skip to content

[Feature Request] Option --ignoreTestFiles cli argument not exposed by the cypress builder #3439

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

Closed
jacobworrel opened this issue Jul 30, 2020 · 3 comments · Fixed by #3500
Closed
Labels
community This is a good first issue for contributing outdated scope: testing tools Issues related to Cypress / Jest / Playwright / Vitest support in Nx type: feature

Comments

@jacobworrel
Copy link

jacobworrel commented Jul 30, 2020

Description

I would like to be able to ignore test files via the cypress ignoreTestFiles cli argument. Currently, when trying to run nx e2e my-app-e2e --ignoreTestFiles=path/to/file, I get the following error:

Schema validation failed with the following errors:
  Data path "" should NOT have additional properties(ignoreTestFiles).

Motivation

I want to ignore tests that run visual regression tests on applitools while running e2e tests in watch mode. My current workaround is to duplicate the cypress.json file with a ignoreTestFiles prop and pass that in via the cypress-config cli flag.

Suggested Implementation

The implementation for #1192 should work for this flag as well.

@FrozenPandaz FrozenPandaz added community This is a good first issue for contributing scope: testing tools Issues related to Cypress / Jest / Playwright / Vitest support in Nx labels Jul 31, 2020
@FrozenPandaz
Copy link
Collaborator

That sounds like an important flag for the builder to accept.

As mentioned, this is a relatively simple feature and someone from the community can send a PR! 🙏

@mehrad-rafigh
Copy link
Contributor

@FrozenPandaz @jacobworrel I will take a try at this and open a PR

vsavkin pushed a commit that referenced this issue Aug 13, 2020

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
ISSUES CLOSED: #3439
FrozenPandaz pushed a commit that referenced this issue Aug 13, 2020

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
ISSUES CLOSED: #3439
FrozenPandaz pushed a commit that referenced this issue Aug 13, 2020

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
ISSUES CLOSED: #3439
FrozenPandaz pushed a commit that referenced this issue Aug 18, 2020

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
ISSUES CLOSED: #3439
Doginal pushed a commit to Doginal/nx that referenced this issue Nov 25, 2020

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
ISSUES CLOSED: nrwl#3439
@github-actions
Copy link

This issue has been closed for more than 30 days. If this issue is still occuring, please open a new issue with more recent context.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
community This is a good first issue for contributing outdated scope: testing tools Issues related to Cypress / Jest / Playwright / Vitest support in Nx type: feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants