Skip to content

Commit 9422090

Browse files
committed
fix: Revert bad change
1 parent 91826df commit 9422090

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
@@ -405,7 +405,11 @@ func runAgent(t *testing.T, client *codersdk.Client, userID uuid.UUID) ([]coders
405405
clitest.SetupConfig(t, client, root)
406406
errC := make(chan error)
407407
agentCtx, agentCancel := context.WithCancel(ctx)
408-
defer agentCancel()
408+
t.Cleanup(func() {
409+
agentCancel()
410+
err := <-errC
411+
require.NoError(t, err)
412+
})
409413
go func() {
410414
errC <- cmd.ExecuteContext(agentCtx)
411415
}()
@@ -414,9 +418,6 @@ func runAgent(t *testing.T, client *codersdk.Client, userID uuid.UUID) ([]coders
414418
resources, err := client.WorkspaceResourcesByBuild(context.Background(), workspace.LatestBuild.ID)
415419
require.NoError(t, err)
416420

417-
err := <-errC
418-
require.NoError(t, err)
419-
420421
return resources, workspace
421422
}
422423

0 commit comments

Comments
 (0)