directive wip

Signed-off-by: Kent Rancourt <kent.rancourt@gmail.com>

internal/directives/git_clone_directive.go

@@ -108,7 +108,7 @@ func (g *gitCloneDirective) run(
 		case checkout.Branch != "":
 			ref = checkout.Branch
 			if err = ensureRemoteBranch(repo, ref); err != nil {
-				return fmt.Errorf("error preparing branch %s: %w", ref, err)
+				return fmt.Errorf("error ensuring existence of remote branch %s: %w", ref, err)
 			}
 		case checkout.FromFreight:
 			var desiredOrigin *kargoapi.FreightOrigin
@@ -151,7 +151,7 @@ func (g *gitCloneDirective) run(
 		}
 	}
 	// Note: We do NOT defer repo.Close() because we want to keep the repository
-	// around onf the FS for subsequent directives to use. The directive execution
+	// around on the FS for subsequent directives to use. The directive execution
 	// engine will handle all work dir cleanup.
 	return nil
 }
@@ -181,12 +181,6 @@ func ensureRemoteBranch(repo git.BareRepo, branch string) error {
 		)
 	}
 	defer workTree.Close()
-	if err = workTree.Clear(); err != nil {
-		return fmt.Errorf(
-			"error clearing temporary working tree for branch %q of repo %s: %w",
-			branch, repo.URL(), err,
-		)
-	}
 	if err = workTree.Commit(
 		"Initial commit",
 		&git.CommitOptions{AllowEmpty: true},

internal/directives/git_commit_directive_test.go

@@ -89,3 +89,7 @@ func TestGitCommitDirective_Validate(t *testing.T) {
 		})
 	}
 }
+
+func TestGitCommitDirective_Run(t *testing.T) {
+	// TODO: Test this
+}

internal/directives/git_overwrite_directive_test.go

@@ -63,3 +63,7 @@ func TestGitOverwriteDirective_Validate(t *testing.T) {
 		})
 	}
 }
+
+func TestGitOverwriteDirective_Run(t *testing.T) {
+	// TODO: Test this
+}

internal/directives/git_push_directive_test.go

@@ -122,3 +122,7 @@ func TestGitPushDirective_Validate(t *testing.T) {
 		})
 	}
 }
+
+func TestGitPushDirective_Run(t *testing.T) {
+	// TODO: Test this
+}