From 897776fc8f3e9d4053a34155ee92c2f603d3a8e2 Mon Sep 17 00:00:00 2001
From: Will Roden <will@roden.cc>
Date: Mon, 27 Nov 2023 15:09:02 -0600
Subject: [PATCH] use binName() when installing

---
 internal/bindown/config.go      |  2 +-
 internal/bindown/config_test.go | 39 +++++++++++++++++++++++++++++++++
 2 files changed, 40 insertions(+), 1 deletion(-)

diff --git a/internal/bindown/config.go b/internal/bindown/config.go
index 83017c1..e221f46 100644
--- a/internal/bindown/config.go
+++ b/internal/bindown/config.go
@@ -453,7 +453,7 @@ func (c *Config) InstallDependencies(deps []string, system System, opts *ConfigI
 		}
 		target := opts.TargetFile
 		if target == "" {
-			target = filepath.Join(targetDir, name)
+			target = filepath.Join(targetDir, dep.binName())
 		}
 		out, err := install(dep, target, c.Cache, opts.Force, opts.ToCache, opts.AllowMissingChecksum)
 		if err != nil {
diff --git a/internal/bindown/config_test.go b/internal/bindown/config_test.go
index 87fbb53..0a862c6 100644
--- a/internal/bindown/config_test.go
+++ b/internal/bindown/config_test.go
@@ -264,6 +264,45 @@ dependencies:
 		require.Equal(t, wantMode, stat.Mode().Perm()&0o750)
 	})
 
+	t.Run("renames bin", func(t *testing.T) {
+		dir := t.TempDir()
+		servePath := filepath.Join("testdata", "downloadables", "fooinroot.tar.gz")
+		ts := testutil.ServeFile(t, servePath, "/foo/fooinroot.tar.gz", "")
+		depURL := ts.URL + "/foo/fooinroot.tar.gz"
+		binDir := filepath.Join(dir, "bin")
+		require.NoError(t, os.MkdirAll(binDir, 0o755))
+		cacheDir := filepath.Join(dir, ".bindown")
+		config := mustConfigFromYAML(t, fmt.Sprintf(`
+install_dir: %q
+cache: %q
+url_checksums:
+  "%s": 27dcce60d1ed72920a84dd4bc01e0bbd013e5a841660e9ee2e964e53fb83c0b3
+dependencies:
+  foo:
+    url: %q
+    archive_path: foo
+    bin: baz
+`, binDir, cacheDir, depURL, depURL))
+		t.Cleanup(func() { require.NoError(t, config.ClearCache()) })
+		wantBin := filepath.Join(binDir, "baz")
+		var stdout bytes.Buffer
+		wantStdout := fmt.Sprintf("installed foo to %s\n", wantBin)
+		err := config.InstallDependencies([]string{"foo"}, "darwin/amd64", &ConfigInstallDependenciesOpts{
+			Stdout: &stdout,
+		})
+		require.NoError(t, err)
+		require.Equal(t, wantStdout, stdout.String())
+		require.True(t, FileExists(wantBin))
+		stat, err := os.Stat(wantBin)
+		require.NoError(t, err)
+		require.False(t, stat.IsDir())
+		wantMode := os.FileMode(0o750)
+		if runtime.GOOS == "windows" {
+			wantMode = 0o640
+		}
+		require.Equal(t, wantMode, stat.Mode().Perm()&0o750)
+	})
+
 	t.Run("wrong checksum", func(t *testing.T) {
 		dir := t.TempDir()
 		servePath := filepath.Join("testdata", "downloadables", "fooinroot.tar.gz")