diff options
| author | Peter Stone <thepeterstone@gmail.com> | 2026-01-25 20:55:58 -1000 |
|---|---|---|
| committer | Peter Stone <thepeterstone@gmail.com> | 2026-01-25 20:55:58 -1000 |
| commit | f5b997bfc4c77ef262726d14b30d387eb7acd1c6 (patch) | |
| tree | 740879da10f3ddcd62bf276cd0632134b53a23c8 /internal/handlers/tab_state_test.go | |
| parent | fa95c71494458070b78270e3d9170076028fc974 (diff) | |
Fix all static analysis errors (golangci-lint)
- Fix errcheck: handle all error return values in production code
- Fix errcheck: handle all error return values in test files
- Fix staticcheck: replace deprecated WithCredentialsFile with WithAuthCredentialsFile
- Remove unused code: authHeaders, planToEatPlannerItem, planToEatResponse
- Use defer func() { _ = x.Close() }() pattern for ignored close errors
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Diffstat (limited to 'internal/handlers/tab_state_test.go')
| -rw-r--r-- | internal/handlers/tab_state_test.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/internal/handlers/tab_state_test.go b/internal/handlers/tab_state_test.go index 60f1340..71c6ed8 100644 --- a/internal/handlers/tab_state_test.go +++ b/internal/handlers/tab_state_test.go @@ -17,7 +17,7 @@ func TestHandleDashboard_TabState(t *testing.T) { if err != nil { t.Fatalf("Failed to create test database: %v", err) } - defer db.Close() + defer func() { _ = db.Close() }() // Create mock API clients todoistClient := api.NewTodoistClient("test-key") @@ -75,7 +75,7 @@ func TestHandleDashboard_TabState(t *testing.T) { h.HandleDashboard(w, req) resp := w.Result() - defer resp.Body.Close() + defer func() { _ = resp.Body.Close() }() if resp.StatusCode != http.StatusOK { t.Errorf("Expected status 200, got %d", resp.StatusCode) |