add test

packages/node/src/otel/instrument.ts

@@ -1,7 +1,8 @@
 import type { Instrumentation } from '@opentelemetry/instrumentation';
 import { addOpenTelemetryInstrumentation } from '@sentry/opentelemetry';
 
-const INSTRUMENTED: Record<string, Instrumentation> = {};
+/** Exported only for tests. */
+export const INSTRUMENTED: Record<string, Instrumentation> = {};
 
 /**
  * Instrument an OpenTelemetry instrumentation once.

packages/node/test/integrations/tracing/graphql.test.ts

@@ -0,0 +1,46 @@
+import { GraphQLInstrumentation } from '@opentelemetry/instrumentation-graphql';
+import { graphqlIntegration, instrumentGraphql } from '../../../src/integrations/tracing/graphql';
+import { INSTRUMENTED } from '../../../src/otel/instrument';
+
+jest.mock('@opentelemetry/instrumentation-graphql');
+
+describe('GraphQL', () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+    delete INSTRUMENTED.Graphql;
+
+    (GraphQLInstrumentation as unknown as jest.SpyInstance).mockImplementation(() => {
+      return {
+        setTracerProvider: () => undefined,
+        setMeterProvider: () => undefined,
+        getConfig: () => ({}),
+        setConfig: () => ({}),
+        enable: () => undefined,
+      };
+    });
+  });
+
+  it('defaults are correct for instrumentGraphql', () => {
+    instrumentGraphql({ ignoreTrivialResolveSpans: false });
+
+    expect(GraphQLInstrumentation).toHaveBeenCalledTimes(1);
+    expect(GraphQLInstrumentation).toHaveBeenCalledWith({
+      ignoreResolveSpans: true,
+      ignoreTrivialResolveSpans: false,
+      useOperationNameForRootSpan: true,
+      responseHook: expect.any(Function),
+    });
+  });
+
+  it('defaults are correct for _graphqlIntegration', () => {
+    graphqlIntegration({ ignoreTrivialResolveSpans: false }).setupOnce!();
+
+    expect(GraphQLInstrumentation).toHaveBeenCalledTimes(1);
+    expect(GraphQLInstrumentation).toHaveBeenCalledWith({
+      ignoreResolveSpans: true,
+      ignoreTrivialResolveSpans: false,
+      useOperationNameForRootSpan: true,
+      responseHook: expect.any(Function),
+    });
+  });
+});