unit-test builder (Vitest): key `fsModuleCache` on the served build output instead of forcing it off
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 20/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- angular, typescript
- Domaine
- build-system, testing
Piste de recherche
Start in the angular:test-in-memory-provider plugin’s configureVitest hook and inspect how buildResultFiles are served and hashed. Check the unit-test builder configuration that forces fsModuleCache off, then trace how the cache key can include the served output and source map hashes. Done means the cache can be enabled without stale results when an imported file changes; the issue reports a fix-ready linked pull request, #34444.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Command
test
Is this a regression?
- Yes, this behavior used to work in the previous version
The previous version in which this bug was not present was
No response
Description
Since f0a74e8f0 (released in 22.2.3), @angular/build:unit-test forces fsModuleCache: false, at the root and on the test project, overriding the runnerConfig. The reason is sound: Vitest keys its file system module cache on the file found on disk, while a test entry point is served from its bundle, which also holds the application code the spec imports. Changing an imported file without touching the spec left the key unchanged, and the cache served a stale bundle.
Vitest provides a hook for exactly this case: a plugin can add its own part to the cache key with defineCacheKeyGenerator, from configureVitest. The angular:test-in-memory-provider plugin already implements configureVitest and holds buildResultFiles, the files its load() serves, each with a content hash. Keying the cache on the served build output (the output file hash plus its source map hash) removes the staleness at its root, so the cache does not need to be turned off.
Since #34337 (also in 22.2.3) the test bundles are byte-identical from one build to the next, so such a key is stable and the cache can be reused across runs and CI pipelines.
Measured on a workspace application with 255 spec files (warm Angular cache, macOS, runs alternated on an idle machine), the builder patched as described:
| builder | Vitest duration | ng test wall time |
|---|---|---|
| 22.2.3 (cache forced off) | 21.4 s, 21.4 s | 36.6 s, 37.9 s |
patched, warm fsModuleCache |
20.0 s, 18.4 s | 35.8 s, 35.8 s |
The gain is modest here, since execution dominates this suite; it grows with the share of transform time. On the same workspace, changing a file imported by a spec (without touching the spec) makes that spec fail as expected, and reverting the change reuses the original cache entry.
I have a fix ready and will open a PR referencing this issue.
Minimal Reproduction
- In a project using
@angular/build:unit-testwith Vitest 5, setfsModuleCache: trueandfsModuleCachePath: './.vitest-cache'in therunnerConfig. - Run
ng test --no-watch. - No
.vitest-cachedirectory is created: the builder overrides the option.
Exception or Error
No response
Your Environment
Angular CLI : 22.2.3
Angular : 22.2.3
Node.js : 24.21.0
Package Manager : npm 11.19.0
Operating System : darwin arm64
@angular/build : 22.2.3
vitest : 5.0.4
Anything else relevant?
Follow-up of #34335 / #34337 (deterministic test discovery) and of f0a74e8f0 (fsModuleCache disabled to avoid stale bundles).
- Langage dominant
- TypeScript
- Étoiles
- 27k
- Forks
- 11.8k
- Merge moyen
- 1 j 5 h
- PR mergées (30 j)
- 244
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de angular/angular-cli
-
Can't use an array of hostnames in --allowedHosts cli parameter in @angular/build:dev-serverOuvertearea: @angular/build gemini-triaged
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
angular/angular-cli#33955 ·
Les mainteneurs répondent en général sous 1 jour
-
area: @angular/cli gemini-triaged
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
angular/angular-cli#33055 · 1 commentaire · 3 réactions ·
Les mainteneurs répondent en général sous 1 jour
-
Angular SSR Unescaped prerender parameters enable RCE through generated manifests and XSS in redirect pagesPeut-être pris @SkyZeroZx l’a pris il y a 151 jours. Ouvertearea: @angular/ssr
Difficulté 4/5 3-5 jours Accessibilité débutants 53/100
angular/angular-cli#34474 ·
Les mainteneurs répondent en général sous 1 jour
-
`ng generate app-shell` and `ng generate module --route` produce broken imports when a component `type` is configuredPeut-être pris @MeAkib l’a pris il y a 1 jour. Ouvertearea: @schematics/angular
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
angular/angular-cli#34449 ·
Les mainteneurs répondent en général sous 1 jour
-
Tailwind CSS v4 classes in new or moved files are not picked up by ng serve until another rebuildPeut-être pris @MeAkib l’a pris il y a 3 jours. Ouvertearea: @angular/build
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
angular/angular-cli#34446 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de angular/angular-cli
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Nimblesite/SharpLsp#468 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Bundled uv is never updated and trusts the compromised eMudhra certificatesPeut-être pris @FabienDanieau l’a pris aujourd’hui. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
-
bg_wait's blocking mode (stopOnAttention: false) is under-documented and effectively undiscoverableOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
nicobailon/pi-subagents#3019 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
db-ux-design-system/core-web#8613 ·
Les mainteneurs répondent en général sous 1 jour