Merge pull request #180 from erizocosmico/fix/migrations-dir

fix migration command directory in windows

Author: Miguel Molina

generator/cli/kallax/cmd/migrate.go

@@ -3,6 +3,7 @@ package cmd
 import (
 	"fmt"
 	"path/filepath"
+	"strings"
 
 	"github.com/mattes/migrate"
 	_ "github.com/mattes/migrate/database/postgres"
@@ -146,7 +147,7 @@ func runMigrationAction(fn runMigrationFunc) cli.ActionFunc {
 			return fmt.Errorf("kallax: cannot get absolute path of `dir`: %s", err)
 		}
 
-		m, err := migrate.New(fmt.Sprintf("file://%s", dir), fmt.Sprintf("postgres://%s", dsn))
+		m, err := migrate.New(pathToFileURL(dir), fmt.Sprintf("postgres://%s", dsn))
 		if err != nil {
 			return fmt.Errorf("kallax: unable to open a connection with the database: %s", err)
 		}
@@ -155,6 +156,14 @@ func runMigrationAction(fn runMigrationFunc) cli.ActionFunc {
 	}
 }
 
+func pathToFileURL(path string) string {
+	path = strings.Replace(path, "\\", "/", -1)
+	if !strings.HasPrefix(path, "/") {
+		path = "/" + path
+	}
+	return fmt.Sprintf("file://%s", path)
+}
+
 func migrateAction(c *cli.Context) error {
 	dirs := c.StringSlice("input")
 	dir := c.String("out")

generator/cli/kallax/cmd/migrate_test.go

@@ -0,0 +1,21 @@
+package cmd
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/require"
+)
+
+func TestPathToFileURL(t *testing.T) {
+	cases := []struct {
+		input    string
+		expected string
+	}{
+		{`c:\foo\bar\baz`, "file:///c:/foo/bar/baz"},
+		{"/foo/bar/baz", "file:///foo/bar/baz"},
+	}
+
+	for _, tt := range cases {
+		require.Equal(t, tt.expected, pathToFileURL(tt.input), tt.input)
+	}
+}