feat(dashboard): remove deprecated /incomplete and /recent-activity endpoints

GET /api/documents/incomplete and GET /api/documents/recent-activity are
superseded by the new dashboard endpoints (GET /api/dashboard/activity etc.)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

backend/src/main/java/org/raddatz/familienarchiv/controller/DocumentController.java

@@ -17,7 +17,6 @@ import org.raddatz.familienarchiv.dto.DocumentSearchResult;
 import org.raddatz.familienarchiv.dto.DocumentUpdateDTO;
 import org.raddatz.familienarchiv.dto.TagOperator;
 import org.raddatz.familienarchiv.dto.DocumentVersionSummary;
-import org.raddatz.familienarchiv.dto.IncompleteDocumentDTO;
 import org.raddatz.familienarchiv.exception.DomainException;
 import org.raddatz.familienarchiv.exception.ErrorCode;
 import org.raddatz.familienarchiv.model.Document;
@@ -198,12 +197,6 @@ public class DocumentController {
         return Map.of("count", documentService.getIncompleteCount());
     }
 
-    @GetMapping("/incomplete")
-    public List<IncompleteDocumentDTO> getIncomplete(
-            @Parameter(description = "Maximum number of results") @RequestParam(defaultValue = "10") int size) {
-        return documentService.findIncompleteDocuments(size);
-    }
-
     @GetMapping("/incomplete/next")
     public ResponseEntity<Document> getNextIncomplete(@RequestParam UUID excludeId) {
         return documentService.findNextIncompleteDocument(excludeId)
@@ -211,12 +204,6 @@ public class DocumentController {
                 .orElse(ResponseEntity.noContent().build());
     }
 
-    @GetMapping("/recent-activity")
-    public ResponseEntity<List<Document>> getRecentActivity(
-            @RequestParam(defaultValue = "5") int size) {
-        return ResponseEntity.ok(documentService.getRecentActivity(size));
-    }
-
     @GetMapping("/search")
     public ResponseEntity<DocumentSearchResult> search(
             @RequestParam(required = false) String q,

backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java

@@ -3,7 +3,6 @@ package org.raddatz.familienarchiv.controller;
 import org.junit.jupiter.api.Test;
 import org.raddatz.familienarchiv.dto.DocumentSearchResult;
 import org.raddatz.familienarchiv.dto.DocumentVersionSummary;
-import org.raddatz.familienarchiv.dto.IncompleteDocumentDTO;
 import org.raddatz.familienarchiv.exception.DomainException;
 import org.raddatz.familienarchiv.exception.ErrorCode;
 import org.raddatz.familienarchiv.model.Document;
@@ -390,47 +389,14 @@ class DocumentControllerTest {
                 .andExpect(jsonPath("$.count").value(3));
     }
 
-    // ─── GET /api/documents/incomplete ───────────────────────────────────────
+    // ─── GET /api/documents/incomplete (removed — superseded by dashboard) ────
-
-    @Test
-    void getIncomplete_returns401_whenUnauthenticated() throws Exception {
-        mockMvc.perform(get("/api/documents/incomplete"))
-                .andExpect(status().isUnauthorized());
-    }
 
     @Test
     @WithMockUser
-    void getIncomplete_returns200_withDTOList() throws Exception {
+    void getIncomplete_endpointRemoved() throws Exception {
-        UUID id = UUID.randomUUID();
+        // The path hits /{id} and fails UUID conversion — not a 200 anymore
-        IncompleteDocumentDTO dto = new IncompleteDocumentDTO(id, "Unvollständig");
-        when(documentService.findIncompleteDocuments(anyInt())).thenReturn(List.of(dto));
-
         mockMvc.perform(get("/api/documents/incomplete"))
-                .andExpect(status().isOk())
+                .andExpect(status().is4xxClientError());
-                .andExpect(jsonPath("$[0].id").value(id.toString()))
-                .andExpect(jsonPath("$[0].title").value("Unvollständig"));
-    }
-
-    @Test
-    @WithMockUser
-    void getIncomplete_withSizeParam_passesItToService() throws Exception {
-        when(documentService.findIncompleteDocuments(5)).thenReturn(List.of());
-
-        mockMvc.perform(get("/api/documents/incomplete").param("size", "5"))
-                .andExpect(status().isOk());
-
-        verify(documentService).findIncompleteDocuments(5);
-    }
-
-    @Test
-    @WithMockUser
-    void getIncomplete_usesDefaultSizeWhenNotSpecified() throws Exception {
-        when(documentService.findIncompleteDocuments(anyInt())).thenReturn(List.of());
-
-        mockMvc.perform(get("/api/documents/incomplete"))
-                .andExpect(status().isOk());
-
-        verify(documentService).findIncompleteDocuments(10);
     }
 
     // ─── GET /api/documents/incomplete/next ──────────────────────────────────
@@ -467,36 +433,14 @@ class DocumentControllerTest {
                 .andExpect(status().isNoContent());
     }
 
-    // ─── GET /api/documents/recent-activity ──────────────────────────────────
+    // ─── GET /api/documents/recent-activity (removed — superseded by dashboard)
-
-    @Test
-    void getRecentActivity_returns401_whenUnauthenticated() throws Exception {
-        mockMvc.perform(get("/api/documents/recent-activity"))
-                .andExpect(status().isUnauthorized());
-    }
 
     @Test
     @WithMockUser
-    void getRecentActivity_returnsOkWithDocuments() throws Exception {
+    void getRecentActivity_endpointRemoved() throws Exception {
-        Document doc1 = Document.builder().id(UUID.randomUUID()).title("Alpha").originalFilename("a.pdf").build();
+        // The path hits /{id} and fails UUID conversion — not a 200 anymore
-        Document doc2 = Document.builder().id(UUID.randomUUID()).title("Beta").originalFilename("b.pdf").build();
-        when(documentService.getRecentActivity(5)).thenReturn(List.of(doc1, doc2));
-
-        mockMvc.perform(get("/api/documents/recent-activity").param("size", "5"))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$[0].title").value("Alpha"))
-                .andExpect(jsonPath("$[1].title").value("Beta"));
-    }
-
-    @Test
-    @WithMockUser
-    void getRecentActivity_appliesDefaultSizeOfFive_whenSizeParamOmitted() throws Exception {
-        when(documentService.getRecentActivity(5)).thenReturn(List.of());
-
         mockMvc.perform(get("/api/documents/recent-activity"))
-                .andExpect(status().isOk());
+                .andExpect(status().is4xxClientError());
-
-        verify(documentService).getRecentActivity(5);
     }
 
     // ─── GET /api/documents/{id}/versions ────────────────────────────────────