From 7ba0eb9cdfe1a4721a635f22ee3123c360f7f2ad Mon Sep 17 00:00:00 2001 From: Jeremy Meng Date: Wed, 8 May 2024 12:46:21 -0700 Subject: [PATCH] [test][CI] move test building into test:browser scripts for non-core packages in js - core pipelines. This PR is a follow-up of PR https://github.com/Azure/azure-sdk-for-js/pull/29411 for non-core packages. --- sdk/identity/identity/package.json | 6 +++--- sdk/servicebus/service-bus/package.json | 8 +++----- sdk/synapse/synapse-access-control/package.json | 8 ++++---- 3 files changed, 10 insertions(+), 12 deletions(-) diff --git a/sdk/identity/identity/package.json b/sdk/identity/identity/package.json index 3c8bf2e00649..c48105607dd8 100644 --- a/sdk/identity/identity/package.json +++ b/sdk/identity/identity/package.json @@ -48,7 +48,7 @@ "scripts": { "audit": "node ../../../common/scripts/rush-audit.js && rimraf node_modules package-lock.json && npm i --package-lock-only 2>&1 && npm audit", "build:samples": "echo Obsolete.", - "build:test": "tsc -p . && dev-tool run bundle", + "build:test": "echo skipped. actual commands inlined in browser test scripts", "build": "npm run clean && npm run extract-api && tsc -p . && dev-tool run bundle", "clean": "rimraf --glob dist dist-* types *.tgz *.log", "execute:samples": "dev-tool samples run samples-dev", @@ -62,8 +62,8 @@ "lint:fix": "eslint package.json api-extractor.json src test --ext .ts --fix --fix-type [problem,suggestion]", "lint": "eslint package.json api-extractor.json src test --ext .ts", "pack": "npm pack 2>&1", - "test:browser": "npm run clean && npm run build:test && npm run unit-test:browser && npm run integration-test:browser", - "test:node": "npm run clean && npm run build:test && npm run unit-test:node && npm run integration-test:node", + "test:browser": "npm run clean && tsc -p . && dev-tool run bundle && npm run unit-test:browser && npm run integration-test:browser", + "test:node": "npm run clean && npm run unit-test:node && npm run integration-test:node", "test": "npm run clean && npm run build:test && npm run unit-test && npm run integration-test", "unit-test:browser": "dev-tool run test:browser", "unit-test:node": "dev-tool run test:node-ts-input -- --timeout 300000 --exclude 'test/**/browser/**/*.spec.ts' 'test/**/**/*.spec.ts'", diff --git a/sdk/servicebus/service-bus/package.json b/sdk/servicebus/service-bus/package.json index dcc8e657e3a6..ed593eeac756 100644 --- a/sdk/servicebus/service-bus/package.json +++ b/sdk/servicebus/service-bus/package.json @@ -45,9 +45,7 @@ "bundle": "tsc -p . && dev-tool run bundle --polyfill-node false --inject-node-polyfills true --ignore-missing-node-builtins true", "build:browser": "npm run bundle", "build:node": "npm run bundle", - "build:test:browser": "npm run bundle", - "build:test:node": "npm run bundle", - "build:test": "npm run bundle", + "build:test": "echo skipped. actual commands inlined in browser test scripts", "build": "npm run clean && npm run bundle && npm run extract-api", "check-format": "dev-tool run vendored prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"samples-dev/**/*.ts\" \"*.{js,json}\"", "clean": "rimraf --glob dist dist-* types *.tgz *.log coverage coverage-browser .nyc_output", @@ -62,8 +60,8 @@ "lint": "eslint package.json api-extractor.json README.md src test --ext .ts,.javascript,.js", "pack": "npm pack 2>&1", "perf-test:node": "tsc -p . --module \"commonjs\" && node dist-esm/test/perf/track-2/index.spec.js", - "test:browser": "npm run clean && npm run build:test:browser && npm run integration-test:browser", - "test:node": "npm run clean && npm run build:test:node && npm run integration-test:node", + "test:browser": "npm run clean && npm run bundle && npm run integration-test:browser", + "test:node": "npm run clean && npm run integration-test:node", "test": "npm run test:node && npm run test:browser", "unit-test:browser": "echo skipped", "unit-test:node": "dev-tool run test:node-ts-input --no-test-proxy -- --timeout 1200000 --full-trace \"test/internal/unit/{,!(browser)/**/}*.spec.ts\"", diff --git a/sdk/synapse/synapse-access-control/package.json b/sdk/synapse/synapse-access-control/package.json index 2180e23ddcb9..3767b91d5837 100644 --- a/sdk/synapse/synapse-access-control/package.json +++ b/sdk/synapse/synapse-access-control/package.json @@ -100,16 +100,16 @@ "build": "tsc && dev-tool run bundle && npm run minify && npm run extract-api", "minify": "uglifyjs -c -m --comments --source-map \"content='./dist/index.js.map'\" -o ./dist/index.min.js ./dist/index.js", "pack": "npm pack 2>&1", - "build:test": "tsc -p . && dev-tool run bundle", + "build:test": "echo skipped. actual commands inlined in browser test scripts", "integration-test:browser": "echo skip", "integration-test:node": "echo skip", "integration-test": "echo skip", "lint": "echo skip", "format": "echo skip format as this package has only auto generated code", "check-format": "echo skip format check as this package has only auto generated code", - "test:browser": "npm run clean && npm run build:test && npm run unit-test:browser", - "test:node": "npm run clean && npm run build:test && npm run unit-test:node", - "test": "npm run clean && npm run build:test && npm run unit-test", + "test:browser": "npm run clean && tsc -p . && dev-tool run bundle && npm run unit-test:browser", + "test:node": "npm run unit-test:node", + "test": "npm run unit-test", "unit-test:browser": "dev-tool run test:browser", "unit-test:node": "dev-tool run test:node-ts-input -- --timeout 1200000 --exclude 'test/**/browser/*.spec.ts' 'test/**/*.spec.ts'", "unit-test": "npm run unit-test:node && npm run unit-test:browser",