From d8f492db2dcb9b9a84ae2646e1de03d66451f0a7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20Kr=C3=BChlmann?= <andreas@kruhlmann.dev>
Date: Fri, 24 Apr 2026 16:16:40 +0200
Subject: [PATCH] test: add AccountServiceTest (6 tests) covering deleteAccount
 behaviour

---
 .../imgfloat/service/AccountServiceTest.java  | 124 ++++++++++++++++++
 1 file changed, 124 insertions(+)
 create mode 100644 src/test/java/dev/kruhlmann/imgfloat/service/AccountServiceTest.java

diff --git a/src/test/java/dev/kruhlmann/imgfloat/service/AccountServiceTest.java b/src/test/java/dev/kruhlmann/imgfloat/service/AccountServiceTest.java
new file mode 100644
index 0000000..2f444fd
--- /dev/null
+++ b/src/test/java/dev/kruhlmann/imgfloat/service/AccountServiceTest.java
@@ -0,0 +1,124 @@
+package dev.kruhlmann.imgfloat.service;
+
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.anyString;
+import static org.mockito.ArgumentMatchers.contains;
+import static org.mockito.ArgumentMatchers.eq;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.never;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+
+import dev.kruhlmann.imgfloat.model.AssetType;
+import dev.kruhlmann.imgfloat.model.db.imgfloat.Asset;
+import dev.kruhlmann.imgfloat.model.db.imgfloat.ScriptAssetFile;
+import dev.kruhlmann.imgfloat.repository.AssetRepository;
+import dev.kruhlmann.imgfloat.repository.ChannelRepository;
+import dev.kruhlmann.imgfloat.repository.MarketplaceScriptHeartRepository;
+import dev.kruhlmann.imgfloat.repository.ScriptAssetFileRepository;
+import dev.kruhlmann.imgfloat.repository.SystemAdministratorRepository;
+import java.util.List;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.springframework.jdbc.core.JdbcTemplate;
+
+class AccountServiceTest {
+
+    private ChannelDirectoryService channelDirectoryService;
+    private AssetRepository assetRepository;
+    private ChannelRepository channelRepository;
+    private ScriptAssetFileRepository scriptAssetFileRepository;
+    private MarketplaceScriptHeartRepository heartRepository;
+    private SystemAdministratorRepository sysadminRepository;
+    private AssetStorageService assetStorageService;
+    private JdbcTemplate jdbcTemplate;
+    private AuditLogService auditLogService;
+    private AccountService service;
+
+    @BeforeEach
+    void setup() {
+        channelDirectoryService = mock(ChannelDirectoryService.class);
+        assetRepository = mock(AssetRepository.class);
+        channelRepository = mock(ChannelRepository.class);
+        scriptAssetFileRepository = mock(ScriptAssetFileRepository.class);
+        heartRepository = mock(MarketplaceScriptHeartRepository.class);
+        sysadminRepository = mock(SystemAdministratorRepository.class);
+        assetStorageService = mock(AssetStorageService.class);
+        jdbcTemplate = mock(JdbcTemplate.class);
+        auditLogService = mock(AuditLogService.class);
+
+        service = new AccountService(
+            channelDirectoryService, assetRepository, channelRepository,
+            scriptAssetFileRepository, heartRepository, sysadminRepository,
+            assetStorageService, jdbcTemplate, auditLogService
+        );
+    }
+
+    @Test
+    void deleteAccountNormalizesUsername() {
+        when(assetRepository.findByBroadcaster("uppercase")).thenReturn(List.of());
+        when(scriptAssetFileRepository.findByBroadcaster("uppercase")).thenReturn(List.of());
+
+        service.deleteAccount("UPPERCASE");
+
+        verify(assetRepository).findByBroadcaster("uppercase");
+        verify(channelRepository).deleteById("uppercase");
+    }
+
+    @Test
+    void deleteAccountSkipsBlankUsername() {
+        service.deleteAccount("");
+        service.deleteAccount("   ");
+        service.deleteAccount(null);
+
+        verify(assetRepository, never()).findByBroadcaster(any());
+    }
+
+    @Test
+    void deleteAccountDeletesAllAssets() {
+        Asset asset = new Asset("user", AssetType.IMAGE);
+        when(assetRepository.findByBroadcaster("user")).thenReturn(List.of(asset));
+        when(scriptAssetFileRepository.findByBroadcaster("user")).thenReturn(List.of());
+
+        service.deleteAccount("user");
+
+        verify(channelDirectoryService).deleteAsset(asset.getId(), "user");
+    }
+
+    @Test
+    void deleteAccountDeletesScriptAssetFilesFromDisk() {
+        when(assetRepository.findByBroadcaster("user")).thenReturn(List.of());
+        ScriptAssetFile file = new ScriptAssetFile("user", AssetType.SCRIPT);
+        file.setMediaType("application/javascript");
+        when(scriptAssetFileRepository.findByBroadcaster("user")).thenReturn(List.of(file));
+
+        service.deleteAccount("user");
+
+        verify(assetStorageService).deleteAsset(eq("user"), eq(file.getId()), eq("application/javascript"), eq(false));
+        verify(scriptAssetFileRepository).delete(file);
+    }
+
+    @Test
+    void deleteAccountCleansUpAllRepositories() {
+        when(assetRepository.findByBroadcaster("user")).thenReturn(List.of());
+        when(scriptAssetFileRepository.findByBroadcaster("user")).thenReturn(List.of());
+
+        service.deleteAccount("user");
+
+        verify(heartRepository).deleteByUsername("user");
+        verify(sysadminRepository).deleteByTwitchUsername("user");
+        verify(channelRepository).deleteById("user");
+        verify(auditLogService).deleteEntriesForBroadcaster("user");
+    }
+
+    @Test
+    void deleteAccountCleansUpSessionTables() {
+        when(assetRepository.findByBroadcaster("user")).thenReturn(List.of());
+        when(scriptAssetFileRepository.findByBroadcaster("user")).thenReturn(List.of());
+
+        service.deleteAccount("user");
+
+        verify(jdbcTemplate).update(contains("SPRING_SESSION_ATTRIBUTES"), eq("user"));
+        verify(jdbcTemplate).update(eq("DELETE FROM SPRING_SESSION WHERE PRINCIPAL_NAME = ?"), eq("user"));
+    }
+}