fix selecting dialect (#692)

Author: jbking

cmd/goose/main.go

@@ -130,15 +130,6 @@ func main() {
 	}
 
 	driver, dbstring, command := args[0], args[1], args[2]
-	// To avoid breaking existing consumers. An implementation detail
-	// that consumers should not care which underlying driver is used.
-	switch driver {
-	case "sqlite3":
-		//  Internally uses the CGo-free port of SQLite: modernc.org/sqlite
-		driver = "sqlite"
-	case "postgres", "redshift":
-		driver = "pgx"
-	}
 	db, err := goose.OpenDBWithDriver(driver, normalizeDBString(driver, dbstring, *certfile, *sslcert, *sslkey))
 	if err != nil {
 		log.Fatalf("-dbstring=%q: %v\n", dbstring, err)

db.go

@@ -12,13 +12,20 @@ func OpenDBWithDriver(driver string, dbstring string) (*sql.DB, error) {
 		return nil, err
 	}
 
+	// To avoid breaking existing consumers. An implementation detail
+	// that consumers should not care which underlying driver is used.
 	switch driver {
 	case "mssql":
 		driver = "sqlserver"
 	case "tidb":
 		driver = "mysql"
 	case "turso":
 		driver = "libsql"
+	case "sqlite3":
+		//  Internally uses the CGo-free port of SQLite: modernc.org/sqlite
+		driver = "sqlite"
+	case "postgres", "redshift":
+		driver = "pgx"
 	}
 
 	switch driver {