test: New integration tests (#896)

Author: timfish

.github/workflows/checks.yml

@@ -145,13 +145,36 @@ jobs:
         with:
           path: "**/node_modules"
           key: ${{ runner.os }}-${{ hashFiles('**/package.json') }}-${{ hashFiles('**/yarn.lock') }}
-      - name: Use build cache
+      - 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@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@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+      - name: Use dependency cache
         uses: actions/cache@v4
+        id: dependency-cache
         with:
-          path: .nxcache
-          key: build-cache-key-${{ runner.os }}-${{ github.run_id }}
-          restore-keys: |
-            build-cache-key-${{ runner.os }}-
+          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'
@@ -160,7 +183,7 @@ jobs:
         with:
           name: dist-artifacts-${{ github.run_id }}
           path: packages
-      - run: yarn test:integration
+      - run: yarn test:integration-next
 
   test-e2e:
     # We only run E2E tests for non-fork PRs because the E2E tests require secrets to work and they can't be accessed from forks

.vscode/settings.json

@@ -1,3 +1,3 @@
 {
-  "editor.defaultFormatter": "oxc.oxc-vscode",
+  "editor.defaultFormatter": "oxc.oxc-vscode"
 }

nx.json

@@ -1,30 +1,28 @@
 {
   "$schema": "./node_modules/nx/schemas/nx-schema.json",
   "namedInputs": {
+    "default": ["{projectRoot}/**/*", "sharedGlobals"],
     "sharedGlobals": ["{workspaceRoot}/*.js", "{workspaceRoot}/*.json", "{workspaceRoot}/yarn.lock"]
   },
   "targetDefaults": {
     "build": {
-      "inputs": ["sharedGlobals"],
+      "inputs": ["default"],
       "dependsOn": ["^build"],
       "outputs": ["{projectRoot}/dist"],
       "cache": true
     },
     "lint": {
-      "inputs": ["sharedGlobals"],
+      "inputs": ["default"],
       "dependsOn": ["^build", "build"],
-      "outputs": [],
       "cache": true
     },
     "test": {
-      "inputs": ["sharedGlobals"],
-      "outputs": [],
-      "cache": true
+      "inputs": ["default"],
+      "cache": false
     },
     "check:types": {
-      "inputs": ["sharedGlobals"],
-      "dependsOn": ["^build"],
-      "outputs": []
+      "inputs": ["default"],
+      "dependsOn": ["^build"]
     },
     "build:npm": {
       "dependsOn": ["build", "^build"]
@@ -33,5 +31,8 @@
   "cacheDirectory": ".nxcache",
   "tui": {
     "autoExit": true
+  },
+  "nxCloudOptions": {
+    "detectFlakyTasks": false
   }
 }

package.json

@@ -17,9 +17,12 @@
     "packages/rollup-plugin",
     "packages/tsconfigs",
     "packages/vite-plugin",
-    "packages/webpack-plugin"
+    "packages/webpack-plugin",
+    "packages/integration-tests-next",
+    "packages/integration-tests-next/fixtures/rolldown"
   ],
   "scripts": {
+    "postinstall": "patch-package",
     "build": "nx run-many --target=build --all",
     "build:watch": "nx run-many --target=build:watch --all",
     "build:graph": "nx graph",
@@ -29,8 +32,9 @@
     "clean:all": "nx run-many --target=clean:all --all && yarn",
     "test": "nx run-many --target=test --all --exclude=@sentry-internal/bundler-plugin-e2e-tests",
     "test:all": "nx run-many --target=test --all",
-    "test:unit": "nx run-many --target=test --all --exclude=@sentry-internal/integration-tests,@sentry-internal/bundler-plugin-e2e-tests",
+    "test:unit": "nx run-many --target=test --all --exclude=@sentry-internal/integration-tests,@sentry-internal/integration-tests-next,@sentry-internal/bundler-plugin-e2e-tests",
     "test:integration": "nx run @sentry-internal/integration-tests:test",
+    "test:integration-next": "nx run @sentry-internal/integration-tests-next:test",
     "test:e2e": "nx run @sentry-internal/bundler-plugin-e2e-tests:test",
     "lint": "nx run-many --target=lint --all",
     "check:formatting": "oxfmt --check .",
@@ -41,7 +45,8 @@
     "npm-run-all": "^4.1.5",
     "nx": "22.5.2",
     "oxfmt": "^0.33.0",
-    "ts-node": "^10.9.2"
+    "ts-node": "^10.9.2",
+    "patch-package": "^8.0.1"
   },
   "volta": {
     "node": "22.22.0",

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

@@ -58,7 +58,7 @@
     "eslint": "^8.18.0",
     "vitest": "^4.0.0",
     "premove": "^4.0.0",
-    "rolldown": "^1.0.0-rc.4",
+    "rolldown": "^1.0.0-rc.6",
     "ts-node": "^10.9.1",
     "typescript": "^4.7.4"
   },

packages/bundler-plugin-core/package.json

@@ -72,7 +72,7 @@
     "eslint": "^8.18.0",
     "vitest": "^4.0.0",
     "premove": "^4.0.0",
-    "rolldown": "^1.0.0-rc.4",
+    "rolldown": "^1.0.0-rc.6",
     "typescript": "^4.7.4"
   },
   "volta": {

packages/bundler-plugin-core/src/sentry/transports.ts

@@ -119,6 +119,15 @@ export function makeOptionallyEnabledNodeTransport(
         }
 
         if (await shouldSendTelemetry) {
+          if (process.env["SENTRY_TEST_OUT_DIR"]) {
+            // eslint-disable-next-line @typescript-eslint/unbound-method
+            const { join } = await import("node:path");
+            const { appendFileSync } = await import("node:fs");
+            const path = join(process.env["SENTRY_TEST_OUT_DIR"], "sentry-telemetry.json");
+            appendFileSync(path, JSON.stringify(request) + ",\n");
+            return { statusCode: 200 };
+          }
+
           return nodeTransport.send(request);
         }
 

packages/esbuild-plugin/package.json

@@ -57,7 +57,7 @@
     "eslint": "^8.18.0",
     "vitest": "^4.0.0",
     "premove": "^4.0.0",
-    "rolldown": "^1.0.0-rc.4",
+    "rolldown": "^1.0.0-rc.6",
     "ts-node": "^10.9.1",
     "typescript": "^4.7.4"
   },

packages/integration-tests-next/.eslintrc.js

@@ -0,0 +1,16 @@
+/** @type {import('eslint').ESLint.Options} */
+module.exports = {
+  root: true,
+  extends: ["@sentry-internal/eslint-config/base"],
+  ignorePatterns: [".eslintrc.js", "fixtures/*/out", "fixtures/*/src"],
+  parserOptions: {
+    tsconfigRootDir: __dirname,
+    project: ["./tsconfig.json"],
+  },
+  env: {
+    node: true,
+  },
+  rules: {
+    "@typescript-eslint/explicit-function-return-type": "off",
+  },
+};

packages/integration-tests-next/.gitignore

@@ -0,0 +1 @@
+fixtures/*/out/**