Ensure git is always given an absolute repo path

Add simple unit test for readFile

Ensure all repo paths are absolute

cmd/console.go

@@ -196,7 +196,11 @@ func GetConsoleCommand() *cobra.Command {
 				}
 
 				if f.IsDir() {
-					repo := p
+					repo, err := absoluteRepoPath(p)
+					if err != nil {
+						pterm.Error.Println(err.Error())
+						return err
+					}
 					p = args[1]
 					f, err = os.Stat(filepath.Join(repo, p))
 					if err != nil {

cmd/html_report.go

@@ -218,7 +218,11 @@ func GetHTMLReportCommand() *cobra.Command {
 				}
 
 				if f.IsDir() {
-					repo := p
+					repo, err := absoluteRepoPath(p)
+					if err != nil {
+						pterm.Error.Println(err.Error())
+						return err
+					}
 					p = args[1]
 					f, err = os.Stat(filepath.Join(repo, p))
 					if err != nil {

cmd/paths.go

@@ -0,0 +1,20 @@
+package cmd
+
+import (
+	"fmt"
+	"os"
+	"path"
+	"path/filepath"
+)
+
+func absoluteRepoPath(p string) (string, error) {
+	if !path.IsAbs(p) {
+		wd, err := os.Getwd()
+		if err != nil {
+			return "", fmt.Errorf("get working dir: %v", err)
+		}
+		return filepath.Join(wd, p), nil
+	}
+
+	return p, nil
+}

cmd/paths_test.go

@@ -0,0 +1,26 @@
+package cmd
+
+import (
+	"path/filepath"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestAbsoluteRepoPath(t *testing.T) {
+	t.Run("makes relative path absolute", func(t *testing.T) {
+		p := "repo"
+
+		have, err := absoluteRepoPath(p)
+		assert.NoError(t, err)
+		assert.True(t, filepath.IsAbs(have))
+	})
+
+	t.Run("returns absolute path", func(t *testing.T) {
+		p := "/home/user/repo"
+
+		have, err := absoluteRepoPath(p)
+		assert.NoError(t, err)
+		assert.Equal(t, p, have)
+	})
+}

cmd/report.go

@@ -143,13 +143,10 @@ func GetReportCommand() *cobra.Command {
 				}
 
 				if f.IsDir() {
-					repo := p
-					if !path.IsAbs(repo) {
-						wd, err := os.Getwd()
-						if err != nil {
-							return fmt.Errorf("get working dir: %v", err)
-						}
-						repo = filepath.Join(wd, repo)
+					repo, err := absoluteRepoPath(p)
+					if err != nil {
+						pterm.Error.Println(err.Error())
+						return err
 					}
 					p = args[1]
 					f, err = os.Stat(filepath.Join(repo, p))

cmd/summary.go

@@ -216,8 +216,11 @@ func GetSummaryCommand() *cobra.Command {
 				}
 
 				if f.IsDir() {
-
-					repo := p
+					repo, err := absoluteRepoPath(p)
+					if err != nil {
+						pterm.Error.Println(err.Error())
+						return err
+					}
 					p = args[1]
 					f, err = os.Stat(filepath.Join(repo, p))
 					if err != nil {

git/read_local.go

@@ -132,10 +132,10 @@ func PopulateHistoryWithChanges(commitHistory []*model.Commit, limit int, limitT
 	progressChan chan *model.ProgressUpdate, errorChan chan model.ProgressError, base string, remote bool) ([]*model.Commit, []error) {
 
 	for c := range commitHistory {
-		var err []error
+		var err error
 		commitHistory[c].Data, err = readFile(commitHistory[c].RepoDirectory, commitHistory[c].Hash, commitHistory[c].FilePath)
 		if err != nil {
-			return nil, err
+			return nil, []error{err}
 		}
 		model.SendProgressUpdate("population",
 			fmt.Sprintf("Extracting %d bytes extracted from commit '%s'",
@@ -293,15 +293,15 @@ func ExtractPathAndFile(location string) (string, string) {
 
 // readFile reads the specified file at the specified commit hash from the
 // specified git repository.
-func readFile(repoDir, hash, filePath string) ([]byte, []error) {
+func readFile(repoDir, hash, filePath string) ([]byte, error) {
 	cmd := exec.Command(GIT, NOPAGER, SHOW, fmt.Sprintf("%s:%s", hash, filePath))
 	var ou, er bytes.Buffer
 	cmd.Stdout = &ou
 	cmd.Stderr = &er
 	cmd.Dir = repoDir
 	err := cmd.Run()
 	if err != nil {
-		return nil, []error{fmt.Errorf("read file from git: %v", err)}
+		return nil, fmt.Errorf("read file from git: %v", err)
 	}
 
 	return ou.Bytes(), nil

git/read_local_test.go

@@ -88,3 +88,9 @@ func TestGetTopLevel(t *testing.T) {
 	assert.NoError(t, err)
 	assert.NotEmpty(t, str)
 }
+
+func TestReadFile(t *testing.T) {
+	contentRaw, err := readFile("../", "HEAD", "./git/read_local.go")
+	assert.NoError(t, err)
+	assert.NotEmpty(t, contentRaw)
+}