test: Remove old integration tests

Author: timfish

.github/workflows/checks.yml

@@ -136,36 +136,6 @@ jobs:
         node-version: [18, 20, 22, 24]
         os: [ubuntu-latest, windows-latest]
     runs-on: ${{ matrix.os }}
-    steps:
-      - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
-      - uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
-        with:
-          node-version: ${{ matrix.node-version }}
-      - name: Use dependency cache
-        uses: actions/cache@0057852bfaa89a56745cba8c7296529d2fc39830 # v4
-        id: dependency-cache
-        with:
-          path: "**/node_modules"
-          key: ${{ runner.os }}-${{ hashFiles('**/package.json') }}-${{ hashFiles('**/yarn.lock') }}
-      - name: Install dependencies
-        run: yarn --frozen-lockfile --ignore-engines
-        if: steps.dependency-cache.outputs.cache-hit != 'true'
-      - name: Download build artifacts
-        uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # v4
-        with:
-          name: dist-artifacts-${{ github.run_id }}
-          path: packages
-      - run: yarn test:integration
-
-  test-integration-next:
-    needs: build
-    name: "Integration Tests Next (Node ${{ matrix.node-version }}, OS ${{ matrix.os }})"
-    strategy:
-      fail-fast: false
-      matrix:
-        node-version: [18, 20, 22, 24]
-        os: [ubuntu-latest, windows-latest]
-    runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
       - uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
@@ -188,7 +158,7 @@ jobs:
         with:
           name: dist-artifacts-${{ github.run_id }}
           path: packages
-      - run: yarn test:integration-next
+      - run: yarn test:integration
 
   lint:
     needs: build

package.json

@@ -11,7 +11,6 @@
     "packages/dev-utils",
     "packages/esbuild-plugin",
     "packages/eslint-configs",
-    "packages/integration-tests",
     "packages/playground",
     "packages/rollup-plugin",
     "packages/tsconfigs",
@@ -28,9 +27,8 @@
     "clean": "nx run-many --target=clean --all",
     "clean:all": "nx run-many --target=clean:all --all && yarn",
     "test": "nx run-many --target=test --all",
-    "test:unit": "nx run-many --target=test --all --exclude=@sentry-internal/integration-tests,@sentry-internal/integration-tests-next",
-    "test:integration": "nx run @sentry-internal/integration-tests:test",
-    "test:integration-next": "nx run @sentry-internal/integration-tests-next:test",
+    "test:unit": "nx run-many --target=test --all --exclude=@sentry-internal/integration-tests-next",
+    "test:integration": "nx run @sentry-internal/integration-tests-next:test",
     "lint": "nx run-many --target=lint --all",
     "check:formatting": "oxfmt --check .",
     "fix:formatting": "oxfmt ."

packages/babel-plugin-component-annotate/package.json

