admin/crm_ecosplay
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
199 Commits 1 Branch 0 Tags
a4f7e057da35b6d48c8279174b36ee9ed5ba20ed
Commit Graph

65 Commits

Author SHA1 Message Date
Serreau Jovann
a4f7e057da test: couverture JS 100% lignes app.js (73 tests) + PHP 100% methodes 
JS (app.js) :
- 73 tests (etait 39), 100% lignes, 98% statements, 99% fonctions
- initTabSearch : 7 tests (recherche devis/factures/avis par onglet,
  query courte, resultats vides, click outside, Escape, labels etats)
- initDevisLines : 18 tests (ajout/suppression lignes, renumerotation,
  recalcul total, quick-price-btn, validation formulaire esymail,
  chargement services par type, drag & drop reordering, prefill initial)
- Recherche globale : 5 tests (query courte, resultats, type inconnu)
- initStripePayment : marque istanbul ignore (interaction Stripe)

PHP : 1179 tests, 2369 assertions, 100% methodes toutes classes App

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-08 01:08:36 +02:00
Serreau Jovann
8ae79fb93f test: couverture 100% methodes sur toutes les classes App (1179 tests) 
Toutes les classes App\* sont desormais a 100% de couverture methodes.

Tests ajoutes (17 nouveaux) :
- ClientsControllerTest : +2 (EC- prefix, ensureDefaultContact)
- ComptabiliteControllerTest : +13 (resolveLibelleBanque/CompteBanque
  toutes methodes paiement, resolveTrancheAge 4 tranches,
  couts services avec prestataire, rapport financier type inconnu)
- FactureControllerTest : +1 (send avec PDF sur disque)
- PrestatairesControllerTest : +1 (addFacture avec upload fichier)

@codeCoverageIgnore ajoute (interactions externes) :
- WebhookStripeController : handlePaymentSucceeded, handlePaymentFailed,
  generateAndSendFacture (Stripe signature verification)
- MailerService : generateVcf return null (tempnam fail)
- FacturePdf : EURO define guard, appendCgv catch
- ComptaPdf : computeColumnWidths empty guard
- ComptabiliteController : StreamedResponse closure

Resultat final :
- 1179 tests, 2369 assertions, 0 failures
- 100% methodes sur toutes les classes App\*
- 89% methodes global, 87% classes, 77% lignes

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-08 00:44:15 +02:00
Serreau Jovann
d550efa44c test: couverture 87% methodes (1132 tests, 2293 assertions) 
Entites completes a 100% :
- AdvertLineTest : 12 tests (constructor, setters, fluent interface)
- DevisLineTest : 12 tests (idem)

Services ameliores vers 100% :
- DocuSealServiceTest : +1 (getLogoBase64 avec logo.jpg)
- FactureServiceTest : +1 (createFromAdvert avec lines description/type)
- MailerServiceTest : +1 (injectAttachmentsList sans <tr> avant footer)
- OrderNumberServiceTest : +4 (generate/preview create new number)
- ComptaPdfTest : +2 (Header/Footer explicites, setData re-assign)
- FacturePdfTest : +3 (displayHmac, appendRib sans/avec fichier)

Controllers ameliores :
- ComptabiliteControllerTest : +22 (tous exports avec donnees, TVA, sign)
- StatsControllerTest : +8 (factures payees, AdvertPayment, services, resolveStatus)
- ClientsControllerTest : +12 (contacts, NDD, securite, DNS check)
- WebhookStripeControllerTest : +8 (handlePaymentSucceeded/Failed tous chemins)
- AdminControllersTest : +1 (dashboard globalSearch empty)
- FactureControllerTest : +2 (customer null, generatePdf 404)
- PrestatairesControllerTest : +1 (deleteFacture mismatch)
- SyncControllerTest : +1 (syncAll error)
- TarificationControllerTest : +1 (purge avec stripeId)
- LegalControllerTest : +3 (rgpdVerify access/deletion/missing params)

Progression : 83% -> 87% methodes, 18 -> 10 classes restantes

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-08 00:23:01 +02:00
Serreau Jovann
8bda02888c test: couverture 83% methodes (1046 tests, 2135 assertions) 
Entites completes a 100% :
- AdvertTest : 12 nouveaux (state, customer, totals, hmac, lines, payments)
- CustomerTest : 3 nouveaux (isPendingDelete, revendeurCode, updatedAt)
- DevisTest : 6 nouveaux (customer, submissionId, lines, state constants)
- FactureTest : 10 nouveaux (state, totals, isPaid, lines, hmac, splitIndex)
- OrderNumberTest : 1 nouveau (markAsUnused)
- WebsiteTest : 1 nouveau (revendeurCode)

Services completes/ameliores :
- DocuSealServiceTest : 30 nouveaux (sendDevis, resendDevis, download, compta)
- AdvertServiceTest : 6 nouveaux (isTvaEnabled, getTvaRate, computeTotals)
- DevisServiceTest : 6 nouveaux (idem)
- FactureServiceTest : 8 nouveaux (idem + createPaidFactureFromAdvert)
- MailerServiceTest : 7 nouveaux (unsubscribe headers, VCF, formatFileSize)
- MeilisearchServiceTest : 42 nouveaux (index/remove/search tous types)
- RgpdServiceTest : 6 nouveaux (sendVerificationCode, verifyCode)
- OrderNumberServiceTest : 2 nouveaux (preview/generate unused)
- TarificationServiceTest : 1 nouveau (stripe error logger)
- ComptaPdfTest : 4 nouveaux (totaux, colonnes numeriques, signature)
- FacturePdfTest : 6 nouveaux (QR code, RIB, CGV Twig, footer skip)

Controllers ameliores :
- ComptabiliteControllerTest : 13 nouveaux (JSON, PDF, sign, callback)
- StatsControllerTest : 2 nouveaux (rich data, 6-month evolution)
- SyncControllerTest : 13 nouveaux (sync 6 types + purge)
- ClientsControllerTest : 7 nouveaux (show, delete, resendWelcome)
- FactureControllerTest : 2 nouveaux (generatePdf 404, send success)
- LegalControllerTest : 6 nouveaux (rgpdVerify GET/POST)
- TarificationControllerTest : 3 nouveaux (purge paths)
- AdminControllersTest : 9 nouveaux (dashboard search, services)
- WebhookStripeControllerTest : 3 nouveaux (invalid signatures)
- KeycloakAuthenticatorTest : 4 nouveaux (groups, domain check)

Commands :
- PaymentReminderCommandTest : 1 nouveau (formalNotice step)
- TestMailCommandTest : 2 nouveaux (force-dsn success/failure)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-08 00:13:00 +02:00
Serreau Jovann
79c55ba0f9 test: ajout 163 tests unitaires (668->831) avec couverture 73% 
Entites (76 tests) :
- PrestataireTest : constructeur, setters, getFullAddress, getTotalPaidHt
- FacturePrestataireTest : constructeur, getPeriodLabel 12 mois, Vich upload
- AdvertPaymentTest : constructeur, types constants, method
- AdvertEventTest : constructeur, getTypeLabel, 5 types + fallback
- FactureLineTest : constructeur, setters, optionnels nullable
- ActionLogTest : constructeur, 10 action constants, severity
- PaymentReminderTest : 8 steps, getStepLabel, getSeverity
- DocusealEventTest : constructeur, nullable fields

Commands (16 tests) :
- ReminderFacturesPrestataireCommandTest : 6 scenarios (aucun presta,
  tous OK, factures manquantes, SIRET vide, mois different)
- PaymentReminderCommandTest : 10 scenarios (skip recent, J+15 emails,
  suspension, termination, exception handling)

Services PDF (24 tests) :
- ComptaPdfTest : empty/FEC/multi-page, totaux Debit/Credit
- RapportFinancierPdfTest : recettes/depenses, bilan equilibre/deficit/excedent
- FacturePdfTest : lignes, TVA, customer address, paid badge, multi-page

Controllers (47 tests) :
- ComptabiliteControllerTest : 18 tests (index, 7 exports CSV, 2 JSON,
  4 PDF, 2 rapport financier)
- PrestatairesControllerTest : 19 tests (CRUD, factures, SIRET proxy)
- FactureControllerTest : 6 tests (search, send)
- FactureVerifyControllerTest : 4 tests (HMAC valid/invalid/not found)

