agile-planner-mcp-server/docs/development/TEST-REFACTORING.md

Serveur MCP pour la génération d'artefacts agiles (backlogs, features, user stories) avec IA - compatible Windsurf, Claude et Cursor

# Plan de refactorisation des tests - Agile Planner MCP Server

Ce document liste les tests qui nécessitent une refactorisation pour résoudre les problèmes d'asynchronisme et autres erreurs. Il sert de guide pour améliorer la qualité et la fiabilité des tests.

## Problèmes identifiés

### 1. Asynchronisme et timeouts

Les tests end-to-end échouent souvent à cause d'opérations asynchrones non correctement gérées. Ces erreurs apparaissent comme :
- "Cannot log after tests are done"
- "A worker process has failed to exit gracefully"
- Timeouts lorsque des promesses ne sont pas proprement résolues

### 2. Erreurs de parsing liées aux caractères français

Certains fichiers contiennent des caractères accentués qui causent des erreurs de parsing :
- `tests/backlog-generator.test.js` (ligne 108) : "Unexpected token erreur"
- `tests/integration/cli-integration.test.js` (ligne 54) : "Unexpected token exécution"

## Tests à refactoriser

| Fichier | Problème | Priorité | Assigné à |
|---------|----------|----------|-----------|
| `tests/integration/mcp.e2e.test.js` | Asynchronisme | Haute | - |
| `tests/integration/mcp.ultra-minimal.test.js` | Asynchronisme | Haute | - |
| `tests/integration/isolated/generateFeature-cli.test.js` | Asynchronisme | Moyenne | - |
| `tests/integration/isolated/generateBacklog-cli.test.js` | Asynchronisme | Moyenne | - |
| `tests/backlog-generator.test.js` | Tokens français | Basse | - |
| `tests/integration/cli-integration.test.js` | Tokens français | Basse | - |

## Approche recommandée

### Pour les problèmes d'asynchronisme :

1. **Améliorer la gestion des promesses** :
   ```javascript
   // Avant
   test('should generate a backlog', () => {
     const result = generateBacklog();
     expect(result).toBeDefined();
   });
   
   // Après
   test('should generate a backlog', async () => {
     const result = await generateBacklog();
     expect(result).toBeDefined();
   });
   ```

2. **Utiliser `done` avec timeout explicit** :
   ```javascript
   test('should handle async operations', (done) => {
     someAsyncOperation().then(result => {
       expect(result).toBe('expected');
       done();
     });
   }, 10000); // timeout explicite en ms
   ```

3. **Nettoyer les ressources** :
   ```javascript
   let server;
   
   beforeEach(() => {
     server = startServer();
   });
   
   afterEach(() => {
     server.close(); // Crucial pour éviter les fuites
   });
   ```

### Pour les erreurs de parsing :

1. **Échapper les caractères spéciaux** :
   ```javascript
   // Avant
   expect(result).toContain("L'exécution a échoué");
   
   // Après
   expect(result).toContain("L\\'exécution a échoué");
   // ou
   expect(result).toContain(`L'exécution a échoué`);
   ```

2. **Utiliser des caractères ASCII lorsque possible** :
   ```javascript
   // Avant
   const errorMessage = "L'opération a échoué";
   
   // Après
   const errorMessage = "L'operation a echoue";
   ```

## Tests à ajouter

- [ ] Test unitaire pour `adaptResultForMarkdown`
- [ ] Test unitaire pour `validateFeatureParams`
- [ ] Test d'intégration pour le workflow complet de génération via CLI
- [ ] Test E2E ultra-minimal pour la compatibilité multi-LLM

## Suivi des progrès

Mettez à jour ce document au fur et à mesure que les tests sont refactorisés en ajoutant des check [x] dans les cases à cocher et en mettant à jour la colonne "Assigné à".