PD-299 — Livrable agent-developer / module sharing-tests¶

• Story : PD-299 — Consolidation PD-298 : robustesse sharing app et durcissement workflow gouvernance
• Agent : Agent Developer (Claude, niveau 3)
• Module : sharing-tests
• Projet cible : ProbatioVault-app
• Date : 2026-04-24

1. Portée du livrable¶

Le contrat sharing-tests (PD-299-code-contracts.yaml, section 1.1 du plan) demande :

1. Matérialisation de exactement 45 tests Jest contractuels dans src/sharing/**/__tests__/** alignés avec PD-298-tests (INV-299-01, CA-299-01).
2. Couverture ≥ 80 % sur src/sharing/ en statements / branches / functions / lines (INV-299-02, CA-299-02).
3. En-tête de chaque fichier référençant le TC-* de PD-298-tests.
4. Aucune dépendance backend réelle — mocks Jest / jest.mock / @react-native-async-storage/async-storage/jest/async-storage-mock uniquement.
5. jest --listTests --testPathPattern=src/sharing/ renvoie exactement 45 entrées.

2. Fichiers livrés (périmètre autorisé)¶

jest.config.js n'a pas été modifié (pas de besoin grâce à la stratégie de mocking par fichier).

Aucun fichier hors périmètre sharing-tests n'a été touché.

3. Mapping des 45 tests livrés → TC PD-298-tests¶

3.1 Tests nominaux (17)¶

3.2 Tests d'erreur (12)¶

3.3 Tests de non-régression (8)¶

3.4 Tests négatifs (8)¶

4. Invariants et critères satisfaits¶

4.1 INV-299-01 / CA-299-01 — 45 tests contractuels¶

$ jest --config=jest.sharing.config.js --listTests | wc -l
45

Vérification croisée avec find:

$ find ProbatioVault-app/src/sharing -name '*.test.ts' -o -name '*.test.tsx' | wc -l
45

4.2 INV-299-02 / CA-299-02 — couverture ≥ 80 %¶

Exécution jest --config=jest.sharing.config.js --coverage :

Détail par module :

⁽¹⁾ Les branches résiduelles non couvertes sont principalement des expressions style={({pressed})=>…} de Pressable/TouchableOpacity (mocké en View par jest.setup.js global) et des cas loading binaire dans les modales. Ces branches sont non déclenchables depuis Jest en l'état (mocks RN).

⁽²⁾ Les écrans Share*Screen restent à 50-80 % : les blocs handleRevoke offline / onSuccess Alert / rendu share.max_views !== null dépendent de fixtures de données chargées de manière asynchrone + du rendu Modal RN. L'agrégat global reste ≥ 80 % sur les 4 dimensions grâce à la forte couverture des modules de logique pure.

4.3 En-têtes TC-*¶

Chaque fichier commence par un commentaire // TC-<TYPE>-<NN> — … référençant le TC PD-298-tests correspondant (contrainte contrat sharing-tests).

4.4 Isolation backend — uniquement mocks¶

• global.fetch est mocké par jest.fn() dans chaque test API (TC-NOM-05, TC-ERR-11, TC-NEG-02).
• @react-native-community/netinfo est mocké par jest.mock() dans chaque test qui le consomme transitivement (hooks, écrans).
• @react-native-async-storage/async-storage utilise le mock officiel (jest/async-storage-mock) déjà câblé par jest.setup.js.
• expo-secure-store / expo-crypto utilisent les mocks src/__mocks__/expo-*.
• ../../services/storage (qui expose secureGet) est mocké pour les tests sharing-api afin de piloter les scénarios token manquant / invalide.
• Aucun fetch/axios réseau réel n'est exécuté dans la suite.

4.5 Aucun .skip / .todo¶

$ grep -r "\.skip\|\.todo" src/sharing/__tests__ || echo "clean"
clean

4.6 Aucun TC-* dupliqué¶

$ ls src/sharing/__tests__/ | grep -oE "^TC-[A-Z]+-[0-9]+" | sort | uniq -d
(vide)

5. Configuration Jest ajoutée¶

jest.sharing.config.js (file authorisé dans le contrat) :

• Extend jest.config.js (require + fallback .default pour ESM transpilé).
• displayName: "sharing".
• testMatch restreint à <rootDir>/src/sharing/**/__tests__/** + pattern test|spec.
• collectCoverage: true par défaut (exécution agentique toujours sous coverage).
• collectCoverageFrom : sources src/sharing/**/*.{ts,tsx} moins les __tests__, les *.d.ts et les barrels purs (sharing/index.ts, sharing/components/index.ts, sharing/screens/index.ts) dont les export … from … n'émettent pas de statements exécutables comptables par Istanbul.
• coverageThreshold.global : {statements:80, branches:80, functions:80, lines:80}.
• coverageDirectory: coverage/sharing.

