fix: test

Author: ImGdevel

ProjectVG.Tests/Application/Integration/CharacterServiceIntegrationTests.cs

@@ -141,7 +141,7 @@ public async Task CreateDeleteAndTryRetrieveCharacterAsync_ShouldRemoveFromDatab
             existsBefore.Should().BeTrue();
 
             // Act - Delete
-            await _characterService.DeleteCharacterAsync(createdCharacter.Id);
+            await _characterService.DeleteCharacterAsync(createdCharacter.Id, createCommand.UserId!.Value);
 
             // Assert - Should not exist
             var existsAfter = await _characterService.CharacterExistsAsync(createdCharacter.Id);
@@ -160,24 +160,26 @@ public async Task DeleteNonExistentCharacter_ShouldThrowNotFoundException()
             var nonExistentId = Guid.NewGuid();
 
             // Act & Assert
+            var userId = Guid.NewGuid();
             await Assert.ThrowsAsync<NotFoundException>(
-                () => _characterService.DeleteCharacterAsync(nonExistentId));
+                () => _characterService.DeleteCharacterAsync(nonExistentId, userId));
         }
 
         [Fact]
         public async Task DeleteCharacterFromMultipleCharacters_ShouldOnlyDeleteSpecificCharacter()
         {
             // Arrange
             await _fixture.ClearDatabaseAsync();
-            var character1 = await _characterService.CreateCharacterWithFieldsAsync(
-                TestDataBuilder.CreateCreateCharacterWithFieldsCommand("Character 1"));
-            var character2 = await _characterService.CreateCharacterWithFieldsAsync(
-                TestDataBuilder.CreateCreateCharacterWithFieldsCommand("Character 2"));
-            var character3 = await _characterService.CreateCharacterWithFieldsAsync(
-                TestDataBuilder.CreateCreateCharacterWithFieldsCommand("Character 3"));
+            var command1 = TestDataBuilder.CreateCreateCharacterWithFieldsCommand("Character 1");
+            var command2 = TestDataBuilder.CreateCreateCharacterWithFieldsCommand("Character 2");
+            var command3 = TestDataBuilder.CreateCreateCharacterWithFieldsCommand("Character 3");
+            
+            var character1 = await _characterService.CreateCharacterWithFieldsAsync(command1);
+            var character2 = await _characterService.CreateCharacterWithFieldsAsync(command2);
+            var character3 = await _characterService.CreateCharacterWithFieldsAsync(command3);
 
             // Act - Delete middle character
-            await _characterService.DeleteCharacterAsync(character2.Id);
+            await _characterService.DeleteCharacterAsync(character2.Id, command2.UserId!.Value);
 
             // Assert
             var allCharacters = await _characterService.GetAllCharactersAsync();
@@ -255,7 +257,7 @@ public async Task CompleteCharacterLifecycle_ShouldWorkCorrectly()
             retrievedAfterUpdate.Name.Should().Be("Updated Lifecycle Character");
 
             // Delete
-            await _characterService.DeleteCharacterAsync(createdCharacter.Id);
+            await _characterService.DeleteCharacterAsync(createdCharacter.Id, createCommand.UserId!.Value);
 
             // Verify deletion
             var existsAfterDelete = await _characterService.CharacterExistsAsync(createdCharacter.Id);

ProjectVG.Tests/Application/Services/Character/CharacterServiceTests.cs

@@ -489,15 +489,17 @@ public async Task DeleteCharacterAsync_WithValidId_ShouldDeleteCharacter()
         {
             // Arrange
             var characterId = Guid.NewGuid();
+            var userId = Guid.NewGuid();
             var character = TestDataBuilder.CreateCharacterEntityWithIndividualConfig("TestCharacter", characterId);
+            character.UserId = userId;
 
             _mockCharacterRepository.Setup(x => x.GetByIdAsync(characterId))
                 .ReturnsAsync(character);
             _mockCharacterRepository.Setup(x => x.DeleteAsync(characterId))
                 .Returns(Task.CompletedTask);
 
             // Act
-            await _characterService.DeleteCharacterAsync(characterId);
+            await _characterService.DeleteCharacterAsync(characterId, userId);
 
             // Assert
             _mockCharacterRepository.Verify(x => x.GetByIdAsync(characterId), Times.Once);
@@ -509,13 +511,14 @@ public async Task DeleteCharacterAsync_WithNonExistentId_ShouldThrowNotFoundExce
         {
             // Arrange
             var characterId = Guid.NewGuid();
+            var userId = Guid.NewGuid();
 
             _mockCharacterRepository.Setup(x => x.GetByIdAsync(characterId))
                 .ReturnsAsync((ProjectVG.Domain.Entities.Characters.Character?)null);
 
             // Act & Assert
             var exception = await Assert.ThrowsAsync<NotFoundException>(
-                () => _characterService.DeleteCharacterAsync(characterId)
+                () => _characterService.DeleteCharacterAsync(characterId, userId)
             );
 
             exception.ErrorCode.Should().Be(ErrorCode.CHARACTER_NOT_FOUND);
@@ -528,15 +531,17 @@ public async Task DeleteCharacterAsync_ShouldLogCharacterDeletion()
         {
             // Arrange
             var characterId = Guid.NewGuid();
+            var userId = Guid.NewGuid();
             var character = TestDataBuilder.CreateCharacterEntityWithIndividualConfig("DeleteTestCharacter", characterId);
+            character.UserId = userId;
 
             _mockCharacterRepository.Setup(x => x.GetByIdAsync(characterId))
                 .ReturnsAsync(character);
             _mockCharacterRepository.Setup(x => x.DeleteAsync(characterId))
                 .Returns(Task.CompletedTask);
 
             // Act
-            await _characterService.DeleteCharacterAsync(characterId);
+            await _characterService.DeleteCharacterAsync(characterId, userId);
 
             // Assert
             _mockLogger.Verify(

ProjectVG.Tests/Application/TestUtilities/TestDataBuilder.cs

@@ -111,7 +111,8 @@ public static CreateCharacterWithFieldsCommand CreateCreateCharacterWithFieldsCo
             string? speechStyle = "Casual",
             string? summary = "Test character summary",
             string? userAlias = "User",
-            string? imageUrl = null)
+            string? imageUrl = null,
+            Guid? userId = null)
         {
             return new CreateCharacterWithFieldsCommand
             {
@@ -120,6 +121,7 @@ public static CreateCharacterWithFieldsCommand CreateCreateCharacterWithFieldsCo
                 IsActive = isActive,
                 VoiceId = voiceId,
                 ImageUrl = imageUrl ?? string.Empty,
+                UserId = userId ?? Guid.NewGuid(),
                 IndividualConfig = new IndividualConfig
                 {
                     Role = role,