Replace test.sh

Author: timfish

.github/workflows/build.yml

@@ -199,7 +199,7 @@ jobs:
       - name: Run tests
         env:
           NODE_VERSION: ${{ matrix.node }}
-        run: ./scripts/test.sh
+        run: yarn test-ci
       - name: Compute test coverage
         uses: codecov/codecov-action@v1
 

package.json

@@ -22,7 +22,8 @@
     "pack:changed": "lerna run pack --since",
     "prepublishOnly": "lerna run --stream --concurrency 1 prepublishOnly",
     "postpublish": "make publish-docs && lerna run --stream --concurrency 1 postpublish",
-    "test": "lerna run --ignore @sentry-internal/browser-integration-tests --stream --concurrency 1 --sort test"
+    "test": "lerna run --ignore @sentry-internal/browser-integration-tests --stream --concurrency 1 --sort test",
+    "test-ci": "ts-node scripts/test.ts"
   },
   "volta": {
     "node": "14.17.0",

scripts/test.sh

@@ -0,0 +1,52 @@
+const { spawnSync } = require('child_process');
+const { join } = require('path');
+
+function run(cmd: string, cwd: string = '') {
+  const result = spawnSync(cmd, { shell: true, stdio: 'inherit', cwd: join(__dirname, '..', cwd) });
+
+  if (result.status !== 0) {
+    process.exit(result.status);
+  }
+}
+
+const nodeMajorVersion = parseInt(process.version.split('.')[0].replace('v', ''), 10);
+
+// control which packages we test on each version of node
+if (nodeMajorVersion <= 6) {
+  // install legacy versions of packages whose current versions don't support node 6
+  // ignoring engines and scripts lets us get away with having incompatible things installed for packages we're not testing
+  run('yarn add --dev --ignore-engines --ignore-scripts [email protected]', 'packages/node');
+  run('yarn add --dev --ignore-engines --ignore-scripts [email protected]', 'packages/tracing');
+  run('yarn add --dev --ignore-engines --ignore-scripts [email protected]', 'packages/utils');
+
+  // only test against @sentry/node and its dependencies - node 6 is too old for anything else to work
+  const scope = ['@sentry/core', '@sentry/hub', '@sentry/minimal', '@sentry/node', '@sentry/utils', '@sentry/tracing']
+    .map(dep => `--scope="${dep}"`)
+    .join(' ');
+
+  run(`yarn test ${scope}`);
+} else if (nodeMajorVersion <= 8) {
+  // install legacy versions of packages whose current versions don't support node 8
+  // ignoring engines and scripts lets us get away with having incompatible things installed for packages we're not testing
+  run('yarn add --dev --ignore-engines --ignore-scripts [email protected]', 'packages/tracing');
+  run('yarn add --dev --ignore-engines --ignore-scripts [email protected]', 'packages/utils');
+
+  // ember tests happen separately, and the rest fail on node 8 for various syntax or dependency reasons
+  const ignore = [
+    '@sentry/ember',
+    '@sentry-internal/eslint-plugin-sdk',
+    '@sentry/react',
+    '@sentry/wasm',
+    '@sentry/gatsby',
+    '@sentry/serverless',
+    '@sentry/nextjs',
+  ]
+    .map(dep => `--ignore="${dep}"`)
+    .join(' ');
+
+  run(`yarn test ${ignore}`);
+} else {
+  run('yarn test --ignore="@sentry/ember"');
+}
+
+process.exit(0);