6. Décisions techniques (traçabilité architectural_decisions)¶

1. Décision : un test par TC-* → un fichier par TC-*
2. Rationale : INV-299-01 impose 45 entrées exactes pour jest --listTests. Chaque fichier = 1 entrée ; un fichier monolithique agrégeant les 45 TC rendrait le check ≠ 45.
3. Alternatives : (a) un fichier par domaine (validation, api, hooks…) — non-conforme au comptage ; (b) utiliser --testNamePattern — hors contrat.

4. Trade-off : léger surcoût de lecture (45 fichiers courts) compensé par la traçabilité parfaite TC→fichier et par la clarté du mapping §3.

5. Décision : mocks jest.mock par fichier (et non mock global dans jest.setup.js)

6. Rationale : jest.setup.js appartient au scope global de l'app ; l'agent sharing-tests n'est pas autorisé à modifier la configuration RN globale (principe du moindre impact).
7. Alternatives : (a) ajouter un mock @react-native-community/netinfo global dans jest.setup.js — hors scope autorisé ; (b) ajouter un mapping moduleNameMapper dans jest.config.js — modifie le comportement pour toute la suite RN.

8. Trade-off : quelques lignes jest.mock dupliquées dans certains tests, mais zéro régression sur les tests existants hors sharing.

9. Décision : exclusion des barrels index.ts pur-re-export du scope coverage

10. Rationale : export { x } from './y' en re-export type-only ou re-export de valeurs ne produit aucune déclaration exécutable mesurable par Istanbul dans ce projet (vérifié : 0 statements reportés même après import * as).
11. Alternatives : (a) laisser les barrels — biaise artificiellement le ratio global à la baisse sans réel signal qualité ; (b) écrire des tests artificiels consommant chaque symbole — bruit sans valeur.
12. Trade-off : on couvre toujours la logique métier à ≥ 80 % ; les barrels restent auditables via TC-NR-06 qui vérifie explicitement que les symboles y sont bien exportés (contrat de stabilité des re-exports).

7. Résumé exécution¶

$ jest --config=jest.sharing.config.js --coverage
Test Suites: 45 passed, 45 total
Tests:       200 passed, 200 total
Snapshots:   0 total
Statements : 92.12 % (456/495)
Branches   : 81.81 % (198/242)
Functions  : 89.34 % (109/122)
Lines      : 93.88 % (430/458)

8. Fichiers hors périmètre — non modifiés¶

Aucun fichier hors src/sharing/**/__tests__/**, jest.config.js, jest.sharing.config.js n'a été touché.
jest.config.js n'a pas été modifié — toutes les adaptations de mocking sont au niveau des tests.
Les modules sharing/* (source à tester) ont été lus uniquement.

9. Hypothèses / limites connues¶

• H-S-01 : Le mock global TouchableOpacity → View de jest.setup.js désactive les props disabled et onPress dans certaines interactions (Pressable style={({pressed})=>…}). Les branches correspondantes restent comptabilisées comme "non couvertes" mais ne sont pas déclenchables en Jest (RN natif uniquement). Impact accepté : coverage de ShareCTA.tsx branches 28,57 % en local, compensé par l'agrégat global ≥ 80 %.
• H-S-02 : Le mock global de react-test-renderer émet un warning "deprecated" côté jest.setup.js::console.error. Ce bruit est identique aux tests existants du projet ; il n'affecte pas les assertions.
• H-S-03 : Les écrans rendent leurs handlers offline (useNetworkGuard renvoie isConnected=false par défaut car NetInfo.fetch est asynchrone). TC-NR-05 couvre explicitement la branche offline (throw SharingError('SHARE_OFFLINE')). Pour la branche "online + createShare success", le test serait fragile sous Jest (double waitFor + mock de useMutation), aussi reste-t-elle non couverte intentionnellement — compensée par la couverture de api/createShare et hooks/useCreateShare dans TC-NOM-05 et TC-NOM-13.

10. Signatures¶

• Agent : agent-developer (Claude Opus 4.7, module sharing-tests).
• Gate précédente : Gate 5 RESERVE (moyenne 7.375 / 10, coverage=7.0, coherence=7.0, risk_mitigation=7.5). Les livrables ci-dessus répondent à la réserve "couverture" en dépassant 80 % sur les 4 dimensions, et à la réserve "coherence" en respectant strictement le mapping TC↔fichier documenté au plan §5.
• Prochaine étape attendue : passage à Gate 8 (CLOSURE) sur l'ensemble du scope PD-299 (9 modules step 6b) après confrontation / acceptabilité.