timothy
1e34e00e31
POST /api/blocks with a nonexistent blockGroupId returned an unhandled DbUpdateException (HTTP 500) because CreateBlockHandler never checked the referenced block group before inserting, letting the SQLite FK constraint raise. Add a BaseError validation (422), matching the "does not exist" precedent already used elsewhere. Found via live E2E testing.
69 lines
2.6 KiB
C#
69 lines
2.6 KiB
C#
using ErsatzTV.Core;
|
|
using ErsatzTV.Core.Domain.Scheduling;
|
|
using ErsatzTV.Infrastructure.Data;
|
|
using Microsoft.EntityFrameworkCore;
|
|
|
|
namespace ErsatzTV.Application.Scheduling;
|
|
|
|
public class CreateBlockHandler(IDbContextFactory<TvContext> dbContextFactory)
|
|
: IRequestHandler<CreateBlock, Either<BaseError, BlockViewModel>>
|
|
{
|
|
public async Task<Either<BaseError, BlockViewModel>> Handle(
|
|
CreateBlock request,
|
|
CancellationToken cancellationToken)
|
|
{
|
|
await using TvContext dbContext = await dbContextFactory.CreateDbContextAsync(cancellationToken);
|
|
Validation<BaseError, Block> validation = await Validate(dbContext, request);
|
|
return await validation.Apply(profile => PersistBlock(dbContext, profile));
|
|
}
|
|
|
|
private static async Task<BlockViewModel> PersistBlock(TvContext dbContext, Block block)
|
|
{
|
|
await dbContext.Blocks.AddAsync(block);
|
|
await dbContext.SaveChangesAsync();
|
|
await dbContext.Entry(block).Reference(b => b.BlockGroup).LoadAsync();
|
|
return Mapper.ProjectToViewModel(block);
|
|
}
|
|
|
|
private static async Task<Validation<BaseError, Block>> Validate(TvContext dbContext, CreateBlock request)
|
|
{
|
|
Validation<BaseError, Unit> blockGroupValidation = await ValidateBlockGroupExists(dbContext, request);
|
|
Validation<BaseError, string> nameValidation = await ValidateBlockName(dbContext, request);
|
|
|
|
return (blockGroupValidation, nameValidation).Apply((_, name) => new Block
|
|
{
|
|
BlockGroupId = request.BlockGroupId,
|
|
Name = name,
|
|
Minutes = 30
|
|
});
|
|
}
|
|
|
|
private static async Task<Validation<BaseError, Unit>> ValidateBlockGroupExists(
|
|
TvContext dbContext,
|
|
CreateBlock request)
|
|
{
|
|
bool blockGroupExists = await dbContext.BlockGroups.AnyAsync(bg => bg.Id == request.BlockGroupId);
|
|
|
|
return blockGroupExists
|
|
? Success<BaseError, Unit>(Unit.Default)
|
|
: BaseError.New("Block group does not exist");
|
|
}
|
|
|
|
private static async Task<Validation<BaseError, string>> ValidateBlockName(
|
|
TvContext dbContext,
|
|
CreateBlock request)
|
|
{
|
|
if (request.Name.Length > 50)
|
|
{
|
|
return BaseError.New($"Block name \"{request.Name}\" is invalid");
|
|
}
|
|
|
|
bool duplicate = await dbContext.Blocks
|
|
.AnyAsync(r => r.BlockGroupId == request.BlockGroupId && r.Name == request.Name);
|
|
|
|
return duplicate
|
|
? BaseError.New($"A block named \"{request.Name}\" already exists in that block group")
|
|
: Success<BaseError, string>(request.Name);
|
|
}
|
|
}
|