add tests for get path

Author: garrettkrohn

cloner/clone_test.go

@@ -0,0 +1,36 @@
+package cloner
+
+import (
+	"os"
+	"testing"
+
+	"github.com/joshmedeski/sesh/model"
+	"github.com/stretchr/testify/assert"
+)
+
+func TestListSessions(t *testing.T) {
+	t.Run("get path with both cmd and repo", func(t *testing.T) {
+		mockOpts := model.GitCloneOptions{Repo: "https://www.github.comtest/repo.git", CmdDir: "cmdDir", Dir: "dir"}
+		actual := getPath(mockOpts)
+		assert.Equal(t, "cmdDir/dir", actual)
+	})
+	t.Run("get path cmdDir", func(t *testing.T) {
+		mockOpts := model.GitCloneOptions{Repo: "https://www.github.comtest/repo.git", CmdDir: "cmdDir"}
+		actual := getPath(mockOpts)
+		assert.Equal(t, "cmdDir/repo", actual)
+	})
+	t.Run("get path dir", func(t *testing.T) {
+		mockOpts := model.GitCloneOptions{Repo: "https://www.github.comtest/repo.git", Dir: "dir"}
+		actual := getPath(mockOpts)
+		expected, _ := os.Getwd()
+		expected = expected + "/" + "dir"
+		assert.Equal(t, expected, actual)
+	})
+	t.Run("get path with no dir or cmdDir", func(t *testing.T) {
+		mockOpts := model.GitCloneOptions{Repo: "https://www.github.comtest/repo.git"}
+		actual := getPath(mockOpts)
+		expected, _ := os.Getwd()
+		expected = expected + "/" + "repo"
+		assert.Equal(t, expected, actual)
+	})
+}

cloner/cloner.go

@@ -31,6 +31,18 @@ func (c *RealCloner) Clone(opts model.GitCloneOptions) (string, error) {
 		return "", err
 	}
 
+	path := getPath(opts)
+
+	newOpts := model.ConnectOpts{}
+	if _, err := c.connector.Connect(path, newOpts); err != nil {
+		return "", err
+	}
+
+	return "", nil
+
+}
+
+func getPath(opts model.GitCloneOptions) string {
 	var path string
 	if opts.CmdDir != "" {
 		path = opts.CmdDir
@@ -44,14 +56,7 @@ func (c *RealCloner) Clone(opts model.GitCloneOptions) (string, error) {
 		repoName := getRepoName(opts.Repo)
 		path = path + "/" + repoName
 	}
-
-	newOpts := model.ConnectOpts{}
-	if _, err := c.connector.Connect(path, newOpts); err != nil {
-		return "", err
-	}
-
-	return "", nil
-
+	return path
 }
 
 func getRepoName(url string) string {