test: integration tests for useCloudSave — issue #371

[thomasgermain93]

Summary

• Crée src/test/useCloudSave.test.ts avec 23 tests couvrant tous les scénarios demandés dans l'issue
• Mock Supabase via vi.mock('@/integrations/supabase/client') — aucune dépendance réseau
• 246 tests passent (23 nouveaux)

Scénarios couverts

• Utilisateur non authentifié : loadFromCloud retourne null sans appeler Supabase ; saveStatsToCloud n'envoie rien
• Flow nominal load : données cloud retournées correctement, storyProgress/dailyQuests inclus, null pour nouveau joueur, valeurs par défaut pour champs manquants
• Migration universalShards : absent → initialisé à 0 via getDefaultPlayerData() ; présent → conservé
• Résolution de conflit : données cloud prioritaires quand l'utilisateur est authentifié ; tutorialStep présent → conservé ; absent (tutoriel terminé) → undefined
• Erreurs Supabase : player_saves en erreur → exception player_saves_load_failed ; player_heroes en erreur → exception player_heroes_load_failed
• Debounce : 3 appels rapides à saveStatsToCloud → 1 seul upsert après 3 s ; canWriteCloud=false → aucun appel
• saveHeroesToCloud : liste vide → pas d'appel ; canWriteCloud=false → pas d'appel ; cas nominal → upsert déclenché
• removeHeroesFromCloud : liste vide → pas d'appel ; cas nominal → delete().eq().in() appelé
• Fallback migration JSONB : save_data.heroes migré dans player_heroes si table vide
• isSyncing : false par défaut ; retourne à false après load

Test plan

• npm run test → 246 passed, 0 failed
• Branche test/issue-371-cloud-save
• Aucun over-engineering — tests pragmatiques avec mocks agressifs

Closes #371

🤖 Generated with Claude Code