test: fix shell for windows tests (#72)

Author: OrKoN

.github/workflows/run-tests.yml

@@ -35,13 +35,16 @@ jobs:
           node-version: ${{ matrix.node }}
 
       - name: Install dependencies
+        shell: bash
         run: npm ci
 
       - name: Disable AppArmor
         if: ${{ matrix.os == 'ubuntu-latest' }}
+        shell: bash
         run: echo 0 | sudo tee /proc/sys/kernel/apparmor_restrict_unprivileged_userns
 
       - name: Run tests
+        shell: bash
         run: npm run test
 
   # Gating job for branch protection.

package.json

@@ -16,10 +16,10 @@
     "docs:generate": "node --experimental-strip-types scripts/generate-docs.ts",
     "start": "npm run build && node build/src/index.js",
     "start-debug": "DEBUG=mcp:* DEBUG_COLORS=false npm run build && node build/src/index.js",
-    "test": "npm run build && node --require ./build/tests/setup.js --test-reporter spec --test-force-exit --test 'build/tests/**/*.test.js'",
-    "test:only": "npm run build && node --require ./build/tests/setup.js --test-reporter spec --test-force-exit --test --test-only 'build/tests/**/*.test.js'",
-    "test:only:no-build": "node --require ./build/tests/setup.js --test-reporter spec --test-force-exit --test --test-only 'build/tests/**/*.test.js'",
-    "test:update-snapshots": "npm run build && node --require ./build/tests/setup.js --test-force-exit --test --test-update-snapshots 'build/tests/**/*.test.js'",
+    "test": "npm run build && node --require ./build/tests/setup.js --test-reporter spec --test-force-exit --test \"build/tests/**/*.test.js\"",
+    "test:only": "npm run build && node --require ./build/tests/setup.js --test-reporter spec --test-force-exit --test --test-only \"build/tests/**/*.test.js\"",
+    "test:only:no-build": "node --require ./build/tests/setup.js --test-reporter spec --test-force-exit --test --test-only \"build/tests/**/*.test.js\"",
+    "test:update-snapshots": "npm run build && node --require ./build/tests/setup.js --test-force-exit --test --test-update-snapshots \"build/tests/**/*.test.js\"",
     "prepare": "node --experimental-strip-types scripts/prepare.ts"
   },
   "files": [

tests/setup.ts

@@ -4,12 +4,13 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 import {it} from 'node:test';
+import path from 'node:path';
 
 // This is run by Node when we execute the tests via the --require flag.
 it.snapshot.setResolveSnapshotPath(testPath => {
   // By default the snapshots go into the build directory, but we want them
   // in the tests/ directory.
-  const correctPath = testPath?.replace('/build/tests', '/tests');
+  const correctPath = testPath?.replace(path.join('build', 'tests'), 'tests');
   return correctPath + '.snapshot';
 });
 

tests/tools/screenshot.test.ts

@@ -64,7 +64,7 @@ describe('screenshot', () => {
       await withBrowser(async (response, context) => {
         const page = context.getSelectedPage();
         await page.setContent(
-          `<div style="color:blue;">test</div>`.repeat(5_000),
+          `<div style="color:blue;">test</div>`.repeat(7_000),
         );
         await screenshot.handler(
           {params: {format: 'png', fullPage: true}},