more tests

Author: nfx

libs/python/interpreters.go

@@ -77,6 +77,7 @@ func DetectInterpreters(ctx context.Context) (allInterpreters, error) {
 			// help here.
 			//
 			// See https://github.com/databricks/cli/pull/805#issuecomment-1735403952
+			log.Debugf(ctx, "%s is world-writeable (%s), skipping for security reasons", prefix, perm)
 			continue
 		}
 		entries, err := os.ReadDir(prefix)

libs/python/interpreters_test.go

@@ -3,6 +3,7 @@ package python
 import (
 	"context"
 	"os"
+	"path/filepath"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -27,7 +28,19 @@ func TestNoInterpretersFound(t *testing.T) {
 }
 
 func TestFilteringInterpreters(t *testing.T) {
-	t.Setenv("PATH", "testdata/other-binaries-filtered"+string(os.PathListSeparator)+"testdata/world-writeable")
+	rogueBin := filepath.Join(t.TempDir(), "rogue-bin")
+	err := os.Mkdir(rogueBin, 0o777)
+	assert.NoError(t, err)
+	os.Chmod(rogueBin, 0o777)
+
+	raw, err := os.ReadFile("testdata/world-writeable/python8.4")
+	assert.NoError(t, err)
+
+	binary := filepath.Join(rogueBin, "python8.4")
+	err = os.WriteFile(binary, raw, 00777)
+	assert.NoError(t, err)
+
+	t.Setenv("PATH", "testdata/other-binaries-filtered"+string(os.PathListSeparator)+rogueBin)
 
 	ctx := context.Background()
 	all, err := DetectInterpreters(ctx)