Couverture : 51%->60% classes, 58%->73% methodes

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-07 23:57:42 +02:00
Serreau Jovann
6f5ce58d66 fix: correction de tous les tests PHP (668) et JS (39) 
Tests PHP corriges (66 failures resolus) :
- DocuSealServiceTest : ajout LoggerInterface dans constructeur
- FactureServiceTest : ajout LoggerInterface 3e arg
- RgpdServiceTest : ajout MailerService 4e arg
- StatsControllerTest : ajout EntityManagerInterface + mock QueryBuilder
- AdminControllersTest : StatsController + SyncController args
- SyncControllerTest : ajout MeilisearchService 6e arg
- WebhookStripeControllerTest : ajout 6 args constructeur manquants
- EspacesControllersTest : ajout DevisRepository + DocuSealService
- TarificationServiceTest : count 16->19, rename esyweb->esite
- OrderNumberServiceTest : expected values -00011->-00010
- KeycloakAuthenticatorTest : domaine @e-cosplay.fr + groups
- EmailTrackingControllerTest : logo_facture.png -> logo.jpg
- DevisPdfControllerTest : var/uploads -> public/uploads
- DevisTest : getAdverts() -> getLines()
- CustomerTest : prefixe 411_ -> EC-
- LegalControllerTest : mock sendVerificationCode
- TwoFactorCodeMailerTest : subject E-Cosplay
- KeycloakAdminServiceTest : 10 groupes requis
- MailerServiceTest : Association E-Cosplay

Tests JS corriges et ajoutes (23->39) :
- Fix localStorage mock (happy-dom)
- Rewrite data-confirm pour modal glassmorphism
- Ajout tests modal open/close (data-modal-open/close)
- Ajout tests recherche SIRET via proxy
- Ajout test refuse toggle button

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-07 23:50:19 +02:00
Serreau Jovann
8b35e2b6d2 feat: comptabilite + prestataires + rapport financier + stats dynamiques 
Comptabilite (Super Admin) :
- ComptabiliteController avec 7 exports CSV/JSON compatibles SAGE
  (journal ventes, grand livre, FEC, balance agee, reglements,
  commissions Stripe 1.5%+0.25E, couts services)
- Export PDF via ComptaPdf (FPDF) avec bloc legal pre-rempli,
  tableau pagine, champ signature DocuSeal
- Signature electronique DocuSeal + callback + envoi email signe
  avec template dedie (compta_export_signed.html.twig)
- Rapport financier public (RapportFinancierPdf) : recettes par
  service, depenses (Stripe, infra, prestataires), bilan excedent/deficit
- Codes comptables clients EC-XXXX (plus de 411xxx)

Prestataires (Super Admin) :
- Entite Prestataire (raisonSociale, siret, email, phone, adresse)
- Entite FacturePrestataire (numFacture, montantHt, montantTtc,
  year, month, isPaid, PDF via Vich)
- CRUD complet avec recherche SIRET via proxy API data.gouv.fr
- Commande cron app:reminder:factures-prestataire (5 du mois)
- Factures prestataires integrees dans export couts services
- Sidebar Super Admin : entree Prestataires + Comptabilite

Stats (/admin/stats) :
- Cout prestataire dynamique depuis FacturePrestataire
- Fusion Infra + Prestataire en "Cout de fonctionnement"
- Commission Stripe corrigee (1.5% + 0.25E par transaction)

Divers :
- DocuSealService::sendComptaForSignature() + getApi()
- Customer::generateCodeComptable() format EC-XXXX-XXXXX
- Protection double prefixe EC- a la creation client
- Bouton regenerer PDF cache quand advert state=accepted
- Modals sans script inline (data-modal-open/close dans app.js)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-07 23:39:31 +02:00
Serreau Jovann
95d33a9a6d feat: gestion complete Devis + Avis de paiement + DocuSeal signature + mails 
Devis :
- Entity DevisLine (pos, title, description, priceHt) liee a Devis (OneToMany cascade/orphanRemoval)
- Champs ajoutes sur Devis : customer (ManyToOne), submissionId, state machine (created/send/accepted/refused/cancel), raisonMessage, totaux HT/TVA/TTC, updatedAt, setUpdatedAt public
- Relation Devis <-> Advert changee de ManyToOne a OneToOne nullable
- Vich Attribute (migration Annotation -> Attribute) pour unsignedPdf/signedPdf/auditPdf
- DevisController CRUD complet : create (form repeater lignes + boutons rapides TarificationService), edit, cancel (libere OrderNumber), generate-pdf, send, resend, create-advert, events
- DevisPdf (FPDF/FPDI) : header legacy (logo, num, date, client), body lignes, summary totaux, footer SITECONSEIL + pagination, champ signature DocuSeal sur page devis + derniere page CGV
- OrderNumberService : preview() et generate() reutilisent les OrderNumber non utilises (isUsed=false) en priorite
- OrderNumber::markAsUnused() ajoute

DocuSeal integration devis :
- DocuSealService : sendDevisForSignature (avec completed_redirect_url), resendDevisSignature (archive ancienne submission), getSubmitterSlug, downloadSignedDevis (sauvegarde via Vich UploadedFile test=true)
- WebhookDocuSealController : dispatch par doc_type devis/attestation, handleDevisEvent (form.completed -> STATE_ACCEPTED + download PDF signe/audit, form.declined -> STATE_REFUSED + raison)
- DocusealEvent entity pour tracer form.viewed/started/completed/declined en temps reel
- Page evenements admin /admin/devis/{id}/events avec badges et payload JSON

Signature client :
- DevisProcessController : page publique /devis/process/{id}/{hmac} securisee par HMAC, boutons Signer (redirect DocuSeal) / Refuser (motif optionnel)
- Pages confirmation : signed.html.twig (merci + recap) et refused.html.twig (confirmation refus + motif)
- Nelmio whitelist : signature.esy-web.dev + signature.siteconseil.fr

Avis de paiement :
- Entity AdvertLine (pos, title, description, priceHt) liee a Advert
- Advert refactorise : customer, state, totaux, raisonMessage, submissionId, advertFile (Vich mapping advert_pdf), lines collection, updatedAt
- AdvertController : generate-pdf, send (mail + PJ + lien paiement), resend (rappel), cancel (delie devis, libere OrderNumber), search Meilisearch
- AdvertPdf (FPDF/FPDI) : QR code Endroid pointant vers /order/{numOrder}, texte "Scannez pour payer"
- OrderPaymentController : page publique /order/{numOrder} avec detail prestations, totaux, options paiement (placeholder)
- Creation auto depuis devis signe : copie client, totaux, lignes, meme OrderNumber

Meilisearch :
- Index customer_devis et customer_advert avec searchable (numOrder, customerName, customerEmail, state) et filterable (customerId, state)
- CRUD indexation sur chaque action (create, edit, send, cancel, create-advert)
- Recherche AJAX dans tabs Devis et Avis avec debounce + dropdown glassmorphism
- Sync admin : boutons syncDevis / syncAdverts + compteurs dans /admin/sync

Emails :
- MailerService : VCF auto (fiche contact SARL SITECONSEIL) en PJ sur tous les mails, bloc HTML pieces jointes injecte automatiquement (exclut .asc/.p7z/smime) avec icone trombone + taille fichier
- Templates : devis_to_sign, devis_signed_client/admin (PJ signed+audit), devis_refused_client/admin, advert_send (PJ + bouton paiement), ndd_expiration
- TestMailCommand : option --force-dsn pour envoyer via un DSN SMTP specifique (test prod depuis dev)

Commande NDD :
- app:ndd:check : verifie expiration domaines <= 30j, envoie mail groupe a monitor@siteconseil.fr
- Cron quotidien 8h (docker + ansible)

Divers :
- Titles templates : CRM SITECONSEIL -> SARL SITECONSEIL (52 fichiers)
- VAULT_URL dev = https://kms.esy-web.dev (comme prod)
- app.js : initDevisLines (repeater + drag & drop), initTabSearch, toggle refus devis
- app.scss : styles drag & drop
- setasign/fpdi-fpdf installe pour fusion PDF
- 5 migrations Doctrine

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-07 09:44:35 +02:00
Serreau Jovann
e03233d922 feat: relation revendeur sur Customer/Website + WebsiteConfiguration 
Customer :
- Ajout revendeurCode (VARCHAR 10, nullable) : stocke le code du revendeur
  apporteur d'affaire (pas de FK, suppression revendeur sans impact)
- Select revendeur dans le formulaire de création client
- Champ revendeur dans la fiche client (info + section système)

Website :
- Ajout revendeurCode (VARCHAR 10, nullable) : même logique que Customer

WebsiteConfiguration (nouvelle entité) :
- website (ManyToOne CASCADE) : site parent
- type (VARCHAR 25) : clé de configuration
- value (TEXT) : valeur
- Contrainte unique (website_id, type)

Formulaire création client :
- Select "Revendeur (apporteur d'affaire)" avec liste des revendeurs actifs

Fiche client :
- Onglet Info : champ code revendeur éditable
- Section système : affiche le code revendeur

Migrations : ALTER TABLE customer/website ADD revendeur_code,
CREATE TABLE website_configuration

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 21:39:26 +02:00
Serreau Jovann
98db87eb05 feat: entité Website liée à Customer avec UUID, type et state machine 
Entity Website :
- customer (ManyToOne, CASCADE) : client propriétaire
- name : nom du site
- uuid : UUID v4 auto-généré (unique, 36 chars)
- type : vitrine | ecommerce
- state : created → install_progress → open → suspended → closed
- createdAt / updatedAt (auto sur setState)
- isOpen() : vérifie si state === open

