Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
44 changes: 44 additions & 0 deletions src/Application/Folders/Commands/RemoveFolder.cs
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
using Application.Common.Exceptions;
using Application.Common.Interfaces;
using Application.Common.Models.Dtos.Physical;
using AutoMapper;
using Domain.Entities.Physical;
using FluentValidation;
using MediatR;
using Microsoft.EntityFrameworkCore;

namespace Application.Folders.Commands;

Expand All@@ -9,4 +15,42 @@ public record Command : IRequest<FolderDto>
{
public Guid FolderId { get; init; }
}

public class CommandHandler : IRequestHandler<Command, FolderDto>
{
private readonly IApplicationDbContext _context;
private readonly IMapper _mapper;

public CommandHandler(IApplicationDbContext context, IMapper mapper)
{
_context = context;
_mapper = mapper;
}

public async Task<FolderDto> Handle(Command request, CancellationToken cancellationToken)
{
var folder = await _context.Folders
.Include(x => x.Locker)
.FirstOrDefaultAsync(x => x.Id.Equals(request.FolderId), cancellationToken);

if (folder is null)
{
throw new KeyNotFoundException("Folder does not exist.");
}

var containDocument = folder.NumberOfDocuments > 0;

if (containDocument)
{
throw new ConflictException("Folder cannot be removed because it contains documents.");
}

Comment thread
kaitoz11 marked this conversation as resolved.
var locker = folder.Locker;
var result = _context.Folders.Remove(folder);
locker.NumberOfFolders -= 1;

await _context.SaveChangesAsync(cancellationToken);
return _mapper.Map<FolderDto>(result.Entity);
}
}
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,94 @@
using Application.Common.Exceptions;
using Application.Folders.Commands;
using Domain.Entities;
using Domain.Entities.Physical;
using FluentAssertions;
using Xunit;

namespace Application.Tests.Integration.Folders.Commands;

public class RemoveFolderTests : BaseClassFixture
{
public RemoveFolderTests(CustomApiFactory apiFactory) : base(apiFactory)
{
}

[Fact]
public async Task ShouldRemoveFolder_WhenFolderHasNoDocuments()
{
// Arrange
var department = CreateDepartment();
var folder = CreateFolder();
var locker = CreateLocker(folder);
var room = CreateRoom(department, locker);
await Add(room);

var command = new RemoveFolder.Command()
{
FolderId = folder.Id,
};

// Act
var result = await SendAsync(command);

// Assert
result.Id.Should().Be(folder.Id);
var removedFolder = await FindAsync<Folder>(folder.Id);
removedFolder.Should().BeNull();
var lockerOfRemovedFolder = await FindAsync<Locker>(locker.Id);
locker.NumberOfFolders.Should().Be(lockerOfRemovedFolder!.NumberOfFolders + 1);

// Cleanup
Remove(lockerOfRemovedFolder);
Remove(await FindAsync<Room>(room.Id));
Remove(await FindAsync<Department>(department.Id));
}

[Fact]
public async Task ShouldThrowConflictException_WhenFolderStillHasDocuments()
{
// Arrange
var department = CreateDepartment();
var documents = CreateNDocuments(1);
var folder = CreateFolder(documents);
var locker = CreateLocker(folder);
var room = CreateRoom(department, locker);
await AddAsync(room);

var command = new RemoveFolder.Command()
{
FolderId = folder.Id
};

// Act
var action = async () => await SendAsync(command);

// Assert
await action.Should().ThrowAsync<ConflictException>()
.WithMessage("Folder cannot be removed because it contains documents.");

// Cleanup
Remove(documents.First());
Remove(folder);
Remove(locker);
Remove(room);
Remove(await FindAsync<Department>(department.Id));
}

[Fact]
public async Task ShouldThrowKeyNotFoundException_WhenFolderDoesNotExist()
{
// Arrange
var command = new RemoveFolder.Command()
{
FolderId = Guid.NewGuid(),
};

// Act
var action = async () => await SendAsync(command);

// Assert
await action.Should().ThrowAsync<KeyNotFoundException>()
.WithMessage("Folder does not exist.");
}
}