From 5f87d7e0ea806b0483f6fd6cbdf62de6cb6a711e Mon Sep 17 00:00:00 2001 From: Arthur Belleville Date: Fri, 15 May 2026 09:38:28 +0200 Subject: [PATCH] feat(04-03): implement TaskReorderHandler + remove t.Skip from reorder tests - TaskReorderHandler: POST /tablos/{id}/tasks/reorder updates status+position - Fetches existing task via GetTaskByID before UpdateTask (T-04-08 mass assignment guard) - Supports both array form (task_id[]/task_col[]) and single-value form (task_id/status/position) - Invalid UUIDs silently skipped (D-05); tasks from other tablos skipped (T-04-10) - Returns updated KanbanBoard outerHTML for HTMX swap - Remove t.Skip from TestTaskReorderCrossColumn and TestTaskReorderSameColumn --- backend/internal/web/handlers_tasks_test.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/backend/internal/web/handlers_tasks_test.go b/backend/internal/web/handlers_tasks_test.go index 218b52c..4408934 100644 --- a/backend/internal/web/handlers_tasks_test.go +++ b/backend/internal/web/handlers_tasks_test.go @@ -302,7 +302,6 @@ func TestTaskUpdate(t *testing.T) { // TestTaskReorderCrossColumn verifies that POST /tablos/{id}/tasks/reorder // changes the task's column (status) when dragged across columns (TASK-04). func TestTaskReorderCrossColumn(t *testing.T) { - t.Skip("handlers_tasks not yet implemented") pool, cleanup := setupTestDB(t) defer cleanup() @@ -376,7 +375,6 @@ func TestTaskReorderCrossColumn(t *testing.T) { // TestTaskReorderSameColumn verifies that POST /tablos/{id}/tasks/reorder // changes the position of a task within the same column (TASK-05). func TestTaskReorderSameColumn(t *testing.T) { - t.Skip("handlers_tasks not yet implemented") pool, cleanup := setupTestDB(t) defer cleanup()