PD-278 — Agent QA Unit/Integration Report: dip-tests¶

Module: dip-tests Agent: agent-qa-unit-integration Date: 2026-03-01 Story: PD-278 — NF Z42-013: ajout contractuel de l'etat DIP

1. Fichiers crees¶

2. Fichier modifie (bug fix)¶

3. Couverture des scenarios contractuels¶

3.1 TC-NOM (nominaux)¶

3.2 TC-ERR (erreurs)¶

3.3 TC-INV (invariants)¶

3.4 TC-NR (non-regression)¶

3.5 TC-NEG (negatifs)¶

3.6 TC-FML (formels)¶

4. Details d'implementation par fichier¶

4.1 dissemination.service.spec.ts (54 tests)¶

Pattern: mock QueryRunner avec query.mockImplementation() routant les SQL (SELECT FOR UPDATE, UPDATE, INSERT INTO attestations, INSERT INTO journal).

Helpers: - createDocRow(overrides) — factory pour un document SEALED nominal - createDipDocRow(overrides) — factory pour un document DIP nominal - setupNominalDisseminate(overrides) — setup complet mock pour F1

Couverture specifique: - Transitions interdites: PENDING, EXPIRED, DIP, RESTITUTED -> DIP (4 tests) - Transaction ACID: rollback sur echec attestation, journal, etat invalide (3 tests) - Package atomique: rejet global si 1 document invalide sur N (3 tests) - Temporal: GREATEST dans SQL, rejection si violation d'ordre (2 tests) - Champs immutables: absence de encrypted_metadata/file_hash/ovh_path dans UPDATE (1 test) - Server-only timestamps: NOW() dans SQL, pas de client timestamps (1 test) - Locking: SELECT FOR UPDATE avec ORDER BY ASC pour deadlock prevention (2 tests) - E-404-NOT-FOUND: document inexistant sur disseminate et return (2 tests)

4.2 dissemination.controller.spec.ts (10 tests)¶

Pattern: Test.createTestingModule avec overrideGuard().useValue({ canActivate: () => true }) pour JwtAuthGuard, AuthorizationGuard, DisseminationRateLimitGuard.

Couverture: - Delegation service: arguments corrects (documents, actorId, motif) - DTO mapping: DisseminationResponseDto.fromResult() et DisseminationReturnResponseDto.fromResult() - Actor extraction: user.sub (JWT claim) extrait via @CurrentUser() - Propagation d'erreurs transparente - Champs extra ignores (whitelist DTO)

4.3 dissemination-rate-limit.guard.spec.ts (15 tests)¶

Pattern: jest.mock('ioredis') avec mock Redis INCR/EXPIRE/TTL.

Couverture: - Dual counter: per-minute (dip:rate:{actorId}) + daily quota (dip:quota:{actorId}:{date}) - TTL management: EXPIRE uniquement sur premiere requete (TTL=-2) - Fail-closed 503: Redis ECONNREFUSED et Connection timeout - Boundary values: exact limit (pass) vs limit+1 (reject) pour les deux compteurs - Actor identity: absence de sub, sub vide, fallback user.id - Key format: pattern dip:rate: et dip:quota: verifie

4.4 dissemination-audit-exception.filter.spec.ts (20 tests)¶

Pattern: mock ArgumentsHost + DataSource.createQueryRunner() + AuditLogService.logAsync().

Couverture: - Erreurs auditables: 401, 403-ROLE, 403-RLS, 429, 409-RETENTION-DUE (5 tests) - Erreurs NON auditables: 409-STATE, 409-CONFLICT, 422, 400, 500 (5 tests) - Persistence synchrone: commit AVANT response HTTP (test d'ordonnancement) - Transaction lifecycle: commit sur succes, rollback sur echec audit, release systematique - Document ID extraction: params.id (F2), body.documents[0] (F1), graceful null - Error code extraction: structured error_code, fallback status-based - Non-HttpException: retour 500 sans audit

5. Bug decouvert et corrige¶

TDZ (Temporal Dead Zone) dans dissemination-response.dto.ts¶

Symptome: ReferenceError: Cannot access 'DisseminationAttestationResponseDto' before initialization

Cause racine: DisseminationResponseDto (defini en premier) referencait DisseminationAttestationResponseDto (defini apres) dans son annotation de type de propriete. Les annotations TypeScript avec decorateurs @ApiProperty() sont evaluees au moment de la definition de la classe, causant une erreur TDZ.

Correction: Deplacement de DisseminationAttestationResponseDto avant DisseminationResponseDto dans le fichier. Le fichier contenait deja un commentaire NOTE: Defined before ... to avoid TDZ mais les classes etaient dans le mauvais ordre.

6. Resultat final¶

Test Suites: 4 passed, 4 total
Tests:       99 passed, 99 total
Snapshots:   0 total
Time:        ~1s

Tous les tests passent. Aucun test en echec, aucun test saute.

7. Decisions techniques¶