only set up integrations if they haven't already been set up

Author: lobsterkatie

packages/core/src/baseclient.ts

@@ -207,7 +207,7 @@ export abstract class BaseClient<B extends Backend, O extends Options> implement
    * Sets up the integrations
    */
   public setupIntegrations(): void {
-    if (this._isEnabled()) {
+    if (this._isEnabled() && !this._integrations.initialized) {
       this._integrations = setupIntegrations(this._options);
     }
   }

packages/core/src/integration.ts

@@ -5,9 +5,9 @@ import { logger } from '@sentry/utils';
 export const installedIntegrations: string[] = [];
 
 /** Map of integrations assigned to a client */
-export interface IntegrationIndex {
+export type IntegrationIndex = {
   [key: string]: Integration;
-}
+} & { initialized?: boolean };
 
 /**
  * @private
@@ -74,5 +74,9 @@ export function setupIntegrations<O extends Options>(options: O): IntegrationInd
     integrations[integration.name] = integration;
     setupIntegration(integration);
   });
+  // set the `initialized` flag so we don't run through the process again unecessarily; use `Object.defineProperty`
+  // because by default it creates a property which is nonenumerable, which we want since `initialized` shouldn't be
+  // considered a member of the index the way the actual integrations are
+  Object.defineProperty(integrations, 'initialized', { value: true });
   return integrations;
 }

packages/core/test/lib/base.test.ts

@@ -2,6 +2,7 @@ import { Hub, Scope, Session } from '@sentry/hub';
 import { Event, Severity, Span } from '@sentry/types';
 import { logger, SentryError, SyncPromise } from '@sentry/utils';
 
+import * as integrationModule from '../../src/integration';
 import { TestBackend } from '../mocks/backend';
 import { TestClient } from '../mocks/client';
 import { TestIntegration } from '../mocks/integration';
@@ -872,6 +873,26 @@ describe('BaseClient', () => {
       expect(Object.keys((client as any)._integrations).length).toBe(0);
       expect(client.getIntegration(TestIntegration)).toBeFalsy();
     });
+
+    test('skips installation if integrations are already installed', () => {
+      expect.assertions(4);
+      const client = new TestClient({
+        dsn: PUBLIC_DSN,
+        integrations: [new TestIntegration()],
+      });
+      // note: not the `Client` method `setupIntegrations`, but the free-standing function which that method calls
+      const setupIntegrationsHelper = jest.spyOn(integrationModule, 'setupIntegrations');
+
+      // it should install the first time, because integrations aren't yet installed...
+      client.setupIntegrations();
+      expect(Object.keys((client as any)._integrations).length).toBe(1);
+      expect(client.getIntegration(TestIntegration)).toBeTruthy();
+      expect(setupIntegrationsHelper).toHaveBeenCalledTimes(1);
+
+      // ...but it shouldn't try to install a second time
+      client.setupIntegrations();
+      expect(setupIntegrationsHelper).toHaveBeenCalledTimes(1);
+    });
   });
 
   describe('flush/close', () => {