Skip to content

Commit

Permalink
fix: set maxWorkers to 1 for jest integration tests to see if it help…
Browse files Browse the repository at this point in the history
…s flakiness (#1013)

We are seeing a lot of timeouts in CI test runs, and the only guess
that I have is that there is a lot of CPU contention on the tiny runner
machines due to running too many tests concurrently. This commit sets
the max number of workers to 1 to see if that cuts down on timeouts.
  • Loading branch information
cprice404 authored Nov 7, 2023
1 parent 88bbd04 commit 9ba2fac
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
8 changes: 4 additions & 4 deletions packages/client-sdk-nodejs/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,11 @@
},
"scripts": {
"prebuild": "eslint . --ext .ts",
"test": "jest --testPathIgnorePatterns auth-client.test.ts --reporters=jest-silent-reporter",
"integration-test-auth": "jest auth-client.test.ts --reporters=jest-silent-reporter",
"integration-test-vector": "jest vector-control-plane.test.ts vector-data-plane.test.ts --reporters=jest-silent-reporter",
"test": "jest --testPathIgnorePatterns auth-client.test.ts --reporters=jest-silent-reporter --maxWorkers 1",
"integration-test-auth": "jest auth-client.test.ts --reporters=jest-silent-reporter --maxWorkers 1",
"integration-test-vector": "jest vector-control-plane.test.ts vector-data-plane.test.ts --reporters=jest-silent-reporter --maxWorkers 1",
"unit-test": "jest unit --reporters=jest-silent-reporter",
"integration-test": "jest integration --reporters=jest-silent-reporter --testPathIgnorePatterns \"auth-client.test.ts|vector-control-plane.test.ts|vector-data-plane.test.ts\"",
"integration-test": "jest integration --reporters=jest-silent-reporter --testPathIgnorePatterns \"auth-client.test.ts|vector-control-plane.test.ts|vector-data-plane.test.ts\" --maxWorkers 1",
"build-deps": "cd ../core && npm run build && cd - && cd ../common-integration-tests && npm run build && cd -",
"build-and-run-tests": "npm run build-deps && jest --testPathIgnorePatterns auth-client.test.ts --maxWorkers 1",
"lint": "eslint . --ext .ts",
Expand Down
8 changes: 4 additions & 4 deletions packages/client-sdk-web/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,11 @@
},
"scripts": {
"prebuild": "eslint . --ext .ts",
"test": "jest --testPathIgnorePatterns auth-client.test.ts --reporters=jest-silent-reporter",
"test": "jest --testPathIgnorePatterns auth-client.test.ts --reporters=jest-silent-reporter --maxWorkers 1",
"unit-test": "jest unit --reporters=jest-silent-reporter",
"integration-test-auth": "jest auth-client.test.ts --reporters=jest-silent-reporter",
"integration-test-vector": "jest vector-control-plane.test.ts vector-data-plane.test.ts --reporters=jest-silent-reporter",
"integration-test": "jest integration --reporters=jest-silent-reporter --testPathIgnorePatterns \"auth-client.test.ts|vector-control-plane.test.ts|vector-data-plane.test.ts\"",
"integration-test-auth": "jest auth-client.test.ts --reporters=jest-silent-reporter --maxWorkers 1",
"integration-test-vector": "jest vector-control-plane.test.ts vector-data-plane.test.ts --reporters=jest-silent-reporter --maxWorkers 1",
"integration-test": "jest integration --reporters=jest-silent-reporter --testPathIgnorePatterns \"auth-client.test.ts|vector-control-plane.test.ts|vector-data-plane.test.ts\" --maxWorkers 1",
"build-deps": "cd ../core && npm run build && cd - && cd ../common-integration-tests && npm run build && cd -",
"build-and-run-tests": "npm run build-deps && jest --testPathIgnorePatterns auth-client.test.ts --maxWorkers 1",
"lint": "eslint . --ext .ts",
Expand Down

0 comments on commit 9ba2fac

Please sign in to comment.