Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
6 changes: 4 additions & 2 deletions .github/workflows/generator-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -96,8 +96,8 @@ jobs:
echo "--- Testing library $lib ---"
cd "library/.test-out-$lib"
pnpm install --no-frozen-lockfile --ignore-workspace --silent
pnpm run prepublishOnly
pnpm test
pnpm run compile
pnpm run system-test
) &
PIDS+=($!)
Expand Down Expand Up @@ -130,8 +130,8 @@ jobs:
echo "--- Testing ESM library $lib ---"
cd "library/.test-out-$lib-esm"
pnpm install --no-frozen-lockfile --ignore-workspace --silent
pnpm run prepublishOnly
pnpm test
pnpm run compile
pnpm run system-test
) &
PIDS+=($!)
Expand Down Expand Up @@ -161,6 +161,7 @@ jobs:
set -ex
cd generator/gapic-generator-typescript/test-fixtures/google-cloud-speech
pnpm install --no-frozen-lockfile --ignore-workspace --silent
pnpm run prepublishOnly
pnpm test
pnpm run system-test

Expand All @@ -176,5 +177,6 @@ jobs:
set -ex
cd generator/gapic-generator-typescript/test-fixtures/google-cloud-tasks
pnpm install --no-frozen-lockfile --ignore-workspace --silent
pnpm run prepublishOnly
pnpm test
pnpm run system-test
40 changes: 25 additions & 15 deletions ci/run_single_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -51,21 +51,31 @@ if command -v cygpath >/dev/null 2>&1; then
PNPMFILE_PATH=$(cygpath -m "${PNPMFILE_PATH}")
fi

echo "pnpm install --engine-strict --pnpmfile \"${PNPMFILE_PATH}\""
if ! pnpm install --engine-strict --pnpmfile "${PNPMFILE_PATH}"; then
echo "::error title=PNPM Install Failed::pnpm install failed in $(pwd)."
echo ""
echo "===================================================================================================="
echo "❌ PNPM Install Failed"
echo ""
echo "If this failure is caused by an outdated lockfile or changed package.json dependencies, run:"
echo " pnpm install --no-frozen-lockfile"
echo " git add pnpm-lock.yaml"
echo " git commit -m \"chore: update pnpm-lock.yaml\""
echo " git push"
echo "===================================================================================================="
echo ""
exit 1
if [ ! -d "node_modules" ]; then
echo "pnpm install --engine-strict --pnpmfile \"${PNPMFILE_PATH}\""
if ! pnpm install --engine-strict --pnpmfile "${PNPMFILE_PATH}"; then
echo "::error title=PNPM Install Failed::pnpm install failed in $(pwd)."
echo ""
echo "===================================================================================================="
echo "❌ PNPM Install Failed"
echo ""
echo "If this failure is caused by an outdated lockfile or changed package.json dependencies, run:"
echo " pnpm install --no-frozen-lockfile"
echo " git add pnpm-lock.yaml"
echo " git commit -m \"chore: update pnpm-lock.yaml\""
echo " git push"
echo "===================================================================================================="
echo ""
exit 1
fi
fi

if [ ! -d "build" ] && [ "${TEST_TYPE}" != "lint" ]; then
if grep -q '"prepublishOnly":' package.json; then
${TEST_CMD} prepublishOnly
elif grep -q '"compile":' package.json; then
${TEST_CMD} compile
fi
fi


Expand Down
2 changes: 1 addition & 1 deletion core/common/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
"scripts": {
"docs": "jsdoc -c .jsdoc.js",
"test": "c8 mocha build/test",
"prepare": "npm run compile",
"prepublishOnly": "npm run compile",
"pretest": "npm run compile",
"compile": "tsc -p .",
"fix": "gts fix",
Expand Down
2 changes: 1 addition & 1 deletion core/dev-packages/pack-n-play/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
"docs": "echo no docs needed",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile",
"prepublishOnly": "npm run compile",
"pretest": "npm run compile",
"samples-test": "echo 'no test'",
"system-test": "echo 'no test'",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha build/cjs/system-test",
"system-test:esm": "c8 mocha build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"docs": "jsdoc -c .jsdoc.js",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/system-test",
"test": "c8 mocha --config ../../.mocharc.cjs build/test"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"postpack": "minifyProtoJson build/cjs && minifyProtoJson build/esm",
"fix": "gts fix",
"lint": "gts check",
"prepare": "npm run compile-protos && npm run compile",
"prepublishOnly": "npm run compile-protos && npm run compile",
"system-test:cjs": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/cjs/system-test",
"system-test:esm": "c8 mocha --config ../../.mocharc.cjs --no-parallel build/esm/system-test",
"system-test": "npm run system-test:esm && npm run system-test:cjs",
Expand Down
Loading
Loading