Backend (7 fixes):
- wallet-service: remove conflicting EF Ignore() calls for mapped backing fields
- fnb-engine: remove KitchenTicket short constructor that set productId=orderItemId
- fnb-engine: replace fire-and-forget Task.Run with direct await for inventory deduction
- TenantMiddleware: implement PostgreSQL RLS SET LOCAL in 4 services (wallet, fnb, inventory, catalog)
- order-service: fix SQL injection pattern in TenantMiddleware with Guid.ToString("D")
- order-service: add ValidateShopAccess() authorization check in SignalR PosHub
- 4 services: register IDbConnection (NpgsqlConnection) in DI for RLS middleware
Frontend (3 fixes):
- PosDataService: return Success=false (not true) when PayOrder response parsing fails
- QrPayment: add _disposed guard to prevent timer race condition after component disposal
- BFF OrderController: add [Authorize] attribute to require JWT for all endpoints
Infrastructure (3 fixes):
- docker-compose: upgrade PostgreSQL 15-alpine to 16-alpine per project spec
- init-databases.sh: add 4 missing marketing service databases (mkt_*)
- Traefik routes: add wallet, catalog, booking routers and /api/v1/stock path
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
109 lines
3.8 KiB
C#
109 lines
3.8 KiB
C#
using FluentAssertions;
|
|
using FnbEngine.API.Application.Commands;
|
|
using FnbEngine.Domain.AggregatesModel.KitchenAggregate;
|
|
using FnbEngine.Domain.SeedWork;
|
|
using Moq;
|
|
using Xunit;
|
|
|
|
namespace FnbEngine.UnitTests.Application.Commands;
|
|
|
|
/// <summary>
|
|
/// EN: Unit tests for UpdateTicketStatusCommandHandler.
|
|
/// VI: Unit tests cho UpdateTicketStatusCommandHandler.
|
|
/// </summary>
|
|
public class UpdateTicketStatusCommandHandlerTests
|
|
{
|
|
private readonly Mock<IKitchenTicketRepository> _repoMock;
|
|
private readonly UpdateTicketStatusCommandHandler _handler;
|
|
|
|
public UpdateTicketStatusCommandHandlerTests()
|
|
{
|
|
_repoMock = new Mock<IKitchenTicketRepository>();
|
|
_repoMock.Setup(r => r.UnitOfWork.SaveEntitiesAsync(It.IsAny<CancellationToken>()))
|
|
.ReturnsAsync(true);
|
|
_handler = new UpdateTicketStatusCommandHandler(_repoMock.Object);
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData("inprogress", "InProgress")]
|
|
[InlineData("InProgress", "InProgress")]
|
|
[InlineData("ready", "Ready")]
|
|
[InlineData("Ready", "Ready")]
|
|
[InlineData("served", "Served")]
|
|
[InlineData("Served", "Served")]
|
|
public async Task Handle_WithValidStatus_ShouldUpdateTicketStatus(string inputStatus, string expectedStatus)
|
|
{
|
|
// Arrange
|
|
var ticketId = Guid.NewGuid();
|
|
var ticket = new KitchenTicket(Guid.NewGuid(), Guid.NewGuid(), Guid.NewGuid(), "Pho Bo");
|
|
_repoMock.Setup(r => r.GetByIdAsync(ticketId, It.IsAny<CancellationToken>()))
|
|
.ReturnsAsync(ticket);
|
|
|
|
var command = new UpdateTicketStatusCommand(ticketId, inputStatus);
|
|
|
|
// Act
|
|
var result = await _handler.Handle(command, CancellationToken.None);
|
|
|
|
// Assert
|
|
result.Should().BeTrue();
|
|
ticket.Status.Should().Be(expectedStatus);
|
|
_repoMock.Verify(r => r.Update(ticket), Times.Once);
|
|
_repoMock.Verify(r => r.UnitOfWork.SaveEntitiesAsync(It.IsAny<CancellationToken>()), Times.Once);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Handle_WithNonExistentTicket_ShouldThrowInvalidOperationException()
|
|
{
|
|
// Arrange
|
|
var ticketId = Guid.NewGuid();
|
|
_repoMock.Setup(r => r.GetByIdAsync(ticketId, It.IsAny<CancellationToken>()))
|
|
.ReturnsAsync((KitchenTicket?)null);
|
|
|
|
var command = new UpdateTicketStatusCommand(ticketId, "Ready");
|
|
|
|
// Act
|
|
var action = () => _handler.Handle(command, CancellationToken.None);
|
|
|
|
// Assert
|
|
await action.Should().ThrowAsync<InvalidOperationException>()
|
|
.WithMessage($"*{ticketId}*not found*");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Handle_WithInvalidStatus_ShouldThrowArgumentException()
|
|
{
|
|
// Arrange
|
|
var ticketId = Guid.NewGuid();
|
|
var ticket = new KitchenTicket(Guid.NewGuid(), Guid.NewGuid(), Guid.NewGuid(), "Pho Bo");
|
|
_repoMock.Setup(r => r.GetByIdAsync(ticketId, It.IsAny<CancellationToken>()))
|
|
.ReturnsAsync(ticket);
|
|
|
|
var command = new UpdateTicketStatusCommand(ticketId, "InvalidStatus");
|
|
|
|
// Act
|
|
var action = () => _handler.Handle(command, CancellationToken.None);
|
|
|
|
// Assert
|
|
await action.Should().ThrowAsync<ArgumentException>()
|
|
.WithMessage("*Invalid status*");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Handle_WithServedStatus_ShouldRaiseDomainEvent()
|
|
{
|
|
// Arrange
|
|
var ticketId = Guid.NewGuid();
|
|
var ticket = new KitchenTicket(Guid.NewGuid(), Guid.NewGuid(), Guid.NewGuid(), "Pho Bo");
|
|
_repoMock.Setup(r => r.GetByIdAsync(ticketId, It.IsAny<CancellationToken>()))
|
|
.ReturnsAsync(ticket);
|
|
|
|
var command = new UpdateTicketStatusCommand(ticketId, "Served");
|
|
|
|
// Act
|
|
await _handler.Handle(command, CancellationToken.None);
|
|
|
|
// Assert
|
|
ticket.DomainEvents.Should().ContainSingle();
|
|
}
|
|
}
|