The GitHub Actions job "Required Checks" on texera.git/feat/vitest-migration has failed. Run started by GitHub user Yicong-Huang (triggered by Yicong-Huang).
Head commit for run: 956cc1250e97bc9dd1a2d50b2246d43d8f4fa925 / Yicong Huang <[email protected]> feat(frontend): scaffold Vitest test runner (Karma → Vitest, step 1) Switches the `test` Architect target from `@angular-builders/custom-webpack:karma` to `@angular/build:unit-test` with the vitest runner. Karma is officially deprecated; this lands the runner switch and the supporting config so follow-up PRs can do the mechanical Jasmine → Vitest spec sweep. Changes: - `frontend/angular.json`: replace test builder; reference `vitest.config.ts` via `runnerConfig`; reuse `gui:build` for app compilation; keep `src/tsconfig.spec.json` as the spec tsconfig. - `frontend/vitest.config.ts` (new): enable `globals: true` so existing Jasmine-style specs continue to use bare `describe/it/expect`. - `frontend/src/vitest-globals.d.ts` (new): triple-slash reference to `vitest/globals` types. Used because the parent tsconfig pins `typeRoots` to `node_modules/@types/`, and Vitest publishes its own types — putting `vitest/globals` directly in `types: [...]` doesn't resolve under that constraint. - `frontend/src/tsconfig.spec.json`: drop `files: ["test.ts"]` (the unit-test builder auto-generates the TestBed init); include the new vitest-globals.d.ts. - `frontend/package.json`: add `[email protected]` and `jsdom`; remove `karma`, `karma-chrome-launcher`, `karma-coverage`, `karma-jasmine`, `jasmine-core`, `@types/jasmine`, `@types/karma-coverage`. - `frontend/karma.conf.js`, `frontend/src/test.ts`: deleted. - `.github/workflows/build.yml`: switch the frontend test invocation from `--code-coverage` (Karma builder flag) to `--coverage --coverage-reporters=lcovonly` (unit-test builder flags). Coverage output path under `frontend/coverage/lcov.info` is unchanged, so the Codecov upload glob still matches. Spec migration is intentionally NOT in this PR — `toBeTrue` / `toBeFalse` / `jasmine.createSpy` / `.and.returnValue(...)` / etc. need a mechanical sweep across 73 spec files (~300 patterns). That sweep lands separately to keep the runner-switch diff reviewable. Tracking: #4861 Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]> Report URL: https://github.com/apache/texera/actions/runs/25283518156 With regards, GitHub Actions via GitBox
