ref(tracing): update hasTracingEnabled return value (#4225)

Author: AbhiPrasad

packages/tracing/src/utils.ts

@@ -19,10 +19,7 @@ export function hasTracingEnabled(
     .getClient()
     ?.getOptions(),
 ): boolean {
-  if (!options) {
-    return false;
-  }
-  return 'tracesSampleRate' in options || 'tracesSampler' in options;
+  return !!options && ('tracesSampleRate' in options || 'tracesSampler' in options);
 }
 
 /**

packages/tracing/test/utils.test.ts

@@ -1,4 +1,21 @@
-import { extractTraceparentData } from '../src/utils';
+import { extractTraceparentData, hasTracingEnabled } from '../src/utils';
+
+describe('hasTracingEnabled', () => {
+  const tracesSampler = () => 1;
+  const tracesSampleRate = 1;
+  it.each([
+    ['No options', undefined, false],
+    ['No tracesSampler or tracesSampleRate', {}, false],
+    ['With tracesSampler', { tracesSampler }, true],
+    ['With tracesSampleRate', { tracesSampleRate }, true],
+    ['With tracesSampler and tracesSampleRate', { tracesSampler, tracesSampleRate }, true],
+  ])(
+    '%s',
+    (_: string, input: Parameters<typeof hasTracingEnabled>[0], output: ReturnType<typeof hasTracingEnabled>) => {
+      expect(hasTracingEnabled(input)).toBe(output);
+    },
+  );
+});
 
 describe('extractTraceparentData', () => {
   test('no sample', () => {