4 Commits

Author SHA1 Message Date
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
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
Serreau Jovann
6fa970e60d refactor: rebrand project to CRM SITECONSEIL (SARL SITECONSEIL)
- Rename all references from E-Cosplay/Ecosplay to SITECONSEIL
- Update entity from Association to SARL SITECONSEIL (Siret: 418664058)
- Update address to 27 rue Le Serurier, 02100 Saint-Quentin
- Update emails: contact@siteconseil.fr, rgpd@siteconseil.fr
- Update hosting from GCP to OVHcloud (Roubaix, Gravelines, Strasbourg, Paris)
- Update legal pages: mentions legales, CGV, RGPD, conformite, hebergement, cookies, CGU
- Add tarifs page with tabs: Site Internet, E-Commerce, Nom de domaine, Esy-Mail, Esy-Mailer, Esy-Tchat, Esy-Meet, Esy-Defender
- Add Discord webhook notification workflow
- Disable deploy and sonarqube workflows
- Update OAuth Keycloak realm to master
- Update logo references to logo_facture.png
- Remove forced image sizing in Liip Imagine filters
- Update SonarQube project key and badge token
- Update tribunal competent to Saint-Quentin
- Move tarif tabs JS to app.js (CSP compliance)

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