Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Simplify test assertion

Co-authored-by: Paweł Gronowski <[email protected]>
Signed-off-by: Matthew MacLeod <[email protected]>
  • Loading branch information
mattmacleod and vvoland committed Nov 28, 2024
1 parent 37d2fb6 commit 24fb803
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions cli/connhelper/connhelper_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -86,9 +86,7 @@ func TestDockerSSHBinaryOverride(t *testing.T) {
t.Run(tc.name, func(t *testing.T) {
t.Setenv(DockerSSHRemoteBinaryEnv, tc.env)
result := dockerSSHRemoteBinary()
if result != tc.expected {
t.Errorf("expected %q, got %q", tc.expected, result)
}
assert.Check(t, is.Equal(result, tc.expected))
})
}
}

0 comments on commit 24fb803

Please sign in to comment.