Skip to content

Commit

Permalink
chore: split jest e2e tests and cleanup configuration
Browse files Browse the repository at this point in the history
  • Loading branch information
leosvelperez committed Apr 30, 2024
1 parent b1f088b commit 51abb92
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 50 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci_checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
- run: |
npx nx-cloud start-ci-run \
--distribute-on="3 linux-medium-js" \
--stop-agents-after="e2e"
--stop-agents-after="e2e-ci"
- uses: browser-actions/setup-chrome@v1
- run: chrome --version
- name: Use Node.js ${{ env.node_version }}
Expand All @@ -41,7 +41,7 @@ jobs:
env:
NX_CLOUD_DISTRIBUTED_EXECUTION: false
- run: yarn nx affected --t=lint,test,build --configuration=ci --exclude=nx-console --verbose
- run: yarn nx affected --t=e2e,e2e-ci --configuration=ci --exclude=nx-console --parallel=1 --verbose
- run: yarn nx affected --t=e2e-ci --exclude=nx-console --parallel=1 --verbose
main-windows:
name: Main Windows
runs-on: windows-latest
Expand Down Expand Up @@ -81,5 +81,5 @@ jobs:
cache: gradle
# there's no need to check formatting & linting again on windows
- run: yarn nx affected --t=test,build --configuration=ci --exclude=nx-console --verbose
- run: yarn nx affected --t=e2e,e2e-ci --configuration=ci --exclude=nx-console --parallel=1 --verbose
- run: yarn nx affected --t=e2e-local --exclude=nx-console --parallel=1 --verbose
timeout-minutes: 45
3 changes: 0 additions & 3 deletions apps/intellij/project.json
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,6 @@
"command": "node ./apps/intellij/run-gradle.js :apps:intellij:check"
}
},
"lint": {
"executor": "nx:noop"
},
"x-lint": {
"//": "Run inspections take a long time on the CI. Disable for now",
"executor": "nx:run-commands",
Expand Down
15 changes: 1 addition & 14 deletions apps/nxls-e2e/project.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,20 +3,7 @@
"$schema": "../../node_modules/nx/schemas/project-schema.json",
"sourceRoot": "apps/nxls-e2e/src",
"projectType": "application",
"targets": {
"test": {
"command": "jest --detectOpenHandles --runInBand --verbose",
"options": {
"cwd": "apps/nxls-e2e"
},
"dependsOn": [
{
"projects": "nxls",
"target": "build"
}
]
}
},
"targets": {},
"tags": ["type:lsp"],
"implicitDependencies": ["nxls"]
}
6 changes: 1 addition & 5 deletions libs/generate-ui/styles/project.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,7 @@
"$schema": "../../../node_modules/nx/schemas/project-schema.json",
"projectType": "library",
"sourceRoot": "libs/generate-ui/styles/src",
"targets": {
"lint": {
"executor": "nx:noop"
}
},
"targets": {},
"generators": {},
"tags": ["type:generate-ui"]
}
34 changes: 9 additions & 25 deletions nx.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,14 +36,6 @@
"inputs": ["default", "^production", "{workspaceRoot}/jest.preset.js"],
"cache": true
},
"lint": {
"inputs": ["default", "{workspaceRoot}/.eslintrc.json"],
"cache": true
},
"e2e": {
"inputs": ["default", "^production"],
"cache": true
},
"build-storybook": {
"inputs": [
"production",
Expand All @@ -60,23 +52,6 @@
},
"extract-dependencies": {
"cache": true
},
"@nx/jest:jest": {
"inputs": ["default", "^production", "{workspaceRoot}/jest.preset.js"],
"cache": true,
"options": {
"passWithNoTests": true
},
"configurations": {
"ci": {
"ci": true,
"codeCoverage": true
}
}
},
"@nx/eslint:lint": {
"inputs": ["default", "{workspaceRoot}/.eslintrc.json"],
"cache": true
}
},
"namedInputs": {
Expand Down Expand Up @@ -109,10 +84,19 @@
},
{
"plugin": "@nx/jest/plugin",
"exclude": ["apps/nxls-e2e/**/*"],
"options": {
"targetName": "test"
}
},
{
"plugin": "@nx/jest/plugin",
"include": ["apps/nxls-e2e/**/*"],
"options": {
"targetName": "e2e-local",
"ciTargetName": "e2e-ci"
}
},
{
"plugin": "@nx/cypress/plugin",
"options": {
Expand Down

0 comments on commit 51abb92

Please sign in to comment.