integration tests

Author: Lms24

packages/bundler-plugin-core/src/debug-id-upload.ts

@@ -168,7 +168,7 @@ export function createDebugIdUploadFunction({
                   });
 
                   await cliInstance.releases.uploadSourceMaps(
-                    releaseName ?? "undefined", // unfortunetly this needs a value for now but it will not matter since debug IDs overpower releases anyhow
+                    releaseName ?? "undefined", // unfortunately this needs a value for now but it will not matter since debug IDs overpower releases anyhow
                     {
                       include: [
                         {

packages/integration-tests/fixtures/error-no-handler/error-no-handler.test.ts

@@ -0,0 +1,18 @@
+import { sentryEsbuildPlugin } from "@sentry/esbuild-plugin";
+import { build } from "esbuild";
+import pluginOptions from "./plugin-options";
+import path from "path";
+
+build({
+  entryPoints: [path.join(__dirname, "input", "bundle.js")],
+  outdir: "./out/esbuild",
+  plugins: [sentryEsbuildPlugin(pluginOptions)],
+  minify: true,
+  bundle: true,
+  format: "cjs",
+  sourcemap: true,
+}).catch((e) => {
+  // eslint-disable-next-line no-console
+  console.error(e);
+  process.exit(1);
+});

packages/integration-tests/fixtures/error-no-handler/setup.ts

@@ -0,0 +1,24 @@
+import { sentryRollupPlugin } from "@sentry/rollup-plugin";
+import * as path from "path";
+import * as rollup from "rollup";
+import pluginOptions from "./plugin-options";
+
+rollup
+  .rollup({
+    input: {
+      index: path.join(__dirname, "input", "bundle.js"),
+    },
+    plugins: [sentryRollupPlugin(pluginOptions)],
+  })
+  .then((bundle) =>
+    bundle.write({
+      dir: path.join(__dirname, "out", "rollup"),
+      format: "cjs",
+      exports: "named",
+    })
+  )
+  .catch((e) => {
+    // eslint-disable-next-line no-console
+    console.error(e);
+    process.exit(1);
+  });

packages/integration-tests/fixtures/errorhandling/build-esbuild.ts

@@ -0,0 +1,27 @@
+import { sentryVitePlugin } from "@sentry/vite-plugin";
+import * as path from "path";
+import * as vite from "vite";
+import pluginOptions from "./plugin-options";
+
+vite
+  .build({
+    clearScreen: false,
+    build: {
+      outDir: path.join(__dirname, "out", "vite"),
+      rollupOptions: {
+        input: {
+          index: path.join(__dirname, "input", "bundle.js"),
+        },
+        output: {
+          format: "cjs",
+          entryFileNames: "[name].js",
+        },
+      },
+    },
+    plugins: [sentryVitePlugin(pluginOptions)],
+  })
+  .catch((e) => {
+    // eslint-disable-next-line no-console
+    console.error(e);
+    process.exit(1);
+  });

packages/integration-tests/fixtures/errorhandling/build-rollup.ts

@@ -0,0 +1,26 @@
+import { sentryWebpackPlugin } from "@sentry/webpack-plugin";
+import * as path from "path";
+import { default as webpack4 } from "webpack4";
+import pluginOptions from "./plugin-options";
+
+webpack4(
+  {
+    devtool: "source-map",
+    cache: false,
+    entry: {
+      index: path.join(__dirname, "input", "bundle.js"),
+    },
+    output: {
+      path: path.join(__dirname, "out", "webpack4"),
+      libraryTarget: "commonjs",
+    },
+    mode: "production",
+    target: "node", // needed for webpack 4 so we can access node api
+    plugins: [sentryWebpackPlugin(pluginOptions)],
+  },
+  (err) => {
+    if (err) {
+      process.exit(1);
+    }
+  }
+);

packages/integration-tests/fixtures/errorhandling/build-vite.ts

@@ -0,0 +1,27 @@
+import { sentryWebpackPlugin } from "@sentry/webpack-plugin";
+import * as path from "path";
+import { webpack as webpack5 } from "webpack5";
+import pluginOptions from "./plugin-options";
+
+webpack5(
+  {
+    devtool: "source-map",
+    cache: false,
+    entry: {
+      index: path.join(__dirname, "input", "bundle.js"),
+    },
+    output: {
+      path: path.join(__dirname, "out", "webpack5"),
+      library: {
+        type: "commonjs",
+      },
+    },
+    mode: "production",
+    plugins: [sentryWebpackPlugin(pluginOptions)],
+  },
+  (err) => {
+    if (err) {
+      process.exit(1);
+    }
+  }
+);

packages/integration-tests/fixtures/errorhandling/build-webpack4.ts

@@ -0,0 +1,52 @@
+/* eslint-disable jest/no-standalone-expect */
+/* eslint-disable jest/expect-expect */
+import path from "path";
+import { spawn } from "child_process";
+
+jest.setTimeout(10_000);
+
+describe("Error throwing by default (no errorHandler)", () => {
+  const FAKE_SENTRY_PORT = "9876";
+
+  const sentryServer = spawn("node", ["fakeSentry.js"], {
+    cwd: __dirname,
+    stdio: "ignore", // <-- set to "inherit" to get server logs. Deactivated to avoid test logs.
+    env: { ...process.env, FAKE_SENTRY_PORT },
+  });
+
+  beforeAll(async () => {
+    await new Promise<void>((resolve) =>
+      sentryServer.on("spawn", () => {
+        resolve();
+      })
+    );
+  });
+
+  afterAll(() => {
+    sentryServer.kill();
+  });
+
+  test.each(["vite", "rollup", "webpack5", "webpack4", "esbuild"])(
+    "doesn't throw when Sentry server responds with error code for %s",
+    async (bundler) => {
+      const buildProcess = spawn("yarn", ["ts-node", path.join(__dirname, `build-${bundler}.ts`)], {
+        env: { ...process.env, FAKE_SENTRY_PORT },
+        stdio: "ignore", // <-- set to "inherit" to get build output. Deactivated to avoid spamming test logs.
+      });
+
+      const exitCode = await new Promise<number>((resolve, reject) => {
+        buildProcess.on("exit", (code) => {
+          resolve(code ?? 99);
+        });
+
+        buildProcess.on("error", (err) => {
+          reject(err);
+        });
+      });
+
+      expect(exitCode).toBe(0);
+
+      buildProcess.kill();
+    }
+  );
+});

packages/integration-tests/fixtures/errorhandling/build-webpack5.ts

@@ -1,13 +1,15 @@
-import { createServer } from "http";
+// eslint-disable-next-line @typescript-eslint/no-var-requires
+const { createServer } = require("http");
 
 const port = process.env["FAKE_SENTRY_PORT"] || 3000;
 
 const server = createServer((req, res) => {
+  console.log("[SANTRY] incoming request", req.url);
   res.statusCode = 503;
   res.end("Error: Santry unreachable");
 });
 
 server.listen(port, () => {
   // eslint-disable-next-line no-console
-  console.log(`Santry running on http://localhost:${port}/`);
+  console.log(`[SANTRY] running on http://localhost:${port}/`);
 });