Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Paolo Chilà <[email protected]>
Co-authored-by: Andrzej Stencel <[email protected]>
  • Loading branch information
3 people authored Sep 12, 2024
1 parent 725cd15 commit 7bfd624
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion pkg/testing/fixture_install.go
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ func (f *Fixture) installNoPkgManager(ctx context.Context, installOpts *InstallO
f.setClient(c)

f.t.Cleanup(func() {
f.DumpProcesses("-pre-uninstall")
f.DumpProcesses("-post-uninstall")
})

f.t.Cleanup(func() {
Expand Down
4 changes: 2 additions & 2 deletions testing/integration/proxy_url_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -733,8 +733,8 @@ func TestProxyURL(t *testing.T) {
Key: args.key,
}})
t.Logf("elastic-agent install output: \n%s\n", string(out))
for proxy, p := range proxies {
t.Logf("Proxy %s requests: %v", proxy, p.ProxiedRequests())
for proxyName, proxy := range proxies {
t.Logf("Proxy %s requests: %v", proxyName, proxy.ProxiedRequests())
}
if tt.wantErr(t, err, "elastic-agent install returned an unexpected error") {
tt.assertFunc(ctx, t, fixture, proxies, mockFleet)
Expand Down

0 comments on commit 7bfd624

Please sign in to comment.