From d595fbcafb93a7ef99d984310f0155be335679bc Mon Sep 17 00:00:00 2001 From: alex289 Date: Sun, 1 Oct 2023 13:11:07 +0200 Subject: [PATCH] fix: Await Test --- .github/workflows/dependency-updates.yml | 2 +- .gitignore | 2 ++ .../CreateTenantCommandHandlerTests.cs | 13 ++++++------ .../DeleteTenantCommandHandlerTests.cs | 13 ++++++------ .../UpdateTenantCommandHandlerTests.cs | 13 ++++++------ .../CreateUserCommandHandlerTests.cs | 21 ++++++++++--------- .../DeleteUserCommandHandlerTests.cs | 13 ++++++------ 7 files changed, 42 insertions(+), 35 deletions(-) diff --git a/.github/workflows/dependency-updates.yml b/.github/workflows/dependency-updates.yml index 76f9c9d..1b28a04 100644 --- a/.github/workflows/dependency-updates.yml +++ b/.github/workflows/dependency-updates.yml @@ -53,7 +53,7 @@ jobs: branch: "update/${{ env.date }}" labels: dependencies delete-branch: true - commit-message: "Update packages" + commit-message: "chore: Update dependencies" assignees: ${{ env.assignee }} base: ${{ env.baseBranch }} title: "Automatic Package Update ${{ env.date }}" diff --git a/.gitignore b/.gitignore index 0899f53..40b5261 100644 --- a/.gitignore +++ b/.gitignore @@ -6,3 +6,5 @@ riderModule.iml .idea .vs CleanArchitecture.sln.DotSettings.user +frontend +.DS_Store \ No newline at end of file diff --git a/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/CreateTenant/CreateTenantCommandHandlerTests.cs b/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/CreateTenant/CreateTenantCommandHandlerTests.cs index 0ac8c00..979831b 100644 --- a/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/CreateTenant/CreateTenantCommandHandlerTests.cs +++ b/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/CreateTenant/CreateTenantCommandHandlerTests.cs @@ -1,4 +1,5 @@ using System; +using System.Threading.Tasks; using CleanArchitecture.Domain.Commands.Tenants.CreateTenant; using CleanArchitecture.Domain.Errors; using CleanArchitecture.Shared.Events.Tenant; @@ -11,13 +12,13 @@ public sealed class CreateTenantCommandHandlerTests private readonly CreateTenantCommandTestFixture _fixture = new(); [Fact] - public void Should_Create_Tenant() + public async Task Should_Create_Tenant() { var command = new CreateTenantCommand( Guid.NewGuid(), "Test Tenant"); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoDomainNotification() @@ -28,7 +29,7 @@ public sealed class CreateTenantCommandHandlerTests } [Fact] - public void Should_Not_Create_Tenant_Insufficient_Permissions() + public async Task Should_Not_Create_Tenant_Insufficient_Permissions() { _fixture.SetupUser(); @@ -36,7 +37,7 @@ public sealed class CreateTenantCommandHandlerTests Guid.NewGuid(), "Test Tenant"); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() @@ -48,7 +49,7 @@ public sealed class CreateTenantCommandHandlerTests } [Fact] - public void Should_Not_Create_Tenant_Already_Exists() + public async Task Should_Not_Create_Tenant_Already_Exists() { var command = new CreateTenantCommand( Guid.NewGuid(), @@ -56,7 +57,7 @@ public sealed class CreateTenantCommandHandlerTests _fixture.SetupExistingTenant(command.AggregateId); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() diff --git a/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/DeleteTenant/DeleteTenantCommandHandlerTests.cs b/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/DeleteTenant/DeleteTenantCommandHandlerTests.cs index 68134d9..b3e5579 100644 --- a/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/DeleteTenant/DeleteTenantCommandHandlerTests.cs +++ b/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/DeleteTenant/DeleteTenantCommandHandlerTests.cs @@ -1,4 +1,5 @@ using System; +using System.Threading.Tasks; using CleanArchitecture.Domain.Commands.Tenants.DeleteTenant; using CleanArchitecture.Domain.Errors; using CleanArchitecture.Shared.Events.Tenant; @@ -11,13 +12,13 @@ public sealed class DeleteTenantCommandHandlerTests private readonly DeleteTenantCommandTestFixture _fixture = new(); [Fact] - public void Should_Delete_Tenant() + public async Task Should_Delete_Tenant() { var tenant = _fixture.SetupTenant(); var command = new DeleteTenantCommand(tenant.Id); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoDomainNotification() @@ -26,13 +27,13 @@ public sealed class DeleteTenantCommandHandlerTests } [Fact] - public void Should_Not_Delete_Non_Existing_Tenant() + public async Task Should_Not_Delete_Non_Existing_Tenant() { _fixture.SetupTenant(); var command = new DeleteTenantCommand(Guid.NewGuid()); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() @@ -44,14 +45,14 @@ public sealed class DeleteTenantCommandHandlerTests } [Fact] - public void Should_Not_Delete_Tenant_Insufficient_Permissions() + public async Task Should_Not_Delete_Tenant_Insufficient_Permissions() { var tenant = _fixture.SetupTenant(); _fixture.SetupUser(); var command = new DeleteTenantCommand(tenant.Id); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() diff --git a/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/UpdateTenant/UpdateTenantCommandHandlerTests.cs b/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/UpdateTenant/UpdateTenantCommandHandlerTests.cs index 980b97b..e03eee0 100644 --- a/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/UpdateTenant/UpdateTenantCommandHandlerTests.cs +++ b/CleanArchitecture.Domain.Tests/CommandHandler/Tenant/UpdateTenant/UpdateTenantCommandHandlerTests.cs @@ -1,4 +1,5 @@ using System; +using System.Threading.Tasks; using CleanArchitecture.Domain.Commands.Tenants.UpdateTenant; using CleanArchitecture.Domain.Errors; using CleanArchitecture.Shared.Events.Tenant; @@ -11,7 +12,7 @@ public sealed class UpdateTenantCommandHandlerTests private readonly UpdateTenantCommandTestFixture _fixture = new(); [Fact] - public void Should_Update_Tenant() + public async Task Should_Update_Tenant() { var command = new UpdateTenantCommand( Guid.NewGuid(), @@ -19,7 +20,7 @@ public sealed class UpdateTenantCommandHandlerTests _fixture.SetupExistingTenant(command.AggregateId); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyCommit() @@ -30,7 +31,7 @@ public sealed class UpdateTenantCommandHandlerTests } [Fact] - public void Should_Not_Update_Tenant_Insufficient_Permissions() + public async Task Should_Not_Update_Tenant_Insufficient_Permissions() { var command = new UpdateTenantCommand( Guid.NewGuid(), @@ -38,7 +39,7 @@ public sealed class UpdateTenantCommandHandlerTests _fixture.SetupUser(); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() @@ -50,13 +51,13 @@ public sealed class UpdateTenantCommandHandlerTests } [Fact] - public void Should_Not_Update_Tenant_Not_Existing() + public async Task Should_Not_Update_Tenant_Not_Existing() { var command = new UpdateTenantCommand( Guid.NewGuid(), "Tenant Name"); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() diff --git a/CleanArchitecture.Domain.Tests/CommandHandler/User/CreateUser/CreateUserCommandHandlerTests.cs b/CleanArchitecture.Domain.Tests/CommandHandler/User/CreateUser/CreateUserCommandHandlerTests.cs index ef81bc5..25108ee 100644 --- a/CleanArchitecture.Domain.Tests/CommandHandler/User/CreateUser/CreateUserCommandHandlerTests.cs +++ b/CleanArchitecture.Domain.Tests/CommandHandler/User/CreateUser/CreateUserCommandHandlerTests.cs @@ -1,4 +1,5 @@ using System; +using System.Threading.Tasks; using CleanArchitecture.Domain.Commands.Users.CreateUser; using CleanArchitecture.Domain.Enums; using CleanArchitecture.Domain.Errors; @@ -13,7 +14,7 @@ public sealed class CreateUserCommandHandlerTests private readonly CreateUserCommandTestFixture _fixture = new(); [Fact] - public void Should_Create_User() + public async Task Should_Create_User() { _fixture.SetupCurrentUser(); @@ -28,7 +29,7 @@ public sealed class CreateUserCommandHandlerTests "Email", "Po=PF]PC6t.?8?ks)A6W"); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoDomainNotification() @@ -37,7 +38,7 @@ public sealed class CreateUserCommandHandlerTests } [Fact] - public void Should_Not_Create_Already_Existing_User() + public async Task Should_Not_Create_Already_Existing_User() { _fixture.SetupCurrentUser(); @@ -51,7 +52,7 @@ public sealed class CreateUserCommandHandlerTests "Email", "Po=PF]PC6t.?8?ks)A6W"); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() @@ -63,7 +64,7 @@ public sealed class CreateUserCommandHandlerTests } [Fact] - public void Should_Not_Create_Already_Existing_Email() + public async Task Should_Not_Create_Already_Existing_Email() { _fixture.SetupCurrentUser(); @@ -86,7 +87,7 @@ public sealed class CreateUserCommandHandlerTests "Email", "Po=PF]PC6t.?8?ks)A6W"); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() @@ -98,7 +99,7 @@ public sealed class CreateUserCommandHandlerTests } [Fact] - public void Should_Not_Create_User_Tenant_Does_Not_Exist() + public async Task Should_Not_Create_User_Tenant_Does_Not_Exist() { _fixture.SetupCurrentUser(); @@ -112,7 +113,7 @@ public sealed class CreateUserCommandHandlerTests "Email", "Po=PF]PC6t.?8?ks)A6W"); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() @@ -124,7 +125,7 @@ public sealed class CreateUserCommandHandlerTests } [Fact] - public void Should_Not_Create_User_Insufficient_Permissions() + public async Task Should_Not_Create_User_Insufficient_Permissions() { _fixture.SetupUser(); @@ -136,7 +137,7 @@ public sealed class CreateUserCommandHandlerTests "Email", "Po=PF]PC6t.?8?ks)A6W"); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() diff --git a/CleanArchitecture.Domain.Tests/CommandHandler/User/DeleteUser/DeleteUserCommandHandlerTests.cs b/CleanArchitecture.Domain.Tests/CommandHandler/User/DeleteUser/DeleteUserCommandHandlerTests.cs index 7dc1c8d..19344e8 100644 --- a/CleanArchitecture.Domain.Tests/CommandHandler/User/DeleteUser/DeleteUserCommandHandlerTests.cs +++ b/CleanArchitecture.Domain.Tests/CommandHandler/User/DeleteUser/DeleteUserCommandHandlerTests.cs @@ -1,4 +1,5 @@ using System; +using System.Threading.Tasks; using CleanArchitecture.Domain.Commands.Users.DeleteUser; using CleanArchitecture.Domain.Errors; using CleanArchitecture.Shared.Events.User; @@ -11,13 +12,13 @@ public sealed class DeleteUserCommandHandlerTests private readonly DeleteUserCommandTestFixture _fixture = new(); [Fact] - public void Should_Delete_User() + public async Task Should_Delete_User() { var user = _fixture.SetupUser(); var command = new DeleteUserCommand(user.Id); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoDomainNotification() @@ -26,13 +27,13 @@ public sealed class DeleteUserCommandHandlerTests } [Fact] - public void Should_Not_Delete_Non_Existing_User() + public async Task Should_Not_Delete_Non_Existing_User() { _fixture.SetupUser(); var command = new DeleteUserCommand(Guid.NewGuid()); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit() @@ -44,7 +45,7 @@ public sealed class DeleteUserCommandHandlerTests } [Fact] - public void Should_Not_Delete_User_Insufficient_Permissions() + public async Task Should_Not_Delete_User_Insufficient_Permissions() { var user = _fixture.SetupUser(); @@ -52,7 +53,7 @@ public sealed class DeleteUserCommandHandlerTests var command = new DeleteUserCommand(user.Id); - _fixture.CommandHandler.Handle(command, default).Wait(); + await _fixture.CommandHandler.Handle(command, default); _fixture .VerifyNoCommit()