Tests manquants du balayage des .tmp (revue PR #37) #38

New issue

Closed

opened 2026-09-30 00:49:08 +00:00 by maximus · 0 comments

maximus commented

2026-09-30 00:49:08 +00:00

Owner

Constats non bloquants de la revue de la PR #37 (commentaire 17080), à ne pas perdre avec la PR fermée. Deux promesses de sweepStaleTempFiles() ne sont tenues par aucun test.

Symlink (index.js, lstatSync dans sweepStaleTempFiles) : remplacer lstatSync par statSync laisse toute la suite verte, alors que le commentaire promet qu'un symlink posé sur le montage n'est « ni suivi ni supprimé ». Test proposé dans __tests__/hosts.test.js : un lien .thinkpad.4242.1700000000000.tmp vers un fichier vieux de deux heures ; le lien et sa cible survivent au balayage, sans console.error. Prouver le rouge avec la mutation statSync.
Ancre ^ de TEMP_FILE_RE : la retirer laisse la suite verte. Ajouter prefix.thinkpad.4242.1700000000000.tmp aux cas « never touches ». Prouver le rouge en retirant l'ancre.

Hors périmètre (cosmétique, noté pour mémoire) : le seuil d'exactement une heure (<= contre <) n'est pas testé.

Constats non bloquants de la revue de la PR #37 (commentaire 17080), à ne pas perdre avec la PR fermée. Deux promesses de `sweepStaleTempFiles()` ne sont tenues par aucun test. 1. **Symlink** (`index.js`, `lstatSync` dans `sweepStaleTempFiles`) : remplacer `lstatSync` par `statSync` laisse toute la suite verte, alors que le commentaire promet qu'un symlink posé sur le montage n'est « ni suivi ni supprimé ». Test proposé dans `__tests__/hosts.test.js` : un lien `.thinkpad.4242.1700000000000.tmp` vers un fichier vieux de deux heures ; le lien et sa cible survivent au balayage, sans `console.error`. Prouver le rouge avec la mutation `statSync`. 2. **Ancre `^` de `TEMP_FILE_RE`** : la retirer laisse la suite verte. Ajouter `prefix.thinkpad.4242.1700000000000.tmp` aux cas « never touches ». Prouver le rouge en retirant l'ancre. Hors périmètre (cosmétique, noté pour mémoire) : le seuil d'exactement une heure (`<=` contre `<`) n'est pas testé.