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

feat(nx): cypress supportFile handling #1920

Merged
merged 1 commit into from
Nov 15, 2019

Conversation

bcabanes
Copy link
Member

@bcabanes bcabanes commented Oct 4, 2019

This update the Cypress schematic to handle by default the supportFile option in cypress.json with an example for custom commands.

Removing the fork-ts-checker-webpack-plugin preventing Cypress to successfully load spec files. Furthermore, fork-ts-checker-webpack-plugin is supporting TSLint that has been deprecated.

Fixes #1609
Fixes #1976
Fixes #2063

@bcabanes bcabanes force-pushed the feat/cypress/enable-supportfile branch 4 times, most recently from 03a7423 to b9364fb Compare October 6, 2019 13:58
@intellix
Copy link

When supportFile is introduced, it could introduce problems: #1976

@bcabanes bcabanes force-pushed the feat/cypress/enable-supportfile branch 7 times, most recently from 3ed736e to 3e3e5e3 Compare October 22, 2019 15:40
@bcabanes bcabanes marked this pull request as ready for review October 22, 2019 16:09
@bcabanes bcabanes mentioned this pull request Nov 1, 2019
4 tasks
@vsavkin
Copy link
Member

vsavkin commented Nov 8, 2019

@bcabanes is the performance issue addressed?

@bcabanes bcabanes force-pushed the feat/cypress/enable-supportfile branch from 3e3e5e3 to 68dee71 Compare November 15, 2019 14:18
@bcabanes
Copy link
Member Author

@vsavkin Yes, the performance problem comes from the fact fork-ts-checker-webpack-plugin blocks Cypress from doing E2E test, waiting in some cases more than 4 minutes before giving up. By removing the Ts fork, we can now safely use the supportFile feature from Cypress.

The PR has been rebased.

This update the Cypress schematic to handle by default the `supportFile` option
in `cypress.json` with an example for _custom commands_.

Related to nrwl#1609
@bcabanes bcabanes force-pushed the feat/cypress/enable-supportfile branch from 68dee71 to 735fc33 Compare November 15, 2019 15:09
@github-actions
Copy link

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants