Revert "fix tests/integration/recipe_new.bats"

This reverts commit 0aac464ded.

cli/internal/cli.go

@@ -238,22 +238,6 @@ var RemoteUserFlag = &cli.StringFlag{
 	Destination: &RemoteUser,
 }
 
-var GitUser string
-var GitUserFlag = &cli.StringFlag{
-	Name:        "git-user, gu",
-	Value:       "",
-	Usage:       "Git user name to do commits with",
-	Destination: &GitUser,
-}
-
-var GitEmail string
-var GitEmailFlag = &cli.StringFlag{
-	Name:        "git-email, ge",
-	Value:       "",
-	Usage:       "Git email name to do commits with",
-	Destination: &GitEmail,
-}
-
 // SubCommandBefore wires up pre-action machinery (e.g. --debug handling).
 func SubCommandBefore(c *cli.Context) error {
 	if Debug {

cli/recipe/new.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"errors"
 	"fmt"
+	"io/ioutil"
 	"os"
 	"path"
 	"text/template"
@@ -36,8 +37,6 @@ var recipeNewCommand = cli.Command{
 		internal.DebugFlag,
 		internal.NoInputFlag,
 		internal.OfflineFlag,
-		internal.GitUserFlag,
-		internal.GitEmailFlag,
 	},
 	Before:    internal.SubCommandBefore,
 	Usage:     "Create a new recipe",
@@ -93,14 +92,14 @@ recipe and domain in the sample environment config).
 				logrus.Fatal(err)
 			}
 
-			if err := os.WriteFile(path, templated.Bytes(), 0644); err != nil {
+			if err := ioutil.WriteFile(path, templated.Bytes(), 0644); err != nil {
 				logrus.Fatal(err)
 			}
 
 		}
 
 		newGitRepo := path.Join(config.RECIPES_DIR, recipeName)
-		if err := git.Init(newGitRepo, true, internal.GitUser, internal.GitEmail); err != nil {
+		if err := git.Init(newGitRepo, true); err != nil {
 			logrus.Fatal(err)
 		}
 

pkg/git/init.go

@@ -2,13 +2,13 @@ package git
 
 import (
 	"github.com/go-git/go-git/v5"
-	"github.com/go-git/go-git/v5/plumbing/object"
+	gitPkg "github.com/go-git/go-git/v5"
 	"github.com/sirupsen/logrus"
 )
 
 // Init inits a new repo and commits all the stuff if you want
-func Init(repoPath string, commit bool, gitUser, gitEmail string) error {
-	if _, err := git.PlainInit(repoPath, false); err != nil {
+func Init(repoPath string, commit bool) error {
+	if _, err := gitPkg.PlainInit(repoPath, false); err != nil {
 		logrus.Fatal(err)
 	}
 	logrus.Debugf("initialised new git repo in %s", repoPath)
@@ -28,12 +28,7 @@ func Init(repoPath string, commit bool, gitUser, gitEmail string) error {
 			return err
 		}
 
-		var author *object.Signature
-		if gitUser != "" && gitEmail != "" {
-			author = &object.Signature{Name: gitUser, Email: gitEmail}
-		}
-
-		if _, err = commitWorktree.Commit("init", &git.CommitOptions{Author: author}); err != nil {
+		if _, err = commitWorktree.Commit("init", &git.CommitOptions{}); err != nil {
 			return err
 		}
 		logrus.Debugf("init committed all files for new git repo in %s", repoPath)

tests/integration/recipe_new.bats

@@ -23,14 +23,14 @@ teardown(){
 }
 
 @test "create new recipe" {
-  run $ABRA recipe new foobar --git-user foo --git-email foo@example.com
+  run $ABRA recipe new foobar
   assert_success
   assert_output --partial 'Your new foobar recipe has been created'
   assert_exists "$ABRA_DIR/recipes/foobar"
 }
 
 @test "create new app from new recipe" {
-  run $ABRA recipe new foobar --git-user foo --git-email foo@example.com
+  run $ABRA recipe new foobar
   assert_success
 
   run $ABRA app new foobar \