From a4d9e44c61c970d7b61612e4b9eb0865be91b473 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=D7=A0=CF=85=CE=B1=CE=B7=20=D7=A0=CF=85=CE=B1=CE=B7=D1=95?=
 =?UTF-8?q?=CF=83=CE=B7?= <john@dewey.ws>
Date: Thu, 7 Mar 2024 21:17:50 -0800
Subject: [PATCH] chore: coverage++

Increased coverage on git worktree.
---
 internal/git/git.go      | 13 +++++-
 internal/git/git_test.go | 91 ++++++++++++++++++++++++++++++++++++++++
 2 files changed, 103 insertions(+), 1 deletion(-)
 create mode 100644 internal/git/git_test.go

diff --git a/internal/git/git.go b/internal/git/git.go
index 1bdb062..1d08687 100644
--- a/internal/git/git.go
+++ b/internal/git/git.go
@@ -31,6 +31,13 @@ import (
 	"github.com/retr0h/gilt/v2/internal"
 )
 
+// AbsFn function to switch when testing.
+// The only time `Abs` will return an error is when `os.Getwd()` returns an
+// error.  Given this situation is rare, but "it's a weird, wild world out there"
+// opted to retain he error handling, but clumsily swap the test function for
+// coverage++.
+var AbsFn = (*Git).abs
+
 // New factory to create a new Git instance.
 func New(
 	appFs avfs.VFS,
@@ -69,7 +76,7 @@ func (g *Git) Worktree(
 	version string,
 	dstDir string,
 ) error {
-	dst, err := g.appFs.Abs(dstDir)
+	dst, err := AbsFn(g, dstDir)
 	if err != nil {
 		return err
 	}
@@ -94,3 +101,7 @@ func (g *Git) Worktree(
 	}
 	return err
 }
+
+func (g *Git) abs(path string) (string, error) {
+	return g.appFs.Abs(path)
+}
diff --git a/internal/git/git_test.go b/internal/git/git_test.go
new file mode 100644
index 0000000..e13ad01
--- /dev/null
+++ b/internal/git/git_test.go
@@ -0,0 +1,91 @@
+// Copyright (c) 2023 John Dewey
+
+// Permission is hereby granted, free of charge, to any person obtaining a copy
+// of this software and associated documentation files (the "Software"), to
+// deal in the Software without restriction, including without limitation the
+// rights to use, copy, modify, merge, publish, distribute, sublicense, and/or
+// sell copies of the Software, and to permit persons to whom the Software is
+// furnished to do so, subject to the following conditions:
+
+// The above copyright notice and this permission notice shall be included in
+// all copies or substantial portions of the Software.
+
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+// FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+// DEALINGS IN THE SOFTWARE.
+
+package git
+
+import (
+	"fmt"
+	"log/slog"
+	"os"
+	"testing"
+
+	"github.com/avfs/avfs/vfs/memfs"
+	"github.com/golang/mock/gomock"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/suite"
+
+	"github.com/retr0h/gilt/v2/internal"
+	"github.com/retr0h/gilt/v2/internal/mocks/exec"
+)
+
+type GitManagerTestSuite struct {
+	suite.Suite
+
+	ctrl     *gomock.Controller
+	mockExec *exec.MockExecManager
+
+	gitURL     string
+	gitVersion string
+	cloneDir   string
+	dstDir     string
+
+	gm internal.GitManager
+}
+
+func (suite *GitManagerTestSuite) NewTestGitManager() internal.GitManager {
+	return New(
+		memfs.New(),
+		suite.mockExec,
+		slog.New(slog.NewTextHandler(os.Stdout, nil)),
+	)
+}
+
+func (suite *GitManagerTestSuite) SetupTest() {
+	suite.ctrl = gomock.NewController(suite.T())
+	suite.mockExec = exec.NewMockExecManager(suite.ctrl)
+
+	suite.gitURL = "https://example.com/user/repo.git"
+	suite.gitVersion = "abc123"
+	suite.cloneDir = "/cloneDir"
+	suite.dstDir = "/dstDir"
+
+	suite.gm = suite.NewTestGitManager()
+}
+
+func (suite *GitManagerTestSuite) TearDownTest() {
+	defer suite.ctrl.Finish()
+}
+
+func (suite *GitManagerTestSuite) TestWorktreeErrorWhenAbsErrors() {
+	originalAbsFn := AbsFn
+	AbsFn = func(g *Git, _ string) (string, error) {
+		return "", fmt.Errorf("failed to get abs path")
+	}
+	defer func() { AbsFn = originalAbsFn }()
+
+	err := suite.gm.Worktree(suite.cloneDir, suite.gitVersion, suite.dstDir)
+	assert.Error(suite.T(), err)
+}
+
+// In order for `go test` to run this suite, we need to create
+// a normal test function and pass our suite to suite.Run.
+func TestGitManagerTestSuite(t *testing.T) {
+	suite.Run(t, new(GitManagerTestSuite))
+}