Skip to content

Commit 8d12589

Browse files
RoyLinRoyLin
authored andcommitted
ci(publish-ts): glob-free test script + skip npm test on Windows
Windows PowerShell doesn't expand the test/*.test.mjs shell glob (bash does) → 'could not find'. Use the explicit file, and skip the npm test step on Windows (the suite uses a unix mock-agent .sh + unix-absolute paths). The Windows BUILD still runs + is what's published.
1 parent d212d9b commit 8d12589

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

.github/workflows/publish-ts.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,8 @@ jobs:
3535
- run: npm install
3636
- run: npm run build -- --target ${{ matrix.settings.target }}
3737
- run: npm test
38+
if: runner.os != 'Windows' # the suite uses a unix mock-agent .sh + unix-absolute paths
39+
3840
- uses: actions/upload-artifact@v4
3941
with:
4042
name: bindings-${{ matrix.settings.target }}

sdk/typescript/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
"scripts": {
3131
"build": "napi build --platform --release",
3232
"build:debug": "napi build --platform",
33-
"test": "node --test test/*.test.mjs",
33+
"test": "node --test test/sdk.test.mjs",
3434
"soak": "node --expose-gc soak.mjs",
3535
"coverage": "bash coverage.sh"
3636
},

0 commit comments

Comments
 (0)