@@ -1220,7 +1220,7 @@ func (s *DockerSuite) TestContainerAPIDeleteRemoveVolume(c *testing.T) {

 	assert.NilError(c, err)

 	_, err = os.Stat(source)

-	assert.Assert(c, os.IsNotExist(err), fmt.Sprintf("expected to get ErrNotExist error, got %v", err))

+	assert.Assert(c, os.IsNotExist(err), "expected to get ErrNotExist error, got %v", err)

 }

 // Regression test for https://github.com/docker/docker/issues/6231

@@ -4,7 +4,6 @@ package main

 import (

 	"bufio"

-	"fmt"

 	"io/ioutil"

 	"os/exec"

 	"strings"

@@ -52,7 +51,7 @@ func (s *DockerSuite) TestAttachClosedOnContainerStop(c *testing.T) {

 	case err := <-errChan:

 		tty.Close()

 		out, _ := ioutil.ReadAll(pty)

-		assert.Assert(c, err == nil, fmt.Sprintf("out: %v", string(out)))

+		assert.Assert(c, err == nil, "out: %v", string(out))

 	case <-time.After(attachWait):

 		c.Fatal("timed out without attach returning")

 	}

@@ -4738,7 +4738,7 @@ func (s *DockerSuite) TestBuildTagEvent(c *testing.T) {

 		}

 	}

-	assert.Assert(c, foundTag, fmt.Sprintf("No tag event found:\n%s", out))

+	assert.Assert(c, foundTag, "No tag event found:\n%s", out)

 }

 // #15780

