feat: Make sure that Debug integration is always setup as the last one

kamilogorek · kamilogorek · commit a909d94b33f3 · 2019-11-08T14:54:30.000+01:00
diff --git a/packages/core/src/integration.ts b/packages/core/src/integration.ts
@@ -40,7 +40,14 @@ export function getIntegrationsToSetup(options: Options): Integration[] {
     integrations = userIntegrations(defaultIntegrations);
     integrations = Array.isArray(integrations) ? integrations : [integrations];
   } else {
-    return [...defaultIntegrations];
+    integrations = [...defaultIntegrations];
+  }
+
+  // Make sure that if present, `Debug` integration will always run last
+  const integrationsNames = integrations.map(i => i.name);
+  const alwaysLastToRun = 'Debug';
+  if (integrationsNames.indexOf(alwaysLastToRun) !== -1) {
+    integrations.push(...integrations.splice(integrationsNames.indexOf(alwaysLastToRun), 1));
   }
 
   return integrations;
diff --git a/packages/core/test/lib/integration.test.ts b/packages/core/test/lib/integration.test.ts
@@ -122,4 +122,27 @@ describe('getIntegrationsToSetup', () => {
     expect((integrations[0] as any).order).toEqual('firstUser');
     expect((integrations[1] as any).order).toEqual('secondUser');
   });
+
+  it('always moves Debug integration to the end of the list', () => {
+    let integrations = getIntegrationsToSetup({
+      defaultIntegrations: [new MockIntegration('Debug'), new MockIntegration('foo')],
+      integrations: [new MockIntegration('bar')],
+    });
+
+    expect(integrations.map(i => i.name)).toEqual(['foo', 'bar', 'Debug']);
+
+    integrations = getIntegrationsToSetup({
+      defaultIntegrations: [new MockIntegration('foo')],
+      integrations: [new MockIntegration('Debug'), new MockIntegration('bar')],
+    });
+
+    expect(integrations.map(i => i.name)).toEqual(['foo', 'bar', 'Debug']);
+
+    integrations = getIntegrationsToSetup({
+      defaultIntegrations: [new MockIntegration('Debug')],
+      integrations: [new MockIntegration('foo')],
+    });
+
+    expect(integrations.map(i => i.name)).toEqual(['foo', 'Debug']);
+  });
 });
