From dccef4ea61bf4d7fb58a7f80e63941c6afa51d79 Mon Sep 17 00:00:00 2001 From: Chirag Madlani <12962843+chirag-madlani@users.noreply.github.com> Date: Mon, 9 Jan 2023 22:20:30 +0530 Subject: [PATCH 1/3] update max worker to 8 for increasing test speed --- openmetadata-ui/src/main/resources/ui/package.json | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/openmetadata-ui/src/main/resources/ui/package.json b/openmetadata-ui/src/main/resources/ui/package.json index 48e97888b76c..fc4a93c4102a 100644 --- a/openmetadata-ui/src/main/resources/ui/package.json +++ b/openmetadata-ui/src/main/resources/ui/package.json @@ -104,11 +104,11 @@ "preinstall": "cd ../../../../.. && yarn install --frozen-lockfile", "postinstall": "yarn run build-check", "pre-commit": "lint-staged --concurrent false", - "test": "jest --passWithNoTests --maxWorkers=3 ", + "test": "jest --passWithNoTests --maxWorkers=8 ", "prepare": "cd ../../../../.. && husky install openmetadata-ui/src/main/resources/ui/.husky", - "test:watch": "jest --passWithNoTests --watch --maxWorkers=3", - "test:coverage": "jest --passWithNoTests --coverage --maxWorkers=3", - "test:cov-summary": "jest --passWithNoTests --coverage --coverageReporters json-summary --maxWorkers=3", + "test:watch": "jest --passWithNoTests --watch --maxWorkers=8", + "test:coverage": "jest --passWithNoTests --coverage --maxWorkers=8", + "test:cov-summary": "jest --passWithNoTests --coverage --coverageReporters json-summary --maxWorkers=8", "lint": "eslint \"./src/**/*.{js,jsx,ts,tsx,json}\"", "lint:fix": "eslint './src/**/*.{js,jsx,ts,tsx,json}' --fix", "pretty": "prettier . --config './.prettierrc.yaml' --ignore-path './.prettierignore' --write", From 7ea11fb12d53e5ebadf9783e1812c92a4f5d0afb Mon Sep 17 00:00:00 2001 From: Chirag Madlani <12962843+chirag-madlani@users.noreply.github.com> Date: Tue, 10 Jan 2023 10:19:54 +0530 Subject: [PATCH 2/3] dummy commit to verify unit tests memory dump issue It's a dummy commit that doesn't add any signification to the code, We will run unit tests workflow couple of time to verify whether it has improved in terms of time or not --- .../src/main/resources/ui/src/constants/Teams.constants.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/openmetadata-ui/src/main/resources/ui/src/constants/Teams.constants.ts b/openmetadata-ui/src/main/resources/ui/src/constants/Teams.constants.ts index 3a7d93af1ada..cc39b2b96e39 100644 --- a/openmetadata-ui/src/main/resources/ui/src/constants/Teams.constants.ts +++ b/openmetadata-ui/src/main/resources/ui/src/constants/Teams.constants.ts @@ -15,6 +15,7 @@ import DraggableBodyRow from '@components/TeamDetails/DraggableBodyRow'; export const DRAGGABLE_BODY_ROW = 'DraggableBodyRow'; +// Represents table constants export const TABLE_CONSTANTS = { body: { row: DraggableBodyRow, From c965538210018e056f9317f1c9720c60ef7f8312 Mon Sep 17 00:00:00 2001 From: Chirag Madlani <12962843+chirag-madlani@users.noreply.github.com> Date: Wed, 18 Jan 2023 16:49:25 +0530 Subject: [PATCH 3/3] update maxWorkers to 5 --- openmetadata-ui/src/main/resources/ui/package.json | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/openmetadata-ui/src/main/resources/ui/package.json b/openmetadata-ui/src/main/resources/ui/package.json index 482d51febd70..cea743d82c43 100644 --- a/openmetadata-ui/src/main/resources/ui/package.json +++ b/openmetadata-ui/src/main/resources/ui/package.json @@ -104,11 +104,11 @@ "preinstall": "cd ../../../../.. && yarn install --frozen-lockfile", "postinstall": "yarn run build-check", "pre-commit": "lint-staged --concurrent false", - "test": "jest --passWithNoTests --maxWorkers=3 ", + "test": "jest --passWithNoTests --maxWorkers=5", "prepare": "cd ../../../../.. && husky install openmetadata-ui/src/main/resources/ui/.husky", - "test:watch": "jest --passWithNoTests --watch --maxWorkers=3", - "test:coverage": "jest --passWithNoTests --coverage --maxWorkers=3", - "test:cov-summary": "jest --passWithNoTests --coverage --coverageReporters json-summary --maxWorkers=3", + "test:watch": "jest --passWithNoTests --watch --maxWorkers=5", + "test:coverage": "jest --passWithNoTests --coverage --maxWorkers=5", + "test:cov-summary": "jest --passWithNoTests --coverage --coverageReporters json-summary --maxWorkers=5", "lint": "eslint \"./**/*.{js,jsx,ts,tsx,json}\"", "lint:fix": "eslint './**/*.{js,jsx,ts,tsx,json}' --fix", "pretty": "prettier . --config './.prettierrc.yaml' --ignore-path './.prettierignore' --write",