refactor: pull actions as actions manager

internal/actions/act.go

@@ -2,7 +2,6 @@
 package actions
 
 import (
-	logger "courgette/internal/logging"
 	"strings"
 )
 
@@ -21,13 +20,3 @@ func GetActionKey(url string) string {
 
 	return strings.Join(parts, "__")
 }
-
-// Prefetches and caches the action defined by <actionName> at <destination>.
-//
-// <actionName> is expected to be the full url of the repository where
-// the action lives.
-func PrefetchAction(client CliClient, actionName string, destination string) error {
-	logger.Info("Prefetching action: %s to %s", actionName, destination)
-
-	return client.Clone(actionName, destination)
-}

internal/actions/manager.go

@@ -0,0 +1,25 @@
+package actions
+
+import (
+	logger "courgette/internal/logging"
+)
+
+type ActionsManager struct {
+	git CliClient
+}
+
+func NewActionsManager() ActionsManager {
+	return ActionsManager{
+		git: GitClient{},
+	}
+}
+
+// Prefetches and caches the action defined by <actionName> at <destination>.
+//
+// <actionName> is expected to be the full url of the repository where
+// the action lives.
+func (a ActionsManager) PrefetchAction(actionName string, destination string) error {
+	logger.Info("Prefetching action: %s to %s", actionName, destination)
+
+	return a.git.Clone(actionName, destination)
+}

internal/runner/runner.go

@@ -49,7 +49,7 @@ func (r *Runner) RunWorkflow(workflow workflows.Workflow) TaskTracker {
 		switch n.(type) {
 		case workflows.Step:
 			if useAction := n.(workflows.Step).Use; useAction != "" {
-				actions.PrefetchAction(actions.GitClient{}, useAction, r.Cache.Path(actions.GetActionKey(useAction)))
+				actions.NewActionsManager().PrefetchAction(useAction, r.Cache.Path(actions.GetActionKey(useAction)))
 			}
 		}
 	})