add tests for new integration exports

Author: AbhiPrasad

packages/tracing/test/index.bundle.test.ts

@@ -1,11 +1,9 @@
 import { Integrations } from '../src/index.bundle';
-import { testOnlyIfNodeVersionAtLeast } from './testutils';
 
 describe('Integrations export', () => {
-  // TODO `Object.values` doesn't work on Node < 8
-  testOnlyIfNodeVersionAtLeast(8)('is exported correctly', () => {
-    Object.values(Integrations).forEach(integration => {
-      expect(integration.id).toStrictEqual(expect.any(String));
+  it('is exported correctly', () => {
+    Object.keys(Integrations).forEach(key => {
+      expect(Integrations[key as keyof typeof Integrations].id).toStrictEqual(expect.any(String));
     });
   });
 });

packages/tracing/test/index.test.ts

@@ -0,0 +1,29 @@
+import { getCurrentHub } from '@sentry/hub';
+
+import { BrowserTracing, Integrations } from '../src';
+
+describe('index', () => {
+  describe('Integrations', () => {
+    it('is exported correctly', () => {
+      Object.keys(Integrations).forEach(key => {
+        expect(Integrations[key as keyof typeof Integrations].id).toStrictEqual(expect.any(String));
+      });
+    });
+
+    it('contains BrowserTracing', () => {
+      expect(Integrations.BrowserTracing).toEqual(BrowserTracing);
+    });
+  });
+
+  it('patches the global hub as a side effect', () => {
+    const hub = getCurrentHub();
+    const transaction = hub.startTransaction({ name: 'test', endTimestamp: 123 });
+    expect(transaction).toBeDefined();
+  });
+
+  it('patches the global hub as a side effect', () => {
+    const hub = getCurrentHub();
+    const transaction = hub.startTransaction({ name: 'test', endTimestamp: 123 });
+    expect(transaction).toBeDefined();
+  });
+});

packages/tracing/test/testutils.ts

@@ -42,17 +42,3 @@ export function getSymbolObjectKeyByName(obj: Record<string | symbol, any>, desc
 
   return matches[0] || undefined;
 }
-
-export const testOnlyIfNodeVersionAtLeast = (minVersion: number): jest.It => {
-  const currentNodeVersion = process.env.NODE_VERSION;
-
-  try {
-    if (Number(currentNodeVersion?.split('.')[0]) < minVersion) {
-      return it.skip;
-    }
-  } catch (oO) {
-    // we can't tell, so err on the side of running the test
-  }
-
-  return it;
-};