test(server-nestjs): migrate Vault client spec to MSW + @msw/data - #2670
test(server-nestjs): migrate Vault client spec to MSW + @msw/data#2670shikanime wants to merge 1 commit into
Conversation
Vault testing utils use @msw/data Collection for faker-seeded secret engines and secrets. Server initialized empty, handlers added in beforeEach. Refs #2655 Co-authored-by: Automata <[email protected]> Signed-off-by: Shikanime Deva <[email protected]> Signed-off-by: William Phetsinorath <[email protected]> Change-Id: I1ed87e505cc5a57922bdc0fa62621c596a6a6964
shikanime
left a comment
shikanime
left a comment
There was a problem hiding this comment.
Verdict : Changements demandés
La migration vers MSW stateful + @msw/data est la bonne direction et le durcissement onUnhandledRequest est appréciable. Mais makeVaultSecret change de contrat alors que quatre specs existantes l'appellent toujours à l'ancienne signature — la branche casse la compile du reste du paquet. Réaligner la signature (ou migrer les consommateurs dans la même PR) suffit à débloquer.
| } | ||
|
|
||
| export function makeVaultSecret(overrides: Partial<VaultSecret> = {}): VaultSecret { | ||
| export function makeVaultSecret<T>(data: T): { data: { data: T, metadata: { created_time: string, destroyed: boolean, version: number } } } { |
There was a problem hiding this comment.
[🔴 Bloquant] makeVaultSecret change de contrat : avant makeVaultSecret(overrides) avec data/metadata optionnels, maintenant makeVaultSecret<T>(data: T). Les specs tronc nexus.service.spec.ts, registry.service.spec.ts, sonarqube.service.spec.ts et vault.service.spec.ts appellent makeVaultSecret({ data: {...} }) — avec la nouvelle signature, { data: ... } serait interprété comme la donnée elle-même. Suggestion : conserver la signature par overrides, ou faire évoluer les quatre specs consommatrices dans cette même PR.
| data: { | ||
| data, | ||
| metadata: { | ||
| created_time: new Date().toISOString(), |
There was a problem hiding this comment.
[🟡 Nit] new Date().toISOString() dans une fixture : préférez faker.date.recent().toISOString() (cohérence faker, pas d'horloge réelle dans les données de test). Idem ligne 68.
| let db: ReturnType<typeof makeVaultDb> | ||
|
|
||
| beforeAll(() => server.listen()) | ||
| beforeAll(() => server.listen({ onUnhandledRequest: 'error' })) |
There was a problem hiding this comment.
[✨ Éloge] onUnhandledRequest: 'error' + handlers réinstallés par test : bon durcissement, chaque test porte son propre état réseau.
1 participant
Migrate Vault client spec from inline MSW handlers to @msw/data-based factories.
Stacking on: PR #2669
Refs #2655