Skip to content

Commit d263af2

Browse files
committed
fix: Revert bad change
1 parent 2334d8a commit d263af2

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

cli/portforward_test.go

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -394,7 +394,11 @@ func runAgent(t *testing.T, client *codersdk.Client, userID uuid.UUID) ([]coders
394394
clitest.SetupConfig(t, client, root)
395395
errC := make(chan error)
396396
agentCtx, agentCancel := context.WithCancel(ctx)
397-
defer agentCancel()
397+
t.Cleanup(func() {
398+
agentCancel()
399+
err := <-errC
400+
require.NoError(t, err)
401+
})
398402
go func() {
399403
errC <- cmd.ExecuteContext(agentCtx)
400404
}()
@@ -403,9 +407,6 @@ func runAgent(t *testing.T, client *codersdk.Client, userID uuid.UUID) ([]coders
403407
resources, err := client.WorkspaceResourcesByBuild(context.Background(), workspace.LatestBuild.ID)
404408
require.NoError(t, err)
405409

406-
err := <-errC
407-
require.NoError(t, err)
408-
409410
return resources, workspace
410411
}
411412

0 commit comments

Comments
 (0)