@@ -252,7 +252,7 @@ func (s *DockerRegistrySuite) TestListImagesWithDigests(c *testing.T) {

 	// make sure repo shown, tag=<none>, digest = $digest1

 	re1 := regexp.MustCompile(`\s*` + repoName + `\s*<none>\s*` + digest1.String() + `\s`)

-	assert.Assert(c, re1.MatchString(out), fmt.Sprintf("expected %q: %s", re1.String(), out))

+	assert.Assert(c, re1.MatchString(out), "expected %q: %s", re1.String(), out)

 	// setup image2

 	digest2, err := setupImageWithTag(c, "tag2")

 	//error setting up image

@@ -270,11 +270,11 @@ func (s *DockerRegistrySuite) TestListImagesWithDigests(c *testing.T) {

 	out, _ = dockerCmd(c, "images", "--digests")

 	// make sure repo shown, tag=<none>, digest = $digest1

-	assert.Assert(c, re1.MatchString(out), fmt.Sprintf("expected %q: %s", re1.String(), out))

+	assert.Assert(c, re1.MatchString(out), "expected %q: %s", re1.String(), out)

 	// make sure repo shown, tag=<none>, digest = $digest2

 	re2 := regexp.MustCompile(`\s*` + repoName + `\s*<none>\s*` + digest2.String() + `\s`)

-	assert.Assert(c, re2.MatchString(out), fmt.Sprintf("expected %q: %s", re2.String(), out))

+	assert.Assert(c, re2.MatchString(out), "expected %q: %s", re2.String(), out)

 	// pull tag1

 	dockerCmd(c, "pull", repoName+":tag1")

@@ -284,9 +284,9 @@ func (s *DockerRegistrySuite) TestListImagesWithDigests(c *testing.T) {

 	// make sure image 1 has repo, tag, <none> AND repo, <none>, digest

 	reWithDigest1 := regexp.MustCompile(`\s*` + repoName + `\s*tag1\s*` + digest1.String() + `\s`)

-	assert.Assert(c, reWithDigest1.MatchString(out), fmt.Sprintf("expected %q: %s", reWithDigest1.String(), out))

+	assert.Assert(c, reWithDigest1.MatchString(out), "expected %q: %s", reWithDigest1.String(), out)

 	// make sure image 2 has repo, <none>, digest

-	assert.Assert(c, re2.MatchString(out), fmt.Sprintf("expected %q: %s", re2.String(), out))

+	assert.Assert(c, re2.MatchString(out), "expected %q: %s", re2.String(), out)

 	// pull tag 2

 	dockerCmd(c, "pull", repoName+":tag2")

@@ -295,22 +295,22 @@ func (s *DockerRegistrySuite) TestListImagesWithDigests(c *testing.T) {

 	out, _ = dockerCmd(c, "images", "--digests")

 	// make sure image 1 has repo, tag, digest

-	assert.Assert(c, reWithDigest1.MatchString(out), fmt.Sprintf("expected %q: %s", reWithDigest1.String(), out))

+	assert.Assert(c, reWithDigest1.MatchString(out), "expected %q: %s", reWithDigest1.String(), out)

 	// make sure image 2 has repo, tag, digest

 	reWithDigest2 := regexp.MustCompile(`\s*` + repoName + `\s*tag2\s*` + digest2.String() + `\s`)

-	assert.Assert(c, reWithDigest2.MatchString(out), fmt.Sprintf("expected %q: %s", reWithDigest2.String(), out))

+	assert.Assert(c, reWithDigest2.MatchString(out), "expected %q: %s", reWithDigest2.String(), out)

 	// list images

 	out, _ = dockerCmd(c, "images", "--digests")

 	// make sure image 1 has repo, tag, digest

-	assert.Assert(c, reWithDigest1.MatchString(out), fmt.Sprintf("expected %q: %s", reWithDigest1.String(), out))

+	assert.Assert(c, reWithDigest1.MatchString(out), "expected %q: %s", reWithDigest1.String(), out)

 	// make sure image 2 has repo, tag, digest

-	assert.Assert(c, reWithDigest2.MatchString(out), fmt.Sprintf("expected %q: %s", reWithDigest2.String(), out))

+	assert.Assert(c, reWithDigest2.MatchString(out), "expected %q: %s", reWithDigest2.String(), out)

 	// make sure busybox has tag, but not digest

 	busyboxRe := regexp.MustCompile(`\s*busybox\s*latest\s*<none>\s`)

-	assert.Assert(c, busyboxRe.MatchString(out), fmt.Sprintf("expected %q: %s", busyboxRe.String(), out))

+	assert.Assert(c, busyboxRe.MatchString(out), "expected %q: %s", busyboxRe.String(), out)

 }

 func (s *DockerRegistrySuite) TestListDanglingImagesWithDigests(c *testing.T) {

@@ -328,7 +328,7 @@ func (s *DockerRegistrySuite) TestListDanglingImagesWithDigests(c *testing.T) {

 	// make sure repo shown, tag=<none>, digest = $digest1

 	re1 := regexp.MustCompile(`\s*` + repoName + `\s*<none>\s*` + digest1.String() + `\s`)

-	assert.Assert(c, re1.MatchString(out), fmt.Sprintf("expected %q: %s", re1.String(), out))

+	assert.Assert(c, re1.MatchString(out), "expected %q: %s", re1.String(), out)

 	// setup image2

 	digest2, err := setupImageWithTag(c, "dangle2")

 	//error setting up image

@@ -346,11 +346,11 @@ func (s *DockerRegistrySuite) TestListDanglingImagesWithDigests(c *testing.T) {

 	out, _ = dockerCmd(c, "images", "--digests", "--filter=dangling=true")

 	// make sure repo shown, tag=<none>, digest = $digest1

-	assert.Assert(c, re1.MatchString(out), fmt.Sprintf("expected %q: %s", re1.String(), out))

+	assert.Assert(c, re1.MatchString(out), "expected %q: %s", re1.String(), out)

 	// make sure repo shown, tag=<none>, digest = $digest2

 	re2 := regexp.MustCompile(`\s*` + repoName + `\s*<none>\s*` + digest2.String() + `\s`)

-	assert.Assert(c, re2.MatchString(out), fmt.Sprintf("expected %q: %s", re2.String(), out))

+	assert.Assert(c, re2.MatchString(out), "expected %q: %s", re2.String(), out)

 	// pull dangle1 tag

 	dockerCmd(c, "pull", repoName+":dangle1")

@@ -360,9 +360,9 @@ func (s *DockerRegistrySuite) TestListDanglingImagesWithDigests(c *testing.T) {

 	// make sure image 1 has repo, tag, <none> AND repo, <none>, digest

 	reWithDigest1 := regexp.MustCompile(`\s*` + repoName + `\s*dangle1\s*` + digest1.String() + `\s`)

-	assert.Assert(c, !reWithDigest1.MatchString(out), fmt.Sprintf("unexpected %q: %s", reWithDigest1.String(), out))

+	assert.Assert(c, !reWithDigest1.MatchString(out), "unexpected %q: %s", reWithDigest1.String(), out)

 	// make sure image 2 has repo, <none>, digest

-	assert.Assert(c, re2.MatchString(out), fmt.Sprintf("expected %q: %s", re2.String(), out))

+	assert.Assert(c, re2.MatchString(out), "expected %q: %s", re2.String(), out)

 	// pull dangle2 tag

 	dockerCmd(c, "pull", repoName+":dangle2")

@@ -371,19 +371,19 @@ func (s *DockerRegistrySuite) TestListDanglingImagesWithDigests(c *testing.T) {

 	out, _ = dockerCmd(c, "images", "--digests")

 	// make sure image 1 has repo, tag, digest

-	assert.Assert(c, reWithDigest1.MatchString(out), fmt.Sprintf("expected %q: %s", reWithDigest1.String(), out))

+	assert.Assert(c, reWithDigest1.MatchString(out), "expected %q: %s", reWithDigest1.String(), out)

 	// make sure image 2 has repo, tag, digest

 	reWithDigest2 := regexp.MustCompile(`\s*` + repoName + `\s*dangle2\s*` + digest2.String() + `\s`)

-	assert.Assert(c, reWithDigest2.MatchString(out), fmt.Sprintf("expected %q: %s", reWithDigest2.String(), out))

+	assert.Assert(c, reWithDigest2.MatchString(out), "expected %q: %s", reWithDigest2.String(), out)

 	// list images, no longer dangling, should not match

 	out, _ = dockerCmd(c, "images", "--digests", "--filter=dangling=true")

 	// make sure image 1 has repo, tag, digest

-	assert.Assert(c, !reWithDigest1.MatchString(out), fmt.Sprintf("unexpected %q: %s", reWithDigest1.String(), out))

+	assert.Assert(c, !reWithDigest1.MatchString(out), "unexpected %q: %s", reWithDigest1.String(), out)

 	// make sure image 2 has repo, tag, digest

-	assert.Assert(c, !reWithDigest2.MatchString(out), fmt.Sprintf("unexpected %q: %s", reWithDigest2.String(), out))

+	assert.Assert(c, !reWithDigest2.MatchString(out), "unexpected %q: %s", reWithDigest2.String(), out)

 }

 func (s *DockerRegistrySuite) TestInspectImageWithDigests(c *testing.T) {

@@ -643,7 +643,7 @@ func (s *DockerRegistrySuite) TestPullFailsWithAlteredLayer(c *testing.T) {

 	assert.Assert(c, exitStatus != 0, "expected a non-zero exit status")

 	expectedErrorMsg := fmt.Sprintf("filesystem layer verification failed for digest %s", targetLayerDigest)

-	assert.Assert(c, strings.Contains(out, expectedErrorMsg), fmt.Sprintf("expected error message in output: %s", out))

+	assert.Assert(c, strings.Contains(out, expectedErrorMsg), "expected error message in output: %s", out)

 }

 // TestPullFailsWithAlteredLayer tests that a `docker pull` fails when

@@ -686,5 +686,5 @@ func (s *DockerSchema1RegistrySuite) TestPullFailsWithAlteredLayer(c *testing.T)

 	assert.Assert(c, exitStatus != 0, "expected a non-zero exit status")

 	expectedErrorMsg := fmt.Sprintf("filesystem layer verification failed for digest %s", targetLayerDigest)

-	assert.Assert(c, strings.Contains(out, expectedErrorMsg), fmt.Sprintf("expected error message in output: %s", out))

+	assert.Assert(c, strings.Contains(out, expectedErrorMsg), "expected error message in output: %s", out)

 }

@@ -1,7 +1,6 @@

 package main

 import (

-	"fmt"

 	"strings"

 	"testing"

@@ -72,7 +71,7 @@ func (s *DockerSuite) TestCommitHardlink(c *testing.T) {

 	chunks := strings.Split(strings.TrimSpace(firstOutput), " ")

 	inode := chunks[0]

 	chunks = strings.SplitAfterN(strings.TrimSpace(firstOutput), " ", 2)

-	assert.Assert(c, strings.Contains(chunks[1], chunks[0]), fmt.Sprintf("Failed to create hardlink in a container. Expected to find %q in %q", inode, chunks[1:]))

+	assert.Assert(c, strings.Contains(chunks[1], chunks[0]), "Failed to create hardlink in a container. Expected to find %q in %q", inode, chunks[1:])

 	imageID, _ := dockerCmd(c, "commit", "hardlinks", "hardlinks")

 	imageID = strings.TrimSpace(imageID)

@@ -81,7 +80,7 @@ func (s *DockerSuite) TestCommitHardlink(c *testing.T) {

 	chunks = strings.Split(strings.TrimSpace(secondOutput), " ")

 	inode = chunks[0]

 	chunks = strings.SplitAfterN(strings.TrimSpace(secondOutput), " ", 2)

-	assert.Assert(c, strings.Contains(chunks[1], chunks[0]), fmt.Sprintf("Failed to create hardlink in a container. Expected to find %q in %q", inode, chunks[1:]))

+	assert.Assert(c, strings.Contains(chunks[1], chunks[0]), "Failed to create hardlink in a container. Expected to find %q in %q", inode, chunks[1:])

 }

 func (s *DockerSuite) TestCommitTTY(c *testing.T) {

@@ -1,7 +1,6 @@

 package main

 import (

-	"fmt"

 	"os"

 	"path/filepath"

 	"testing"

@@ -150,7 +149,7 @@ func (s *DockerSuite) TestCpFromCaseB(c *testing.T) {

 	err := runDockerCp(c, srcPath, dstDir, nil)

 	assert.ErrorContains(c, err, "")

-	assert.Assert(c, isCpDirNotExist(err), fmt.Sprintf("expected DirNotExists error, but got %T: %s", err, err))

+	assert.Assert(c, isCpDirNotExist(err), "expected DirNotExists error, but got %T: %s", err, err)

 }

 // C. SRC specifies a file and DST exists as a file. This should overwrite

@@ -195,7 +194,7 @@ func (s *DockerSuite) TestCpFromCaseD(c *testing.T) {

 	// Ensure that dstPath doesn't exist.

 	_, err := os.Stat(dstPath)

-	assert.Assert(c, os.IsNotExist(err), fmt.Sprintf("did not expect dstPath %q to exist", dstPath))

+	assert.Assert(c, os.IsNotExist(err), "did not expect dstPath %q to exist", dstPath)

 	assert.Assert(c, runDockerCp(c, srcPath, dstDir, nil) == nil)

@@ -266,7 +265,7 @@ func (s *DockerSuite) TestCpFromCaseF(c *testing.T) {

 	err := runDockerCp(c, srcDir, dstFile, nil)

 	assert.ErrorContains(c, err, "")

-	assert.Assert(c, isCpCannotCopyDir(err), fmt.Sprintf("expected ErrCannotCopyDir error, but got %T: %s", err, err))

+	assert.Assert(c, isCpCannotCopyDir(err), "expected ErrCannotCopyDir error, but got %T: %s", err, err)

 }

 // G. SRC specifies a directory and DST exists as a directory. This should copy

@@ -358,7 +357,7 @@ func (s *DockerSuite) TestCpFromCaseI(c *testing.T) {

 	err := runDockerCp(c, srcDir, dstFile, nil)

 	assert.ErrorContains(c, err, "")

-	assert.Assert(c, isCpCannotCopyDir(err), fmt.Sprintf("expected ErrCannotCopyDir error, but got %T: %s", err, err))

+	assert.Assert(c, isCpCannotCopyDir(err), "expected ErrCannotCopyDir error, but got %T: %s", err, err)

 }

 // J. SRC specifies a directory's contents only and DST exists as a directory.

@@ -1,7 +1,6 @@

 package main

 import (

-	"fmt"

 	"os"

 	"testing"

@@ -158,7 +157,7 @@ func (s *DockerSuite) TestCpToCaseB(c *testing.T) {

 	err := runDockerCp(c, srcPath, dstDir, nil)

 	assert.ErrorContains(c, err, "")

-	assert.Assert(c, isCpDirNotExist(err), fmt.Sprintf("expected DirNotExists error, but got %T: %s", err, err))

+	assert.Assert(c, isCpDirNotExist(err), "expected DirNotExists error, but got %T: %s", err, err)

 }

 // C. SRC specifies a file and DST exists as a file. This should overwrite

@@ -288,7 +287,7 @@ func (s *DockerSuite) TestCpToCaseF(c *testing.T) {

 	err := runDockerCp(c, srcDir, dstFile, nil)

 	assert.ErrorContains(c, err, "")

-	assert.Assert(c, isCpCannotCopyDir(err), fmt.Sprintf("expected ErrCannotCopyDir error, but got %T: %s", err, err))

+	assert.Assert(c, isCpCannotCopyDir(err), "expected ErrCannotCopyDir error, but got %T: %s", err, err)

 }

 // G. SRC specifies a directory and DST exists as a directory. This should copy

@@ -393,7 +392,7 @@ func (s *DockerSuite) TestCpToCaseI(c *testing.T) {

 	err := runDockerCp(c, srcDir, dstFile, nil)

 	assert.ErrorContains(c, err, "")

-	assert.Assert(c, isCpCannotCopyDir(err), fmt.Sprintf("expected ErrCannotCopyDir error, but got %T: %s", err, err))

+	assert.Assert(c, isCpCannotCopyDir(err), "expected ErrCannotCopyDir error, but got %T: %s", err, err)

 }

 // J. SRC specifies a directory's contents only and DST exists as a directory.

@@ -462,7 +461,7 @@ func (s *DockerSuite) TestCpToErrReadOnlyRootfs(c *testing.T) {

 	err := runDockerCp(c, srcPath, dstPath, nil)

 	assert.ErrorContains(c, err, "")

-	assert.Assert(c, isCpCannotCopyReadOnly(err), fmt.Sprintf("expected ErrContainerRootfsReadonly error, but got %T: %s", err, err))

+	assert.Assert(c, isCpCannotCopyReadOnly(err), "expected ErrContainerRootfsReadonly error, but got %T: %s", err, err)

 	// Ensure that dstPath doesn't exist.

 	assert.Assert(c, containerStartOutputEquals(c, containerID, "") == nil)

@@ -489,7 +488,7 @@ func (s *DockerSuite) TestCpToErrReadOnlyVolume(c *testing.T) {

 	err := runDockerCp(c, srcPath, dstPath, nil)

 	assert.ErrorContains(c, err, "")

-	assert.Assert(c, isCpCannotCopyReadOnly(err), fmt.Sprintf("expected ErrVolumeReadonly error, but got %T: %s", err, err))

+	assert.Assert(c, isCpCannotCopyReadOnly(err), "expected ErrVolumeReadonly error, but got %T: %s", err, err)

 	// Ensure that dstPath doesn't exist.

 	assert.Assert(c, containerStartOutputEquals(c, containerID, "") == nil)

@@ -36,7 +36,7 @@ func (s *DockerSuite) TestCreateArgs(c *testing.T) {

 	}

 	err := json.Unmarshal([]byte(out), &containers)

-	assert.Assert(c, err == nil, fmt.Sprintf("Error inspecting the container: %s", err))

+	assert.Assert(c, err == nil, "Error inspecting the container: %s", err)

 	assert.Equal(c, len(containers), 1)

 	cont := containers[0]

@@ -95,12 +95,12 @@ func (s *DockerSuite) TestCreateHostConfig(c *testing.T) {

 	}

 	err := json.Unmarshal([]byte(out), &containers)

-	assert.Assert(c, err == nil, fmt.Sprintf("Error inspecting the container: %s", err))

+	assert.Assert(c, err == nil, "Error inspecting the container: %s", err)

 	assert.Equal(c, len(containers), 1)

 	cont := containers[0]

-	assert.Assert(c, cont.HostConfig != nil, fmt.Sprintf("Expected HostConfig, got none"))

-	assert.Assert(c, cont.HostConfig.PublishAllPorts, fmt.Sprintf("Expected PublishAllPorts, got false"))

+	assert.Assert(c, cont.HostConfig != nil, "Expected HostConfig, got none")

+	assert.Assert(c, cont.HostConfig.PublishAllPorts, "Expected PublishAllPorts, got false")

 }

 func (s *DockerSuite) TestCreateWithPortRange(c *testing.T) {

@@ -116,12 +116,12 @@ func (s *DockerSuite) TestCreateWithPortRange(c *testing.T) {

 		}

 	}

 	err := json.Unmarshal([]byte(out), &containers)

-	assert.Assert(c, err == nil, fmt.Sprintf("Error inspecting the container: %s", err))

+	assert.Assert(c, err == nil, "Error inspecting the container: %s", err)

 	assert.Equal(c, len(containers), 1)

 	cont := containers[0]

-	assert.Assert(c, cont.HostConfig != nil, fmt.Sprintf("Expected HostConfig, got none"))

+	assert.Assert(c, cont.HostConfig != nil, "Expected HostConfig, got none")

 	assert.Equal(c, len(cont.HostConfig.PortBindings), 4, fmt.Sprintf("Expected 4 ports bindings, got %d", len(cont.HostConfig.PortBindings)))

 	for k, v := range cont.HostConfig.PortBindings {

@@ -146,11 +146,11 @@ func (s *DockerSuite) TestCreateWithLargePortRange(c *testing.T) {

 	}

 	err := json.Unmarshal([]byte(out), &containers)

-	assert.Assert(c, err == nil, fmt.Sprintf("Error inspecting the container: %s", err))

+	assert.Assert(c, err == nil, "Error inspecting the container: %s", err)

 	assert.Equal(c, len(containers), 1)

 	cont := containers[0]

-	assert.Assert(c, cont.HostConfig != nil, fmt.Sprintf("Expected HostConfig, got none"))

+	assert.Assert(c, cont.HostConfig != nil, "Expected HostConfig, got none")

 	assert.Equal(c, len(cont.HostConfig.PortBindings), 65535)

 	for k, v := range cont.HostConfig.PortBindings {

@@ -178,7 +178,7 @@ func (s *DockerSuite) TestCreateVolumesCreated(c *testing.T) {

 	dockerCmd(c, "create", "--name", name, "-v", prefix+slash+"foo", "busybox")

 	dir, err := inspectMountSourceField(name, prefix+slash+"foo")

-	assert.Assert(c, err == nil, fmt.Sprintf("Error getting volume host path: %q", err))

+	assert.Assert(c, err == nil, "Error getting volume host path: %q", err)

 	if _, err := os.Stat(dir); err != nil && os.IsNotExist(err) {

 		c.Fatalf("Volume was not created")

@@ -131,7 +131,7 @@ func (s *DockerDaemonSuite) TestDaemonRestartUnlessStopped(c *testing.T) {

 		var format string

 		for name, shouldRun := range m {

 			out, err := s.d.Cmd("ps")

-			assert.Assert(c, err == nil, fmt.Sprintf("run ps: %v", out))

+			assert.Assert(c, err == nil, "run ps: %v", out)

 			if shouldRun {

 				format = "%scontainer %q is not running"

 			} else {

@@ -217,7 +217,7 @@ func (s *DockerDaemonSuite) TestDaemonRestartWithInvalidBasesize(c *testing.T) {

 	if newBasesizeBytes < oldBasesizeBytes {

 		err := s.d.RestartWithError("--storage-opt", fmt.Sprintf("dm.basesize=%d", newBasesizeBytes))

-		assert.Assert(c, err != nil, fmt.Sprintf("daemon should not have started as new base device size is less than existing base device size: %v", err))

+		assert.Assert(c, err != nil, "daemon should not have started as new base device size is less than existing base device size: %v", err)

 		// 'err != nil' is expected behaviour, no new daemon started,

 		// so no need to stop daemon.

 		if err != nil {

@@ -241,11 +241,11 @@ func (s *DockerDaemonSuite) TestDaemonRestartWithIncreasedBasesize(c *testing.T)

 	}

 	err := s.d.RestartWithError("--storage-opt", fmt.Sprintf("dm.basesize=%d", newBasesizeBytes))

-	assert.Assert(c, err == nil, fmt.Sprintf("we should have been able to start the daemon with increased base device size: %v", err))

+	assert.Assert(c, err == nil, "we should have been able to start the daemon with increased base device size: %v", err)

 	basesizeAfterRestart := getBaseDeviceSize(c, s.d)

 	newBasesize, err := convertBasesize(newBasesizeBytes)

-	assert.Assert(c, err == nil, fmt.Sprintf("Error in converting base device size: %v", err))

+	assert.Assert(c, err == nil, "Error in converting base device size: %v", err)

 	assert.Equal(c, newBasesize, basesizeAfterRestart, "Basesize passed is not equal to Basesize set")

 	s.d.Stop(c)

 }

@@ -1750,7 +1750,7 @@ func (s *DockerDaemonSuite) TestBridgeIPIsExcludedFromAllocatorPool(c *testing.T

 			break

 		}

 		ip, err := s.d.Cmd("inspect", "--format", "'{{.NetworkSettings.IPAddress}}'", contName)

-		assert.Assert(c, err == nil, fmt.Sprintf("%s", ip))

+		assert.Assert(c, err == nil, "%s", ip)

 		assert.Assert(c, ip != bridgeIP)

 		cont++

@@ -1780,7 +1780,7 @@ func (s *DockerDaemonSuite) TestDaemonNoSpaceLeftOnDeviceError(c *testing.T) {

 	// pull a repository large enough to overfill the mounted filesystem

 	pullOut, err := s.d.Cmd("pull", "debian:stretch")

-	assert.Assert(c, err != nil, fmt.Sprintf("%s", pullOut))

+	assert.Assert(c, err != nil, "%s", pullOut)

 	assert.Assert(c, strings.Contains(pullOut, "no space left on device"))

 }

@@ -1855,7 +1855,7 @@ func (s *DockerDaemonSuite) TestDaemonCgroupParent(c *testing.T) {

 	out, err := s.d.Cmd("run", "--name", name, "busybox", "cat", "/proc/self/cgroup")

 	assert.NilError(c, err)

 	cgroupPaths := ParseCgroupPaths(string(out))

-	assert.Assert(c, len(cgroupPaths) != 0, fmt.Sprintf("unexpected output - %q", string(out)))

+	assert.Assert(c, len(cgroupPaths) != 0, "unexpected output - %q", string(out))

 	out, err = s.d.Cmd("inspect", "-f", "{{.Id}}", name)

 	assert.NilError(c, err)

 	id := strings.TrimSpace(string(out))

@@ -1867,7 +1867,7 @@ func (s *DockerDaemonSuite) TestDaemonCgroupParent(c *testing.T) {

 			break

 		}

 	}

-	assert.Assert(c, found, fmt.Sprintf("Cgroup path for container (%s) doesn't found in cgroups file: %s", expectedCgroup, cgroupPaths))

+	assert.Assert(c, found, "Cgroup path for container (%s) doesn't found in cgroups file: %s", expectedCgroup, cgroupPaths)

 }

 func (s *DockerDaemonSuite) TestDaemonRestartWithLinks(c *testing.T) {

@@ -2007,7 +2007,7 @@ func (s *DockerDaemonSuite) TestCleanupMountsAfterDaemonCrash(c *testing.T) {

 	// the following check for mounts being cleared is pointless.

 	skipMountCheck := false

 	mountOut, err := ioutil.ReadFile("/proc/self/mountinfo")

-	assert.Assert(c, err == nil, fmt.Sprintf("Output: %s", mountOut))

+	assert.Assert(c, err == nil, "Output: %s", mountOut)

 	if !strings.Contains(string(mountOut), id) {

 		skipMountCheck = true

 	}

@@ -2032,7 +2032,7 @@ func (s *DockerDaemonSuite) TestCleanupMountsAfterDaemonCrash(c *testing.T) {

 	}

 	// Now, container mounts should be gone.

 	mountOut, err = ioutil.ReadFile("/proc/self/mountinfo")

-	assert.Assert(c, err == nil, fmt.Sprintf("Output: %s", mountOut))

+	assert.Assert(c, err == nil, "Output: %s", mountOut)

 	comment := fmt.Sprintf("%s is still mounted from older daemon start:\nDaemon root repository %s\n%s", id, s.d.Root, mountOut)

 	assert.Equal(c, strings.Contains(string(mountOut), id), false, comment)

 }

@@ -2229,8 +2229,8 @@ func (s *DockerDaemonSuite) TestDaemonDiscoveryBackendConfigReload(c *testing.T)

 	out, err := s.d.Cmd("info")

 	assert.NilError(c, err)

-	assert.Assert(c, strings.Contains(out, fmt.Sprintf("Cluster Store: consul://consuladdr:consulport/some/path")))

-	assert.Assert(c, strings.Contains(out, fmt.Sprintf("Cluster Advertise: 192.168.56.100:0")))

+	assert.Assert(c, strings.Contains(out, "Cluster Store: consul://consuladdr:consulport/some/path"))

+	assert.Assert(c, strings.Contains(out, "Cluster Advertise: 192.168.56.100:0"))

 }

 // Test for #21956

@@ -2376,11 +2376,11 @@ func (s *DockerDaemonSuite) TestDaemonDNSFlagsInHostMode(c *testing.T) {

 	expectedOutput := "nameserver 1.2.3.4"

 	out, _ := s.d.Cmd("run", "--net=host", "busybox", "cat", "/etc/resolv.conf")

-	assert.Assert(c, strings.Contains(out, expectedOutput), fmt.Sprintf("Expected '%s', but got %q", expectedOutput, out))

+	assert.Assert(c, strings.Contains(out, expectedOutput), "Expected '%s', but got %q", expectedOutput, out)

 	expectedOutput = "search example.com"

-	assert.Assert(c, strings.Contains(out, expectedOutput), fmt.Sprintf("Expected '%s', but got %q", expectedOutput, out))

+	assert.Assert(c, strings.Contains(out, expectedOutput), "Expected '%s', but got %q", expectedOutput, out)

 	expectedOutput = "options timeout:3"

-	assert.Assert(c, strings.Contains(out, expectedOutput), fmt.Sprintf("Expected '%s', but got %q", expectedOutput, out))

+	assert.Assert(c, strings.Contains(out, expectedOutput), "Expected '%s', but got %q", expectedOutput, out)

 }

 func (s *DockerDaemonSuite) TestRunWithRuntimeFromConfigFile(c *testing.T) {

@@ -2555,10 +2555,10 @@ func (s *DockerDaemonSuite) TestDaemonRestartWithAutoRemoveContainer(c *testing.

 	// top1 will exist after daemon restarts

 	out, err := s.d.Cmd("run", "-d", "--name", "top1", "busybox:latest", "top")

-	assert.Assert(c, err == nil, fmt.Sprintf("run top1: %v", out))

+	assert.Assert(c, err == nil, "run top1: %v", out)

 	// top2 will be removed after daemon restarts

 	out, err = s.d.Cmd("run", "-d", "--rm", "--name", "top2", "busybox:latest", "top")

-	assert.Assert(c, err == nil, fmt.Sprintf("run top2: %v", out))

+	assert.Assert(c, err == nil, "run top2: %v", out)

 	out, err = s.d.Cmd("ps")

 	assert.NilError(c, err)

@@ -2714,13 +2714,13 @@ func (s *DockerDaemonSuite) TestExecWithUserAfterLiveRestore(c *testing.T) {

 	out1, err := s.d.Cmd("exec", "-u", "test", "top", "id")

 	// uid=100(test) gid=101(test) groups=101(test)

-	assert.Assert(c, err == nil, fmt.Sprintf("Output: %s", out1))

+	assert.Assert(c, err == nil, "Output: %s", out1)

 	// restart daemon.

 	s.d.Restart(c, "--live-restore")

 	out2, err := s.d.Cmd("exec", "-u", "test", "top", "id")

-	assert.Assert(c, err == nil, fmt.Sprintf("Output: %s", out2))

+	assert.Assert(c, err == nil, "Output: %s", out2)

 	assert.Equal(c, out2, out1, fmt.Sprintf("Output: before restart '%s', after restart '%s'", out1, out2))

 	out, err = s.d.Cmd("stop", "top")

@@ -2773,7 +2773,7 @@ func (s *DockerDaemonSuite) TestRestartPolicyWithLiveRestore(c *testing.T) {

 		StartedAt time.Time

 	}

 	out, err = s.d.Cmd("inspect", "-f", "{{json .State}}", id)

-	assert.Assert(c, err == nil, fmt.Sprintf("output: %s", out))

+	assert.Assert(c, err == nil, "output: %s", out)

 	var origState state

 	err = json.Unmarshal([]byte(strings.TrimSpace(out)), &origState)

@@ -2799,7 +2799,7 @@ func (s *DockerDaemonSuite) TestRestartPolicyWithLiveRestore(c *testing.T) {

 		}

 		out, err := s.d.Cmd("inspect", "-f", "{{json .State}}", id)

-		assert.Assert(c, err == nil, fmt.Sprintf("output: %s", out))

+		assert.Assert(c, err == nil, "output: %s", out)

 		var newState state

 		err = json.Unmarshal([]byte(strings.TrimSpace(out)), &newState)

@@ -297,7 +297,7 @@ func (s *DockerExternalVolumeSuite) TestExternalVolumeDriverNamed(c *testing.T)

 	p := hostVolumePath("external-volume-test")

 	_, err = os.Lstat(p)

 	assert.ErrorContains(c, err, "")

-	assert.Assert(c, os.IsNotExist(err), fmt.Sprintf("Expected volume path in host to not exist: %s, %v\n", p, err))

+	assert.Assert(c, os.IsNotExist(err), "Expected volume path in host to not exist: %s, %v\n", p, err)

 	assert.Equal(c, s.ec.activations, 1)

 	assert.Equal(c, s.ec.creations, 1)

@@ -98,7 +98,7 @@ func (s *DockerSuite) TestHistoryHumanOptionFalse(c *testing.T) {

 		sizeString := lines[i][startIndex:endIndex]

 		_, err := strconv.Atoi(strings.TrimSpace(sizeString))

-		assert.Assert(c, err == nil, fmt.Sprintf("The size '%s' was not an Integer", sizeString))

+		assert.Assert(c, err == nil, "The size '%s' was not an Integer", sizeString)

 	}

 }

@@ -52,7 +52,7 @@ func (s *DockerSuite) TestInfoEnsureSucceeds(c *testing.T) {

 	}

 	for _, linePrefix := range stringsToCheck {

-		assert.Assert(c, strings.Contains(out, linePrefix), fmt.Sprintf("couldn't find string %v in output", linePrefix))

+		assert.Assert(c, strings.Contains(out, linePrefix), "couldn't find string %v in output", linePrefix)

 	}

 }

@@ -118,7 +118,7 @@ func (s *DockerSuite) TestInspectTypeFlagWithInvalidValue(c *testing.T) {

 	dockerCmd(c, "run", "--name=busybox", "-d", "busybox", "true")

 	out, exitCode, err := dockerCmdWithError("inspect", "--type=foobar", "busybox")

-	assert.Assert(c, err != nil, fmt.Sprintf("%d", exitCode))

+	assert.Assert(c, err != nil, "%d", exitCode)

 	assert.Equal(c, exitCode, 1, fmt.Sprintf("%s", err))

 	assert.Assert(c, strings.Contains(out, "not a valid value for --type"))

 }

@@ -129,7 +129,7 @@ func (s *DockerSuite) TestInspectImageFilterInt(c *testing.T) {

 	out := inspectField(c, imageTest, "Size")

 	size, err := strconv.Atoi(out)

-	assert.Assert(c, err == nil, fmt.Sprintf("failed to inspect size of the image: %s, %v", out, err))

+	assert.Assert(c, err == nil, "failed to inspect size of the image: %s, %v", out, err)

 	//now see if the size turns out to be the same

 	formatStr := fmt.Sprintf("--format={{eq .Size %d}}", size)

@@ -151,7 +151,7 @@ func (s *DockerSuite) TestInspectContainerFilterInt(c *testing.T) {

 	out = inspectField(c, id, "State.ExitCode")

 	exitCode, err := strconv.Atoi(out)

-	assert.Assert(c, err == nil, fmt.Sprintf("failed to inspect exitcode of the container: %s, %v", out, err))

+	assert.Assert(c, err == nil, "failed to inspect exitcode of the container: %s, %v", out, err)

 	//now get the exit code to verify

 	formatStr := fmt.Sprintf("--format={{eq .State.ExitCode %d}}", exitCode)

@@ -171,12 +171,12 @@ func (s *DockerSuite) TestInspectImageGraphDriver(c *testing.T) {

 	deviceID := inspectField(c, imageTest, "GraphDriver.Data.DeviceId")

 	_, err := strconv.Atoi(deviceID)

-	assert.Assert(c, err == nil, fmt.Sprintf("failed to inspect DeviceId of the image: %s, %v", deviceID, err))

+	assert.Assert(c, err == nil, "failed to inspect DeviceId of the image: %s, %v", deviceID, err)

 	deviceSize := inspectField(c, imageTest, "GraphDriver.Data.DeviceSize")

 	_, err = strconv.ParseUint(deviceSize, 10, 64)

-	assert.Assert(c, err == nil, fmt.Sprintf("failed to inspect DeviceSize of the image: %s, %v", deviceSize, err))

+	assert.Assert(c, err == nil, "failed to inspect DeviceSize of the image: %s, %v", deviceSize, err)

 }

 func (s *DockerSuite) TestInspectContainerGraphDriver(c *testing.T) {

@@ -196,12 +196,12 @@ func (s *DockerSuite) TestInspectContainerGraphDriver(c *testing.T) {

 	assert.Assert(c, imageDeviceID != deviceID)

 	_, err := strconv.Atoi(deviceID)

-	assert.Assert(c, err == nil, fmt.Sprintf("failed to inspect DeviceId of the image: %s, %v", deviceID, err))

+	assert.Assert(c, err == nil, "failed to inspect DeviceId of the image: %s, %v", deviceID, err)

 	deviceSize := inspectField(c, out, "GraphDriver.Data.DeviceSize")

 	_, err = strconv.ParseUint(deviceSize, 10, 64)

-	assert.Assert(c, err == nil, fmt.Sprintf("failed to inspect DeviceSize of the image: %s, %v", deviceSize, err))

+	assert.Assert(c, err == nil, "failed to inspect DeviceSize of the image: %s, %v", deviceSize, err)

 }

 func (s *DockerSuite) TestInspectBindMountPoint(c *testing.T) {

@@ -288,7 +288,7 @@ func (s *DockerSuite) TestInspectLogConfigNoType(c *testing.T) {

 	out := inspectFieldJSON(c, "test", "HostConfig.LogConfig")

 	err := json.NewDecoder(strings.NewReader(out)).Decode(&logConfig)

-	assert.Assert(c, err == nil, fmt.Sprintf("%v", out))

+	assert.Assert(c, err == nil, "%v", out)

 	assert.Equal(c, logConfig.Type, "json-file")

 	assert.Equal(c, logConfig.Config["max-file"], "42", fmt.Sprintf("%v", logConfig))

@@ -27,7 +27,7 @@ func (s *DockerSuite) TestLinksInvalidContainerTarget(c *testing.T) {

 	out, _, err := dockerCmdWithError("run", "--link", "bogus:alias", "busybox", "true")

 	// an invalid container target should produce an error

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 	// an invalid container target should produce an error

 	// note: convert the output to lowercase first as the error string

 	// capitalization was changed after API version 1.32

@@ -169,7 +169,7 @@ func (s *DockerSuite) TestLinksUpdateOnRestart(c *testing.T) {

 	getIP := func(hosts []byte, hostname string) string {

 		re := regexp.MustCompile(fmt.Sprintf(`(\S*)\t%s`, regexp.QuoteMeta(hostname)))

 		matches := re.FindSubmatch(hosts)

-		assert.Assert(c, matches != nil, fmt.Sprintf("Hostname %s have no matches in hosts", hostname))

+		assert.Assert(c, matches != nil, "Hostname %s have no matches in hosts", hostname)

 		return string(matches[1])

 	}

 	ip := getIP(content, "one")

@@ -220,7 +220,7 @@ func (s *DockerSuite) TestLinksNetworkHostContainer(c *testing.T) {

 	out, _, err := dockerCmdWithError("run", "--name", "should_fail", "--link", "host_container:tester", "busybox", "true")

 	// Running container linking to a container with --net host should have failed

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 	// Running container linking to a container with --net host should have failed

 	assert.Assert(c, strings.Contains(out, runconfig.ErrConflictHostNetworkAndLinks.Error()))

 }

@@ -1,7 +1,6 @@

 package main

 import (

-	"fmt"

 	"strings"

 	"testing"

@@ -20,7 +19,7 @@ const stringCheckPS = "PID   USER"

 // stop the tests.

 func dockerCmdWithFail(c *testing.T, args ...string) (string, int) {

 	out, status, err := dockerCmdWithError(args...)

-	assert.Assert(c, err != nil, fmt.Sprintf("%v", out))

+	assert.Assert(c, err != nil, "%v", out)

 	return out, status

 }

@@ -403,7 +403,7 @@ func (s *DockerSuite) TestDockerNetworkDeleteMultiple(c *testing.T) {

 	// contains active container, its deletion should fail.

 	out, _, err := dockerCmdWithError("network", "rm", "testDelMulti0", "testDelMulti1", "testDelMulti2")

 	// err should not be nil due to deleting testDelMulti2 failed.

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 	// testDelMulti2 should fail due to network has active endpoints

 	assert.Assert(c, strings.Contains(out, "has active endpoints"))

 	assertNwNotAvailable(c, "testDelMulti0")

@@ -1298,7 +1298,7 @@ func (s *DockerNetworkSuite) TestDockerNetworkConnectPreferredIP(c *testing.T) {

 	// Still it should fail to connect to the default network with a specified IP (whatever ip)

 	out, _, err := dockerCmdWithError("network", "connect", "--ip", "172.21.55.44", "bridge", "c0")

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 	assert.Assert(c, strings.Contains(out, runconfig.ErrUnsupportedNetworkAndIP.Error()))

 }

@@ -1336,10 +1336,10 @@ func (s *DockerNetworkSuite) TestDockerNetworkUnsupportedRequiredIP(c *testing.T

 	assertNwIsAvailable(c, "n0")

 	out, _, err := dockerCmdWithError("run", "-d", "--ip", "172.28.99.88", "--net", "n0", "busybox", "top")

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 	assert.Assert(c, strings.Contains(out, runconfig.ErrUnsupportedNetworkNoSubnetAndIP.Error()))

 	out, _, err = dockerCmdWithError("run", "-d", "--ip6", "2001:db8:1234::9988", "--net", "n0", "busybox", "top")

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 	assert.Assert(c, strings.Contains(out, runconfig.ErrUnsupportedNetworkNoSubnetAndIP.Error()))

 	dockerCmd(c, "network", "rm", "n0")

 	assertNwNotAvailable(c, "n0")

@@ -1347,7 +1347,7 @@ func (s *DockerNetworkSuite) TestDockerNetworkUnsupportedRequiredIP(c *testing.T

 func checkUnsupportedNetworkAndIP(c *testing.T, nwMode string) {

 	out, _, err := dockerCmdWithError("run", "-d", "--net", nwMode, "--ip", "172.28.99.88", "--ip6", "2001:db8:1234::9988", "busybox", "top")

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 	assert.Assert(c, strings.Contains(out, runconfig.ErrUnsupportedNetworkAndIP.Error()))

 }

@@ -1474,8 +1474,8 @@ func (s *DockerNetworkSuite) TestDockerNetworkDisconnectDefault(c *testing.T) {

 	dockerCmd(c, "start", containerName)

 	assert.Assert(c, waitRun(containerName) == nil)

 	networks := inspectField(c, containerName, "NetworkSettings.Networks")

-	assert.Assert(c, strings.Contains(networks, netWorkName1), fmt.Sprintf(fmt.Sprintf("Should contain '%s' network", netWorkName1)))

-	assert.Assert(c, strings.Contains(networks, netWorkName2), fmt.Sprintf(fmt.Sprintf("Should contain '%s' network", netWorkName2)))

+	assert.Assert(c, strings.Contains(networks, netWorkName1), fmt.Sprintf("Should contain '%s' network", netWorkName1))

+	assert.Assert(c, strings.Contains(networks, netWorkName2), fmt.Sprintf("Should contain '%s' network", netWorkName2))

 	assert.Assert(c, !strings.Contains(networks, "bridge"), "Should not contain 'bridge' network")

 }

@@ -1538,11 +1538,11 @@ func (s *DockerSuite) TestUserDefinedNetworkConnectDisconnectAlias(c *testing.T)

 	// verify the alias option is rejected when running on predefined network

 	out, _, err := dockerCmdWithError("run", "--rm", "--name=any", "--net-alias=any", "busybox:glibc", "top")

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 	assert.Assert(c, strings.Contains(out, runconfig.ErrUnsupportedNetworkAndAlias.Error()))

 	// verify the alias option is rejected when connecting to predefined network

 	out, _, err = dockerCmdWithError("network", "connect", "--alias=any", "bridge", "first")

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 	assert.Assert(c, strings.Contains(out, runconfig.ErrUnsupportedNetworkAndAlias.Error()))

 }

@@ -428,7 +428,7 @@ func (s *DockerSuite) TestPluginUpgrade(c *testing.T) {

 	// make sure "v2" does not exists

 	_, err = os.Stat(filepath.Join(testEnv.DaemonInfo.DockerRootDir, "plugins", id, "rootfs", "v2"))

-	assert.Assert(c, os.IsNotExist(err), fmt.Sprintf("%s", out))

+	assert.Assert(c, os.IsNotExist(err), "%s", out)

 	dockerCmd(c, "plugin", "disable", "-f", plugin)

 	dockerCmd(c, "plugin", "upgrade", "--grant-all-permissions", "--skip-remote-check", plugin, pluginV2)

@@ -104,7 +104,7 @@ func (s *DockerSuite) TestPortList(c *testing.T) {

 			"-p", "9090-9092:80",

 			"busybox", "top")

 		// Exhausted port range did not return an error

-		assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+		assert.Assert(c, err != nil, "out: %s", out)

 		for i := 0; i < 3; i++ {

 			dockerCmd(c, "rm", "-f", IDs[i])

@@ -120,7 +120,7 @@ func (s *DockerSuite) TestPortList(c *testing.T) {

 			"-p", invalidRange,

 			"busybox", "top")

 		// Port range should have returned an error

-		assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+		assert.Assert(c, err != nil, "out: %s", out)

 	}

 	// test host range:container range spec.

@@ -299,7 +299,7 @@ func (s *DockerSuite) TestPortHostBinding(c *testing.T) {

 	out, _, err = dockerCmdWithError("run", "--net=host", "busybox", "nc", "localhost", "9876")

 	// Port is still bound after the Container is removed

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 }

 func (s *DockerSuite) TestPortExposeHostBinding(c *testing.T) {

@@ -311,7 +311,7 @@ func (s *DockerSuite) TestPortExposeHostBinding(c *testing.T) {

 	out, _ = dockerCmd(c, "port", firstID, "80")

 	_, exposedPort, err := net.SplitHostPort(out)

-	assert.Assert(c, err == nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err == nil, "out: %s", out)

 	dockerCmd(c, "run", "--net=host", "busybox",

 		"nc", "localhost", strings.TrimSpace(exposedPort))

@@ -321,7 +321,7 @@ func (s *DockerSuite) TestPortExposeHostBinding(c *testing.T) {

 	out, _, err = dockerCmdWithError("run", "--net=host", "busybox",

 		"nc", "localhost", strings.TrimSpace(exposedPort))

 	// Port is still bound after the Container is removed

-	assert.Assert(c, err != nil, fmt.Sprintf("out: %s", out))

+	assert.Assert(c, err != nil, "out: %s", out)

 }

 func (s *DockerSuite) TestPortBindingOnSandbox(c *testing.T) {

@@ -175,7 +175,7 @@ func (s *DockerSuite) TestPsListContainersSize(c *testing.T) {

 	assert.Equal(c, foundID, id[:12], fmt.Sprintf("Expected id %s, got %s", id[:12], foundID))

 	expectedSize := fmt.Sprintf("%dB", 2+baseBytes)

 	foundSize := lines[1][sizeIndex:]

-	assert.Assert(c, strings.Contains(foundSize, expectedSize), fmt.Sprintf("Expected size %q, got %q", expectedSize, foundSize))

+	assert.Assert(c, strings.Contains(foundSize, expectedSize), "Expected size %q, got %q", expectedSize, foundSize)

 }

 func (s *DockerSuite) TestPsListContainersFilterStatus(c *testing.T) {

@@ -442,11 +442,11 @@ func (s *DockerSuite) TestPsListContainersFilterExited(c *testing.T) {

 	secondZero, _ := dockerCmd(c, "run", "-d", "busybox", "true")

 	out, _, err := dockerCmdWithError("run", "--name", "nonzero1", "busybox", "false")

-	assert.Assert(c, err != nil, fmt.Sprintf("Should fail. out: %s", out))

+	assert.Assert(c, err != nil, "Should fail. out: %s", out)

 	firstNonZero := getIDByName(c, "nonzero1")

 	out, _, err = dockerCmdWithError("run", "--name", "nonzero2", "busybox", "false")

-	assert.Assert(c, err != nil, fmt.Sprintf("Should fail. out: %s", out))

+	assert.Assert(c, err != nil, "Should fail. out: %s", out)

 	secondNonZero := getIDByName(c, "nonzero2")

 	// filter containers by exited=0

@@ -516,7 +516,7 @@ func (s *DockerSuite) TestPsListContainersFilterCreated(c *testing.T) {

 	// Make sure it DOESN'T show up w/o a '-a' for normal 'ps'

 	out, _ = dockerCmd(c, "ps", "-q")

-	assert.Assert(c, !strings.Contains(out, shortCID), fmt.Sprintf("Should have not seen '%s' in ps output:\n%s", shortCID, out))

+	assert.Assert(c, !strings.Contains(out, shortCID), "Should have not seen '%s' in ps output:\n%s", shortCID, out)

 	// Make sure it DOES show up as 'Created' for 'ps -a'

 	out, _ = dockerCmd(c, "ps", "-a")

@@ -526,7 +526,7 @@ func (s *DockerSuite) TestPsListContainersFilterCreated(c *testing.T) {

 			continue

 		}

 		hits++

-		assert.Assert(c, strings.Contains(line, "Created"), fmt.Sprintf("Missing 'Created' on '%s'", line))

+		assert.Assert(c, strings.Contains(line, "Created"), "Missing 'Created' on '%s'", line)

 	}

 	assert.Equal(c, hits, 1, fmt.Sprintf("Should have seen '%s' in ps -a output once:%d\n%s", shortCID, hits, out))

@@ -602,7 +602,7 @@ func (s *DockerSuite) TestPsNotShowPortsOfStoppedContainer(c *testing.T) {

 	out, _ = dockerCmd(c, "ps", "-l")

 	lines = strings.Split(strings.TrimSpace(string(out)), "\n")

 	fields = strings.Fields(lines[1])

-	assert.Assert(c, fields[len(fields)-2] != expected, fmt.Sprintf("Should not got %v", expected))

+	assert.Assert(c, fields[len(fields)-2] != expected, "Should not got %v", expected)

 }

 func (s *DockerSuite) TestPsShowMounts(c *testing.T) {

@@ -26,7 +26,7 @@ func (s *DockerSuite) TestRmiWithContainerFails(c *testing.T) {

 	// Container is using image, should not be able to rmi

 	assert.ErrorContains(c, err, "")

 	// Container is using image, error message should contain errSubstr

-	assert.Assert(c, strings.Contains(out, errSubstr), fmt.Sprintf("Container: %q", cleanedContainerID))

+	assert.Assert(c, strings.Contains(out, errSubstr), "Container: %q", cleanedContainerID)

 	// make sure it didn't delete the busybox name

 	images, _ := dockerCmd(c, "images")

 	// The name 'busybox' should not have been removed from images

@@ -99,7 +99,7 @@ func (s *DockerSuite) TestRmiImgIDMultipleTag(c *testing.T) {

 	imagesAfter = cli.DockerCmd(c, "images", "-a").Combined()

 	// rmi -f failed, image still exists

-	assert.Assert(c, !strings.Contains(imagesAfter, imgID[:12]), fmt.Sprintf("ImageID:%q; ImagesAfter: %q", imgID, imagesAfter))

+	assert.Assert(c, !strings.Contains(imagesAfter, imgID[:12]), "ImageID:%q; ImagesAfter: %q", imgID, imagesAfter)

 }

 func (s *DockerSuite) TestRmiImgIDForce(c *testing.T) {

@@ -209,7 +209,7 @@ func (s *DockerSuite) TestRmiForceWithMultipleRepositories(c *testing.T) {

 	assert.Assert(c, !strings.Contains(out, "Untagged: "+tag1))

 	// Check built image still exists

 	images, _ := dockerCmd(c, "images", "-a")

-	assert.Assert(c, strings.Contains(images, imageName), fmt.Sprintf("Built image missing %q; Images: %q", imageName, images))

+	assert.Assert(c, strings.Contains(images, imageName), "Built image missing %q; Images: %q", imageName, images)

 }

 func (s *DockerSuite) TestRmiBlank(c *testing.T) {

@@ -217,9 +217,9 @@ func (s *DockerSuite) TestRmiBlank(c *testing.T) {

 	// Should have failed to delete ' ' image

 	assert.ErrorContains(c, err, "")

 	// Wrong error message generated

-	assert.Assert(c, !strings.Contains(out, "no such id"), fmt.Sprintf("out: %s", out))

+	assert.Assert(c, !strings.Contains(out, "no such id"), "out: %s", out)

 	// Expected error message not generated

-	assert.Assert(c, strings.Contains(out, "image name cannot be blank"), fmt.Sprintf("out: %s", out))

+	assert.Assert(c, strings.Contains(out, "image name cannot be blank"), "out: %s", out)

 }

 func (s *DockerSuite) TestRmiContainerImageNotFound(c *testing.T) {

@@ -244,7 +244,7 @@ func (s *DockerSuite) TestRmiContainerImageNotFound(c *testing.T) {

 	out, _, err := dockerCmdWithError("rmi", "-f", imageIds[0])

 	// The image of the running container should not be removed.

 	assert.ErrorContains(c, err, "")

-	assert.Assert(c, strings.Contains(out, "image is being used by running container"), fmt.Sprintf("out: %s", out))

+	assert.Assert(c, strings.Contains(out, "image is being used by running container"), "out: %s", out)

 }

 // #13422

@@ -3208,7 +3208,7 @@ func (s *DockerSuite) TestRunCreateContainerFailedCleanUp(c *testing.T) {

 	assert.Assert(c, err != nil, "Expected docker run to fail!")

 	containerID, err := inspectFieldWithError(name, "Id")