client/client_test: Use functional option to create mock client

vvoland · vvoland · commit 6452807fb40d · 2025-08-29T15:17:22.000+02:00
Signed-off-by: Paweł Gronowski &lt;pawel.gronowski@docker.com&gt;
diff --git a/client/client_test.go b/client/client_test.go
@@ -360,16 +360,16 @@ func TestNegotiateAPIVersionConnectionFailure(t *testing.T) {
 
 func TestNegotiateAPIVersionAutomatic(t *testing.T) {
 	var pingVersion string
-	httpClient := newMockClient(func(req *http.Request) (*http.Response, error) {
-		resp := &http.Response{StatusCode: http.StatusOK, Header: http.Header{}}
-		resp.Header.Set("Api-Version", pingVersion)
-		resp.Body = io.NopCloser(strings.NewReader("OK"))
-		return resp, nil
-	})
 
 	ctx := context.Background()
 	client, err := NewClientWithOpts(
-		WithHTTPClient(httpClient),
+		WithHTTPClient(&http.Client{
+			Transport: roundTripFunc(func(req *http.Request) (*http.Response, error) {
+				resp := &http.Response{StatusCode: http.StatusOK, Header: http.Header{}}
+				resp.Header.Set("Api-Version", pingVersion)
+				resp.Body = io.NopCloser(strings.NewReader("OK"))
+				return resp, nil
+			})}),
 		WithAPIVersionNegotiation(),
 	)
 	assert.NilError(t, err)