@@ -51,6 +51,7 @@
   },
   "devDependencies": {
     "@babel/core": "7.18.5",
+    "@types/babel__core": "^7.20.5",
     "@babel/preset-react": "^7.23.3",
     "@sentry-internal/eslint-config": "5.2.1",
     "@sentry-internal/sentry-bundler-plugin-tsconfig": "5.2.1",

packages/babel-plugin-component-annotate/src/experimental.ts

@@ -283,16 +283,13 @@ function processJSX(context: JSXProcessingContext, jsxNode: Babel.NodePath): voi
   // NOTE: I don't know of a case where `openingElement` would have more than one item,
   // but it's safer to always iterate
   const paths = jsxNode.get("openingElement");
-  const openingElements = Array.isArray(paths) ? paths : [paths];
+  const openingElements: Babel.NodePath<Babel.types.JSXOpeningElement>[] = Array.isArray(paths)
+    ? paths
+    : [paths];
 
   const hasInjectedAttributes = openingElements.reduce(
     (prev, openingElement) =>
-      prev ||
-      applyAttributes(
-        context,
-        openingElement as Babel.NodePath<Babel.types.JSXOpeningElement>,
-        context.componentName
-      ),
+      prev || applyAttributes(context, openingElement, context.componentName),
     false
   );
 

packages/integration-tests-next/fixtures/esbuild/utils.ts

@@ -37,7 +37,7 @@ export function test(url: string, callback: TestCallback) {
   // Detect CJS config files by test name suffix
   const configExt = testName.endsWith("-cjs") ? ".config.cjs" : ".config.js";
 
-  vitestTest(`esbuild > ${testName}`, (ctx) =>
+  vitestTest(`pnpm esbuild > ${testName}`, (ctx) =>
     callback({
       outDir,
       runBundler: (env) =>

packages/integration-tests-next/fixtures/rolldown/utils.ts

@@ -39,7 +39,7 @@ export function test(url: string, callback: TestCallback) {
         outDir,
         runBundler: (env) =>
           runBundler(
-            `rolldown --config ${testName}${configExt}`,
+            `pnpm rolldown --config ${testName}${configExt}`,
             {
               cwd,
               env: {

packages/integration-tests-next/fixtures/rollup3/utils.ts

@@ -33,7 +33,7 @@ export function test(url: string, callback: TestCallback) {
       outDir,
       runBundler: (env) =>
         runBundler(
-          `rollup --config ${testName}${configExt}`,
+          `pnpm rollup --config ${testName}${configExt}`,
           {
             cwd,
             env: {

packages/integration-tests-next/fixtures/rollup4/utils.ts

@@ -33,7 +33,7 @@ export function test(url: string, callback: TestCallback) {
       outDir,
       runBundler: (env) =>
         runBundler(
-          `rollup --config ${testName}${configExt}`,
+          `pnpm rollup --config ${testName}${configExt}`,
           {
             cwd,
             env: {

packages/integration-tests-next/fixtures/vite4/after-upload-deletion.test.ts

@@ -5,15 +5,15 @@ test(import.meta.url, ({ runBundler, readOutputFiles, runFileInNode }) => {
   runBundler();
   expect(readOutputFiles()).toMatchInlineSnapshot(`
     {
-      "basic.js": "!(function() {
+      "basic.js": "!function() {
       try {
         var e = "undefined" != typeof window ? window : "undefined" != typeof global ? global : "undefined" != typeof globalThis ? globalThis : "undefined" != typeof self ? self : {};
         e.SENTRY_RELEASE = { id: "CURRENT_SHA" };
         var n = new e.Error().stack;
         n && (e._sentryDebugIds = e._sentryDebugIds || {}, e._sentryDebugIds[n] = "00000000-0000-0000-0000-000000000000", e._sentryDebugIdIdentifier = "sentry-dbid-00000000-0000-0000-0000-000000000000");
       } catch (e2) {
       }
-    })();
+    }();
     console.log("hello world");
     //# sourceMappingURL=basic.js.map
     ",

packages/integration-tests-next/fixtures/vite4/application-key.test.ts

@@ -5,22 +5,24 @@ test(import.meta.url, ({ runBundler, readOutputFiles }) => {
   runBundler();
   expect(readOutputFiles()).toMatchInlineSnapshot(`
     {
-      "basic.js": "!(function() {
+      "basic.js": "!function() {
       try {
         var e = "undefined" != typeof window ? window : "undefined" != typeof global ? global : "undefined" != typeof globalThis ? globalThis : "undefined" != typeof self ? self : {};
         e.SENTRY_RELEASE = { id: "CURRENT_SHA" };
-        e._sentryModuleMetadata = e._sentryModuleMetadata || {}, e._sentryModuleMetadata[new e.Error().stack] = (function(e2) {
+        e._sentryModuleMetadata = e._sentryModuleMetadata || {}, e._sentryModuleMetadata[new e.Error().stack] = function(e2) {
           for (var n2 = 1; n2 < arguments.length; n2++) {
             var a = arguments[n2];
-            if (null != a) for (var t in a) a.hasOwnProperty(t) && (e2[t] = a[t]);
+            if (null != a)
+              for (var t in a)
+                a.hasOwnProperty(t) && (e2[t] = a[t]);
           }
           return e2;
-        })({}, e._sentryModuleMetadata[new e.Error().stack], { "_sentryBundlerPluginAppKey:1234567890abcdef": true });
+        }({}, e._sentryModuleMetadata[new e.Error().stack], { "_sentryBundlerPluginAppKey:1234567890abcdef": true });
         var n = new e.Error().stack;
         n && (e._sentryDebugIds = e._sentryDebugIds || {}, e._sentryDebugIds[n] = "00000000-0000-0000-0000-000000000000", e._sentryDebugIdIdentifier = "sentry-dbid-00000000-0000-0000-0000-000000000000");
       } catch (e2) {
       }
-    })();
+    }();
     console.log("hello world");
     ",
     }