Removes benchmarks from unit tests

Author: antonis

packages/babel-plugin-component-annotate/test/__snapshots__/test-plugin.test.ts.snap

@@ -20,6 +20,22 @@ export default function TestComponent() {
 }"
 `;
 
+exports[`Fragment Detection handles Fragment aliased correctly when used by other non-Fragment components in a different scope 1`] = `
+"import { Fragment as OriginalF } from 'react';
+import { OtherComponent } from 'some-library';
+function TestComponent() {
+  const F = OriginalF;
+
+  // Use Fragment alias - should be ignored
+  return /*#__PURE__*/React.createElement(F, null, /*#__PURE__*/React.createElement(\\"div\\", null, \\"This should NOT have data-sentry-element (Fragment)\\"));
+}
+function AnotherComponent() {
+  // Different component with same alias name in different function scope
+  const F = OtherComponent;
+  return /*#__PURE__*/React.createElement(F, null, /*#__PURE__*/React.createElement(\\"div\\", null, \\"This SHOULD have data-sentry-element (not Fragment)\\"));
+}"
+`;
+
 exports[`Fragment Detection handles complex variable assignment chains 1`] = `
 "import { Fragment } from 'react';
 const MyFragment = Fragment;
@@ -168,22 +184,6 @@ export default function TestComponent() {
 }"
 `;
 
-exports[`Fragment Performance Benchmarks handles Fragment aliased correctly when used by other non-Fragment components in a different scope 1`] = `
-"import { Fragment as OriginalF } from 'react';
-import { OtherComponent } from 'some-library';
-function TestComponent() {
-  const F = OriginalF;
-
-  // Use Fragment alias - should be ignored
-  return /*#__PURE__*/React.createElement(F, null, /*#__PURE__*/React.createElement(\\"div\\", null, \\"This should NOT have data-sentry-element (Fragment)\\"));
-}
-function AnotherComponent() {
-  // Different component with same alias name in different function scope
-  const F = OtherComponent;
-  return /*#__PURE__*/React.createElement(F, null, /*#__PURE__*/React.createElement(\\"div\\", null, \\"This SHOULD have data-sentry-element (not Fragment)\\"));
-}"
-`;
-
 exports[`arrow snapshot matches 1`] = `
 "import React, { Component } from 'react';
 const componentName = () => {

packages/babel-plugin-component-annotate/test/test-plugin.test.ts

@@ -1747,143 +1747,6 @@ export default function TestComponent() {
     expect(result?.code).not.toContain('"data-sentry-element": "MyReact.Fragment"');
     expect(result?.code).toMatchSnapshot();
   });
-});
-
-describe("Fragment Performance Benchmarks", () => {
-  interface TestFile {
-    filename: string;
-    code: string;
-  }
-
-  interface BenchmarkResult {
-    avg: number;
-    min: number;
-    max: number;
-    median: number;
-  }
-
-  const createTestProject = (fileCount: number, fragmentRatio = 0.4): TestFile[] => {
-    const files: TestFile[] = [];
-
-    for (let i = 0; i < fileCount; i++) {
-      const hasFragment = Math.random() < fragmentRatio;
-      const patterns = [
-        `import React, { Fragment } from 'react';`,
-        `import { Fragment } from 'react';`,
-        `import * as React from 'react';`,
-        `import React from 'react';\nconst { Fragment } = React;`,
-        `import MyReact from 'react';\nconst { Fragment: F } = MyReact;`,
-      ];
-
-      const pattern = patterns[Math.floor(Math.random() * patterns.length)] as string;
-      const fragmentType = getFragmentType(pattern);
-
-      // Add imports to other files for realistic import traversal
-      const imports: string[] = [];
-      const numImports = Math.min(3, i);
-      for (let j = 0; j < numImports; j++) {
-        imports.push(`import Component${j} from './component${j}';`);
-      }
-
-      const code = `${pattern}
-${imports.join("\n")}
-
-export default function Component${i}() {
-  return ${
-    hasFragment
-      ? `<${fragmentType}><div>Content ${i}</div></${fragmentType}>`
-      : `<div>Component ${i}</div>`
-  };
-}`;
-
-      files.push({ filename: `component${i}.js`, code });
-    }
-
-    return files;
-  };
-
-  const getFragmentType = (pattern: string): string => {
-    if (pattern.includes("Fragment: F")) return "F";
-    if (pattern.includes("Fragment }")) return "Fragment";
-    if (pattern.includes("* as React")) return "React.Fragment";
-    if (pattern.includes("const { Fragment }")) return "Fragment";
-    return "Fragment";
-  };
-
-  const runBenchmark = (files: TestFile[], iterations = 10): BenchmarkResult => {
-    const times: number[] = [];
-
-    for (let i = 0; i < iterations; i++) {
-      const start = getTime();
-
-      // Transform all files (simulating your plugin processing)
-      files.forEach((file: TestFile) => {
-        const result = transform(file.code, {
-          filename: `/${file.filename}`,
-          configFile: false,
-          presets: ["@babel/preset-react"],
-          plugins: [plugin],
-        });
-        if (!result?.code) throw new Error("Transform failed");
-      });
-
-      const end = getTime();
-      times.push(end - start);
-    }
-
-    const sortedTimes = [...times].sort((a, b) => a - b);
-    return {
-      avg: times.reduce((a, b) => a + b) / times.length,
-      min: Math.min(...times),
-      max: Math.max(...times),
-      median: sortedTimes[Math.floor(sortedTimes.length / 2)] ?? -1,
-    };
-  };
-
-  // Test different project sizes
-  const scenarios = [
-    { name: "small" as const, files: 20, expectedMaxTime: 60 },
-    { name: "medium" as const, files: 50, expectedMaxTime: 300 },
-    { name: "large" as const, files: 200, expectedMaxTime: 600 },
-  ];
-
-  scenarios.forEach((scenario) => {
-    it(`should perform well on ${scenario.name} projects (${scenario.files} files)`, () => {
-      const testFiles = createTestProject(scenario.files);
-      const results = runBenchmark(testFiles);
-
-      expect(results.avg).toBeGreaterThan(0);
-      expect(results.avg).toBeLessThan(scenario.expectedMaxTime);
-      expect(results.median).toBeLessThan(scenario.expectedMaxTime * 0.8);
-    });
-  });
-
-  it("should handle fragment-heavy projects efficiently", () => {
-    const fragmentHeavyFiles = createTestProject(25, 0.8);
-    const normalFiles = createTestProject(25, 0.2);
-
-    const fragmentResults = runBenchmark(fragmentHeavyFiles, 5);
-    const normalResults = runBenchmark(normalFiles, 5);
-
-    const overhead = (fragmentResults.avg - normalResults.avg) / normalResults.avg;
-
-    expect(overhead).toBeLessThan(0.5); // Less than 50% overhead
-    expect(fragmentResults.avg).toBeLessThan(300);
-  });
-
-  it("should not consume excessive memory", () => {
-    const testFiles = createTestProject(50);
-
-    if (global.gc) global.gc();
-
-    const memBefore = process.memoryUsage().heapUsed;
-    runBenchmark(testFiles, 1);
-    const memAfter = process.memoryUsage().heapUsed;
-
-    const memoryDelta = (memAfter - memBefore) / 1024 / 1024;
-
-    expect(memoryDelta).toBeLessThan(50);
-  });
 
   it("handles Fragment aliased correctly when used by other non-Fragment components in a different scope", () => {
     const result = transform(