admin/crm_ecosplay
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: ajout 163 tests unitaires (668->831) avec couverture 73%

Browse Source 
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>
This commit is contained in:
Serreau Jovann
2026-04-07 23:57:42 +02:00
parent 6f5ce58d66
commit 79c55ba0f9
17 changed files with 3093 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

328
tests/Command/PaymentReminderCommandTest.php Normal file
View File

@@ -0,0 +1,328 @@
<?php
namespace App\Tests\Command;
use App\Command\PaymentReminderCommand;
use App\Entity\Advert;
use App\Entity\Customer;
use App\Entity\OrderNumber;
use App\Entity\PaymentReminder;
use App\Service\ActionService;
use App\Service\MailerService;
use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\EntityRepository;
use Doctrine\ORM\Query;
use Doctrine\ORM\QueryBuilder;
use PHPUnit\Framework\TestCase;
use Psr\Log\LoggerInterface;
use Symfony\Component\Console\Tester\CommandTester;
use Twig\Environment;
class PaymentReminderCommandTest extends TestCase
{
private EntityManagerInterface $em;
private MailerService $mailer;
private Environment $twig;
private LoggerInterface $logger;
private ActionService $actionService;
protected function setUp(): void
{
$this->em = $this->createMock(EntityManagerInterface::class);
$this->mailer = $this->createMock(MailerService::class);
$this->twig = $this->createStub(Environment::class);
$this->logger = $this->createMock(LoggerInterface::class);
$this->actionService = $this->createMock(ActionService::class);
}
private function makeCommand(): PaymentReminderCommand
{
return new PaymentReminderCommand(
$this->em,
$this->mailer,
$this->twig,
$this->logger,
$this->actionService,
);
}
private function makeAdvert(string $numOrder, ?string $email = 'client@example.com', int $daysAgo = 20): Advert
{
$orderNumber = new OrderNumber($numOrder);
$customer = $this->createStub(Customer::class);
$customer->method('getEmail')->willReturn($email);
$customer->method('getFullName')->willReturn('Jean Dupont');
$advert = $this->createMock(Advert::class);
$advert->method('getOrderNumber')->willReturn($orderNumber);
$advert->method('getCustomer')->willReturn($customer);
$advert->method('getState')->willReturn(Advert::STATE_SEND);
$updatedAt = new \DateTimeImmutable('-'.$daysAgo.' days');
$advert->method('getUpdatedAt')->willReturn($updatedAt);
$advert->method('getCreatedAt')->willReturn($updatedAt);
return $advert;
}
/**
* Stub the PaymentReminder QueryBuilder so getNextStep returns a specific list of already-done steps.
* Also stubs the Advert repository findBy to return the provided adverts.
*
* @param array<Advert> $adverts
*/
private function stubReminderRepo(array $existingSteps, array $adverts = []): void
{
$query = $this->createStub(Query::class);
$query->method('getSingleColumnResult')->willReturn($existingSteps);
$qb = $this->createStub(QueryBuilder::class);
$qb->method('select')->willReturnSelf();
$qb->method('where')->willReturnSelf();
$qb->method('setParameter')->willReturnSelf();
$qb->method('getQuery')->willReturn($query);
$reminderRepo = $this->createStub(EntityRepository::class);
$reminderRepo->method('createQueryBuilder')->willReturn($qb);
$advertRepo = $this->createStub(EntityRepository::class);
$advertRepo->method('findBy')->willReturn($adverts);
$this->em->method('getRepository')->willReturnCallback(
fn (string $class) => match ($class) {
PaymentReminder::class => $reminderRepo,
default => $advertRepo,
}
);
}
public function testNoAdvertsReturnsSuccess(): void
{
$repo = $this->createStub(EntityRepository::class);
$repo->method('findBy')->willReturn([]);
$this->em->method('getRepository')->with(Advert::class)->willReturn($repo);
$this->mailer->expects($this->never())->method('sendEmail');
$tester = new CommandTester($this->makeCommand());
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('Aucun avis de paiement en attente', $tester->getDisplay());
}
public function testAdvertWithNoCustomerIsSkipped(): void
{
$advert = $this->createStub(Advert::class);
$advert->method('getCustomer')->willReturn(null);
$advertRepo = $this->createStub(EntityRepository::class);
$advertRepo->method('findBy')->willReturn([$advert]);
$this->em->method('getRepository')->willReturn($advertRepo);
$this->mailer->expects($this->never())->method('sendEmail');
$tester = new CommandTester($this->makeCommand());
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('relance(s) envoyee(s)', $tester->getDisplay());
}
public function testAdvertWithNoEmailIsSkipped(): void
{
$advert = $this->makeAdvert('04/2026-00001', null, 20);
$advertRepo = $this->createStub(EntityRepository::class);
$advertRepo->method('findBy')->willReturn([$advert]);
$this->em->method('getRepository')->willReturn($advertRepo);
$this->mailer->expects($this->never())->method('sendEmail');
$tester = new CommandTester($this->makeCommand());
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('relance(s) envoyee(s)', $tester->getDisplay());
}
public function testAdvertWithNoEligibleStepIsSkipped(): void
{
// Only 5 days old — no step requires less than 15 days
$advert = $this->makeAdvert('04/2026-00001', 'client@example.com', 5);
$this->stubReminderRepo([], [$advert]);
$this->mailer->expects($this->never())->method('sendEmail');
$tester = new CommandTester($this->makeCommand());
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('0 relance(s) envoyee(s)', $tester->getDisplay());
}
public function testAdvertWithAllStepsDoneIsSkipped(): void
{
// 30 days old — many steps eligible, but all already done
$advert = $this->makeAdvert('04/2026-00001', 'client@example.com', 30);
// All steps already done
$allSteps = array_keys(PaymentReminder::STEPS_CONFIG);
$this->stubReminderRepo($allSteps, [$advert]);
$this->mailer->expects($this->never())->method('sendEmail');
$tester = new CommandTester($this->makeCommand());
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('0 relance(s) envoyee(s)', $tester->getDisplay());
}
public function testSendsFirstReminderAt15Days(): void
{
// 16 days old, no steps done yet -> should trigger reminder_15
$advert = $this->makeAdvert('04/2026-00001', 'client@example.com', 16);
$this->stubReminderRepo([], [$advert]);
$this->twig->method('render')->willReturn('<p>Email content</p>');
// Expect 2 emails: client + admin notification
$this->mailer->expects($this->exactly(2))->method('sendEmail');
$this->em->method('persist');
$this->em->method('flush');
$tester = new CommandTester($this->makeCommand());
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('1 relance(s) envoyee(s)', $tester->getDisplay());
}
public function testSuspensionStepCallsSuspendCustomer(): void
{
// 30 days old, all steps before suspension already done
$advert = $this->makeAdvert('04/2026-00001', 'client@example.com', 30);
$customer = $advert->getCustomer();
// All steps before STEP_SUSPENSION_1 are done; days=30 >= 29 for suspension_1
$doneSteps = [
PaymentReminder::STEP_REMINDER_15,
PaymentReminder::STEP_WARNING_10,
PaymentReminder::STEP_SUSPENSION_WARNING_5,
PaymentReminder::STEP_FINAL_REMINDER_3,
];
$this->stubReminderRepo($doneSteps, [$advert]);
$this->twig->method('render')->willReturn('<p>Email</p>');
$this->mailer->method('sendEmail');
$this->em->method('persist');
$this->em->method('flush');
$this->actionService->expects($this->once())
->method('suspendCustomer')
->with($customer, $this->stringContains('Impaye'));
$tester = new CommandTester($this->makeCommand());
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('1 relance(s) envoyee(s)', $tester->getDisplay());
}
public function testTerminationWarningStepCallsDisableCustomer(): void
{
// 46 days old -> termination_warning step (>= 45 days)
$advert = $this->makeAdvert('04/2026-00001', 'client@example.com', 46);
$customer = $advert->getCustomer();
// All earlier steps done
$doneSteps = [
PaymentReminder::STEP_REMINDER_15,
PaymentReminder::STEP_WARNING_10,
PaymentReminder::STEP_SUSPENSION_WARNING_5,
PaymentReminder::STEP_FINAL_REMINDER_3,
PaymentReminder::STEP_SUSPENSION_1,
PaymentReminder::STEP_FORMAL_NOTICE,
];
$this->stubReminderRepo($doneSteps, [$advert]);
$this->twig->method('render')->willReturn('<p>Email</p>');
$this->mailer->method('sendEmail');
$this->em->method('persist');
$this->em->method('flush');
$this->actionService->expects($this->once())
->method('disableCustomer')
->with($customer, $this->stringContains('Pre-resiliation'));
$tester = new CommandTester($this->makeCommand());
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
}
public function testTerminationStepCallsMarkForDeletion(): void
{
// 61 days old -> termination_30 step (>= 60 days)
$advert = $this->makeAdvert('04/2026-00001', 'client@example.com', 61);
$customer = $advert->getCustomer();
// All earlier steps done
$doneSteps = [
PaymentReminder::STEP_REMINDER_15,
PaymentReminder::STEP_WARNING_10,
PaymentReminder::STEP_SUSPENSION_WARNING_5,
PaymentReminder::STEP_FINAL_REMINDER_3,
PaymentReminder::STEP_SUSPENSION_1,
PaymentReminder::STEP_FORMAL_NOTICE,
PaymentReminder::STEP_TERMINATION_WARNING,
];
$this->stubReminderRepo($doneSteps, [$advert]);
$this->twig->method('render')->willReturn('<p>Email</p>');
$this->mailer->method('sendEmail');
$this->em->method('persist');
$this->em->expects($this->atLeastOnce())->method('flush');
$this->actionService->expects($this->once())
->method('markForDeletion')
->with($customer, $this->stringContains('Resiliation'));
// setState(STATE_CANCEL) should be called on the advert
$advert->expects($this->once())->method('setState')->with(Advert::STATE_CANCEL);
$tester = new CommandTester($this->makeCommand());
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
}
public function testExceptionInStepIsLoggedAndContinues(): void
{
$advert = $this->makeAdvert('04/2026-00001', 'client@example.com', 20);
$this->stubReminderRepo([], [$advert]);
// Twig throws an exception when rendering
$this->twig->method('render')->willThrowException(new \RuntimeException('Twig error'));
$this->logger->expects($this->once())->method('error');
$tester = new CommandTester($this->makeCommand());
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('0 relance(s) envoyee(s)', $tester->getDisplay());
$this->assertStringContainsString('Erreur', $tester->getDisplay());
}
}

