Skip to content

Commit 1f6d1db

Browse files
authored
Avoid nx jest executor for running unit tests (#33220)
* Avoid nx jest executor for running unit tests The jest executor mangles the "summary of failing tests" from jest. * Remove unneded dep on nx/jest
1 parent 12a3abc commit 1f6d1db

File tree

3 files changed

+3
-62
lines changed

3 files changed

+3
-62
lines changed

apps/web/project.json

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -47,11 +47,9 @@
4747
"dependsOn": ["^build", "^build:playwright"]
4848
},
4949
"test:unit": {
50-
"executor": "@nx/jest:jest",
51-
"options": {
52-
"jestConfig": "{projectRoot}/jest.config.ts",
53-
"cwd": "apps/web"
54-
},
50+
// We avoid the jest executor because it doesn't seem to give any benefit, and it mangles the summary of failing tests.
51+
"command": "jest",
52+
"options": { "cwd": "apps/web" },
5553
"dependsOn": ["^build"]
5654
},
5755
"test:playwright": {

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@
2828
"@action-validator/cli": "^0.6.0",
2929
"@action-validator/core": "^0.6.0",
3030
"@nx-tools/nx-container": "^7.2.1",
31-
"@nx/jest": "^22.5.0",
3231
"@playwright/test": "catalog:",
3332
"@types/node": "22",
3433
"cronstrue": "^3.0.0",

pnpm-lock.yaml

Lines changed: 0 additions & 56 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)