-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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(testing): add projects into jest config #3766
Conversation
20ac78c
to
a9ce7e8
Compare
Nx Cloud ReportCI ran the following commands. Click to see the status, the terminal output, and the build insights.
Sent with 💌 from NxCloud. |
@@ -1,3 +1,4 @@ | |||
import { tags } from '@angular-devkit/core'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks unused.
@@ -67,6 +69,7 @@ function run( | |||
useStderr: options.useStderr, | |||
watch: options.watch, | |||
watchAll: options.watchAll, | |||
projects: [options.jestConfig], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this was necessary.
* feat(testing): add projects into jest config * chore(testing): update unit tests and fix presets with spreading a default * chore(testing): fix node e2e * chore(testing): review comment changes
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. |
Current Behavior
Jest configs are independant, and cannot be run from the root of the workspace with just
jest
. This causes some issues with VsCode extensions.Expected Behavior
The root jest.config.js file now contains references to all jest projects in the workspace. A new jest.preset.js file is created as well where each project config has this as a preset.
Related Issue(s)
Closes #747
Closes #1506
Closes #2377
Closes #2344
Closes #2635
Closes #3507