Merge pull request #789 from TransactionProcessing/copilot/add-estate-request-handler-tests

Add unit tests for EstateRequestHandler to achieve full coverage

Author: StuartFerguson

EstateManagementUI.BlazorServer.Tests/RequestHandlers/EstateRequestHandlerTests.cs

@@ -0,0 +1,192 @@
+using EstateManagementUI.BusinessLogic.Client;
+using EstateManagementUI.BusinessLogic.Models;
+using EstateManagementUI.BusinessLogic.RequestHandlers;
+using EstateManagementUI.BusinessLogic.Requests;
+using Moq;
+using Shouldly;
+using SimpleResults;
+
+namespace EstateManagementUI.BlazorServer.Tests.RequestHandlers;
+
+public class EstateRequestHandlerTests
+{
+    private readonly Mock<IApiClient> _mockApiClient;
+    private readonly EstateRequestHandler _handler;
+
+    public EstateRequestHandlerTests()
+    {
+        _mockApiClient = new Mock<IApiClient>();
+        _handler = new EstateRequestHandler(_mockApiClient.Object);
+    }
+
+    [Fact]
+    public async Task Handle_GetEstateQuery_ReturnsSuccess_WhenApiClientSucceeds()
+    {
+        // Arrange
+        var estateId = Guid.NewGuid();
+        var query = new EstateQueries.GetEstateQuery(CorrelationIdHelper.New(), estateId);
+        var estateModel = new EstateModels.EstateModel
+        {
+            EstateId = estateId,
+            EstateName = "Test Estate"
+        };
+
+        _mockApiClient
+            .Setup(c => c.GetEstate(query, It.IsAny<CancellationToken>()))
+            .ReturnsAsync(Result.Success(estateModel));
+
+        // Act
+        var result = await _handler.Handle(query, CancellationToken.None);
+
+        // Assert
+        result.IsSuccess.ShouldBeTrue();
+        result.Data.ShouldNotBeNull();
+        result.Data!.EstateId.ShouldBe(estateId);
+        result.Data.EstateName.ShouldBe("Test Estate");
+
+        _mockApiClient.Verify(c => c.GetEstate(query, It.IsAny<CancellationToken>()), Times.Once);
+    }
+
+    [Fact]
+    public async Task Handle_GetEstateQuery_ReturnsFailure_WhenApiClientFails()
+    {
+        // Arrange
+        var query = new EstateQueries.GetEstateQuery(CorrelationIdHelper.New(), Guid.NewGuid());
+
+        _mockApiClient
+            .Setup(c => c.GetEstate(query, It.IsAny<CancellationToken>()))
+            .ReturnsAsync(Result.Failure("api error"));
+
+        // Act
+        var result = await _handler.Handle(query, CancellationToken.None);
+
+        // Assert
+        result.IsFailed.ShouldBeTrue();
+
+        _mockApiClient.Verify(c => c.GetEstate(query, It.IsAny<CancellationToken>()), Times.Once);
+    }
+
+    [Fact]
+    public async Task Handle_AddOperatorToEstateCommand_ReturnsSuccess_WhenApiClientSucceeds()
+    {
+        // Arrange
+        var command = new EstateCommands.AddOperatorToEstateCommand(CorrelationIdHelper.New(), Guid.NewGuid(), Guid.NewGuid());
+
+        _mockApiClient
+            .Setup(c => c.AddEstateOperator(command, It.IsAny<CancellationToken>()))
+            .ReturnsAsync(Result.Success());
+
+        // Act
+        var result = await _handler.Handle(command, CancellationToken.None);
+
+        // Assert
+        result.IsSuccess.ShouldBeTrue();
+
+        _mockApiClient.Verify(c => c.AddEstateOperator(command, It.IsAny<CancellationToken>()), Times.Once);
+    }
+
+    [Fact]
+    public async Task Handle_AddOperatorToEstateCommand_ReturnsFailure_WhenApiClientFails()
+    {
+        // Arrange
+        var command = new EstateCommands.AddOperatorToEstateCommand(CorrelationIdHelper.New(), Guid.NewGuid(), Guid.NewGuid());
+
+        _mockApiClient
+            .Setup(c => c.AddEstateOperator(command, It.IsAny<CancellationToken>()))
+            .ReturnsAsync(Result.Failure("api error"));
+
+        // Act
+        var result = await _handler.Handle(command, CancellationToken.None);
+
+        // Assert
+        result.IsFailed.ShouldBeTrue();
+
+        _mockApiClient.Verify(c => c.AddEstateOperator(command, It.IsAny<CancellationToken>()), Times.Once);
+    }
+
+    [Fact]
+    public async Task Handle_RemoveOperatorFromEstateCommand_ReturnsSuccess_WhenApiClientSucceeds()
+    {
+        // Arrange
+        var command = new EstateCommands.RemoveOperatorFromEstateCommand(CorrelationIdHelper.New(), Guid.NewGuid(), Guid.NewGuid());
+
+        _mockApiClient
+            .Setup(c => c.RemoveEstateOperator(command, It.IsAny<CancellationToken>()))
+            .ReturnsAsync(Result.Success());
+
+        // Act
+        var result = await _handler.Handle(command, CancellationToken.None);
+
+        // Assert
+        result.IsSuccess.ShouldBeTrue();
+
+        _mockApiClient.Verify(c => c.RemoveEstateOperator(command, It.IsAny<CancellationToken>()), Times.Once);
+    }
+
+    [Fact]
+    public async Task Handle_RemoveOperatorFromEstateCommand_ReturnsFailure_WhenApiClientFails()
+    {
+        // Arrange
+        var command = new EstateCommands.RemoveOperatorFromEstateCommand(CorrelationIdHelper.New(), Guid.NewGuid(), Guid.NewGuid());
+
+        _mockApiClient
+            .Setup(c => c.RemoveEstateOperator(command, It.IsAny<CancellationToken>()))
+            .ReturnsAsync(Result.Failure("api error"));
+
+        // Act
+        var result = await _handler.Handle(command, CancellationToken.None);
+
+        // Assert
+        result.IsFailed.ShouldBeTrue();
+
+        _mockApiClient.Verify(c => c.RemoveEstateOperator(command, It.IsAny<CancellationToken>()), Times.Once);
+    }
+
+    [Fact]
+    public async Task Handle_GetAssignedOperatorsQuery_ReturnsSuccess_WhenApiClientSucceeds()
+    {
+        // Arrange
+        var estateId = Guid.NewGuid();
+        var query = new EstateQueries.GetAssignedOperatorsQuery(CorrelationIdHelper.New(), estateId);
+        var operators = new List<OperatorModels.OperatorModel>
+        {
+            new() { OperatorId = Guid.NewGuid(), Name = "Operator 1" },
+            new() { OperatorId = Guid.NewGuid(), Name = "Operator 2" }
+        };
+
+        _mockApiClient
+            .Setup(c => c.GetEstateAssignedOperators(query, It.IsAny<CancellationToken>()))
+            .ReturnsAsync(Result.Success(operators));
+
+        // Act
+        var result = await _handler.Handle(query, CancellationToken.None);
+
+        // Assert
+        result.IsSuccess.ShouldBeTrue();
+        result.Data.ShouldNotBeNull();
+        result.Data!.Count.ShouldBe(2);
+        result.Data[0].Name.ShouldBe("Operator 1");
+        result.Data[1].Name.ShouldBe("Operator 2");
+
+        _mockApiClient.Verify(c => c.GetEstateAssignedOperators(query, It.IsAny<CancellationToken>()), Times.Once);
+    }
+
+    [Fact]
+    public async Task Handle_GetAssignedOperatorsQuery_ReturnsFailure_WhenApiClientFails()
+    {
+        // Arrange
+        var query = new EstateQueries.GetAssignedOperatorsQuery(CorrelationIdHelper.New(), Guid.NewGuid());
+
+        _mockApiClient
+            .Setup(c => c.GetEstateAssignedOperators(query, It.IsAny<CancellationToken>()))
+            .ReturnsAsync(Result.Failure("api error"));
+
+        // Act
+        var result = await _handler.Handle(query, CancellationToken.None);
+
+        // Assert
+        result.IsFailed.ShouldBeTrue();
+
+        _mockApiClient.Verify(c => c.GetEstateAssignedOperators(query, It.IsAny<CancellationToken>()), Times.Once);
+    }
+}