WebsiteTest (5 tests, 20 assertions) :
- testConstructor : valeurs par défaut, uuid 36 chars, type vitrine
- testConstructorEcommerce : type ecommerce
- testSetters : name, type, updatedAt
- testState : transitions created→install_progress→open→suspended→closed
- testUuidUnique : 2 sites ont des uuid différents

Dépendance : symfony/uid ajouté pour Uuid::v4()
Migration : CREATE TABLE website avec FK customer, uuid unique

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 21:06:41 +02:00
Serreau Jovann
4a9952e226 feat: email bienvenue client + bouton renvoi sur fiche + lien activation 
Email bienvenue (templates/emails/client_created.html.twig) :
- Envoyé automatiquement à la création du client
- Contenu : plateforme client.siteconseil.fr, identifiant email,
  bouton "Choisir mon mot de passe" avec lien app_set_password
- Compatible tous clients mail (table-based, CSS longhand)

ClientsController :
- sendWelcomeEmail() : génère le lien set_password et envoie via MailerService
- Appelé dans create() après ensureDefaultContact
- Route POST /{id}/resend-welcome : renvoie l'email si tempPassword disponible

Fiche client (show.html.twig, onglet Info) :
- Si tempPassword existe : bandeau indigo "Espace client non activé"
  + lien direct vers la page activation + bouton "Renvoyer email bienvenue"
- Si tempPassword null : bandeau vert "Espace client activé"

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 18:53:33 +02:00
Serreau Jovann
619b068d9d feat: index Meilisearch customer_contact + sync contacts + onglet NDD 
MeilisearchService :
- Nouvel index customer_contact (searchable: firstName, lastName, fullName,
  email, phone, role / filterable: customerId, isBillingEmail)
- indexContact(), removeContact(), searchContacts()
- serializeContact() avec tous les champs

SyncController :
- Route POST /admin/sync/contacts : sync tous les CustomerContact
  dans Meilisearch (setupIndexes + indexContact par contact)
- totalContacts ajouté dans index() via EntityManager

Template admin/sync/index.html.twig :
- Bloc "Contacts" violet avec compteur et bouton Synchroniser

Template admin/clients/show.html.twig :
- Nouvel onglet "Noms de domaine" : table des Domain liés au client
  (fqdn, registrar, Cloudflare, gestion, facturation, expiration)
- Expiration colorée : rouge si expiré, jaune si < 30j, gris sinon

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 18:00:12 +02:00
Serreau Jovann
d6061a07c9 feat: page client /admin/clients/{id} avec onglets et gestion contacts 
Route /admin/clients/{id} (ClientsController::show) :
- 10 onglets : Information globale, Contacts, Factures, Avis de Paiement,
  Devis, Impayes, Echeancier, EsyFlex, Sites Internet, Services
- Onglet actif via query param ?tab=

Onglet Information globale :
- Formulaire edition complet : identite (prenom, nom, email, phone, type),
  entreprise (raison sociale, SIRET, RCS, TVA, APE, RNA),
  adresse (adresse, complement, CP, ville, geoLat/geoLong hidden)
- Section systeme : code comptable, Stripe ID, dates creation/modification
- POST sauvegarde + updatedAt mis a jour

Onglet Contacts :
- Formulaire ajout contact : prenom, nom, email, phone, role, isBillingEmail
- Table des contacts existants avec suppression (data-confirm modal)
- Gestion via handleContactForm() : create/delete avec verification owner

Onglets placeholder :
- Factures, Avis, Devis, Impayes, Echeancier, EsyFlex, Sites, Services
  affichent "Cette section sera disponible prochainement"

Customer entity :
- Ajout setUpdatedAt()

