[test][CI] move test building into test:browser scripts (#29633)

for non-core packages in js - core pipelines. This PR is a follow-up of PR #29411 for non-core packages.
Azure · May 10, 2024 · 5bdb96b · 5bdb96b
1 parent 242f65c
commit 5bdb96b
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 12 deletions.
diff --git 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
@@ -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
@@ -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",