From 640b64710e73c9978a630b3543e5a242845098d8 Mon Sep 17 00:00:00 2001 From: Steve Cliff Date: Mon, 4 May 2026 19:19:28 +0100 Subject: [PATCH] =?UTF-8?q?store:=20A1=20=E2=80=94=20check=20rows.Err()=20?= =?UTF-8?q?+=20Scan=20err=20in=20migrate=5Ftest?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Code-quality nits flagged in review of e6d965d. Mirrors the existing pattern in host_credentials_test.go. --- internal/store/migrate_test.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/internal/store/migrate_test.go b/internal/store/migrate_test.go index 59fb304..6bed70a 100644 --- a/internal/store/migrate_test.go +++ b/internal/store/migrate_test.go @@ -24,9 +24,14 @@ func TestMigration0013AlertsLastSeen(t *testing.T) { cols := map[string]bool{} for rows.Next() { var n string - _ = rows.Scan(&n) + if err := rows.Scan(&n); err != nil { + t.Fatalf("scan: %v", err) + } cols[n] = true } + if err := rows.Err(); err != nil { + t.Fatalf("rows iter: %v", err) + } if !cols["last_seen_at"] { t.Fatalf("alerts.last_seen_at not present after migration; cols=%v", cols) }