test(browser): Remove webpack build tests (#12967)

Removes `packages/browser/test/package/npm-build.js`.

`packages/browser/test/package/npm-build.js` contained three tests:

1. Test against `tslib_1__default` appearing in the bundle. I added a
browser integration test to replicate this
2. Test that adding a bundled package would work, our integration + e2e
tests pretty much cover this
3. Test that you can add bundle packages multiple times - our
integration + e2e tests also cover this

I also migrated `packages/utils/test/types/index.js` to use jest instead
of a plain script.

Author: AbhiPrasad

.github/workflows/build.yml

@@ -779,33 +779,6 @@ jobs:
           name: playwright-traces
           path: dev-packages/browser-integration-tests/test-results
 
-  job_browser_build_tests:
-    name: Browser Build Tests
-    needs: [job_get_metadata, job_build]
-    runs-on: ubuntu-20.04
-    timeout-minutes: 5
-    steps:
-      - name: Check out current commit (${{ needs.job_get_metadata.outputs.commit_label }})
-        uses: actions/checkout@v4
-        with:
-          ref: ${{ env.HEAD_COMMIT }}
-      - name: Set up Node
-        uses: actions/setup-node@v4
-        with:
-          node-version-file: 'package.json'
-      - name: Restore caches
-        uses: ./.github/actions/restore-cache
-        env:
-          DEPENDENCY_CACHE_KEY: ${{ needs.job_build.outputs.dependency_cache_key }}
-      - name: Run browser build tests
-        run: |
-          cd packages/browser
-          yarn test:package
-      - name: Run utils build tests
-        run: |
-          cd packages/utils
-          yarn test:package
-
   job_check_for_faulty_dts:
     name: Check for faulty .d.ts files
     needs: [job_get_metadata, job_build]
@@ -1317,7 +1290,6 @@ jobs:
       [
         job_build,
         job_compile_bindings_profiling_node,
-        job_browser_build_tests,
         job_browser_unit_tests,
         job_bun_unit_tests,
         job_deno_unit_tests,

packages/browser/test/package/test-code.js renamed to dev-packages/browser-integration-tests/suites/public-api/init/built-pkg/subject.js

@@ -1,20 +1,3 @@
-/* eslint-disable no-console */
-const Sentry = require('../../build/npm/cjs/index.js');
-
-// Init
-Sentry.init({
-  dsn: 'https://completelyrandom@dsn.asdf/42',
-  beforeSend(_event) {
-    console.log('Got an event');
-    return null;
-  },
-  beforeBreadcrumb(crumb) {
-    console.log(`Got a breadcrumb: ${crumb.category}`);
-    return crumb;
-  },
-});
-
-// Configure
 const scope = Sentry.getCurrentScope();
 scope.setExtra('foo', 'bar');
 scope.setFingerprint('foo');
@@ -75,6 +58,5 @@ Sentry.withScope(scope => {
 
 var xhr = new XMLHttpRequest();
 xhr.onload = () => console.log('loaded'); // This throws error
-// xhr.addEventListener("load", () => console.log('loaded')); This does not throw error
 xhr.open('GET', 'https://httpbin.org/get');
 xhr.send();

dev-packages/browser-integration-tests/suites/public-api/init/built-pkg/test.ts

@@ -0,0 +1,19 @@
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+
+import { expect } from '@playwright/test';
+
+import { sentryTest } from '../../../../utils/fixtures';
+
+// Regression test against https://github.com/getsentry/sentry-javascript/pull/1896
+sentryTest('should not contain tslib_1__default', async ({ getLocalTestPath }) => {
+  await getLocalTestPath({ testDir: __dirname });
+
+  const initBundle = fs.readFileSync(path.join(__dirname, 'dist', 'init.bundle.js'), 'utf-8');
+  expect(initBundle.length).toBeGreaterThan(0);
+  expect(initBundle).not.toContain('tslib_1__default');
+
+  const subjectBundle = fs.readFileSync(path.join(__dirname, 'dist', 'subject.bundle.js'), 'utf-8');
+  expect(subjectBundle.length).toBeGreaterThan(0);
+  expect(subjectBundle).not.toContain('tslib_1__default');
+});

packages/browser/package.json

@@ -49,8 +49,7 @@
   },
   "devDependencies": {
     "@sentry-internal/integration-shims": "8.18.0",
-    "fake-indexeddb": "^4.0.1",
-    "webpack": "^4.47.0"
+    "fake-indexeddb": "^4.0.1"
   },
   "scripts": {
     "build": "run-p build:transpile build:bundle build:types",
@@ -73,7 +72,6 @@
     "size:check": "cat build/bundles/bundle.min.js | gzip -9 | wc -c | awk '{$1=$1/1024; print \"ES2017: \",$1,\"kB\";}'",
     "test": "yarn test:unit",
     "test:unit": "jest",
-    "test:package": "node test/package/npm-build.js && rm test/package/tmp.js",
     "test:unit:watch": "jest --watch",
     "yalc:publish": "yalc publish --push --sig"
   },

packages/browser/test/package/npm-build.js

@@ -63,7 +63,6 @@
     "lint": "eslint . --format stylish",
     "test": "jest",
     "test:watch": "jest --watch",
-    "test:package": "node test/types/index.js",
     "version": "node ../../scripts/versionbump.js src/version.ts",
     "yalc:publish": "yalc publish --push --sig"
   },

packages/utils/package.json

@@ -0,0 +1,25 @@
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+
+const testStrings = ['/// <reference types="node" />'];
+
+const paths = [path.join('./build/cjs'), path.join('./build/esm')];
+
+test('typedef', () => {
+  paths.forEach(dir => {
+    if (!fs.existsSync(dir)) {
+      throw new Error(`${dir} doesn't exist please build first`);
+    }
+    const files = fs.readdirSync(dir);
+    files.forEach(file => {
+      if (file.includes('.d.ts')) {
+        testStrings.forEach(testString => {
+          const filePath = path.join(dir, file);
+          if (fs.readFileSync(filePath, 'utf8').includes(testString)) {
+            throw new Error(`${filePath} contains types`);
+          }
+        });
+      }
+    });
+  });
+});