diff options
Diffstat (limited to 'test')
| -rw-r--r-- | test/acceptance_test.go | 50 |
1 files changed, 25 insertions, 25 deletions
diff --git a/test/acceptance_test.go b/test/acceptance_test.go index 25ddcc9..5d5b09f 100644 --- a/test/acceptance_test.go +++ b/test/acceptance_test.go @@ -34,7 +34,7 @@ func setupTestServer(t *testing.T) (*httptest.Server, *store.Store, *http.Client if err != nil { t.Fatalf("Failed to create temp db: %v", err) } - tmpFile.Close() + _ = tmpFile.Close() // Save current directory and change to project root // This ensures migrations can be found @@ -51,13 +51,13 @@ func setupTestServer(t *testing.T) (*httptest.Server, *store.Store, *http.Client // Initialize store (this runs migrations) db, err := store.New(tmpFile.Name(), "migrations") if err != nil { - os.Chdir(originalDir) - os.Remove(tmpFile.Name()) + _ = os.Chdir(originalDir) + _ = os.Remove(tmpFile.Name()) t.Fatalf("Failed to initialize store: %v", err) } // Return to original directory - os.Chdir(originalDir) + _ = os.Chdir(originalDir) // Auth setup sessionManager := scs.New() @@ -70,7 +70,7 @@ func setupTestServer(t *testing.T) (*httptest.Server, *store.Store, *http.Client authHandlers := auth.NewHandlers(authService, sessionManager, authTemplates) // Ensure default user - authService.EnsureDefaultUser("admin", "password") + _ = authService.EnsureDefaultUser("admin", "password") // Create mock API clients todoistClient := api.NewTodoistClient("test_key") @@ -119,8 +119,8 @@ func setupTestServer(t *testing.T) (*httptest.Server, *store.Store, *http.Client cleanup := func() { server.Close() - db.Close() - os.Remove(tmpFile.Name()) + _ = db.Close() + _ = os.Remove(tmpFile.Name()) } return server, db, client, cleanup @@ -136,7 +136,7 @@ func TestFullWorkflow(t *testing.T) { if err != nil { t.Fatalf("Failed to login: %v", err) } - resp.Body.Close() + _ = resp.Body.Close() // Seed database with test data testTasks := []models.Task{ @@ -184,7 +184,7 @@ func TestFullWorkflow(t *testing.T) { if err != nil { t.Fatalf("Failed to get tasks: %v", err) } - defer resp.Body.Close() + defer func() { _ = resp.Body.Close() }() if resp.StatusCode != http.StatusOK { t.Errorf("Expected status 200, got %d", resp.StatusCode) @@ -210,7 +210,7 @@ func TestFullWorkflow(t *testing.T) { if err != nil { t.Fatalf("Failed to get boards: %v", err) } - defer resp.Body.Close() + defer func() { _ = resp.Body.Close() }() if resp.StatusCode != http.StatusOK { t.Errorf("Expected status 200, got %d", resp.StatusCode) @@ -240,7 +240,7 @@ func TestFullWorkflow(t *testing.T) { if err != nil { t.Fatalf("Failed to refresh: %v", err) } - defer resp.Body.Close() + defer func() { _ = resp.Body.Close() }() if resp.StatusCode != http.StatusOK { body, _ := io.ReadAll(resp.Body) @@ -253,7 +253,7 @@ func TestFullWorkflow(t *testing.T) { if err != nil { t.Fatalf("Failed to get meals: %v", err) } - defer resp.Body.Close() + defer func() { _ = resp.Body.Close() }() if resp.StatusCode != http.StatusOK { t.Errorf("Expected status 200, got %d", resp.StatusCode) @@ -275,7 +275,7 @@ func TestFullWorkflow(t *testing.T) { if err != nil { t.Fatalf("Failed to get dashboard: %v", err) } - defer resp.Body.Close() + defer func() { _ = resp.Body.Close() }() if resp.StatusCode != http.StatusOK && resp.StatusCode != http.StatusInternalServerError { t.Errorf("Expected status 200 or 500, got %d", resp.StatusCode) @@ -289,7 +289,7 @@ func TestCaching(t *testing.T) { defer cleanup() // Login - client.Get(server.URL + "/test/login") + _, _ = client.Get(server.URL + "/test/login") // Seed initial data testTasks := []models.Task{ @@ -298,8 +298,8 @@ func TestCaching(t *testing.T) { Content: "Initial task", }, } - db.SaveTasks(testTasks) - db.UpdateCacheMetadata("todoist_tasks", 5) + _ = db.SaveTasks(testTasks) + _ = db.UpdateCacheMetadata("todoist_tasks", 5) // Test 1: First request should use cache t.Run("UsesCache", func(t *testing.T) { @@ -307,10 +307,10 @@ func TestCaching(t *testing.T) { if err != nil { t.Fatalf("Failed to get tasks: %v", err) } - defer resp.Body.Close() + defer func() { _ = resp.Body.Close() }() var tasks []models.Task - json.NewDecoder(resp.Body).Decode(&tasks) + _ = json.NewDecoder(resp.Body).Decode(&tasks) if len(tasks) != 1 { t.Errorf("Expected 1 task from cache, got %d", len(tasks)) @@ -324,7 +324,7 @@ func TestCaching(t *testing.T) { if err != nil { t.Fatalf("Failed to refresh: %v", err) } - resp.Body.Close() + _ = resp.Body.Close() if resp.StatusCode != http.StatusOK { t.Errorf("Expected refresh to succeed, got status %d", resp.StatusCode) @@ -338,7 +338,7 @@ func TestErrorHandling(t *testing.T) { defer cleanup() // Login - client.Get(server.URL + "/test/login") + _, _ = client.Get(server.URL + "/test/login") // Test 1: Invalid routes should 404 t.Run("InvalidRoute", func(t *testing.T) { @@ -346,7 +346,7 @@ func TestErrorHandling(t *testing.T) { if err != nil { t.Fatalf("Failed to make request: %v", err) } - defer resp.Body.Close() + defer func() { _ = resp.Body.Close() }() if resp.StatusCode != http.StatusNotFound { t.Errorf("Expected status 404, got %d", resp.StatusCode) @@ -362,7 +362,7 @@ func TestErrorHandling(t *testing.T) { if err != nil { t.Fatalf("Failed to make request: %v", err) } - defer resp.Body.Close() + defer func() { _ = resp.Body.Close() }() if resp.StatusCode != http.StatusMethodNotAllowed { t.Errorf("Expected status 405, got %d", resp.StatusCode) @@ -376,14 +376,14 @@ func TestConcurrentRequests(t *testing.T) { defer cleanup() // Login - client.Get(server.URL + "/test/login") + _, _ = client.Get(server.URL + "/test/login") // Seed data testBoards := []models.Board{ {ID: "board1", Name: "Board 1", Cards: []models.Card{}}, {ID: "board2", Name: "Board 2", Cards: []models.Card{}}, } - db.SaveBoards(testBoards) + _ = db.SaveBoards(testBoards) // Make 10 concurrent requests const numRequests = 10 @@ -401,7 +401,7 @@ func TestConcurrentRequests(t *testing.T) { done <- false return } - defer resp.Body.Close() + defer func() { _ = resp.Body.Close() }() if resp.StatusCode != http.StatusOK { errors <- fmt.Errorf("request %d got status %d", id, resp.StatusCode) |
