#16: Fix alt root lookup by using lookup function. Also refactors to …

…new function name to reflect usage

drushlauncher/drushlauncher.go

@@ -6,7 +6,7 @@ import (
 	"path/filepath"
 )
 
-func FindDrupalRoot(path string) (string, error) {
+func FindDrushExecutable(path string) (string, error) {
 	// Check if the vendor/bin/drush directory exists in the current directory
 	drushDir := filepath.Join(path, "vendor", "bin", "drush")
 	if _, err := os.Stat(drushDir); err == nil {
@@ -29,5 +29,5 @@ func FindDrupalRoot(path string) (string, error) {
 	}
 
 	// Recursively continue searching in the parent directory
-	return FindDrupalRoot(parentDir)
+	return FindDrushExecutable(parentDir)
 }

drushlauncher/drushlauncher_test.go

@@ -6,7 +6,7 @@ import (
 	"testing"
 )
 
-func TestFindDrupalRoot(t *testing.T) {
+func TestFindDrushExecutable(t *testing.T) {
 	// Set up a temporary directory for the tests
 	tmpDir := os.TempDir()
 	defer os.RemoveAll(tmpDir)
@@ -15,7 +15,7 @@ func TestFindDrupalRoot(t *testing.T) {
 	testDir1 := filepath.Join(tmpDir, "testDir1", "vendor", "bin", "drush")
 	os.MkdirAll(filepath.Dir(testDir1), os.ModePerm)
 	os.Create(testDir1)
-	drupalRoot, err := FindDrupalRoot(testDir1)
+	drupalRoot, err := FindDrushExecutable(testDir1)
 	if err != nil || drupalRoot != filepath.Join(tmpDir, "testDir1") {
 		t.Errorf("Test case 1 failed: Expected Drupal root %s, got %s, error: %v", filepath.Join(tmpDir, "testDir1"), drupalRoot, err)
 	}
@@ -24,14 +24,14 @@ func TestFindDrupalRoot(t *testing.T) {
 	testDir2 := filepath.Join(tmpDir, "testDir2", "vendor", "bin", "drush")
 	os.MkdirAll(filepath.Dir(testDir2), os.ModePerm)
 	os.Create(testDir2)
-	drupalRoot, err = FindDrupalRoot(testDir2)
+	drupalRoot, err = FindDrushExecutable(testDir2)
 	if err != nil || drupalRoot != filepath.Join(tmpDir, "testDir2") {
 		t.Errorf("Test case 2 failed: Expected Drupal root %s, got %s, error: %v", filepath.Join(tmpDir, "testDir2"), drupalRoot, err)
 	}
 
 	// Test case 3: Drupal root not found
 	testDir3 := filepath.Join(tmpDir, "testDir3", "some", "random", "path")
-	drupalRoot, err = FindDrupalRoot(testDir3)
+	drupalRoot, err = FindDrushExecutable(testDir3)
 	if err == nil || drupalRoot != "" {
 		t.Errorf("Test case 3 failed: Expected no Drupal root, got %s, error: %v", drupalRoot, err)
 	}

main.go

@@ -39,15 +39,20 @@ func main() {
 
 	// Use the alternative Drupal root if provided
 	if altRoot != "" {
-		drupalRoot = altRoot
+		var err error
+		drupalRoot, err = drushlauncher.FindDrushExecutable(altRoot)
+		if err != nil {
+			fmt.Println(err)
+			os.Exit(1)
+		}
 	} else {
 		// If no alternative root provided, find the Drupal root from the current directory
 		cwd, err := os.Getwd()
 		if err != nil {
 			fmt.Println("Error getting current directory:", err)
 			os.Exit(1)
 		}
-		drupalRoot, err = drushlauncher.FindDrupalRoot(cwd)
+		drupalRoot, err = drushlauncher.FindDrushExecutable(cwd)
 		if err != nil {
 			fmt.Println(err)
 			os.Exit(1)