separate run test

don't depend on binary

ghbackup/run_test.go

@@ -0,0 +1,43 @@
+package ghbackup_test
+
+import (
+	"io/ioutil"
+	"os"
+	"testing"
+
+	"qvl.io/ghbackup/ghbackup"
+)
+
+func TestRun(t *testing.T) {
+	dir, err := ioutil.TempDir("", "qvl-backup")
+	if err != nil {
+		t.Error(err)
+	}
+	defer func() {
+		err := os.RemoveAll(dir)
+		if err != nil {
+			t.Error(err)
+		}
+	}()
+
+	updates := make(chan ghbackup.Update)
+	go func() {
+		for u := range updates {
+			switch u.Type {
+			case ghbackup.UErr:
+				t.Error("Unexpected error:", u.Message)
+			}
+		}
+	}()
+
+	err = ghbackup.Run(ghbackup.Config{
+		Account: "qvl",
+		Dir:     dir,
+		Secret:  os.Getenv("SECRET"),
+		Updates: updates,
+	})
+
+	if err != nil {
+		t.Error("Unexpected error:", err)
+	}
+}

main_test.go

@@ -1,9 +1,7 @@
-package main
+package main_test
 
 import (
 	"bytes"
-	"io/ioutil"
-	"os"
 	"os/exec"
 	"testing"
 )
@@ -48,40 +46,6 @@ func TestHelp(t *testing.T) {
 	}
 }
 
-func TestRun(t *testing.T) {
-	dir, err := ioutil.TempDir("", "qvl-backup")
-	if err != nil {
-		t.Error(err)
-	}
-	defer func() {
-		err := os.RemoveAll(dir)
-		if err != nil {
-			t.Error(err)
-		}
-	}()
-
-	// Use secret from environment if available.
-	// Prevents rate limiting on CI server.
-	secret := os.Getenv("SECRET")
-	var args []string
-	if secret == "" {
-		args = []string{"-account", "qvl", dir}
-	} else {
-		args = []string{"-account", "qvl", "-secret", secret, dir}
-	}
-
-	stdout, stderr, ok := run(args)
-	if !ok {
-		t.Error("Non-zero exit code")
-	}
-	if stdout != "" {
-		t.Error("Unexpected stdout:", stdout)
-	}
-	if stderr != "" {
-		t.Error("Unexpected stderr:", stderr)
-	}
-}
-
 func run(a []string) (string, string, bool) {
 	var outbuf, errbuf bytes.Buffer
 	cmd := exec.Command("ghbackup", a...)