encapsulate paths with quotes to escape spaces

runner/config.go

@@ -9,6 +9,7 @@ import (
 	"reflect"
 	"regexp"
 	"runtime"
+	"strings"
 	"time"
 
 	"dario.cat/mergo"
@@ -305,9 +306,7 @@ func (c *Config) preprocess() error {
 	if c.TestDataDir == "" {
 		c.TestDataDir = "testdata"
 	}
-	if err != nil {
-		return err
-	}
+
 	ed := c.Build.ExcludeDir
 	for i := range ed {
 		ed[i] = cleanPath(ed[i])
@@ -328,6 +327,9 @@ func (c *Config) preprocess() error {
 	// CMD will not recognize relative path like ./tmp/server
 	c.Build.Bin, err = filepath.Abs(c.Build.Bin)
 
+	// Account for spaces in filepath
+	c.Build.Bin = fmt.Sprintf("%q", c.Build.Bin)
+
 	return err
 }
 
@@ -363,7 +365,8 @@ func (c *Config) killDelay() time.Duration {
 }
 
 func (c *Config) binPath() string {
-	return filepath.Join(c.Root, c.Build.Bin)
+	bin := strings.Trim(c.Build.Bin, "\"")
+	return fmt.Sprintf("%q", filepath.Join(c.Root, bin))
 }
 
 func (c *Config) tmpPath() string {

runner/config_test.go

@@ -117,16 +117,49 @@ func TestReadConfByName(t *testing.T) {
 }
 
 func TestConfPreprocess(t *testing.T) {
-	t.Setenv(airWd, "_testdata/toml")
-	df := defaultConfig()
-	err := df.preprocess()
-	if err != nil {
-		t.Fatalf("preprocess error %v", err)
+	tests := []struct {
+		name   string
+		space  bool
+		suffix string
+	}{
+		{
+			name:   "no spaces",
+			space:  false,
+			suffix: "/_testdata/toml/tmp/main\"",
+		},
+		{
+			name:   "with spaces",
+			space:  true,
+			suffix: "/_testdata/toml/tmp space/main\"",
+		},
 	}
-	suffix := "/_testdata/toml/tmp/main"
-	binPath := df.Build.Bin
-	if !strings.HasSuffix(binPath, suffix) {
-		t.Fatalf("bin path is %s, but not have suffix  %s.", binPath, suffix)
+
+	for _, tt := range tests {
+
+		oWD, err := os.Getwd()
+		if err != nil {
+			t.Fatalf("failed to get currWD: %v", err)
+		}
+
+		t.Setenv(airWd, "_testdata/toml")
+		df := defaultConfig()
+		if tt.space {
+			df.Build.Bin = "./tmp space/main"
+		}
+		err = df.preprocess()
+		if err != nil {
+			t.Fatalf("%s: preprocess error %v", tt.name, err)
+		}
+
+		binPath := df.Build.Bin
+		if !strings.HasSuffix(binPath, tt.suffix) {
+			t.Fatalf("%s: bin path is %s, but not have suffix  %s.", tt.name, binPath, tt.suffix)
+		}
+
+		err = os.Chdir(oWD)
+		if err != nil {
+			t.Fatalf("failed to change back to original WD: %v", err)
+		}
 	}
 }