feat(client-portal) : phase 3 — ticket notifications
LST-69 (3.2) phase 3. Wires the existing notification system to client-ticket events (the bell/useNotifications/endpoints already existed). - Notification.relatedTicket (ManyToOne ClientTicketInterface, SET NULL) + additive migration + notification:read group. - NotifierInterface::notify() gains a backward-compatible optional relatedTicket param (existing callers unchanged). - ClientTicketNumberProcessor (POST): notifies all ROLE_ADMIN users (ticket_created), tolerant try/catch after flush. ClientTicketStatusProcessor (PATCH): notifies submittedBy on status change (ticket_status_changed). - Front: notification DTO relatedTicket; NotificationBell navigates to /admin (admin) or /portal (client) on ticket notifications. 180 tests green (178 + 2), nuxt build passes, cs-fixer clean.
This commit is contained in:
@@ -6,8 +6,11 @@ namespace App\Tests\Functional\Module\ClientPortal;
|
||||
|
||||
use App\Module\ClientPortal\Domain\Entity\ClientTicket;
|
||||
use App\Module\ClientPortal\Domain\Enum\ClientTicketStatus;
|
||||
use App\Module\ClientPortal\Domain\Enum\ClientTicketType;
|
||||
use App\Module\Core\Domain\Entity\Notification;
|
||||
use App\Module\Core\Domain\Entity\User;
|
||||
use App\Module\ProjectManagement\Domain\Entity\Project;
|
||||
use DateTimeImmutable;
|
||||
use Doctrine\ORM\EntityManagerInterface;
|
||||
use Symfony\Bundle\FrameworkBundle\KernelBrowser;
|
||||
use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
|
||||
@@ -123,6 +126,84 @@ final class ClientTicketApiTest extends WebTestCase
|
||||
self::assertGreaterThanOrEqual(1, $data['number']);
|
||||
}
|
||||
|
||||
public function testCreatingTicketNotifiesAdmins(): void
|
||||
{
|
||||
$client = self::createClient();
|
||||
$em = self::getContainer()->get(EntityManagerInterface::class);
|
||||
|
||||
$projectIri = $this->sirhProjectIri($em);
|
||||
$this->loginClient($client, 'client-liot');
|
||||
|
||||
$title = 'Notif test '.uniqid('', true);
|
||||
$client->request('POST', '/api/client_tickets', server: [
|
||||
'CONTENT_TYPE' => 'application/ld+json',
|
||||
], content: json_encode([
|
||||
'type' => 'other',
|
||||
'title' => $title,
|
||||
'description' => 'Trigger an admin notification.',
|
||||
'project' => $projectIri,
|
||||
]));
|
||||
|
||||
self::assertResponseStatusCodeSame(201);
|
||||
$data = json_decode($client->getResponse()->getContent(), true);
|
||||
$ticketId = $data['id'];
|
||||
|
||||
$admin = $em->getRepository(User::class)->findOneBy(['username' => 'admin']);
|
||||
self::assertNotNull($admin);
|
||||
|
||||
$notification = $em->getRepository(Notification::class)->findOneBy([
|
||||
'user' => $admin,
|
||||
'type' => 'ticket_created',
|
||||
], ['createdAt' => 'DESC']);
|
||||
|
||||
self::assertInstanceOf(Notification::class, $notification);
|
||||
self::assertNotNull($notification->getRelatedTicket());
|
||||
self::assertSame($ticketId, $notification->getRelatedTicket()->getId());
|
||||
}
|
||||
|
||||
public function testChangingStatusNotifiesSubmitter(): void
|
||||
{
|
||||
$client = self::createClient();
|
||||
$em = self::getContainer()->get(EntityManagerInterface::class);
|
||||
|
||||
// Set up a fresh `new` ticket whose submitter is a known client user,
|
||||
// so we can assert the submitter is the notification recipient.
|
||||
$submitter = $em->getRepository(User::class)->findOneBy(['username' => 'client-liot']);
|
||||
self::assertNotNull($submitter);
|
||||
$project = $em->getRepository(Project::class)->findOneBy(['code' => 'SIRH']);
|
||||
self::assertNotNull($project);
|
||||
|
||||
$ticket = new ClientTicket();
|
||||
$ticket->setType(ClientTicketType::Other);
|
||||
$ticket->setTitle('Status notif '.uniqid('', true));
|
||||
$ticket->setDescription('Will be moved to in_progress.');
|
||||
$ticket->setProject($project);
|
||||
$ticket->setSubmittedBy($submitter);
|
||||
$ticket->setStatus(ClientTicketStatus::New);
|
||||
$ticket->setNumber(9000 + random_int(1, 999));
|
||||
$ticket->setCreatedAt(new DateTimeImmutable());
|
||||
$ticket->setUpdatedAt(new DateTimeImmutable());
|
||||
$em->persist($ticket);
|
||||
$em->flush();
|
||||
$ticketId = $ticket->getId();
|
||||
|
||||
// Admin moves it to in_progress.
|
||||
$this->loginClient($client, 'admin');
|
||||
$client->request('PATCH', '/api/client_tickets/'.$ticketId, server: [
|
||||
'CONTENT_TYPE' => 'application/merge-patch+json',
|
||||
], content: json_encode(['status' => 'in_progress']));
|
||||
self::assertResponseIsSuccessful();
|
||||
|
||||
$notification = $em->getRepository(Notification::class)->findOneBy([
|
||||
'user' => $submitter,
|
||||
'type' => 'ticket_status_changed',
|
||||
], ['createdAt' => 'DESC']);
|
||||
|
||||
self::assertInstanceOf(Notification::class, $notification);
|
||||
self::assertNotNull($notification->getRelatedTicket());
|
||||
self::assertSame($ticketId, $notification->getRelatedTicket()->getId());
|
||||
}
|
||||
|
||||
public function testAdminCannotCreateTicket(): void
|
||||
{
|
||||
$client = self::createClient();
|
||||
|
||||
Reference in New Issue
Block a user