db migration fixes

- fix offbyone for executing migrations
- fix user_version setter not working
- add one more migration for WIP weather table

internal/db/db.go

@@ -65,7 +65,7 @@ func (tdb *TelemDb) GetVersion() (int, error) {
 }
 
 func (tdb *TelemDb) SetVersion(version int) error {
-	stmt := fmt.Sprintf("PRAGMA user_version %d", version)
+	stmt := fmt.Sprintf("PRAGMA user_version = %d", version)
 	_, err := tdb.db.Exec(stmt)
 	return err
 }

internal/db/migration.go

@@ -100,7 +100,7 @@ func RunMigrations(tdb *TelemDb) (finalVer int, err error) {
 	if err != nil {
 		return
 	}
-	for v := currentVer + 1; v < finalVer; v++ {
+	for v := currentVer + 1; v <= finalVer; v++ {
 		// attempt to get the "up" migration.
 		mMap, ok := migrations[v]
 		if !ok {

internal/db/migration_test.go

@@ -6,6 +6,8 @@ import (
 	"testing"
 )
 
+// import just the first and second migrations to ensure stability.
+//
 //go:embed migrations/1_*.sql
 //go:embed migrations/2_*.sql
 var testFs embed.FS

internal/db/migrations/3_weather_down.sql

@@ -0,0 +1 @@
+DROP TABLE "weather_station_logs";

internal/db/migrations/3_weather_up.sql

@@ -0,0 +1,6 @@
+CREATE TABLE "weather_station_logs" (
+    "ts" INTEGER NOT NULL,
+    "wind_speed" REAL,
+    "ground speed" REAL,
+    "heading" REAL
+);