Add after-upload-deletion-promise tests

Author: timfish

packages/integration-tests-next/fixtures/configs/after-upload-deletion-promise.config.js

@@ -0,0 +1,16 @@
+// Config that uses a Promise for filesToDeleteAfterUpload
+// This tests that the plugin can handle async file deletion patterns
+export function getSentryConfig(outDir) {
+  const fileDeletionPromise = new Promise((resolve) => {
+    setTimeout(() => {
+      resolve([`${outDir}/basic.js.map`]);
+    }, 100);
+  });
+
+  return {
+    telemetry: false,
+    sourcemaps: {
+      filesToDeleteAfterUpload: fileDeletionPromise,
+    },
+  };
+}

packages/integration-tests-next/fixtures/esbuild/after-upload-deletion-promise.config.js

@@ -0,0 +1,15 @@
+import * as esbuild from "esbuild";
+import { sentryEsbuildPlugin } from "@sentry/esbuild-plugin";
+import { getSentryConfig } from "../configs/after-upload-deletion-promise.config.js";
+
+const outDir = "./out/after-upload-deletion-promise";
+
+await esbuild.build({
+  entryPoints: ["./src/basic.js"],
+  bundle: true,
+  outfile: `${outDir}/basic.js`,
+  minify: false,
+  format: "iife",
+  sourcemap: true,
+  plugins: [sentryEsbuildPlugin(getSentryConfig(outDir))],
+});

packages/integration-tests-next/fixtures/esbuild/after-upload-deletion-promise.test.ts

@@ -0,0 +1,16 @@
+import { expect } from "vitest";
+import { test } from "./utils";
+import { existsSync } from "node:fs";
+import { join } from "node:path";
+
+test(import.meta.url, ({ runBundler, outDir, runFileInNode }) => {
+  runBundler();
+  
+  // Verify the JS file exists and works
+  const output = runFileInNode("basic.js");
+  expect(output).toBe("hello world\n");
+  
+  // Verify the sourcemap was deleted (by the Promise)
+  const sourcemapPath = join(outDir, "basic.js.map");
+  expect(existsSync(sourcemapPath)).toBe(false);
+});

packages/integration-tests-next/fixtures/rolldown/after-upload-deletion-promise.config.ts

@@ -0,0 +1,14 @@
+import { sentryRollupPlugin } from "@sentry/rollup-plugin";
+import { defineConfig } from "rolldown";
+import { getSentryConfig } from "../configs/after-upload-deletion-promise.config.js";
+
+const outDir = "out/after-upload-deletion-promise";
+
+export default defineConfig({
+  input: "src/basic.js",
+  output: {
+    file: `${outDir}/basic.js`,
+    sourcemap: true,
+  },
+  plugins: [sentryRollupPlugin(getSentryConfig(outDir))],
+});

packages/integration-tests-next/fixtures/rolldown/after-upload-deletion-promise.test.ts

@@ -0,0 +1,16 @@
+import { expect } from "vitest";
+import { test } from "./utils";
+import { existsSync } from "node:fs";
+import { join } from "node:path";
+
+test(import.meta.url, ({ runBundler, outDir, runFileInNode }) => {
+  runBundler();
+  
+  // Verify the JS file exists and works
+  const output = runFileInNode("basic.js");
+  expect(output).toBe("hello world\n");
+  
+  // Verify the sourcemap was deleted (by the Promise)
+  const sourcemapPath = join(outDir, "basic.js.map");
+  expect(existsSync(sourcemapPath)).toBe(false);
+});

packages/integration-tests-next/fixtures/rollup3/after-upload-deletion-promise.config.js

@@ -0,0 +1,14 @@
+import { sentryRollupPlugin } from "@sentry/rollup-plugin";
+import { defineConfig } from "rollup";
+import { getSentryConfig } from "../configs/after-upload-deletion-promise.config.js";
+
+const outDir = "out/after-upload-deletion-promise";
+
+export default defineConfig({
+  input: "src/basic.js",
+  output: {
+    file: `${outDir}/basic.js`,
+    sourcemap: true,
+  },
+  plugins: [sentryRollupPlugin(getSentryConfig(outDir))],
+});

packages/integration-tests-next/fixtures/rollup3/after-upload-deletion-promise.test.ts

@@ -0,0 +1,16 @@
+import { expect } from "vitest";
+import { test } from "./utils";
+import { existsSync } from "node:fs";
+import { join } from "node:path";
+
+test(import.meta.url, ({ runBundler, outDir, runFileInNode }) => {
+  runBundler();
+  
+  // Verify the JS file exists and works
+  const output = runFileInNode("basic.js");
+  expect(output).toBe("hello world\n");
+  
+  // Verify the sourcemap was deleted (by the Promise)
+  const sourcemapPath = join(outDir, "basic.js.map");
+  expect(existsSync(sourcemapPath)).toBe(false);
+});

packages/integration-tests-next/fixtures/rollup4/after-upload-deletion-promise.config.js

@@ -0,0 +1,14 @@
+import { sentryRollupPlugin } from "@sentry/rollup-plugin";
+import { defineConfig } from "rollup";
+import { getSentryConfig } from "../configs/after-upload-deletion-promise.config.js";
+
+const outDir = "out/after-upload-deletion-promise";
+
+export default defineConfig({
+  input: "src/basic.js",
+  output: {
+    file: `${outDir}/basic.js`,
+    sourcemap: true,
+  },
+  plugins: [sentryRollupPlugin(getSentryConfig(outDir))],
+});

packages/integration-tests-next/fixtures/rollup4/after-upload-deletion-promise.test.ts

@@ -0,0 +1,16 @@
+import { expect } from "vitest";
+import { test } from "./utils";
+import { existsSync } from "node:fs";
+import { join } from "node:path";
+
+test(import.meta.url, ({ runBundler, outDir, runFileInNode }) => {
+  runBundler();
+  
+  // Verify the JS file exists and works
+  const output = runFileInNode("basic.js");
+  expect(output).toBe("hello world\n");
+  
+  // Verify the sourcemap was deleted (by the Promise)
+  const sourcemapPath = join(outDir, "basic.js.map");
+  expect(existsSync(sourcemapPath)).toBe(false);
+});

packages/integration-tests-next/fixtures/vite4/after-upload-deletion-promise.config.ts

@@ -0,0 +1,20 @@
+import { sentryVitePlugin } from "@sentry/vite-plugin";
+import { defineConfig } from "vite";
+import { getSentryConfig } from "../configs/after-upload-deletion-promise.config.js";
+
+const outDir = "out/after-upload-deletion-promise";
+
+export default defineConfig({
+  build: {
+    minify: false,
+    sourcemap: true,
+    rollupOptions: {
+      input: "src/basic.js",
+      output: {
+        dir: outDir,
+        entryFileNames: "[name].js",
+      },
+    },
+  },
+  plugins: [sentryVitePlugin(getSentryConfig(outDir))],
+});