Enable running scripts in parallel (#139)

* Make it possible to run multiple scripts in parallel
* Code review changes
* Fix references after rebase on main

Author: mads-hartmann

cmd/run.go

@@ -1,23 +1,37 @@
 package cmd
 
 import (
+	"errors"
+
 	log "github.com/sirupsen/logrus"
 	"github.com/spf13/cobra"
+	"golang.org/x/sync/errgroup"
 )
 
 // runCmd represents the version command
 var runCmd = &cobra.Command{
-	Use:   "run <script>",
-	Short: "Executes a script",
-	Args:  cobra.ExactArgs(1),
+	Use:   "run [scripts]",
+	Short: "Executes one or more scripts in parallel",
+	Long: `Executes one or more scripts in parallel
+All scripts will run to completion, regardless of whether or not the other scripts exit with errors.
+
+Should any of the scripts fail Leeway will exit with an exit code of 1 once all scripts are done executing.
+`,
+	Args: cobra.MinimumNArgs(1),
 	Run: func(cmd *cobra.Command, args []string) {
-		_, _, script, _ := getTarget(args, true)
-		if script == nil {
-			log.Fatal("tree needs a package")
+		g := new(errgroup.Group)
+		for _, scriptName := range args {
+			scriptName := scriptName
+			g.Go(func() error {
+				_, _, script, _ := getTarget([]string{scriptName}, true)
+				if script == nil {
+					return errors.New("run needs a script")
+				}
+				opts, _ := getBuildOpts(cmd)
+				return script.Run(opts...)
+			})
 		}
-
-		opts, _ := getBuildOpts(cmd)
-		err := script.Run(opts...)
+		err := g.Wait()
 		if err != nil {
 			log.Fatal(err)
 		}

pkg/leeway/build_test.go

@@ -36,7 +36,7 @@ fi
 `
 
 func TestBuildDockerDeps(t *testing.T) {
-	if *dut {
+	if *testutil.Dut {
 		pth, err := os.MkdirTemp("", "")
 		if err != nil {
 			t.Fatal(err)
@@ -50,9 +50,9 @@ func TestBuildDockerDeps(t *testing.T) {
 		os.Setenv("PATH", pth+":"+os.Getenv("PATH"))
 		log.WithField("path", os.Getenv("PATH")).Debug("modified path to use dummy docker")
 	}
-	runDUT()
+	testutil.RunDUT()
 
-	tests := []*CommandFixtureTest{
+	tests := []*testutil.CommandFixtureTest{
 		{
 			Name:        "docker dependency",
 			T:           t,

pkg/leeway/fixtures/scripts.yaml

@@ -253,7 +253,7 @@ func executeBashScript(script string, wd string, env []string) error {
 	err = cmd.Run()
 	if exiterr, ok := err.(*exec.ExitError); ok {
 		if status, ok := exiterr.Sys().(syscall.WaitStatus); ok {
-			os.Exit(status.ExitStatus())
+			return xerrors.Errorf("failed with exit code %d", status.ExitStatus())
 		}
 	}
 	return err