Removed enableTracing getsentry/sentry-javascript#15078

Author: antonis

packages/core/src/js/integrations/default.ts

@@ -91,10 +91,7 @@ export function getDefaultIntegrations(options: ReactNativeClientOptions): Integ
   // hasTracingEnabled from `@sentry/core` only check if tracesSampler or tracesSampleRate keys are present
   // that's different from prev imp here and might lead misconfiguration
   // `tracesSampleRate: undefined` should not enable tracing
-  const hasTracingEnabled =
-    options.enableTracing ||
-    typeof options.tracesSampleRate === 'number' ||
-    typeof options.tracesSampler === 'function';
+  const hasTracingEnabled = typeof options.tracesSampleRate === 'number' || typeof options.tracesSampler === 'function';
   if (hasTracingEnabled && options.enableAppStartTracking) {
     integrations.push(appStartIntegration());
   }

packages/core/test/feedback.test.ts

@@ -297,7 +297,7 @@ describe('captureFeedback', () => {
       getDefaultTestClientOptions({
         dsn: 'https://[email protected]/1',
         enableSend: true,
-        enableTracing: true,
+        tracesSampleRate: 1.0,
         // We don't care about transactions here...
         beforeSendTransaction() {
           return null;
@@ -360,7 +360,7 @@ describe('captureFeedback', () => {
       getDefaultTestClientOptions({
         dsn: 'https://[email protected]/1',
         enableSend: true,
-        enableTracing: true,
+        tracesSampleRate: 1.0,
         // We don't care about transactions here...
         beforeSendTransaction() {
           return null;

packages/core/test/profiling/integration.test.ts

@@ -343,7 +343,7 @@ function initTestClient(
   const transportSendMock = jest.fn<ReturnType<Transport['send']>, Parameters<Transport['send']>>();
   const options: Sentry.ReactNativeOptions = {
     dsn: MOCK_DSN,
-    enableTracing: true,
+    tracesSampleRate: 1.0,
     enableNativeFramesTracking: false,
     profilesSampleRate: 1,
     integrations: integrations => {

packages/core/test/trace.test.ts

@@ -7,7 +7,7 @@ describe('parentSpanIsAlwaysRootSpan', () => {
 
   it('creates a span as child of root span if parentSpanIsAlwaysRootSpan=true', () => {
     const options = getDefaultTestClientOptions({
-      tracesSampleRate: 1,
+      tracesSampleRate: 1.0,
       parentSpanIsAlwaysRootSpan: true,
     });
     client = new TestClient(options);
@@ -27,7 +27,7 @@ describe('parentSpanIsAlwaysRootSpan', () => {
 
   it('does not creates a span as child of root span if parentSpanIsAlwaysRootSpan=false', () => {
     const options = getDefaultTestClientOptions({
-      tracesSampleRate: 1,
+      tracesSampleRate: 1.0,
       parentSpanIsAlwaysRootSpan: false,
     });
     client = new TestClient(options);

packages/core/test/tracing/reactnavigation.ttid.test.tsx

@@ -614,7 +614,7 @@ function initSentry(sut: ReturnType<typeof Sentry.reactNavigationIntegration>):
   const transportSendMock = jest.fn<ReturnType<Transport['send']>, Parameters<Transport['send']>>();
   const options: Sentry.ReactNativeOptions = {
     dsn: MOCK_DSN,
-    enableTracing: true,
+    tracesSampleRate: 1.0,
     enableStallTracking: false,
     integrations: [
       sut,

samples/expo/app/_layout.tsx

@@ -67,7 +67,6 @@ Sentry.init({
   // For testing, session close when 5 seconds (instead of the default 30) in the background.
   sessionTrackingIntervalMillis: 5000,
   // This will capture ALL TRACES and likely use up all your quota
-  enableTracing: true,
   tracesSampleRate: 1.0,
   tracePropagationTargets: ['localhost', /^\//, /^https:\/\//, /^http:\/\//],
   attachStacktrace: true,

samples/react-native-macos/src/App.tsx

@@ -95,7 +95,6 @@ Sentry.init({
   // For testing, session close when 5 seconds (instead of the default 30) in the background.
   sessionTrackingIntervalMillis: 30000,
   // This will capture ALL TRACES and likely use up all your quota
-  enableTracing: true,
   tracesSampleRate: 1.0,
   tracePropagationTargets: ['localhost', /^\//, /^https:\/\//, /^http:\/\//],
   attachStacktrace: true,

samples/react-native/src/App.tsx

@@ -110,7 +110,6 @@ Sentry.init({
   // For testing, session close when 5 seconds (instead of the default 30) in the background.
   sessionTrackingIntervalMillis: 30000,
   // This will capture ALL TRACES and likely use up all your quota
-  enableTracing: true,
   tracesSampleRate: 1.0,
   tracePropagationTargets: ['localhost', /^\//, /^https:\/\//, /^http:\/\//],
   attachStacktrace: true,