- extracted defaultBin constant to an extra package as it is reused i…

…n main and lib.

- added detection of home directory in case we are using windows.
- updated tests accordingly.

.github/workflows/go.yml

@@ -19,7 +19,7 @@ jobs:
     - uses: actions/checkout@v4
 
     - name: Set up Go
-      uses: actions/setup-go@v4
+      uses: actions/setup-go@v5
       with:
         go-version: ${{ matrix.go_version }}
 

go.mod

@@ -5,35 +5,41 @@ go 1.22
 require (
 	github.com/hashicorp/go-version v1.6.0
 	github.com/hashicorp/hcl2 v0.0.0-20191002203319-fb75b3253c80
-	github.com/hashicorp/terraform-config-inspect v0.0.0-20211115214459-90acf1ca460f
+	github.com/hashicorp/terraform-config-inspect v0.0.0-20231204233900-a34142ec2a72
 	github.com/manifoldco/promptui v0.9.0
 	github.com/mitchellh/go-homedir v1.1.0
 	github.com/pborman/getopt v1.1.0
-	github.com/spf13/viper v1.12.0
+	github.com/spf13/viper v1.18.2
 	golang.org/x/sys v0.18.0
 )
 
 require (
 	github.com/agext/levenshtein v1.2.2 // indirect
 	github.com/apparentlymart/go-textseg v1.0.0 // indirect
 	github.com/chzyer/readline v0.0.0-20180603132655-2972be24d48e // indirect
-	github.com/fsnotify/fsnotify v1.5.4 // indirect
-	github.com/google/go-cmp v0.5.8 // indirect
+	github.com/fsnotify/fsnotify v1.7.0 // indirect
+	github.com/google/go-cmp v0.5.9 // indirect
 	github.com/hashicorp/hcl v1.0.0 // indirect
 	github.com/hashicorp/hcl/v2 v2.0.0 // indirect
-	github.com/magiconair/properties v1.8.6 // indirect
+	github.com/magiconair/properties v1.8.7 // indirect
 	github.com/mitchellh/go-wordwrap v1.0.0 // indirect
 	github.com/mitchellh/mapstructure v1.5.0 // indirect
 	github.com/pelletier/go-toml v1.9.5 // indirect
-	github.com/pelletier/go-toml/v2 v2.0.1 // indirect
-	github.com/spf13/afero v1.8.2 // indirect
-	github.com/spf13/cast v1.5.0 // indirect
+	github.com/pelletier/go-toml/v2 v2.1.0 // indirect
+	github.com/sagikazarmark/locafero v0.4.0 // indirect
+	github.com/sagikazarmark/slog-shim v0.1.0 // indirect
+	github.com/sourcegraph/conc v0.3.0 // indirect
+	github.com/spf13/afero v1.11.0 // indirect
+	github.com/spf13/cast v1.6.0 // indirect
 	github.com/spf13/jwalterweatherman v1.1.0 // indirect
 	github.com/spf13/pflag v1.0.5 // indirect
-	github.com/subosito/gotenv v1.3.0 // indirect
+	github.com/subosito/gotenv v1.6.0 // indirect
 	github.com/zclconf/go-cty v1.1.0 // indirect
-	golang.org/x/text v0.7.0 // indirect
-	gopkg.in/ini.v1 v1.66.4 // indirect
+	go.uber.org/atomic v1.9.0 // indirect
+	go.uber.org/multierr v1.9.0 // indirect
+	golang.org/x/exp v0.0.0-20230905200255-921286631fa9 // indirect
+	golang.org/x/text v0.14.0 // indirect
+	gopkg.in/ini.v1 v1.67.0 // indirect
 	gopkg.in/yaml.v2 v2.4.0 // indirect
-	gopkg.in/yaml.v3 v3.0.0 // indirect
+	gopkg.in/yaml.v3 v3.0.1 // indirect
 )

lib/defaults.go

@@ -0,0 +1,20 @@
+package lib
+
+import (
+	"github.com/mitchellh/go-homedir"
+	"log"
+	"runtime"
+)
+
+// GetDefaultBin Get default binary path
+func GetDefaultBin() string {
+	var defaultBin = "/usr/local/bin/terraform"
+	if runtime.GOOS == "windows" {
+		home, err := homedir.Dir()
+		if err != nil {
+			log.Fatalf(`Could not detect home directory.`)
+		}
+		defaultBin = home + `/bin/terraform.exe`
+	}
+	return defaultBin
+}

lib/download_test.go

@@ -2,13 +2,14 @@ package lib_test
 
 import (
 	"fmt"
+	"github.com/mitchellh/go-homedir"
 	"log"
 	"net/url"
 	"os"
 	"path/filepath"
+	"runtime"
 	"testing"
 
-	"github.com/mitchellh/go-homedir"
 	"github.com/warrensbox/terraform-switcher/lib"
 )
 
