ci: fix workflow indentation and Codecov v5; lint: errcheck fixes; tests: add config URI builder tests

Author: amitbasuri

cmd/worker/main.go

@@ -69,6 +69,8 @@ func main() {
 	ctx, stop := signal.NotifyContext(context.Background(), syscall.SIGINT, syscall.SIGTERM)
 	defer stop()
 
-	w.Start(ctx)
+	if err := w.Start(ctx); err != nil && err != context.Canceled {
+		slog.Error("Worker stopped with error", "error", err)
+	}
 	slog.Info("Worker stopped gracefully")
 }

internal/api/dashboard.go

@@ -49,7 +49,10 @@ func (h *Handler) StreamTasks(c *gin.Context) {
 			}
 
 			// SSE format: "event: stats\ndata: <json>\n\n"
-			fmt.Fprintf(c.Writer, "event: stats\ndata: %s\n\n", string(data))
+			if _, err := fmt.Fprintf(c.Writer, "event: stats\ndata: %s\n\n", string(data)); err != nil {
+				slog.Error("Failed to write SSE data", "error", err)
+				continue
+			}
 			flusher.Flush()
 		}
 	}

internal/config/config_test.go

@@ -0,0 +1,38 @@
+package config
+
+import "testing"
+
+func TestDatabase_ToDbConnectionUri(t *testing.T) {
+    d := Database{
+        Username:     "user",
+        Password:     "pass",
+        Host:         "localhost",
+        Port:         "5432",
+        Database:     "tasks",
+        SSLMode:      "disable",
+        PoolMaxConns: 5,
+    }
+
+    got := d.ToDbConnectionUri()
+    want := "postgres://user:pass@localhost:5432/tasks?sslmode=disable&pool_max_conns=5"
+    if got != want {
+        t.Fatalf("ToDbConnectionUri() = %q, want %q", got, want)
+    }
+}
+
+func TestDatabase_ToMigrationUri(t *testing.T) {
+    d := Database{
+        Username: "user",
+        Password: "pass",
+        Host:     "localhost",
+        Port:     "5432",
+        Database: "tasks",
+        SSLMode:  "require",
+    }
+
+    got := d.ToMigrationUri()
+    want := "pgx5://user:pass@localhost:5432/tasks?sslmode=require"
+    if got != want {
+        t.Fatalf("ToMigrationUri() = %q, want %q", got, want)
+    }
+}