Remove redundant tests

Author: onurtemizkan

test/react-router/codemods/server-entry.test.ts

@@ -10,7 +10,6 @@ import {
   instrumentServerEntry,
   instrumentHandleRequest,
   instrumentHandleError,
-  instrumentUnstableInstrumentations,
 } from '../../../src/react-router/codemods/server-entry';
 
 // @ts-expect-error - magicast is ESM and TS complains about that. It works though
@@ -668,63 +667,3 @@ export const unstable_instrumentations = [Sentry.createSentryServerInstrumentati
     expect(count).toBe(1);
   });
 });
-
-describe('instrumentUnstableInstrumentations', () => {
-  let tmpDir: string;
-
-  beforeEach(() => {
-    tmpDir = path.join(
-      __dirname,
-      'fixtures',
-      'tmp',
-      `unstable-instrumentations-${Date.now()}-${Math.random()
-        .toString(36)
-        .substring(2, 11)}`,
-    );
-    fs.mkdirSync(tmpDir, { recursive: true });
-  });
-
-  afterEach(() => {
-    if (fs.existsSync(tmpDir)) {
-      fs.rmSync(tmpDir, { recursive: true });
-    }
-  });
-
-  it('should add unstable_instrumentations export to the end of the file', async () => {
-    const content = `
-import { ServerRouter } from 'react-router';
-import * as Sentry from '@sentry/react-router';
-
-export default function handleRequest() {}
-`;
-    const tempFile = path.join(tmpDir, 'entry.server.tsx');
-    fs.writeFileSync(tempFile, content);
-
-    const mod = await loadFile(tempFile);
-    instrumentUnstableInstrumentations(mod);
-
-    const modifiedCode = generateCode(mod.$ast).code;
-
-    expect(modifiedCode).toContain(
-      'export const unstable_instrumentations = [Sentry.createSentryServerInstrumentation()];',
-    );
-  });
-
-  it('should not add unstable_instrumentations if already present', async () => {
-    const content = `
-import * as Sentry from '@sentry/react-router';
-
-export default function handleRequest() {}
-export const unstable_instrumentations = [Sentry.createSentryServerInstrumentation()];
-`;
-    const tempFile = path.join(tmpDir, 'entry.server.tsx');
-    fs.writeFileSync(tempFile, content);
-
-    const mod = await loadFile(tempFile);
-    const originalBodyLength = (mod.$ast as any).body.length;
-
-    instrumentUnstableInstrumentations(mod);
-
-    expect((mod.$ast as any).body.length).toBe(originalBodyLength);
-  });
-});

test/react-router/sdk-setup.test.ts

@@ -262,11 +262,6 @@ describe('React Router SDK Setup', () => {
     });
 
     it('should handle semver range specifiers correctly', () => {
-      expect(
-        supportsInstrumentationAPI({
-          dependencies: { '@react-router/dev': '^7.9.5' },
-        }),
-      ).toBe(true);
       expect(
         supportsInstrumentationAPI({
           dependencies: { '@react-router/dev': '~7.9.5' },

test/react-router/templates.test.ts

@@ -295,13 +295,6 @@ describe('React Router Templates', () => {
         expect(result).not.toContain('unstable_instrumentations');
         expect(result).not.toContain('createSentryServerInstrumentation');
       });
-
-      it('should default to no instrumentation API when parameter is omitted', () => {
-        const result = getManualServerEntryContent();
-
-        expect(result).not.toContain('unstable_instrumentations');
-        expect(result).not.toContain('createSentryServerInstrumentation');
-      });
     });
   });