From 7dcc70b448d0b9d8d67c712dca068b7c4620e604 Mon Sep 17 00:00:00 2001 From: John O'Keefe Date: Tue, 10 Feb 2026 18:15:28 -0500 Subject: [PATCH] fix: remove redundant getTestUserID call in TestKoboInitialization The TestKoboInitialization test was calling getTestUserID() explicitly on line 25, but loginTestUser() already calls this internally. This caused the test user to be deleted and recreated after login, leading to inconsistent state and HTTP 500 errors when creating libraries. After removing the redundant call: - TestKoboInitialization now passes - All Kobo sync tests pass successfully --- cmd/server/tests/kobo_test.go | 1 - 1 file changed, 1 deletion(-) diff --git a/cmd/server/tests/kobo_test.go b/cmd/server/tests/kobo_test.go index 3a86475..1fc4464 100644 --- a/cmd/server/tests/kobo_test.go +++ b/cmd/server/tests/kobo_test.go @@ -22,7 +22,6 @@ func TestKoboInitialization(t *testing.T) { setup := setupTestServer(t) token := loginTestUser(t, setup.Server, setup.DB) - _ = getTestUserID(t, setup.DB) _ = createTestMediaItemID(t, setup.Server, token) log.Printf("[DEBUG] Kobo test setup: creating device and media")