optimizely / javascript-sdk / 42643088294

77%

push

github

[FSSDK-13413] Remove the old browser/UMD test setup and Karma leftovers (#1187)

## Summary
Follow-up to #1186. `tests/browser` now runs browser and UMD tests in CI, so this removes the setup it replaced, plus leftovers from the Karma era.

- **Remove the old root browser/UMD setup.**
  - Deleted: `vitest.browser.config.mts`, `vitest.umd.config.mts`, `scripts/run-browser-tests.js`, `scripts/run-umd-tests.js`, `scripts/patch-vitest-viewport.js`, `vitest/umd-plugin.ts`.
  - Root `package.json`: removed the `test-browser*` / `test-umd*` scripts and the `@vitest/browser`, `browserstack-local` and `webdriverio` devDependencies.
  - Root lockfile: 220 entries removed, none added. `npm uninstall` alone would have kept `@vitest/browser`/`webdriverio` and their dependencies as optional peers of the root Vitest, so those were pruned too. Re-resolving moved `@babel/runtime` from 7.28.4 to 7.29.13 (in range).
- **Make `tests/browser` self-contained.** It no longer imports from the root `vitest/` folder, which is gone.
  - The request logger and `public/console-capture.js` moved into it.
  - The console-capture plugin was merged into the local one: it keeps the `/__vitest_console__` endpoint and injects the script via `transformIndexHtml`.
  - The UMD bundle is copied to `tests/browser/public/dist`, which is gitignored via `dist/`.
- **Remove Karma leftovers.**
  - Deleted `lib/index.browser.umdtests.js`. Nothing ran it, and two of its imports no longer exist.
  - Removed the `*.umdtests.js` patterns from `.eslintignore`, both tsconfigs, `.platform-isolation.config.js` and `scripts/platform-utils.js`.
  - Removed the `BROWSER_STACK_*` env var fallbacks and the commented-out `crossbrowser_and_umd_unit_tests` CI job.
- **Update docs and comments.**
  - Comments and the `tests/browser` README describe the current setup instead of comparing it with the removed one.
  - `vitest3-compat.setup.ts` is renamed to `restore-all-mocks.setup.ts`.
  - The root README's testing ... (continued)

1564 of 3059 branches covered (51.13%)

Branch coverage included in aggregate %.

9001 of 10571 relevant lines covered (85.15%)

261.14 hits per line