diff options
| author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-06-09 14:54:47 +0300 |
|---|---|---|
| committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-06-15 08:55:30 +0300 |
| commit | 348aff19d79a6045c170d94541165db1dd975e04 (patch) | |
| tree | cb4923c8144efc03ad89e643349b1b91d4b901bb | |
| parent | 5420c0472e72ee0c867ff7f28835f02a7e363d5d (diff) | |
ssh: Stop setting the PATH environment variable in tests
There is no reason for us to set the `PATH` environment variable
explicitly in our tests, so let's stop doing that.
| -rw-r--r-- | internal/gitaly/service/ssh/upload_pack_test.go | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/internal/gitaly/service/ssh/upload_pack_test.go b/internal/gitaly/service/ssh/upload_pack_test.go index 15474097b..43adab207 100644 --- a/internal/gitaly/service/ssh/upload_pack_test.go +++ b/internal/gitaly/service/ssh/upload_pack_test.go @@ -55,7 +55,6 @@ func runClone( fmt.Sprintf("GITALY_ADDRESS=%s", cfg.SocketPath), fmt.Sprintf("GITALY_PAYLOAD=%s", payload), fmt.Sprintf("GITALY_FEATUREFLAGS=%s", strings.Join(featureflag.AllFlags(ctx), ",")), - fmt.Sprintf("PATH=.:%s", os.Getenv("PATH")), fmt.Sprintf(`GIT_SSH_COMMAND=%s upload-pack`, filepath.Join(cfg.BinDir, "gitaly-ssh")), } if withSidechannel { @@ -452,7 +451,6 @@ func testUploadPackWithoutSideband(t *testing.T, opts ...testcfg.Option) { uploadPack.Env = []string{ fmt.Sprintf("GITALY_ADDRESS=%s", cfg.SocketPath), fmt.Sprintf("GITALY_PAYLOAD=%s", payload), - fmt.Sprintf("PATH=.:%s", os.Getenv("PATH")), } uploadPack.Stdin = negotiation |