@@ -21,14 +22,19 @@ func TestDownloadFromURL_FileNameMatch(t *testing.T) {
 	installPath := fmt.Sprintf(tempDir + string(os.PathSeparator) + ".terraform.versions_test")
 	macOS := "_darwin_amd64.zip"
 
-	// get current user
-	homedir, errCurr := homedir.Dir()
-	if errCurr != nil {
-		log.Fatal(errCurr)
+	home, err := homedir.Dir()
+	if err != nil {
+		log.Fatalf(`Could not detect home directory.`)
 	}
 
-	fmt.Printf("Current home directory: %v \n", homedir)
-	installLocation := filepath.Join(homedir, installPath)
+	fmt.Printf("Current user homedir: %v \n", home)
+	var installLocation = ""
+	if runtime.GOOS != "windows" {
+		installLocation = filepath.Join(home, installPath)
+	} else {
+		installLocation = installPath
+	}
+	fmt.Printf("Install Location: %v \n", installLocation)
 
 	// create /.terraform.versions_test/ directory to store code
 	if _, err := os.Stat(installLocation); os.IsNotExist(err) {
@@ -44,7 +50,7 @@ func TestDownloadFromURL_FileNameMatch(t *testing.T) {
 	lowestVersion := "0.11.0"
 
 	url := hashiURL + lowestVersion + "/" + installVersion + lowestVersion + macOS
-	expectedFile := filepath.Join(homedir, installPath, installVersion+lowestVersion+macOS)
+	expectedFile := filepath.Join(installLocation, installVersion+lowestVersion+macOS)
 	installedFile, errDownload := lib.DownloadFromURL(installLocation, url)
 
 	if errDownload != nil {
@@ -63,7 +69,7 @@ func TestDownloadFromURL_FileNameMatch(t *testing.T) {
 	}
 
 	//check file name is what is expected
-	_, err := os.Stat(expectedFile)
+	_, err = os.Stat(expectedFile)
 	if err != nil {
 		t.Logf("Expected file does not exist %v", expectedFile)
 	}

lib/install.go

@@ -32,7 +32,7 @@ func initialize(binPath string) {
 	cmd := NewCommand(binPath)
 	next := cmd.Find()
 
-	/* overrride installation default binary path if terraform is already installed */
+	/* override installation default binary path if terraform is already installed */
 	/* find the last bin path */
 	for path := next(); len(path) > 0; path = next() {
 		binPath = path
@@ -79,7 +79,7 @@ func Install(tfversion string, binPath string, mirrorURL string) {
 
 	/* Check to see if user has permission to the default bin location which is  "/usr/local/bin/terraform"
 	 * If user does not have permission to default bin location, proceed to create $HOME/bin and install the tfswitch there
-	 * Inform user that they dont have permission to default location, therefore tfswitch was installed in $HOME/bin
+	 * Inform user that they don't have permission to default location, therefore tfswitch was installed in $HOME/bin
 	 * Tell users to add $HOME/bin to their path
 	 */
 	binPath = InstallableBinLocation(binPath)
@@ -226,7 +226,7 @@ func GetRecentVersions() ([]string, error) {
 
 		for _, line := range lines {
 			/* 	checks if versions in the recent file are valid.
-			If any version is invalid, it will be consider dirty
+			If any version is invalid, it will be considered dirty
 			and the recent file will be removed
 			*/
 			if !ValidVersionFormat(line) {
@@ -279,7 +279,7 @@ func InstallableBinLocation(userBinPath string) string {
 	binDir := Path(userBinPath)           //get path directory from binary path
 	binPathExist := CheckDirExist(binDir) //the default is /usr/local/bin but users can provide custom bin locations
 
-	if binPathExist { //if bin path exist - check if we can write to to it
+	if binPathExist == true { //if bin path exist - check if we can write to it
 
 		binPathWritable := false //assume bin path is not writable
 		if runtime.GOOS != "windows" {

lib/symlink_test.go

@@ -1,51 +1,74 @@
 package lib_test
 
 import (
+	"github.com/mitchellh/go-homedir"
 	"log"
 	"os"
 	"path/filepath"
+	"runtime"
 	"testing"
 
-	"github.com/mitchellh/go-homedir"
 	"github.com/warrensbox/terraform-switcher/lib"
 )
 
 // TestCreateSymlink : check if symlink exist-remove if exist,
 // create symlink, check if symlink exist, remove symlink
 func TestCreateSymlink(t *testing.T) {
 
+	testSymlinkDest := "/test-tfswitcher-dest"
 	testSymlinkSrc := "/test-tfswitcher-src"
+	if runtime.GOOS == "windows" {
+		testSymlinkSrc = "/test-tfswitcher-src.exe"
+	}
 
-	testSymlinkDest := "/test-tfswitcher-dest"
+	home, err := homedir.Dir()
+	if err != nil {
+		log.Fatalf(`Could not detect home directory.`)
+	}
+	symlinkPathSrc := filepath.Join(home, testSymlinkSrc)
+	symlinkPathDest := filepath.Join(home, testSymlinkDest)
 
-	homedir, errCurr := homedir.Dir()
-	if errCurr != nil {
-		log.Fatal(errCurr)
+	// Create file for test as windows does not like no source
+	create, err := os.Create(symlinkPathDest)
+	if err != nil {
+		log.Fatalf(`Could not create test dest file for symlink at %v`, symlinkPathDest)
 	}
-	symlinkPathSrc := filepath.Join(homedir, testSymlinkSrc)
-	symlinkPathDest := filepath.Join(homedir, testSymlinkDest)
+	defer create.Close()
 
-	ln, _ := os.Readlink(symlinkPathSrc)
+	if runtime.GOOS != "windows" {
+		ln, _ := os.Readlink(symlinkPathSrc)
 
-	if ln != symlinkPathDest {
-		t.Logf("Symlink does not exist %v [expected]", ln)
-	} else {
-		t.Logf("Symlink exist %v [expected]", ln)
-		os.Remove(symlinkPathSrc)
-		t.Logf("Removed existing symlink for testing purposes")
+		if ln != symlinkPathDest {
+			t.Logf("Symlink does not exist %v [expected]", ln)
+		} else {
+			t.Logf("Symlink exist %v [expected]", ln)
+			os.Remove(symlinkPathSrc)
+			t.Logf("Removed existing symlink for testing purposes")
+		}
 	}
 
 	lib.CreateSymlink(symlinkPathDest, symlinkPathSrc)
 
-	lnCheck, _ := os.Readlink(symlinkPathSrc)
-	if lnCheck == symlinkPathDest {
-		t.Logf("Symlink exist %v [expected]", lnCheck)
+	if runtime.GOOS == "windows" {
+		_, err := os.Stat(symlinkPathSrc + ".exe")
+		if err != nil {
+			t.Logf("Could not stat file copy at %v. [unexpected]", symlinkPathSrc)
+			t.Error("File copy was not created.")
+		} else {
+			t.Logf("File copy exists at %v [expected]", symlinkPathSrc)
+		}
 	} else {
-		t.Logf("Symlink does not exist %v [unexpected]", lnCheck)
-		t.Error("Symlink was not created")
+		lnCheck, _ := os.Readlink(symlinkPathSrc)
+		if lnCheck == symlinkPathDest {
+			t.Logf("Symlink exist %v [expected]", lnCheck)
+		} else {
+			t.Logf("Symlink does not exist %v [unexpected]", lnCheck)
+			t.Error("Symlink was not created")
+		}
 	}
 
 	os.Remove(symlinkPathSrc)
+	os.Remove(symlinkPathDest)
 }
 
 // TestRemoveSymlink : check if symlink exist-create if does not exist,

main.go

@@ -7,7 +7,7 @@ package main
 
 /*** OPERATION WORKFLOW ***/
 /*
-* 1- Create /usr/local/terraform directory if does not exist
+* 1- Create /usr/local/terraform directory if it does not exist
 * 2- Download zip file from url to /usr/local/terraform
 * 3- Unzip the file to /usr/local/terraform
 * 4- Rename the file from `terraform` to `terraform_version`
@@ -40,7 +40,6 @@ import (
 
 const (
 	defaultMirror = "https://releases.hashicorp.com/terraform"
-	defaultBin    = "/usr/local/bin/terraform" //default bin installation dir
 	defaultLatest = ""
 	tfvFilename   = ".terraform-version"
 	rcFilename    = ".tfswitchrc"
@@ -53,7 +52,7 @@ var version = "0.12.0\n"
 
 func main() {
 	dir := lib.GetCurrentDirectory()
-	custBinPath := getopt.StringLong("bin", 'b', lib.ConvertExecutableExt(defaultBin), "Custom binary path. Ex: tfswitch -b "+lib.ConvertExecutableExt("/Users/username/bin/terraform"))
+	custBinPath := getopt.StringLong("bin", 'b', lib.ConvertExecutableExt(lib.GetDefaultBin()), "Custom binary path. Ex: tfswitch -b "+lib.ConvertExecutableExt("/Users/username/bin/terraform"))
 	listAllFlag := getopt.BoolLong("list-all", 'l', "List all versions of terraform - including beta and rc")
 	latestPre := getopt.StringLong("latest-pre", 'p', defaultLatest, "Latest pre-release implicit version. Ex: tfswitch --latest-pre 0.13 downloads 0.13.0-rc1 (latest)")
 	showLatestPre := getopt.StringLong("show-latest-pre", 'P', defaultLatest, "Show latest pre-release implicit version. Ex: tfswitch --show-latest-pre 0.13 prints 0.13.0-rc1 (latest)")
@@ -387,8 +386,8 @@ func getParamsTOML(binPath string, dir string) (string, string) {
 		os.Exit(1) // exit immediately if config file provided but it is unable to read it
 	}
 
-	bin := viper.Get("bin")                                            // read custom binary location
-	if binPath == lib.ConvertExecutableExt(defaultBin) && bin != nil { // if the bin path is the same as the default binary path and if the custom binary is provided in the toml file (use it)
+	bin := viper.Get("bin")                                                     // read custom binary location
+	if binPath == lib.ConvertExecutableExt(lib.GetDefaultBin()) && bin != nil { // if the bin path is the same as the default binary path and if the custom binary is provided in the toml file (use it)
 		binPath = os.ExpandEnv(bin.(string))
 	}
 	//fmt.Println(binPath) //uncomment this to debug