Add test

Author: Luca Forstner

packages/integration-tests/fixtures/disabled-debug-id-injection/disabled-debug-id-injection.test.ts

@@ -0,0 +1,54 @@
+/* eslint-disable jest/no-standalone-expect */
+/* eslint-disable jest/expect-expect */
+import childProcess from "child_process";
+import path from "path";
+import { testIfNodeMajorVersionIsLessThan18 } from "../../utils/testIf";
+
+function checkBundle(bundlePath1: string, bundlePath2: string) {
+  const process1Output = childProcess.execSync(`node ${bundlePath1}`, { encoding: "utf-8" });
+  const debugIdMap1 = JSON.parse(process1Output) as Record<string, string>;
+  const debugIds1 = Object.values(debugIdMap1);
+  expect(debugIds1).toHaveLength(0);
+
+  const process2Output = childProcess.execSync(`node ${bundlePath2}`, { encoding: "utf-8" });
+  const debugIdMap2 = JSON.parse(process2Output) as Record<string, string>;
+  const debugIds2 = Object.values(debugIdMap2);
+  expect(debugIds2).toHaveLength(0);
+}
+
+describe("should not inject debug IDs when `sourcemaps.disable` is `true`", () => {
+  test("esbuild bundle", () => {
+    checkBundle(
+      path.join(__dirname, "out", "esbuild", "bundle1.js"),
+      path.join(__dirname, "out", "esbuild", "bundle2.js")
+    );
+  });
+
+  test("rollup bundle", () => {
+    checkBundle(
+      path.join(__dirname, "out", "rollup", "bundle1.js"),
+      path.join(__dirname, "out", "rollup", "bundle2.js")
+    );
+  });
+
+  test("vite bundle", () => {
+    checkBundle(
+      path.join(__dirname, "out", "vite", "bundle1.js"),
+      path.join(__dirname, "out", "vite", "bundle2.js")
+    );
+  });
+
+  testIfNodeMajorVersionIsLessThan18("webpack 4 bundle", () => {
+    checkBundle(
+      path.join(__dirname, "out", "webpack4", "bundle1.js"),
+      path.join(__dirname, "out", "webpack4", "bundle2.js")
+    );
+  });
+
+  test("webpack 5 bundle", () => {
+    checkBundle(
+      path.join(__dirname, "out", "webpack5", "bundle1.js"),
+      path.join(__dirname, "out", "webpack5", "bundle2.js")
+    );
+  });
+});

packages/integration-tests/fixtures/disabled-debug-id-injection/input/bundle1.js

@@ -0,0 +1,5 @@
+// eslint-disable-next-line no-console
+console.log(JSON.stringify(global._sentryDebugIds));
+
+// Just so the two bundles generate different hashes:
+global.iAmBundle1 = 1;

packages/integration-tests/fixtures/disabled-debug-id-injection/input/bundle2.js

@@ -0,0 +1,5 @@
+// eslint-disable-next-line no-console
+console.log(JSON.stringify(global._sentryDebugIds));
+
+// Just so the two bundles generate different hashes:
+global.iAmBundle2 = 2;

packages/integration-tests/fixtures/disabled-debug-id-injection/setup.ts

@@ -0,0 +1,18 @@
+import * as path from "path";
+import { createCjsBundles } from "../../utils/create-cjs-bundles";
+
+const outputDir = path.resolve(__dirname, "out");
+
+createCjsBundles(
+  {
+    bundle1: path.resolve(__dirname, "input", "bundle1.js"),
+    bundle2: path.resolve(__dirname, "input", "bundle2.js"),
+  },
+  outputDir,
+  {
+    telemetry: false,
+    sourcemaps: {
+      disable: true,
+    },
+  }
+);