refactor adding new git hook

internal/command/install.go

@@ -24,11 +24,8 @@ var InstallCmd = &cli.Command{
 // InstallHookCmd add git hook to .git/hooks/post-commit file
 func InstallHookCmd(c *cli.Context) error {
 	path := c.Path("path")
-	if err := git.ValidateRepo(path); err != nil {
-		return err
-	}
 
-	if err := git.CreateHookDir(path); err != nil {
+	if err := git.ValidateRepo(path); err != nil {
 		return err
 	}
 

pkg/git/git.go

@@ -47,18 +47,12 @@ func LastCommitStatsDiff(repoPath string) ([]byte, error) {
 	return out, nil
 }
 
-func CreateHookDir(path string) error {
-	hookDir := filepath.Join(path, ".git", "hooks")
-	if _, err := os.Stat(hookDir); os.IsNotExist(err) {
-		if err := os.MkdirAll(hookDir, os.ModeDir); err != nil {
-			return fmt.Errorf(hookDir+" directory creation failed: %v", err)
-		}
+// AddHook adds given content to the given hook file
+func AddHook(path string, hook, content string) error {
+	if err := createHookDir(path); err != nil {
+		return err
 	}
 
-	return nil
-}
-
-func AddHook(path string, hook, content string) error {
 	// TODO: prevent duplicate hook
 	// TODO: in test check file is executable
 	hookFilePath := filepath.Join(path, ".git", "hooks", hook)
@@ -75,3 +69,14 @@ func AddHook(path string, hook, content string) error {
 
 	return nil
 }
+
+func createHookDir(path string) error {
+	hookDir := filepath.Join(path, ".git", "hooks")
+	if _, err := os.Stat(hookDir); os.IsNotExist(err) {
+		if err := os.MkdirAll(hookDir, os.ModeDir); err != nil {
+			return fmt.Errorf(hookDir+" directory creation failed: %v", err)
+		}
+	}
+
+	return nil
+}