Only log/test if debug build

Author: aliu39

dev-packages/browser-integration-tests/suites/integrations/featureFlags/unleash/badSignature/test.ts

@@ -8,6 +8,8 @@ sentryTest('Logs and returns if isEnabled does not match expected signature', as
   if (shouldSkipFeatureFlagsTest()) {
     sentryTest.skip();
   }
+  const bundleKey = process.env.PW_BUNDLE || '';
+  const hasDebug = !bundleKey.includes('_min');
 
   await page.route('https://dsn.ingest.sentry.io/**/*', route => {
     return route.fulfill({
@@ -39,17 +41,19 @@ sentryTest('Logs and returns if isEnabled does not match expected signature', as
   expect(results).toEqual(['my-feature', 999, {}]);
 
   // Expected error logs.
-  expect(errorLogs).toEqual(
-    expect.arrayContaining([
-      expect.stringContaining(
-        '[Feature Flags] UnleashClient.isEnabled does not match expected signature. arg0: my-feature (string), result: my-feature (string)',
-      ),
-      expect.stringContaining(
-        '[Feature Flags] UnleashClient.isEnabled does not match expected signature. arg0: 999 (number), result: 999 (number)',
-      ),
-      expect.stringContaining(
-        '[Feature Flags] UnleashClient.isEnabled does not match expected signature. arg0: [object Object] (object), result: [object Object] (object)',
-      ),
-    ]),
-  );
+  if (hasDebug) {
+    expect(errorLogs).toEqual(
+      expect.arrayContaining([
+        expect.stringContaining(
+          '[Feature Flags] UnleashClient.isEnabled does not match expected signature. arg0: my-feature (string), result: my-feature (string)',
+        ),
+        expect.stringContaining(
+          '[Feature Flags] UnleashClient.isEnabled does not match expected signature. arg0: 999 (number), result: 999 (number)',
+        ),
+        expect.stringContaining(
+          '[Feature Flags] UnleashClient.isEnabled does not match expected signature. arg0: [object Object] (object), result: [object Object] (object)',
+        ),
+      ]),
+    );
+  }
 });

packages/browser/src/integrations/featureFlags/unleash/integration.ts

@@ -1,6 +1,7 @@
 import type { Client, Event, EventHint, IntegrationFn } from '@sentry/core';
 
 import { defineIntegration, fill, logger } from '@sentry/core';
+import { DEBUG_BUILD } from '../../../debug-build';
 import { copyFlagsFromScopeToEvent, insertFlagToScope } from '../../../utils/featureFlags';
 import type { UnleashClient, UnleashClientClass } from './types';
 
@@ -64,7 +65,7 @@ function _wrappedIsEnabled(
 
     if (typeof toggleName === 'string' && typeof result === 'boolean') {
       insertFlagToScope(toggleName, result);
-    } else {
+    } else if (DEBUG_BUILD) {
       logger.error(
         `[Feature Flags] UnleashClient.isEnabled does not match expected signature. arg0: ${toggleName} (${typeof toggleName}), result: ${result} (${typeof result})`,
       );