From 7a12b946a6efb925985537aeb3b2639a7cac5fb1 Mon Sep 17 00:00:00 2001 From: Imbus <> Date: Sat, 2 Mar 2024 04:29:50 +0100 Subject: [PATCH] Fixing some issues found by the linter --- backend/cmd/main.go | 2 +- backend/internal/database/db.go | 5 ++++- backend/internal/database/db_test.go | 4 +++- 3 files changed, 8 insertions(+), 3 deletions(-) diff --git a/backend/cmd/main.go b/backend/cmd/main.go index b238470..5762bab 100644 --- a/backend/cmd/main.go +++ b/backend/cmd/main.go @@ -15,7 +15,7 @@ func main() { conf, err := config.ReadConfigFromFile("config.toml") if err != nil { conf = config.NewConfig() - conf.WriteConfigToFile("config.toml") + _ = conf.WriteConfigToFile("config.toml") } // Pretty print the current config diff --git a/backend/internal/database/db.go b/backend/internal/database/db.go index b850289..6e86641 100644 --- a/backend/internal/database/db.go +++ b/backend/internal/database/db.go @@ -104,6 +104,9 @@ func (d *Db) Migrate(dirname string) error { log.Println("Executed SQL file:", file.Name()) } - tr.Commit() + if tr.Commit() != nil { + return err + } + return nil } diff --git a/backend/internal/database/db_test.go b/backend/internal/database/db_test.go index 716bde8..6830668 100644 --- a/backend/internal/database/db_test.go +++ b/backend/internal/database/db_test.go @@ -36,7 +36,9 @@ func TestDbGetUserId(t *testing.T) { if err != nil { t.Error("setupState failed:", err) } - db.AddUser("test", "password") + if db.AddUser("test", "password") != nil { + t.Error("AddUser failed") + } var id int