Files
SIG-CM2.0/tests/SIGCM2.Application.Tests/Permisos/Assign/AssignPermisosToRolCommandHandlerTests.cs
dmolinari a3f01bc6c9 feat(audit): enchufar audit en handlers de Rol (UDT-010 B8)
4 command handlers del módulo Roles + Permisos ahora auditan:

| Handler                              | Action                 |
|--------------------------------------|------------------------|
| CreateRolCommandHandler              | rol.create             |
| UpdateRolCommandHandler              | rol.update             |
| DeactivateRolCommandHandler          | rol.deactivate         |
| AssignPermisosToRolCommandHandler    | rol.permisos_update    |

Mismo patrón que B7 (using block + post-commit reads outside scope).

Metadata:
- rol.create: after={Codigo, Nombre, Descripcion}
- rol.update: {before, after} diff
- rol.permisos_update: {before, after} con arrays de codigos ordenados

AssignPermisosToRolCommandHandler captura 'before' leyendo
GetByRolCodigoAsync antes del TransactionScope para poder emitir el diff.

4 test classes actualizados con mock de IAuditLogger.

Suite: 378/378 Application.Tests + 141/141 Api.Tests = 519/519 passing.

Refs: sdd/udt-010-auditoria-trazabilidad/{spec#REQ-RM-AUD, design, tasks#B8}
2026-04-16 13:54:47 -03:00

126 lines
5.3 KiB
C#

using NSubstitute;
using SIGCM2.Application.Abstractions.Persistence;
using SIGCM2.Application.Audit;
using SIGCM2.Application.Permisos.Assign;
using SIGCM2.Domain.Entities;
using SIGCM2.Domain.Exceptions;
namespace SIGCM2.Application.Tests.Permisos.Assign;
public class AssignPermisosToRolCommandHandlerTests
{
private readonly IRolRepository _rolRepository = Substitute.For<IRolRepository>();
private readonly IPermisoRepository _permisoRepository = Substitute.For<IPermisoRepository>();
private readonly IRolPermisoRepository _rolPermisoRepository = Substitute.For<IRolPermisoRepository>();
private readonly IAuditLogger _audit = Substitute.For<IAuditLogger>();
private readonly AssignPermisosToRolCommandHandler _handler;
public AssignPermisosToRolCommandHandlerTests()
{
_handler = new AssignPermisosToRolCommandHandler(_rolRepository, _permisoRepository, _rolPermisoRepository, _audit);
}
private static Rol MakeRol(int id, string codigo) =>
new(id, codigo, codigo, null, true, DateTime.UtcNow, null);
private static Permiso MakePermiso(int id, string codigo, string modulo = "ventas") =>
Permiso.ForRead(id, codigo, codigo, null, modulo, true, DateTime.UtcNow);
[Fact]
public async Task Handle_HappyPath_CallsReplaceWithCorrectIds()
{
_rolRepository.GetByCodigoAsync("cajero").Returns(MakeRol(5, "cajero"));
var permisoCrear = MakePermiso(1, "ventas:contado:crear");
var permisoFact = MakePermiso(2, "ventas:contado:facturar");
_permisoRepository.GetByCodigosAsync(Arg.Any<IEnumerable<string>>())
.Returns(new List<Permiso> { permisoCrear, permisoFact });
var codigos = new List<string> { "ventas:contado:crear", "ventas:contado:facturar" };
await _handler.Handle(new AssignPermisosToRolCommand("cajero", codigos));
await _rolPermisoRepository.Received(1).ReplaceForRolAsync(
5,
Arg.Is<IEnumerable<int>>(ids => ids.SequenceEqual(new[] { 1, 2 })));
}
[Fact]
public async Task Handle_RolInexistente_ThrowsRolNotFoundException()
{
_rolRepository.GetByCodigoAsync("fantasma").Returns((Rol?)null);
var ex = await Assert.ThrowsAsync<RolNotFoundException>(
() => _handler.Handle(new AssignPermisosToRolCommand("fantasma", new[] { "ventas:contado:crear" })));
Assert.Equal("fantasma", ex.Codigo);
}
[Fact]
public async Task Handle_RolInexistente_DoesNotCallReplace()
{
_rolRepository.GetByCodigoAsync("fantasma").Returns((Rol?)null);
await Assert.ThrowsAsync<RolNotFoundException>(
() => _handler.Handle(new AssignPermisosToRolCommand("fantasma", new[] { "ventas:contado:crear" })));
await _rolPermisoRepository.DidNotReceive().ReplaceForRolAsync(Arg.Any<int>(), Arg.Any<IEnumerable<int>>());
}
[Fact]
public async Task Handle_PermisoInexistente_ThrowsPermisoNotFoundException()
{
_rolRepository.GetByCodigoAsync("cajero").Returns(MakeRol(5, "cajero"));
// Repo devuelve 0 permisos (ningún código matchea en BD)
_permisoRepository.GetByCodigosAsync(Arg.Any<IEnumerable<string>>())
.Returns(new List<Permiso>());
var ex = await Assert.ThrowsAsync<PermisoNotFoundException>(
() => _handler.Handle(new AssignPermisosToRolCommand("cajero", new[] { "permiso:inexistente" })));
Assert.Equal("permiso:inexistente", ex.Codigo);
}
[Fact]
public async Task Handle_PartialPermisoMatch_ThrowsPermisoNotFoundForMissing()
{
_rolRepository.GetByCodigoAsync("cajero").Returns(MakeRol(5, "cajero"));
// Solo devuelve 1 de 2 — el segundo no existe
_permisoRepository.GetByCodigosAsync(Arg.Any<IEnumerable<string>>())
.Returns(new List<Permiso> { MakePermiso(1, "ventas:contado:crear") });
var ex = await Assert.ThrowsAsync<PermisoNotFoundException>(
() => _handler.Handle(new AssignPermisosToRolCommand("cajero",
new[] { "ventas:contado:crear", "permiso:inexistente" })));
Assert.Equal("permiso:inexistente", ex.Codigo);
}
[Fact]
public async Task Handle_EmptyList_CallsReplaceWithEmptyIds()
{
// Para roles no-admin, lista vacía es válida
_rolRepository.GetByCodigoAsync("reportes").Returns(MakeRol(3, "reportes"));
_permisoRepository.GetByCodigosAsync(Arg.Any<IEnumerable<string>>())
.Returns(new List<Permiso>());
await _handler.Handle(new AssignPermisosToRolCommand("reportes", new List<string>()));
await _rolPermisoRepository.Received(1).ReplaceForRolAsync(
3,
Arg.Is<IEnumerable<int>>(ids => !ids.Any()));
}
[Fact]
public async Task Handle_IdempotentCall_CallsReplaceExactlyOnce()
{
_rolRepository.GetByCodigoAsync("cajero").Returns(MakeRol(5, "cajero"));
_permisoRepository.GetByCodigosAsync(Arg.Any<IEnumerable<string>>())
.Returns(new List<Permiso> { MakePermiso(1, "ventas:contado:crear") });
var cmd = new AssignPermisosToRolCommand("cajero", new[] { "ventas:contado:crear" });
await _handler.Handle(cmd);
await _handler.Handle(cmd);
await _rolPermisoRepository.Received(2).ReplaceForRolAsync(Arg.Any<int>(), Arg.Any<IEnumerable<int>>());
}
}