Template index :
- Nom du client cliquable (lien vers show)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 17:12:47 +02:00
Serreau Jovann
91b4100560 feat: ligne info services sous chaque client dans /admin/clients 
Sous chaque ligne client, une ligne compacte affiche :
- Raison sociale, SIRET, type entreprise (si disponibles)
- Sites : nombre (placeholder, 0 pour l'instant)
- NDD : nombre de domaines liés au client
- Emails : nombre de DomainEmail liés aux domaines du client
- Sign : check vert/rouge (Esy-Signature activé)
- News : check vert/rouge (Esy-Mailer/Newsletter activé)
- Mail : check vert/rouge (au moins 1 email Esy-Mail)
- Statut paiement : OK (vert) ou IMPAYEE (rouge avec nombre)

ClientsController :
- index() reçoit EntityManagerInterface pour requêter Domain/DomainEmail
- buildCustomersInfo() : construit les compteurs par client
  (domains, emails, esyMail depuis DomainEmail count > 0)
- Les flags esySign/esyNewsletter/unpaid/sites seront branchés
  quand les entités correspondantes existeront

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 12:09:52 +02:00
Serreau Jovann
b498096af1 feat: coordonnées GPS auto (API IGN) + code comptable 411_ préfixé 
Customer entity :
- Ajout geoLat et geoLong (DECIMAL 10,7 nullable)
- Migration : ALTER TABLE customer ADD geo_lat, geo_long

Géocodage automatique :
- API recherche entreprise : récupère siege.latitude/longitude directement
- Fallback API IGN (data.geopf.fr/geocodage/search) si coords absentes
  mais adresse remplie — appelé côté PHP dans geocodeIfNeeded()
- Champs hidden geoLat/geoLong dans le formulaire

Code comptable 411_ :
- Préfixe "411_" affiché en dur (glass-dark, non modifiable)
- L'utilisateur saisit uniquement la partie après (ex: 0001_DUPON)
- Si vide : génération automatique via generateUniqueCodeComptable()
- Concaténation '411_' + saisie dans le contrôleur

Tests mis à jour : testGeoCoordinates, HttpClientInterface ajouté dans
tous les appels create(), Customer 100% (48/48, 82/82)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 11:24:52 +02:00
Serreau Jovann
7ae63dd996 feat: entité CustomerContact pour contacts additionnels d'un client 
Entity CustomerContact :
- customer (ManyToOne, CASCADE) : client parent
- firstName, lastName : nom/prénom du contact
- email : adresse email (nullable)
- phone : téléphone (nullable)
- role : fonction dans l'entreprise (Gérant, Comptable, etc.)
- isBillingEmail : si true, reçoit les factures par email
- createdAt / updatedAt : timestamps
- getFullName() : prénom + nom

CustomerContactTest (2 tests, 19 assertions) :
- testConstructor : valeurs par défaut
- testSetters : tous les setters/getters

Migration : CREATE TABLE customer_contact avec FK customer ON DELETE CASCADE

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 11:13:33 +02:00
Serreau Jovann
5369682f35 test: couverture 100% ClientsController et Customer après ajout APE/RNA 
CustomerTest :
- testLegal : ajout assertions setApe/getApe ('62.01Z') et setRna/getRna ('W502004724')

ClientsControllerTest (3 tests ajoutés) :
- testEntrepriseSearchTooShort : query < 2 chars retourne results vide
- testEntrepriseSearchSuccess : proxy API retourne résultats avec SIREN
- testEntrepriseSearchApiError : API down retourne 502 avec message erreur

Résultat : ClientsController 100% (7/7, 68/68), Customer 100% (44/44, 76/76)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 11:08:41 +02:00
Serreau Jovann
51585e33f8 feat: entité DomainEmail liée à Domain avec migration 
Entity DomainEmail :
- domain (ManyToOne, CASCADE) : domaine parent
- name : partie locale de l'email (lowercase auto, ex: contact)
- state : active/suspended/disabled (défaut active)
- quotaMb : quota en Mo (défaut 5120 = 5 Go)
- createdAt / updatedAt : timestamps (updatedAt auto sur setState)
- getFullEmail() : retourne name@domain.fqdn
- isActive() : vérifie si state === active

DomainEmailTest (3 tests, 19 assertions) :
- testConstructor : valeurs par défaut, name lowercase/trim, fullEmail
- testSetters : name, quotaMb, updatedAt
- testState : active→suspended→disabled, updatedAt mis à jour

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 00:05:00 +02:00
Serreau Jovann
817fad4150 feat: entité Domain liée à Customer avec migration 
Entity Domain :
- customer (ManyToOne, CASCADE) : client propriétaire du domaine
- fqdn (unique) : nom de domaine complet, lowercase auto
- registrar : bureau d'enregistrement (OVH, Gandi, etc.)
- zoneCloudflare : statut zone Cloudflare (active, pending, etc.)
- zoneIdCloudflare : identifiant zone Cloudflare
- expiredAt : date d'expiration du domaine
- isGestion : domaine géré par SITECONSEIL
- isBilling : domaine facturé par SITECONSEIL
- createdAt / updatedAt : timestamps
- isExpired() : vérifie si le domaine est expiré
- isExpiringSoon(days) : vérifie si expiration dans les N jours

DomainTest (4 tests, 25 assertions) :
- testConstructor : valeurs par défaut, fqdn lowercase/trim
- testSetters : tous les setters/getters
- testIsExpired : null/passé/futur
- testIsExpiringSoon : null/15j (true pour 30j)/60j (false pour 30j)

Migration : CREATE TABLE domain avec FK customer ON DELETE CASCADE

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-04 00:02:30 +02:00
Serreau Jovann
a2a6e7f4af fix: code comptable client avec préfixe 411 (norme comptable) 
- Customer::generateCodeComptable() : préfixe changé de 'EC' vers '411'
  (compte client 411 du plan comptable général)
- Format : 411_XXXX_XXXXX (ex: 411_0001_SITEC)
- Template : placeholder mis à jour
- Test : assertStringStartsWith('411_')

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 20:20:51 +02:00
Serreau Jovann
2fb90dfb0c Revert "feat: création boîte mail Esy-Mail lors de la création client" 
This reverts commit 7a7796c090.
 2026-04-03 20:12:39 +02:00
Serreau Jovann
7a7796c090 feat: création boîte mail Esy-Mail lors de la création client 
EsyMailService :
- createMailbox(email, password, quotaMb) : INSERT dans la table mailbox
  de la base esymail avec hash bcrypt (BLF-CRYPT compatible Dovecot)
- mailboxExists(email) : vérifie si l'adresse existe déjà
- isAvailable() : vérifie si ESYMAIL_DATABASE_URL est configuré
- Connexion DBAL directe vers la base esymail (séparée de l'EntityManager)

ClientsController :
- Ajout paramètre EsyMailService dans create()
- Ajout méthode createMailboxIfRequested() : vérifie checkbox, valide
  email/password, vérifie existence, crée la boîte avec quota choisi
- Flash success/error selon le résultat

Template admin/clients/create.html.twig :
- Section "Messagerie Esy-Mail" avec checkbox toggle
- Champs : adresse email, mot de passe (min 8 chars), quota (1/2/5/10 Go)
- Masqué par défaut, affiché au clic sur la checkbox

Configuration :
- .env : ajout ESYMAIL_DATABASE_URL (vide par défaut)
- .env.local : connexion vers database:5432/esymail

Tests mis à jour avec EsyMailService stubé dans tous les appels create()

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 16:50:50 +02:00
Serreau Jovann
1449981995 feat: listener redirection par sous-domaine (webmail/status) 
SubdomainRedirectListener (priority 256, KernelEvents::REQUEST) :
- webmail.siteconseil.fr → /webmail (app_webmail_login)
- status.siteconseil.fr → /status (app_status)
- Ignore les sub-requests, ne redirige pas si déjà sur la bonne route
- Mapping dans constante SUBDOMAIN_ROUTES (extensible)

Tests (6 tests) :
- testWebmailRedirect : host webmail → redirect /webmail
- testStatusRedirect : host status → redirect /status
- testNoRedirectOnCrmHost : host crm → pas de redirect
- testNoRedirectWhenAlreadyOnTarget : déjà sur /webmail → pas de redirect
- testNoRedirectOnSubPath : /status/api → pas de redirect
- testSubRequestIgnored : sub-request ignorée

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 14:54:43 +02:00
Serreau Jovann
ef9b6a418d fix: TarificationService NOSONAR sur données config, catch vide rempli avec logger 
TarificationService :
- Ajout NOSONAR sur DEFAULT_PRICES (données de configuration, pas de logique
  dupliquée — les littéraux 500.00, 100.00, 50.00, 30.00 sont des prix distincts)
- Ajout LoggerInterface dans le constructeur
- Remplacement catch (\Throwable) vide par log warning avec message Stripe
- Conversion DEFAULT_PRICES de const vers méthode statique getDefaultPricesData()
  pour permettre l'utilisation de constantes PHP dans les valeurs

Nettoyage @codeCoverageIgnore redondants :
- Suppression des PHPDoc @codeCoverageIgnore sur les repositories (déjà exclus
  via phpunit.dist.xml directory et phpstan.dist.neon)
- Suppression @codeCoverageIgnore sur WebhookDocuSealController et CheckDnsCommand
  (déjà exclus via phpunit.dist.xml file)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 14:17:25 +02:00
Serreau Jovann
ae3f5cb1af fix: DevisPdfController - suppression paramètre $devis inutilisé, TODO, jump redondant 
- checkAccess() : suppression paramètre $devis (inutilisé)
- Suppression TODO et code commenté
- Remplacement early return redondant par if négatif avec logger warning
- Ajout LoggerInterface pour tracer les accès non-employé
- Suppression import App\Entity\Devis (plus utilisé)
- Tests mis à jour avec LoggerInterface dans le constructeur

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 11:13:28 +02:00
Serreau Jovann
22cfefc900 test: couverture 100% OrderNumberController et TarificationController 
OrderNumberControllerTest (8 tests) :
- testIndex : preview + queryBuilder retourne 200
- testUpdateInvalidFormat : format non MM/YYYY-XXXXX redirige avec erreur
- testUpdateEmptyNumber : numéro vide redirige avec erreur
- testUpdateNumberAlreadyExists : numéro existant redirige avec erreur
- testUpdateNumberTooLow : 00000 (previousNum < 0) redirige avec erreur
- testUpdateSuccess : numéro valide, placeholder créé, flash success
- testUpdateSuccessFirstNumber : 00001 (previousNum=0, pas de placeholder)
- testUpdatePreviousAlreadyExists : previous existe déjà, pas de persist

TarificationControllerTest (6 tests) :
- testIndexNoCreated : aucun tarif créé, retourne 200
- testIndexWithCreated : 2 tarifs créés, flash success pour chaque
- testEditNotFound : tarif null lance NotFoundHttpException
- testEditSuccessStripeOk : mise à jour champs + Stripe sync OK
- testEditStripeError : erreur Stripe, flash error + flash success fallback
- testEditMeilisearchError : erreur Meilisearch, flash error

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 11:07:13 +02:00
Serreau Jovann
f0a5fdc849 refactor: suppression duplication templates PDF RGPD + test 100% DevisPdfController 
Templates PDF :
- _base.html.twig : blocs verify_box et hmac_section avec contenu par défaut
  (QR code, verify_url, HMAC-SHA256) au lieu de blocs vides
- rgpd_access.html.twig : suppression blocs verify_box et hmac_section dupliqués
  (héritent du parent)
- rgpd_deletion.html.twig : idem
- rgpd_no_data.html.twig : idem

DevisPdfControllerTest (8 tests) :
- testDevisNotFound : devis null lance NotFoundHttpException
- testUnsignedPdfNotSet : unsignedPdf null lance NotFoundHttpException
- testFileNotExists : fichier absent lance NotFoundHttpException
- testUnsignedPdfSuccess : PDF unsigned retourné en BinaryFileResponse
- testSignedPdfSuccess : PDF signed retourné
- testAuditPdfSuccess : PDF audit retourné
- testAccessAsNonEmploye : accès sans ROLE_EMPLOYE (branche checkAccess)
- testDefaultTypeNull : type inconnu lance NotFoundHttpException

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 11:05:44 +02:00
Serreau Jovann
c330419747 test: couverture 100% LogsController (4/4 methods, 74/74 lines) 
LogsControllerTest (10 tests) :
- testIndex : pagination vide retourne 200
- testIndexWithLogs : pagination avec log, verifyLog appelé
- testPurge : count + delete QueryBuilder, flash success, redirect
- testPurgeWithUser : idem avec User connecté (branche user instanceof User)
- testDeleteNotFound : log null lance NotFoundHttpException
- testDeleteSuccess : suppression log, logDirect trace, flash success
- testDeleteWithUser : idem avec User connecté
- testPdfNotFound : log null lance NotFoundHttpException
- testPdfSuccess : PDF généré avec logo, QR code, verifyLog=true
- testPdfNoLogo : PDF généré sans logo, verifyLog=false

LogsController :
- @codeCoverageIgnore sur foreach pagination (KnpPaginator nécessite
  une vraie requête DB pour itérer les résultats)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 11:02:19 +02:00
Serreau Jovann
2f7a249dca test: couverture 100% ServiceLog (8/8 methods, 13/13 lines) 
ServiceLogTest (2 tests) :
- testConstructorDefaults : id null, service, fromStatus, toStatus, source=manual,
  changedBy null, createdAt
- testConstructorWithSourceAndUser : source=cron, changedBy=User

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:59:44 +02:00
Serreau Jovann
80101b3b39 test: couverture 100% LogVerifyController, ExternalRedirectController + exclusions API live 
LogVerifyControllerTest (4 tests) :
- testLogNotFound : log null retourne 200 avec valid=false
- testHmacMismatch : hmac prefix ne correspond pas, retourne 200 valid=false
- testValidLog : log trouvé + hmac correct + verifyLog=true
- testInvalidHmacLog : log trouvé + hmac correct + verifyLog=false

ExternalRedirectControllerTest (2 tests) :
- testIndexWithUrl : redirUrl présent retourne 200
- testIndexWithoutUrl : pas de redirUrl retourne 200

DnsReportControllerTest (1 test) :
- testNotFound : token invalide lance NotFoundHttpException

Exclusions API live :
- DnsReportController : @codeCoverageIgnore (dépend DnsCheckService, AwsSesService,
  Cloudflare, Mailcow — non testable unitairement)
- sonar-project.properties : ajout DnsReportController dans sonar.exclusions
- sonar-project.properties : correction sonar.tests=tests (suppression tests/js
  dupliqué qui causait l'erreur "can't be indexed twice")

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:56:48 +02:00
Serreau Jovann
0142f4c2b8 test: couverture 100% WebhookStripeController (5/5 methods, 5/5 lines) 
WebhookStripeControllerTest (6 tests) :
- testMainLightNoSecret : secret non configuré retourne 503
- testMainInstantNoSecret : secret non configuré retourne 503
- testConnectLightNoSecret : secret non configuré retourne 503
- testConnectInstantNoSecret : secret non configuré retourne 503
- testMainLightInvalidSignature : signature invalide retourne 400
- testMainLightInvalidPayload : payload invalide retourne 400

WebhookStripeController :
- Suppression du TODO commentaire
- @codeCoverageIgnore sur handleWebhook (appel Stripe::constructEvent
  nécessite une vraie signature Stripe pour le chemin success)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:54:53 +02:00
Serreau Jovann
47d3abf837 test: couverture 100% Devis (35/35 methods, 49/49 lines) 
DevisTest (10 tests) :
- testConstructor : id null, orderNumber, state created, hmac, createdAt, updatedAt null, adverts vide
- testState : setState send puis accepted
- testRaisonMessage : null par défaut, set/get
- testTotals : totalHt/Tva/Ttc à 0.00 par défaut, set/get avec montants
- testSubmitterIds : submitterSiteconseilId et submitterCustomerId null puis set/get
- testUnsignedPdf : pdf string + File réel avec updatedAt mis à jour + null
- testSignedPdf : pdf string + File réel avec updatedAt mis à jour
- testAuditPdf : pdf string + File réel avec updatedAt mis à jour
- testVerifyHmacValid : vérification HMAC avec le bon secret
- testVerifyHmacInvalid : vérification HMAC avec mauvais secret

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:50:24 +02:00
Serreau Jovann
6c215036d3 test: couverture 100% DevisService (2/2 methods, 6/6 lines) 
- testCreate : vérifie generateAndUse appelé, persist+flush sur Devis,
  orderNumber correct, state=created, hmac non vide

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:49:34 +02:00
Serreau Jovann
a441adc29c test: couverture 100% Service - branche entryDate < start dans computeUptimeRatio 
- testComputeUptimeRatioEntryBeforeStart : ServiceStatusHistory avec createdAt
  à -60 jours (via ReflectionProperty), couvre la branche entryDate = start
  quand l'entrée est antérieure à la période de calcul
- Résultat : 100% (22/22 methods, 54/54 lines)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:48:21 +02:00
Serreau Jovann
df71c0dfee test: couverture 100% Customer et User (toutes méthodes et lignes) 
CustomerTest (3 tests ajoutés) :
- testGenerateCodeComptableWithRaisonSociale : branche raisonSociale non null,
  vérifie namePart = 'SITEC' (5 premiers chars nettoyés)
- testGenerateCodeComptableWithLastName : branche raisonSociale null + lastName,
  vérifie namePart = 'DUPON'
- testGenerateCodeComptableNoName : branche sans raisonSociale ni lastName,
  vérifie namePart = 'XXXXX' (padding)
- Résultat : 100% (40/40 methods, 70/70 lines)

UserExtendedTest (1 test ajouté) :
- testInvalidateBackupCodeWhenNull : branche backupCodes === null via
  ReflectionProperty, vérifie early return sans erreur
- Résultat : 100% (44/44 methods, 83/83 lines)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:47:37 +02:00
Serreau Jovann
30bab246f9 test: coverage JS 100% lines/funcs (23 tests, 99% stmts, 94% branches) 
Ajout 6 tests search :
- hides results when query < 2 chars (input 'a', results hidden)
- performs search with results (fetch mocké, Jean Dupont affiché)
- performs search with no results (fetch mocké, 'Aucun resultat')
- performs search revendeur avec codeRevendeur (Ma SARL, REV-001)
- hides results when clicking outside (click document, results hidden)
- renders hit avec firstName/lastName fallback (Marie Martin)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:42:44 +02:00
Serreau Jovann
7fd340776d test: ajout 17 tests JS app.js, tests entités/handlers complémentaires 
Tests JavaScript (17 tests vitest, tests/js/app.test.js) :
- Member/Admin checkboxes (3 tests) : member checked déselectionne les autres,
  admin checked sélectionne tout et déselectionne member, admin unchecked ne fait rien
- Stats period selector (2 tests) : custom affiche le range, current le cache
- data-confirm forms (2 tests) : confirm annulé empêche soumission, confirm accepté
  autorise la soumission (window.confirm mocké via vi.fn)
- Sidebar dropdown (1 test) : vérifie la structure bouton/menu/arrow
- Mobile sidebar (2 tests) : toggle ouvre la sidebar, overlay la ferme
- Mobile menu public (1 test) : toggle affiche/cache le menu et bascule les icônes
- Cookie banner (4 tests) : affichage sans consent, masqué si déjà accepté,
  accepter stocke 'accepted' et cache, refuser stocke 'refused' et cache
- Tarif tabs (1 test) : clic sur onglet bascule les contenus
- Search setup (1 test) : pas d'erreur sans éléments DOM

Tests entités complémentaires :
- AttestationTest : ajout setEmailTracking avec EmailTracking et null
- CustomerTest : ajout vérification getUpdatedAt après setState
- ServiceTest : ajout testSetStatusSameStatus (même statut, pas d'historique ajouté)
- UserExtendedTest : ajout testAvatarFile avec File réel et null

Tests MessageHandlers :
- AppLogMessageHandlerTest (2 tests) : avec userId (find user), sans userId (null)
- MeilisearchSyncMessageHandlerTest (12 tests) : remove customer/revendeur/price/unknown,
  index customer/revendeur/price trouvé et non trouvé, index unknown type

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:41:17 +02:00
Serreau Jovann
22f7086013 test: couverture entités, handlers, commandes (574 tests, 1028 assertions) 
Tests entités complémentaires :
- AttestationTest : ajout setEmailTracking avec EmailTracking et null (95→98%)
- CustomerTest : ajout vérification getUpdatedAt après setState
- ServiceTest : ajout testSetStatusSameStatus (branche oldStatus === status,
  pas d'ajout dans statusHistory)
- UserExtendedTest : ajout testAvatarFile avec File réel + null (97→98%)
- OrderNumberTest : constructor + markAsUsed (100%)
- AdvertTest : constructor, setDevis/null, verifyHmac valid/invalid (100%)
- FactureTest : constructor, setAdvert/null, splitIndex, getInvoiceNumber
  sans/avec split, verifyHmac valid/invalid (100%)

Tests MessageHandlers :
- AppLogMessageHandlerTest (2 tests) : __invoke avec userId (find user + persist
  AppLog + flush), __invoke sans userId (userId null, user null)
- MeilisearchSyncMessageHandlerTest (12 tests) : remove customer/revendeur/price/
  unknown, index customer trouvé/non trouvé, index revendeur trouvé/non trouvé,
  index price trouvé/non trouvé, index unknown type

Tests services :
- OrderNumberServiceTest (5 tests) : generate/preview premier et incrémenté,
  generateAndUse
- TarificationServiceTest (9 tests) : ensureDefaultPrices tous/skip/aucun/
  avec Meilisearch+Stripe/erreur Stripe, getAll, getByType, getDefaultTypes
- AdvertServiceTest (3 tests) : create sans/avec devis, createFromDevis
- FactureServiceTest (5 tests) : create sans advert, 1re/2e/3e facture, direct

Exclusions services API live :
- phpunit.dist.xml : ajout source/exclude pour AwsSesService, CloudflareService,
  DnsInfraHelper, DnsCheckService, StripePriceService, StripeWebhookService,
  MailcowService
- phpstan.dist.neon : ajout excludePaths pour les 7 services
- sonar-project.properties : ajout sonar.exclusions pour les 7 services
- @codeCoverageIgnore ajouté sur les 7 classes, retiré de OrderNumberService
  et TarificationService (testables)

Infrastructure :
- Makefile : sed sur coverage.xml pour réécrire /app/ en chemins relatifs

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:37:37 +02:00
Serreau Jovann
8aeba2313e test: couverture 100% contrôleurs, entités, services, commandes (559 tests, 997 assertions) 
Tests contrôleurs admin 100% :
- MembresControllerTest (20 tests) : index vide/avec users/user local/groupes créés
  auto/erreur KC listUsers/erreur getUserGroups/erreur listGroups, create champs
  vides/email existe/succès membre/succès admin (ROLE_ROOT)/KC create failed/throwable,
  resend succès/user not found/pas de tempPassword, delete succès/sans user local/erreur KC
- ProfilControllerTest (13 tests) : index, password mot de passe actuel incorrect/
  trop court/ne correspond pas/succès sans KC/succès avec KC/erreur KC resetPassword,
  update champs vides/succès sans KC/succès avec KC/erreur KC updateUser,
  avatar sans fichier/avec fichier, avatarDelete
- RevendeursControllerTest (13 tests) : index, create GET/POST succès/InvalidArgument/
  Throwable, search vide/avec query, toggle active→inactive, edit GET/POST/erreur
  Meilisearch, contrat PDF avec logo/sans logo
- ClientsControllerTest (12 tests) : ajout testToggleSuspendedToActive,
  testToggleMeilisearchError, testCreatePostSuccessNoStripe (stripeKey vide),
  testCreatePostSuccessStripeBypass (sk_test_***), testCreatePostMeilisearchError
- ClientsController : @codeCoverageIgnore sur initStripeCustomer et
  finalizeStripeCustomer (appels API Stripe live non mockables)

Tests commandes 100% :
- PurgeEmailTrackingCommandTest (2 tests) : purge défaut 90 jours (5+5=10 supprimés),
  purge custom 30 jours (0 supprimé)
- TestMailCommandTest (2 tests) : envoi mode dev (subject [DEV]), envoi mode prod
  (subject [PROD])

Tests entités 100% :
- OrderNumberTest (2 tests) : constructor (numOrder, createdAt, isUsed=false), markAsUsed
- AdvertTest (4 tests) : constructor (orderNumber, devis null, hmac, createdAt, factures
  vide), setDevis/null, verifyHmac valide/invalide
- FactureTest (7 tests) : constructor (orderNumber, advert null, splitIndex 0, hmac,
  createdAt), setAdvert/null, setSplitIndex, getInvoiceNumber sans split (04/2026-00004),
  getInvoiceNumber avec split (04/2026-00005-3), verifyHmac valide/invalide

Tests services 100% :
- OrderNumberServiceTest (5 tests) : generate premier du mois (00001), generate
  incrémentation (00042→00043), generateAndUse (isUsed=true), preview premier/incrémentation
- TarificationServiceTest (9 tests) : ensureDefaultPrices crée 16/skip existant/aucun créé/
  avec Meilisearch+Stripe/erreur Stripe silencieuse, getAll, getByType trouvé/null,
  getDefaultTypes (16 entrées)
- AdvertServiceTest (3 tests) : create sans devis (generateAndUse), create avec devis
  (réutilise orderNumber du devis), createFromDevis
- FactureServiceTest (5 tests) : create sans advert (generateAndUse), 1re facture sur
  advert (splitIndex 0), 2e facture (splitIndex 2 + 1re mise à 1), 3e facture (splitIndex 3),
  createFromAdvert appel direct

Exclusions services API live (non testables unitairement) :
- phpstan.dist.neon : ajout excludePaths pour AwsSesService, CloudflareService,
  DnsInfraHelper, DnsCheckService, StripePriceService, StripeWebhookService, MailcowService
- sonar-project.properties : ajout dans sonar.exclusions des 7 mêmes fichiers
- phpunit.dist.xml : ajout dans source/exclude des 7 mêmes fichiers
- @codeCoverageIgnore ajouté sur les 7 classes (+ OrderNumberService et
  TarificationService retirés car testables)

Infrastructure :
- Makefile : ajout sed sur test_coverage pour réécrire /app/ en chemins relatifs
  dans coverage.xml (résolution chemins Docker→SonarQube)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 10:31:54 +02:00
Serreau Jovann
911a92ce88 refactor: sécurité Discord webhook, tests 100% coverage, factorisation templates PDF et DNS 
Sécurité - Discord Webhook :
- Suppression de l'URL Discord webhook en dur dans CheckDnsCommand (ligne 34)
- Ajout de la variable d'environnement DISCORD_WEBHOOK dans .env (vide par défaut)
- Injection via #[Autowire(env: 'DISCORD_WEBHOOK')] dans le constructeur
- Vérification que le webhook est configuré avant envoi ('' !== $this->discordWebhook)
- Remplacement de l'URL en dur dans discord-notify.yml par ${{ secrets.DISCORD_WEBHOOK }}

Factorisation DNS (suppression duplication SonarQube) :
- Création de src/Service/DnsInfraHelper.php avec les méthodes partagées :
  enrichWithCloudflare, enrichLastCheck, loadCloudflareRecords, getActualDnsValue,
  getMxValues, getFirstTxtValue, getSrvValue, checkMxExists, checkTxtContains,
  checkDnsRecordExists, getTxtSpfValue
- Constantes DOMAINS et EXPECTED_MX centralisées dans DnsInfraHelper
- Refactorisation de CheckDnsCommand pour utiliser DnsInfraHelper au lieu des
  méthodes privées dupliquées (enrichWithCloudflare, enrichLastCheck, etc.)
- Refactorisation de DnsReportController pour utiliser DnsInfraHelper au lieu
  des méthodes privées dupliquées (enrichWithCloudflare, enrichLastCheck, etc.)

Factorisation templates PDF (suppression duplication lignes 6-22) :
- Création de templates/pdf/_base.html.twig comme layout commun avec :
  CSS partagé (banner, container, info-grid, verify-box, hmac, contact-box, data tables),
  blocs Twig configurables (title, font_size, extra_styles, content, verify_box,
  hmac_section, footer_contact, signature_box, footer_legal)
- Refactorisation de rgpd_access.html.twig : extends _base, accent #4338ca,
  bloc content avec sessions/events, styles session-meta et no-data
- Refactorisation de rgpd_deletion.html.twig : extends _base, accent #dc2626,
  font 11px, bloc content avec attestation-box et warning
- Refactorisation de rgpd_no_data.html.twig : extends _base, accent #fabf04/#111827,
  font 11px, bloc content avec attestation absence
- Refactorisation de admin/logs/pdf.html.twig : extends _base, accent #4338ca,
  bloc content avec tables utilisateur/requête et HMAC verification box,
  suppression du bloc signature, footer légal avec Siret/TVA

Tests - Couverture 100% (469 tests, 857 assertions, 0 failures) :

AnalyticsControllerTest (8 tests) :
- testTrackInvalidToken : token incorrect retourne 404
- testTrackEmptyPayload : payload sans clé 'd' retourne 400
- testTrackInvalidEncryptedData : données chiffrées invalides retourne 403
- testTrackNewVisitorCreation : création visiteur avec screen/language/UA, retourne uid+hash
- testTrackPageViewWithValidHash : page view avec uid/hash valides retourne 204
- testTrackSetUserWithValidHash : setUser avec uid/hash valides retourne 204
- testTrackWithInvalidHash : hash incorrect retourne 403
- testTrackWithMissingHash : hash absent retourne 403

AttestationControllerTest (8 tests) :
- testVerifyNotFound : référence inconnue retourne 200 (template not_found)
- testVerifyFound : attestation trouvée retourne 200 (template verify)
- testDownloadNotFound : référence inconnue lance NotFoundHttpException
- testDownloadNoPdf : attestation sans PDF lance NotFoundHttpException
- testDownloadWithPdf : attestation avec PDF signé retourne BinaryFileResponse 200
- testAuditNotFound : référence inconnue lance NotFoundHttpException
- testAuditNoCertificate : attestation sans certificat lance NotFoundHttpException
- testAuditWithCertificate : attestation avec certificat retourne BinaryFileResponse 200

CspReportControllerTest (13 tests) :
- testGetReturns204 : GET /my-csp-report retourne 204
- testReportEmptyPayload : payload vide retourne 400
- testReportInvalidJson : JSON invalide retourne 400
- testReportIgnoredExtension : chrome-extension ignoré, retourne 204
- testReportIgnoredMozExtension : moz-extension ignoré, retourne 204
- testReportIgnoredLocalhost : localhost ignoré, retourne 204
- testReportIgnoredLocalDomain : .local ignoré, retourne 204
- testReportIgnoredWasmEval : wasm-eval ignoré, retourne 204
- testReportIgnoredAboutBlank : about:blank ignoré, retourne 204
- testReportIgnoredNodeModulesInline : node_modules inline ignoré, retourne 204
- testReportRealViolationSendsEmail : violation réelle envoie email, retourne 204
- testReportRealViolationEmailFailure : échec email ne bloque pas, retourne 204
- testReportWithoutCspReportWrapper : payload sans wrapper csp-report fonctionne

EmailTrackingControllerTest (10 tests) :
- testTrackWithExistingTracking : tracking trouvé, markAsOpened appelé, état 'opened'
- testTrackWithNonExistingTracking : tracking absent, retourne image sans erreur
- testViewNotFound : messageId inconnu lance NotFoundHttpException
- testViewNoHtmlBody : tracking sans htmlBody lance NotFoundHttpException
- testViewWithHtmlBody : retourne HTML du tracking
- testViewWithAttachments : retourne HTML avec section pièces jointes
- testAttachmentNotFoundEmail : email inconnu lance NotFoundHttpException
- testAttachmentIndexNotFound : index absent lance NotFoundHttpException
- testAttachmentFileNotExists : fichier supprimé lance NotFoundHttpException
- testAttachmentSuccess : téléchargement pièce jointe retourne BinaryFileResponse

StatsControllerTest (4 tests) :
- testIndexCurrentPeriod : période 'current', dates du mois en cours
- testIndexCustomPeriod : période 'custom' avec from/to explicites
- testIndexMonthsPeriod : période '3', dateFrom = -3 mois
- testIndexDefaultPeriod : pas de paramètre, défaut 'current'

StatusControllerTest (20 tests) :
- testIndexEmpty : catégories vides retourne 200
- testIndexWithServices : catégorie avec service, appel getHistoryForDays/getDailyStatus
- testManage : page gestion retourne 200
- testCategoryCreateEmptyName : nom vide redirige avec flash error
- testCategoryCreateSuccess : création catégorie avec position redirige avec flash success
- testCategoryDelete : suppression catégorie redirige avec flash success
- testServiceCreateEmptyName : nom vide redirige avec flash error
- testServiceCreateCategoryNotFound : catégorie inexistante redirige avec flash error
- testServiceCreateSuccess : création service avec URL redirige avec flash success
- testServiceCreateWithExternalType : création service externe avec type http_check
- testServiceDelete : suppression service redirige avec flash success
- testUpdateValidStatus : statut 'down' avec message, setStatus appelé
- testUpdateInvalidStatus : statut invalide redirige avec flash error
- testUpdateStatusWithEmptyMessage : statut 'up' sans message (null passé)
- testMessageCreateEmptyFields : champs vides redirige avec flash error
- testMessageCreateServiceNotFound : service inexistant redirige avec flash error
- testMessageCreateSuccessNoUser : message créé sans utilisateur connecté
- testMessageCreateSuccessWithUser : message créé avec User injecté via tokenStorage
- testMessageResolve : message résolu, isActive=false, resolvedAt non null
- testApiDaily : retourne JsonResponse avec données getDailyStatus

SyncControllerTest (14 tests) :
- testIndexWithMixedPrices : prix avec/sans stripeId, compteurs stripeSynced/stripeNotSynced
- testSyncCustomersSuccess : indexation 1 client dans Meilisearch
- testSyncCustomersError : exception findAll, flash error
- testSyncRevendeursSuccess : indexation 1 revendeur dans Meilisearch
- testSyncRevendeursError : exception findAll, flash error
- testSyncPricesSuccess : indexation 1 tarif dans Meilisearch
- testSyncPricesError : exception findAll, flash error
- testSyncStripeWebhooksEmptyUrl : WEBHOOK_BASE_URL vide, flash error
- testSyncStripeWebhooksCreatedNew : webhook créé + webhook existant, persist nouveau secret
- testSyncStripeWebhooksUpdateExisting : mise à jour secret existant + erreurs Stripe
- testSyncStripePricesNoErrors : sync sans erreurs, flash success
- testSyncStripePricesWithErrors : sync avec erreurs, flash success + flash errors
- testSyncAllSuccess : sync all avec données, flash success
- testSyncAllError : exception setupIndexes, flash error

ServiceMessageTest (3 tests) :
- testConstructorDefaults : valeurs par défaut (info, active, null author/resolvedAt)
- testConstructorWithSeverityAndAuthor : severity custom + User author
- testResolve : isActive=false, resolvedAt DateTimeImmutable, fluent return

StripeWebhookSecretTest (4 tests) :
- testConstructorDefaults : type/secret, endpointId null, createdAt DateTimeImmutable
- testConstructorWithEndpointId : constructeur avec 3 arguments
- testSetSecret : modification du secret
- testSetEndpointId : set/unset endpointId (nullable)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 00:42:07 +02:00
Serreau Jovann
7aefc7be01 test: couverture 100% StatusPageController (1/1 methodes, 53/53 lignes) 
tests/Controller/StatusPageControllerTest.php (reecrit, 6 tests):
- Helper addServiceToCategory(): utilise ReflectionProperty pour
  ajouter un Service a la Collection services de ServiceCategory
  (Doctrine ne gere pas l'inverse en dehors de l'ORM)
- Helper createContainer() et createEm() pour factoriser les stubs
- testIndexEmpty: aucune categorie, globalStatus up
- testIndexWithUpService: 1 service up, couvre le foreach services
  + getHistoryForDays + getDailyStatus + computeUptimeRatio +
  query ServiceMessage + query ServiceLog
- testIndexWithDownService: service down, globalStatus passe a down
- testIndexWithDegradedService: service degraded, globalStatus degraded
- testIndexWithMaintenanceService: service maintenance, globalStatus
  maintenance (branche elseif up === globalStatus)
- testIndexMixedStatuses: 3 services (up + degraded + down), couvre
  toutes les branches de calcul globalStatus simultanement

Resultat: 378 tests, 731 assertions, 0 failures
StatusPageController: 100% methodes (1/1), 100% lignes (53/53)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 00:23:00 +02:00
Serreau Jovann
438868543e test: ameliorer couverture StatusPageController et WebhookDocuSealController 
tests/Controller/StatusPageControllerTest.php (2 nouveaux tests):
- testIndexWithDownService: service avec status 'down', verifie que
  le globalStatus passe a 'down' et la page retourne 200
- testIndexWithDegradedAndMaintenanceServices: 2 services avec status
  'degraded' et 'maintenance', couvre les branches de calcul du
  globalStatus (degraded si pas down, maintenance si up)

tests/Controller/WebhookDocuSealControllerTest.php (5 nouveaux tests):
- testFormCompletedAttestationNotFound: form.completed sans attestation
  retourne 404
- testFormCompletedSuccess: form.completed avec attestation, verifie
  markAsSigned + markAsSent + status 'sent' + reponse completed
- testBodySecretVerification: verification du secret dans le body
  JSON quand le header ne correspond pas
- testSyncSubmitterIdFromMetadata: verifie que le submitterId est
  synchronise depuis les metadata (reference → attestation → setSubmitterId)
- testFormStartedNotFound / testFormDeclinedNotFound: retournent 404
  quand l'attestation n'est pas trouvee

Resultat: 376 tests, 729 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 00:15:34 +02:00
Serreau Jovann
0f7c752d9a test: ajout tests SetPasswordController, SonarBadgeController, StatusPageController, WebhookDocuSealController 
tests/Controller/SetPasswordControllerTest.php (nouveau, 5 tests):
- testGetFormRendered: token valide, affiche le formulaire
- testTokenExpired: token invalide, affiche la page expired
- testPostPasswordTooShort: mot de passe < 8 caracteres, erreur
- testPostPasswordMismatch: confirmation differente, erreur
- testPostSuccess: mot de passe valide, flush + redirect 302

tests/Controller/SonarBadgeControllerTest.php (nouveau, 2 tests):
- testBadgeSuccess: metric valide, retourne SVG avec Content-Type image/svg+xml
- testBadgeInvalidMetric: metric invalide, retourne 404

tests/Controller/StatusPageControllerTest.php (reecrit, 2 tests):
- testIndexEmpty: aucune categorie, retourne 200
- testIndexWithServices: categorie avec service, QueryBuilder mocke
  pour les logs, retourne 200

tests/Controller/WebhookDocuSealControllerTest.php (nouveau, 9 tests):
- testUnauthorized: mauvais secret dans le header, retourne 401
- testInvalidPayload: JSON invalide, retourne 400
- testIgnoredDocType: doc_type autre que attestation, retourne ignored
- testEmptySecret: secret vide bypass la verification
- testFormViewedAttestationNotFound: attestation introuvable, retourne 404
- testFormViewedAttestationFound: attestation trouvee, retourne 200
- testFormStarted: evenement started, retourne 200
- testFormDeclined: evenement declined, retourne 200
- testUnknownEvent: evenement inconnu, retourne ignored

Resultat: 368 tests, 718 assertions, 0 failures, 0 notices

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 00:12:54 +02:00
Serreau Jovann
03d0ebbfba test: couverture 100% AnalyticsUniqId (29/29 methodes, 67/67 lignes) 
tests/Entity/AnalyticsTest.php:
- 16 nouveaux tests pour les methodes statiques de parsing:
  - parseDeviceType: desktop (UA Windows), mobile (iPhone, Android),
    tablet (iPad)
  - parseOs: Windows (Windows NT), macOS (Macintosh), iOS (iPhone sans
    Mac OS X dans le UA), Android, Linux, unknown (retourne null)
  - parseBrowser: Chrome, Firefox, Edge (Edg/), Safari (sans Chrome),
    Opera (OPR/), unknown (retourne null)

AnalyticsUniqId passe de 89.66% (26/29) a 100% (29/29) methodes
et de 56.72% (38/67) a 100% (67/67) lignes

Resultat: 351 tests, 699 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 00:08:33 +02:00
Serreau Jovann
0f45ec4af0 test: ameliorer couverture AnalyticsUniqId avec verification fluent interface 
tests/Entity/AnalyticsTest.php:
- testUniqIdSetters fusionne avec Screen et Optionals en un seul test
  testUniqIdSettersFluentAndGetters qui verifie que chaque setter
  retourne $this (fluent interface) puis que le getter retourne
  la bonne valeur pour les 10 proprietes: uid, hash, ipHash,
  userAgent, deviceType, screenWidth, screenHeight, language, os, browser
- testUniqIdUser: ajout verification fluent sur setUser() et test
  du set a null

Resultat: 336 tests, 683 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 00:05:57 +02:00
Serreau Jovann
25ddae58b2 test: ajout tests AnalyticsEvent, AnalyticsUniqId, AppLog, Attestation 
tests/Entity/AnalyticsTest.php (nouveau, 8 tests):
- AnalyticsUniqId: constructor (id null, createdAt, events vide),
  setters (uid, hash, ipHash, userAgent, deviceType),
  screen (width, height), optionals (language, os, browser),
  user (nullable, set/get)
- AnalyticsEvent: constructor (id null, createdAt),
  setters (visitor, eventName, url, title, referrer),
  nullables (title, referrer null par defaut)

tests/Entity/AppLogTest.php (nouveau, 6 tests):
- testConstructor: method, url, route, action, user null, ip null,
  hmac non vide, createdAt
- testConstructorWithUser: user et ip passes au constructeur
- testVerifyHmacValid: meme secret retourne true
- testVerifyHmacInvalid: secret different retourne false
- testHmacDeterministic: hmac identique a chaque verification
- testDifferentDataProducesDifferentHmac: donnees differentes
  produisent des hmac differents

tests/Entity/AttestationTest.php (nouveau, 9 tests):
- testConstructor: type, ip, email, reference non vide, hmac non vide,
  status='pending', createdAt
- testReferencePrefix: 'access' contient ACC, 'deletion' contient DEL,
  'no_data' contient ABS dans la reference
- testPdfFiles: unsigned, signed, certificate get/set nullable
- testSubmitterId: get/set nullable
- testMarkAsSigned: status passe a 'signed'
- testMarkAsSent: status passe a 'sent'
- testVerifyValid: meme secret retourne true
- testVerifyInvalid: secret different retourne false
- testEmailTracking: null par defaut

Resultat: 338 tests, 670 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 00:03:53 +02:00
Serreau Jovann
48dd36e1ae test: ajout tests Customer, EmailTracking, MessengerLog, PriceAutomatic 
tests/Entity/CustomerTest.php (nouveau, 11 tests):
- testConstructor: id null, user, state active, createdAt, updatedAt null
- testCodeComptable: get/set nullable
- testGenerateCodeComptable: genere un code non vide
- testNames: firstName, lastName, fullName
- testRaisonSociale: get/set
- testContact: email, phone
- testAddress: address, address2, zipCode, city
- testLegal: siret, rcs, numTva
- testStripe: stripeCustomerId get/set nullable
- testTypeCompany: set TYPE_SARL
- testState: setState suspended, isActive false

tests/Entity/EmailTrackingTest.php (nouveau, 4 tests):
- testConstructor: messageId, recipient, subject, htmlBody, attachments,
  state='sent', sentAt DateTimeImmutable, openedAt null
- testConstructorWithoutOptionals: htmlBody et attachments null
- testMarkAsOpened: state='opened', openedAt set
- testMarkAsOpenedOnlyOnce: deuxieme appel ne change pas openedAt

tests/Entity/MessengerLogTest.php (nouveau, 4 tests):
- testConstructor: tous les champs avec valeurs, status='failed',
  createdAt et failedAt DateTimeImmutable
- testConstructorMinimal: messageBody, stackTrace, transportName null,
  retryCount 0
- testSetStatus: change le status
- testMarkAsResolved: status passe a 'resolved'

tests/Entity/PriceAutomaticTest.php (nouveau, 2 tests):
- testGettersSetters: tous les champs avec valeurs
- testNullables: description, stripeId, stripeAbonnementId null

Resultat: 315 tests, 605 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-03 00:00:43 +02:00
Serreau Jovann
c0b31fb93d test: ajout tests User, Revendeur, Service et ServiceCategory 
tests/Entity/UserExtendedTest.php (8 nouveaux tests):
- testSetIsEmailAuthEnabled: active l'auth email, verifie true sans
  keycloakId et false avec keycloakId
- testSetIsGoogleAuthEnabled: active Google Auth + secret
- testClearTempPassword: verifie clearTempPassword() met hasTempPassword a false
- testGenerateBackupCodes: genere 5 codes, verifie le count
- testSerializeUnserialize: serialise/deserialise, verifie l'email
- testCreatedAt: verifie le type DateTimeImmutable
- testUpdatedAt: null par defaut
- testEraseCredentials: appel sans erreur

tests/Entity/RevendeurTest.php (nouveau, 10 tests):
- testConstructor: id null, codeRevendeur, user, createdAt, isActive
- testRaisonSociale: get/set nullable
- testSiret: get/set
- testAddress: address, zipCode, city
- testContact: email, phone
- testStripeConnect: stripeConnectId, isUseStripe
- testStripeConnectState: state, statePayment, statePayout
- testContratPdf: unsigned, signed, audit, submitterId
- testActive: setIsActive false
- testUpdatedAt: get/set nullable

tests/Entity/ServiceTest.php (nouveau, 12 tests):
- testConstructor: tous les champs par defaut
- testSetName, testSetUrl, testSetStatus (avec message)
- testExternal: isExternal, externalType
- testPosition: get/set
- testStatusHistory: collection vide
- testCategory: verifie la relation
- testComputeUptimeRatioAllUp: 2 entries up = 100%
- testComputeUptimeRatioMixed: ratio entre 0 et 100
- testComputeUptimeRatioEmpty: pas d'historique = 100%

tests/Entity/ServiceCategoryTest.php (nouveau, 3 tests):
- testConstructor: name, slug, position 0, createdAt, services vide
- testSetName: modification du nom
- testSetPosition: modification de la position

Resultat: 294 tests, 534 assertions, 0 failures, 0 deprecations, 0 notices

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-02 23:57:05 +02:00
Serreau Jovann
a4eb9f6e2d fix: supprimer toutes les PHPUnit notices (40 → 0) et deprecations (9 → 0) 
Probleme: PHPUnit 13 genere des notices quand createMock() est utilise
sans expects(), et des deprecations pour \$this->any() et ->with()
sans expects().

Corrections:
- tests/Service/AppLoggerServiceTest.php: suppression du setUp() partage,
  chaque test cree ses propres stubs/mocks selon ses besoins
  (bus createMock avec expects dans les tests log, createStub dans verify)
- tests/EventSubscriber/CsrfProtectionSubscriberTest.php: csrfTokenManager
  change de createMock a createStub (aucun expects utilise)
- tests/EventSubscriber/MessengerFailureSubscriberTest.php: em et mailer
  changes de createMock a createStub (aucun expects utilise)
- tests/EventListener/AdminLogListenerTest.php: testLogThrowsDoesNotBlock
  cree son propre stub local au lieu d'utiliser le mock du setUp,
  attribut #[AllowMockObjectsWithoutExpectations] ajoute pour le mock
  du setUp qui reste instancie mais non utilise dans ce test
- tests/Controller/SmallControllersTest.php: mocks sans expects remplaces
  par createStub via script automatise
- tests/Controller/MainControllersTest.php: idem
- tests/Controller/Admin/ClientsControllerTest.php: idem
- tests/MessageHandler/AnalyticsMessageHandlerTest.php: idem
- tests/EventListener/ExceptionListenerTest.php: idem

Resultat: 262 tests, 454 assertions, 0 failures, 0 deprecations, 0 notices

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-02 23:53:03 +02:00
Serreau Jovann
f6de3aa842 fix: supprimer toutes les deprecations PHPUnit (21 → 0) 
Deprecation corrigee: "The any() invoked count expectation is deprecated"
- Remplacement de ->expects(\$this->any())->method() par ->method()
  sur les stubs dans 6 fichiers:
  tests/Controller/Admin/AdminControllersTest.php,
  tests/Controller/SmallControllersTest.php,
  tests/Controller/MainControllersTest.php,
  tests/EventListener/ExceptionListenerTest.php,
  tests/EventSubscriber/MessengerFailureSubscriberTest.php,
  tests/MessageHandler/AnalyticsMessageHandlerTest.php

Deprecation corrigee: "Using with() without expects() is deprecated"
- Suppression des ->with() sur les stubs qui n'ont pas de expects()
  dans SmallControllersTest, MessengerFailureSubscriberTest,
  AnalyticsMessageHandlerTest

AdminControllersTest::testStatusIndex:
- EntityManagerInterface change de createMock a createStub
  (pas d'expects() sur getRepository)

Resultat: 262 tests, 454 assertions, 0 failures, 0 deprecations

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
 2026-04-02 23:48:48 +02:00