only disable CLI-using plugins, add tests

Author: Lms24

packages/bundler-plugin-core/src/index.ts

@@ -111,7 +111,7 @@ export function sentryUnpluginFactory({
 
     const options = normalizeUserOptions(userOptions);
 
-    if (isDevMode || options.disable) {
+    if (options.disable) {
       return [
         {
           name: "sentry-noop-plugin",
@@ -324,6 +324,8 @@ export function sentryUnpluginFactory({
       logger.debug(
         "No release name provided. Will not create release. Please set the `release.name` option to identify your release."
       );
+    } else if (isDevMode) {
+      logger.debug("Running in development mode. Will not create release.");
     } else if (!options.authToken) {
       logger.warn(
         "No auth token provided. Will not create release. Please set the `authToken` option. You can find information on how to generate a Sentry auth token here: https://docs.sentry.io/api/auth/"
@@ -372,6 +374,8 @@ export function sentryUnpluginFactory({
       logger.debug(
         "Source map upload was disabled. Will not upload sourcemaps using debug ID process."
       );
+    } else if (isDevMode) {
+      logger.debug("Running in development mode. Will not upload sourcemaps.");
     } else if (!options.authToken) {
       logger.warn(
         "No auth token provided. Will not upload source maps. Please set the `authToken` option. You can find information on how to generate a Sentry auth token here: https://docs.sentry.io/api/auth/"

packages/rollup-plugin/test/public-api.test.ts

@@ -7,6 +7,10 @@ test("Rollup plugin should exist", () => {
 });
 
 describe("sentryRollupPlugin", () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
   it("returns an array of rollup plugins", () => {
     const plugins = sentryRollupPlugin({
       authToken: "test-token",
@@ -27,4 +31,41 @@ describe("sentryRollupPlugin", () => {
       "sentry-file-deletion-plugin",
     ]);
   });
+
+  it("doesn't include release management and debug id upload plugins if NODE_ENV is 'development'", () => {
+    const originalNodeEnv = process.env["NODE_ENV"];
+    process.env["NODE_ENV"] = "development";
+
+    const consoleSpy = jest.spyOn(console, "debug").mockImplementation(() => {
+      /* avoid test output pollution */
+    });
+
+    const plugins = sentryRollupPlugin({
+      authToken: "test-token",
+      org: "test-org",
+      project: "test-project",
+      debug: true,
+    }) as Plugin[];
+
+    expect(Array.isArray(plugins)).toBe(true);
+
+    const pluginNames = plugins.map((plugin) => plugin.name);
+
+    expect(pluginNames).toEqual([
+      "sentry-telemetry-plugin",
+      "sentry-rollup-release-injection-plugin",
+      "sentry-rollup-debug-id-injection-plugin",
+      "sentry-file-deletion-plugin",
+    ]);
+
+    expect(consoleSpy).toHaveBeenCalledWith(
+      expect.stringContaining("Running in development mode. Will not create release.")
+    );
+
+    expect(consoleSpy).toHaveBeenCalledWith(
+      expect.stringContaining("Running in development mode. Will not upload sourcemaps.")
+    );
+
+    process.env["NODE_ENV"] = originalNodeEnv;
+  });
 });

packages/vite-plugin/test/public-api.test.ts

@@ -7,6 +7,10 @@ test("Vite plugin should exist", () => {
 });
 
 describe("sentryVitePlugin", () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
   it("returns an array of Vite plugins", () => {
     const plugins = sentryVitePlugin({
       authToken: "test-token",
@@ -27,4 +31,41 @@ describe("sentryVitePlugin", () => {
       "sentry-file-deletion-plugin",
     ]);
   });
+
+  it("doesn't include release management and debug id upload plugins if NODE_ENV is 'development'", () => {
+    const originalNodeEnv = process.env["NODE_ENV"];
+    process.env["NODE_ENV"] = "development";
+
+    const consoleSpy = jest.spyOn(console, "debug").mockImplementation(() => {
+      /* avoid test output pollution */
+    });
+
+    const plugins = sentryVitePlugin({
+      authToken: "test-token",
+      org: "test-org",
+      project: "test-project",
+      debug: true,
+    }) as VitePlugin[];
+
+    expect(Array.isArray(plugins)).toBe(true);
+
+    const pluginNames = plugins.map((plugin) => plugin.name);
+
+    expect(pluginNames).toEqual([
+      "sentry-telemetry-plugin",
+      "sentry-vite-release-injection-plugin",
+      "sentry-vite-debug-id-injection-plugin",
+      "sentry-file-deletion-plugin",
+    ]);
+
+    expect(consoleSpy).toHaveBeenCalledWith(
+      expect.stringContaining("Running in development mode. Will not create release.")
+    );
+
+    expect(consoleSpy).toHaveBeenCalledWith(
+      expect.stringContaining("Running in development mode. Will not upload sourcemaps.")
+    );
+
+    process.env["NODE_ENV"] = originalNodeEnv;
+  });
 });