Revert "fix: test"

This reverts commit 002b1b9.

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

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

@@ -489,17 +489,15 @@ 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, userId);
+            await _characterService.DeleteCharacterAsync(characterId);
 
             // Assert
             _mockCharacterRepository.Verify(x => x.GetByIdAsync(characterId), Times.Once);
@@ -511,14 +509,13 @@ 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, userId)
+                () => _characterService.DeleteCharacterAsync(characterId)
             );
 
             exception.ErrorCode.Should().Be(ErrorCode.CHARACTER_NOT_FOUND);
@@ -531,17 +528,15 @@ 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, userId);
+            await _characterService.DeleteCharacterAsync(characterId);
 
             // Assert
             _mockLogger.Verify(

ProjectVG.Tests/Application/TestUtilities/TestDataBuilder.cs

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