ref: Run node integration tests via runner & skip fastify on node <14

Author: mydea

packages/node/package.json

@@ -62,9 +62,8 @@
     "lint": "run-s lint:prettier lint:eslint",
     "lint:eslint": "eslint . --format stylish",
     "lint:prettier": "prettier --check \"{src,test,scripts}/**/**.ts\"",
-    "test": "run-s test:jest test:express test:fastify test:webpack test:release-health",
-    "test:express": "node test/manual/express-scope-separation/start.js",
-    "test:fastify": "node test/manual/fastify-scope-separation/start.js",
+    "test": "run-s test:jest test:integration test:webpack test:release-health",
+    "test:integration": "node test/manual/integration/runner.js",
     "test:jest": "jest",
     "test:release-health": "node test/manual/release-health/runner.js",
     "test:webpack": "cd test/manual/webpack-async-context/ && yarn --silent && node npm-build.js",

packages/node/test/manual/express-scope-separation/start.js renamed to packages/node/test/manual/integration/express/express-scope-separation.js

@@ -1,8 +1,8 @@
 const http = require('http');
 const express = require('express');
 const app = express();
-const Sentry = require('../../../build/cjs');
-const { colorize } = require('../colorize');
+const Sentry = require('../../../../build/cjs');
+const { colorize } = require('../../colorize');
 const { TextEncoder } = require('util');
 
 // don't log the test errors we're going to throw, so at a quick glance it doesn't look like the test itself has failed

packages/node/test/manual/fastify-scope-separation/start.js renamed to packages/node/test/manual/integration/fastify/fastify-scope-separation.js

@@ -1,8 +1,8 @@
 const http = require('http');
 const fastify = require('fastify');
 const app = fastify();
-const Sentry = require('../../../build/cjs');
-const { colorize } = require('../colorize');
+const Sentry = require('../../../../build/cjs');
+const { colorize } = require('../../colorize');
 const { TextEncoder } = require('util');
 
 // don't log the test errors we're going to throw, so at a quick glance it doesn't look like the test itself has failed

packages/node/test/manual/integration/runner.js

@@ -0,0 +1,59 @@
+const fs = require('fs');
+const path = require('path');
+const { spawn } = require('child_process');
+const { colorize } = require('../colorize');
+
+const nodeVersion = parseInt(process.version.match(/^v(\d+)\./)[1]);
+const scenariosDirs = ['express'];
+const scenarios = [];
+
+// Fastify is only supported on Node 14+
+if (nodeVersion >= 14) {
+  scenariosDirs.push('fastify');
+}
+
+for (const dir of scenariosDirs) {
+  const scenarioDir = path.resolve(__dirname, dir);
+  const filenames = fs.readdirSync(scenarioDir);
+  const paths = filenames.map(filename => [filename, path.resolve(scenarioDir, filename)]);
+  scenarios.push(...paths);
+}
+
+const processes = scenarios.map(([filename, filepath]) => {
+  return new Promise(resolve => {
+    const scenarioProcess = spawn('node', [filepath], { timeout: 10000 });
+    const output = [];
+    const errors = [];
+
+    scenarioProcess.stdout.on('data', data => {
+      output.push(data.toString());
+    });
+
+    scenarioProcess.stderr.on('data', data => {
+      errors.push(data.toString());
+    });
+
+    scenarioProcess.on('exit', code => {
+      if (code === 0) {
+        console.log(colorize(`PASSED: ${filename}`, 'green'));
+      } else {
+        console.log(colorize(`FAILED: ${filename}`, 'red'));
+
+        if (output.length) {
+          console.log(colorize(output.join('\n'), 'yellow'));
+        }
+        if (errors.length) {
+          console.log(colorize(errors.join('\n'), 'yellow'));
+        }
+      }
+
+      resolve(code);
+    });
+  });
+});
+
+Promise.all(processes).then(codes => {
+  if (codes.some(code => code !== 0)) {
+    process.exit(1);
+  }
+});