199
tests/Command/ReminderFacturesPrestataireCommandTest.php Normal file
View File

@@ -0,0 +1,199 @@
<?php
namespace App\Tests\Command;
use App\Command\ReminderFacturesPrestataireCommand;
use App\Entity\FacturePrestataire;
use App\Entity\Prestataire;
use App\Service\MailerService;
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\EntityRepository;
use PHPUnit\Framework\TestCase;
use Symfony\Component\Console\Tester\CommandTester;
class ReminderFacturesPrestataireCommandTest extends TestCase
{
private EntityManagerInterface $em;
private MailerService $mailer;
protected function setUp(): void
{
$this->em = $this->createMock(EntityManagerInterface::class);
$this->mailer = $this->createMock(MailerService::class);
}
private function makePrestataire(string $raisonSociale, ?string $siret = null): Prestataire
{
$p = new Prestataire($raisonSociale);
if (null !== $siret) {
$p->setSiret($siret);
}
return $p;
}
private function makeFacturePrestataire(int $year, int $month): FacturePrestataire
{
$f = $this->createStub(FacturePrestataire::class);
$f->method('getYear')->willReturn($year);
$f->method('getMonth')->willReturn($month);
return $f;
}
private function stubRepository(array $prestataires): void
{
$repo = $this->createStub(EntityRepository::class);
$repo->method('findBy')->willReturn($prestataires);
$this->em->method('getRepository')
->with(Prestataire::class)
->willReturn($repo);
}
public function testNoActivePrestatairesReturnsSuccess(): void
{
$this->stubRepository([]);
$this->mailer->expects($this->never())->method('sendEmail');
$command = new ReminderFacturesPrestataireCommand($this->em, $this->mailer);
$tester = new CommandTester($command);
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('Toutes les factures prestataires', $tester->getDisplay());
}
public function testAllPrestatairesHaveFactureForPreviousMonth(): void
{
$now = new \DateTimeImmutable();
$prev = $now->modify('first day of last month');
$year = (int) $prev->format('Y');
$month = (int) $prev->format('n');
$presta = $this->makePrestataire('ACME Corp', '12345678900011');
$facture = $this->makeFacturePrestataire($year, $month);
$presta->getFactures()->add($facture);
$this->stubRepository([$presta]);
$this->mailer->expects($this->never())->method('sendEmail');
$command = new ReminderFacturesPrestataireCommand($this->em, $this->mailer);
$tester = new CommandTester($command);
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('Toutes les factures prestataires', $tester->getDisplay());
}
public function testMissingFacturesSendsReminderEmail(): void
{
$presta1 = $this->makePrestataire('ACME Corp', '12345678900011');
$presta2 = $this->makePrestataire('Beta SARL');
// presta1 has no factures for the previous month, presta2 has none at all
$this->stubRepository([$presta1, $presta2]);
$this->mailer->expects($this->once())
->method('getAdminEmail')
->willReturn('admin@e-cosplay.fr');
$this->mailer->expects($this->once())
->method('sendEmail')
->with(
'admin@e-cosplay.fr',
$this->stringContains('Rappel : factures prestataires'),
$this->stringContains('ACME Corp'),
null,
null,
false,
);
$command = new ReminderFacturesPrestataireCommand($this->em, $this->mailer);
$tester = new CommandTester($command);
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$display = $tester->getDisplay();
$this->assertStringContainsString('2 facture(s) prestataire(s) manquante(s)', $display);
$this->assertStringContainsString('rappel envoye', $display);
}
public function testPrestataireWithFactureForDifferentMonthStillMissing(): void
{
$now = new \DateTimeImmutable();
$prev = $now->modify('first day of last month');
$year = (int) $prev->format('Y');
$month = (int) $prev->format('n');
// Facture for a different month (two months ago)
$wrongMonth = $month === 1 ? 12 : $month - 1;
$wrongYear = $month === 1 ? $year - 1 : $year;
$presta = $this->makePrestataire('Old Corp', '99999999900011');
$oldFacture = $this->makeFacturePrestataire($wrongYear, $wrongMonth);
$presta->getFactures()->add($oldFacture);
$this->stubRepository([$presta]);
$this->mailer->method('getAdminEmail')->willReturn('admin@e-cosplay.fr');
$this->mailer->expects($this->once())->method('sendEmail');
$command = new ReminderFacturesPrestataireCommand($this->em, $this->mailer);
$tester = new CommandTester($command);
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertStringContainsString('1 facture(s) prestataire(s) manquante(s)', $tester->getDisplay());
}
public function testEmailContainsPrestataireDetails(): void
{
$presta = $this->makePrestataire('Dupont & Cie', '11122233300011');
$this->stubRepository([$presta]);
$this->mailer->method('getAdminEmail')->willReturn('admin@e-cosplay.fr');
$capturedHtml = null;
$this->mailer->method('sendEmail')
->willReturnCallback(function (string $to, string $subject, string $html) use (&$capturedHtml) {
$capturedHtml = $html;
});
$command = new ReminderFacturesPrestataireCommand($this->em, $this->mailer);
$tester = new CommandTester($command);
$tester->execute([]);
$this->assertNotNull($capturedHtml);
$this->assertStringContainsString('Dupont & Cie', $capturedHtml);
$this->assertStringContainsString('11122233300011', $capturedHtml);
$this->assertStringContainsString('crm.e-cosplay.fr/admin/prestataires', $capturedHtml);
}
public function testPrestataireWithoutSiretSendsEmailWithoutSiret(): void
{
$presta = $this->makePrestataire('Anonymous Corp');
$this->stubRepository([$presta]);
$this->mailer->method('getAdminEmail')->willReturn('admin@e-cosplay.fr');
$capturedHtml = null;
$this->mailer->method('sendEmail')
->willReturnCallback(function (string $to, string $subject, string $html) use (&$capturedHtml) {
$capturedHtml = $html;
});
$command = new ReminderFacturesPrestataireCommand($this->em, $this->mailer);
$tester = new CommandTester($command);
$tester->execute([]);
$this->assertSame(0, $tester->getStatusCode());
$this->assertNotNull($capturedHtml);
$this->assertStringContainsString('Anonymous Corp', $capturedHtml);
}
}