equinor / equinor/fusion-framework

Remove legacy `tests/` vitest projects once migrated to `src/__tests__`

Open
#5,142 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
10
Forks
10
Avg merge
19h 40m
Merged PRs (30d)
150

Description

Several packages still configure `vitest.config.ts` to include a legacy `tests/**` directory instead of the standard `src/__tests__/**` convention, with a `TODO remove after __tests__ are deleted!` marker:

- `packages/modules/http/vitest.config.ts`
- `packages/utils/observable/vitest.config.ts`

Once tests in these packages are migrated to `src/__tests__`, remove the legacy `tests/**` include and this tracking issue can be closed.

Contributor guide

Open the contributing guide

Research direction

Start with packages/modules/http/vitest.config.ts and packages/utils/observable/vitest.config.ts, then inspect each package's legacy tests/** directory and its TODO marker. Migrate those tests to src/__tests__, remove the legacy tests/** include from both Vitest configurations, and verify that the packages' test suites still run successfully.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
testing-qa
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
64/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.