ref: Add child run to test webpack build (#2524)

* ref: Add child run to test webpack build

* fix: Restore dynamicRequire but for perf_hooks only

* fix: Tests on node6

Co-authored-by: Kamil Ogórek <kamil.ogorek@gmail.com>

Author: HazAT

packages/browser/test/package/npm-build.js

@@ -2,17 +2,14 @@ const fs = require('fs');
 const path = require('path');
 const webpack = require('webpack');
 const { JSDOM } = require('jsdom');
-// runTests();
+
 webpack(
   {
     entry: path.join(__dirname, 'test-code.js'),
     output: {
       path: __dirname,
       filename: 'tmp.js',
     },
-    // resolve: {
-    //   mainFields: ['main'],
-    // },
     mode: 'development',
   },
   (err, stats) => {
@@ -28,10 +25,12 @@ webpack(
 
     if (stats.hasErrors()) {
       console.error(info.errors);
+      process.exit(1);
     }
 
     if (stats.hasWarnings()) {
       console.warn(info.warnings);
+      process.exit(1);
     }
 
     runTests();

packages/node/package.json

@@ -60,7 +60,7 @@
     "test:jest": "jest",
     "test:watch": "jest --watch",
     "test:express": "node test/manual/express-scope-separation/start.js",
-    "test:webpack": "cd test/manual/webpack-domain/ && yarn && yarn webpack && node dist/bundle.js",
+    "test:webpack": "cd test/manual/webpack-domain/ && yarn && node npm-build.js",
     "version": "node ../../scripts/versionbump.js src/version.ts"
   },
   "jest": {

packages/node/test/manual/webpack-domain/npm-build.js

@@ -0,0 +1,46 @@
+const path = require('path');
+const webpack = require('webpack');
+const { execSync } = require('child_process');
+
+// prettier-ignore
+webpack(
+  {
+    entry: './index.js',
+    output: {
+      path: path.resolve(__dirname, 'dist'),
+      filename: 'bundle.js',
+    },
+    target: 'node',
+    mode: 'development',
+  },
+  function(err, stats) {
+    if (err) {
+      console.error(err.stack || err);
+      if (err.details) {
+        console.error(err.details);
+      }
+      return;
+    }
+
+    const info = stats.toJson();
+
+    if (stats.hasErrors()) {
+      console.error(info.errors);
+      process.exit(1);
+    }
+
+    if (stats.hasWarnings()) {
+      console.warn(info.warnings);
+      process.exit(1);
+    }
+    runTests();
+  }
+);
+
+function runTests() {
+  try {
+    execSync('node ' + path.resolve(__dirname, 'dist', 'bundle.js'));
+  } catch (_) {
+    process.exit(1);
+  }
+}

packages/node/test/manual/webpack-domain/webpack.config.js

@@ -20,6 +20,16 @@ interface SentryGlobal {
   };
 }
 
+/**
+ * Requires a module which is protected against bundler minification.
+ *
+ * @param request The module path to resolve
+ */
+export function dynamicRequire(mod: any, request: string): any {
+  // tslint:disable-next-line: no-unsafe-any
+  return mod.require(request);
+}
+
 /**
  * Checks whether we're in the Node.js or Browser environment
  *
@@ -365,8 +375,7 @@ const performanceFallback: CrossPlatformPerformance = {
 export const crossPlatformPerformance: CrossPlatformPerformance = (() => {
   if (isNodeEnv()) {
     try {
-      const req = require;
-      const perfHooks = req('perf_hooks') as { performance: CrossPlatformPerformance };
+      const perfHooks = dynamicRequire(module, 'perf_hooks') as { performance: CrossPlatformPerformance };
       return perfHooks.performance;
     } catch (_) {
       return performanceFallback;