[#200] Make functions in vcs/command.go public

To be able to isolate git related code in their own subpackage
- rename
- adapt call sites

Author: philou

src/vcs/command.go

@@ -24,12 +24,14 @@ package vcs
 
 import "os/exec"
 
-func isCommandAvailable(command string) bool {
+// IsCommandAvailable indicates if the provided command is available in the path
+func IsCommandAvailable(command string) bool {
 	_, err := exec.LookPath(command)
 	return err == nil
 }
 
-func getCommandPath(command string) string {
+// GetCommandPath returns the full path to the provided command
+func GetCommandPath(command string) string {
 	path, _ := exec.LookPath(command)
 	return path
 }

src/vcs/command_test.go

@@ -30,18 +30,18 @@ import (
 )
 
 func Test_check_command_is_available_for_a_valid_command(t *testing.T) {
-	assert.True(t, isCommandAvailable("ls"))
+	assert.True(t, IsCommandAvailable("ls"))
 }
 
 func Test_check_command_is_available_for_an_invalid_command(t *testing.T) {
-	assert.False(t, isCommandAvailable("unknown-command"))
+	assert.False(t, IsCommandAvailable("unknown-command"))
 }
 
 func Test_check_command_path_for_a_valid_command(t *testing.T) {
-	base := filepath.Base(getCommandPath("ls"))
+	base := filepath.Base(GetCommandPath("ls"))
 	assert.Equal(t, strings.TrimSuffix(base, ".exe"), "ls")
 }
 
 func Test_check_command_path_for_an_invalid_command(t *testing.T) {
-	assert.Zero(t, getCommandPath("unknown-command"))
+	assert.Zero(t, GetCommandPath("unknown-command"))
 }

src/vcs/git_command.go

@@ -32,12 +32,12 @@ import (
 
 // IsGitCommandAvailable indicates if git command is available on local machine
 func IsGitCommandAvailable() bool {
-	return isCommandAvailable("git")
+	return IsCommandAvailable("git")
 }
 
 // GetGitCommandPath returns the path to git command on this machine
 func GetGitCommandPath() string {
-	return getCommandPath("git")
+	return GetCommandPath("git")
 }
 
 // GetGitCommandVersion returns the version of git command on this machine