Merge pull request moby#4932 from tonistiigi/testifylint-fixes1

lint: fix some testifylint warnings

Author: tonistiigi

.golangci.yml

@@ -23,6 +23,7 @@ linters:
     - nolintlint
     - revive
     - staticcheck
+    # - testifylint
     - typecheck
     - unused
     - whitespace
@@ -74,6 +75,12 @@ linters-settings:
       - G601  # Implicit memory aliasing in for loop (false positives)
     config:
       G306: "0644"
+  testifylint:
+    disable:
+      # disable rules that reduce the test condition
+      - "empty"
+      - "bool-compare"
+      - "len"
 
 issues:
   exclude-files:

cache/manager_test.go

@@ -230,7 +230,7 @@ func TestManager(t *testing.T) {
 
 	fi, err := os.Stat(target)
 	require.NoError(t, err)
-	require.Equal(t, fi.IsDir(), true)
+	require.Equal(t, true, fi.IsDir())
 
 	err = lm.Unmount()
 	require.NoError(t, err)
@@ -315,7 +315,7 @@ func TestManager(t *testing.T) {
 
 	checkDiskUsage(ctx, t, cm, 0, 0)
 
-	require.Equal(t, len(buf.all), 2)
+	require.Equal(t, 2, len(buf.all))
 
 	err = cm.Close()
 	require.NoError(t, err)
@@ -506,7 +506,7 @@ func TestSnapshotExtract(t *testing.T) {
 
 	checkDiskUsage(ctx, t, cm, 2, 0)
 
-	require.Equal(t, len(buf.all), 0)
+	require.Equal(t, 0, len(buf.all))
 
 	dirs, err = os.ReadDir(filepath.Join(tmpdir, "snapshots/snapshots"))
 	require.NoError(t, err)
@@ -1823,7 +1823,7 @@ func TestGetRemotes(t *testing.T) {
 			eg.Go(func() error {
 				remotes, err := ir.GetRemotes(egctx, false, refCfg, true, nil)
 				require.NoError(t, err)
-				require.True(t, len(remotes) > 0, "for %s : %d", compressionType, len(remotes))
+				require.Greater(t, len(remotes), 0, "for %s : %d", compressionType, len(remotes))
 				gotMain, gotVariants := remotes[0], remotes[1:]
 
 				// Check the main blob is compatible with all == false

cache/metadata/metadata_test.go

@@ -146,14 +146,14 @@ func TestIndexes(t *testing.T) {
 	require.NoError(t, err)
 	require.Equal(t, 2, len(sis))
 
-	require.Equal(t, sis[0].ID(), "foo1")
-	require.Equal(t, sis[1].ID(), "foo3")
+	require.Equal(t, "foo1", sis[0].ID())
+	require.Equal(t, "foo3", sis[1].ID())
 
 	sis, err = s.Search(ctx, "tag:bax")
 	require.NoError(t, err)
 	require.Equal(t, 1, len(sis))
 
-	require.Equal(t, sis[0].ID(), "foo2")
+	require.Equal(t, "foo2", sis[0].ID())
 
 	err = s.Clear("foo1")
 	require.NoError(t, err)
@@ -162,7 +162,7 @@ func TestIndexes(t *testing.T) {
 	require.NoError(t, err)
 	require.Equal(t, 1, len(sis))
 
-	require.Equal(t, sis[0].ID(), "foo3")
+	require.Equal(t, "foo3", sis[0].ID())
 }
 
 func TestExternalData(t *testing.T) {

client/client_test.go

@@ -1018,7 +1018,7 @@ func testPushByDigest(t *testing.T, sb integration.Sandbox) {
 
 	require.Equal(t, resp.ExporterResponse[exptypes.ExporterImageDigestKey], desc.Digest.String())
 	require.Equal(t, images.MediaTypeDockerSchema2Manifest, desc.MediaType)
-	require.True(t, desc.Size > 0)
+	require.Greater(t, desc.Size, int64(0))
 }
 
 func testSecurityMode(t *testing.T, sb integration.Sandbox) {
@@ -3354,7 +3354,7 @@ func testSourceDateEpochClamp(t *testing.T, sb integration.Sandbox) {
 	require.NoError(t, err)
 	busyboxTms := busyboxTmsX.FromImage
 
-	require.True(t, len(busyboxTms) > 1)
+	require.Greater(t, len(busyboxTms), 1)
 	bboxLayerLen := len(busyboxTms) - 1
 
 	tm, err := time.Parse(time.RFC3339Nano, busyboxTms[1])
@@ -4475,7 +4475,7 @@ func testBuildPushAndValidate(t *testing.T, sb integration.Sandbox) {
 	require.Equal(t, "layers", ociimg.RootFS.Type)
 	require.Equal(t, 3, len(ociimg.RootFS.DiffIDs))
 	require.NotNil(t, ociimg.Created)
-	require.True(t, time.Since(*ociimg.Created) < 2*time.Minute)
+	require.Less(t, time.Since(*ociimg.Created), 2*time.Minute)
 	require.Condition(t, func() bool {
 		for _, env := range ociimg.Config.Env {
 			if strings.HasPrefix(env, "PATH=") {
@@ -7766,7 +7766,7 @@ func checkAllReleasable(t *testing.T, c *Client, sb integration.Sandbox, checkCo
 	retries := 0
 loop0:
 	for {
-		require.True(t, 20 > retries)
+		require.Greater(t, 20, retries)
 		retries++
 		du, err := c.DiskUsage(sb.Context())
 		require.NoError(t, err)
@@ -7814,7 +7814,7 @@ loop0:
 		if count == 0 {
 			break
 		}
-		require.True(t, 20 > retries)
+		require.Less(t, retries, 20)
 		retries++
 		time.Sleep(500 * time.Millisecond)
 	}

client/llb/fileop_test.go

@@ -710,7 +710,7 @@ func parseDef(t *testing.T, def [][]byte) (map[digest.Digest]pb.Op, []pb.Op) {
 }
 
 func last(t *testing.T, arr []pb.Op) (digest.Digest, int) {
-	require.True(t, len(arr) > 1)
+	require.Greater(t, len(arr), 1)
 
 	op := arr[len(arr)-1]
 	require.Equal(t, 1, len(op.Inputs))

client/llb/llbbuild/llbbuild_test.go

@@ -24,7 +24,7 @@ func TestMarshal(t *testing.T) {
 	require.NoError(t, err)
 
 	buildop := op.GetBuild()
-	require.NotEqual(t, buildop, nil)
+	require.NotNil(t, buildop)
 
 	require.Equal(t, len(op.Inputs), 1)
 	require.Equal(t, buildop.Builder, pb.LLBBuilder)

client/llb/llbtest/platform_test.go

@@ -30,7 +30,7 @@ func TestCustomPlatform(t *testing.T) {
 	e, err := llbsolver.Load(context.TODO(), def.ToPB(), nil)
 	require.NoError(t, err)
 
-	require.Equal(t, depth(e), 5)
+	require.Equal(t, 5, depth(e))
 
 	expected := ocispecs.Platform{OS: "windows", Architecture: "amd64"}
 	require.Equal(t, expected, platform(e))
@@ -59,7 +59,7 @@ func TestDefaultPlatform(t *testing.T) {
 	e, err := llbsolver.Load(context.TODO(), def.ToPB(), nil)
 	require.NoError(t, err)
 
-	require.Equal(t, depth(e), 2)
+	require.Equal(t, 2, depth(e))
 
 	// needs extra normalize for default spec
 	// https://github.com/moby/buildkit/pull/2427#issuecomment-952301867
@@ -103,7 +103,7 @@ func TestPlatformMixed(t *testing.T) {
 	e, err := llbsolver.Load(context.TODO(), def.ToPB(), nil)
 	require.NoError(t, err)
 
-	require.Equal(t, depth(e), 4)
+	require.Equal(t, 4, depth(e))
 
 	expectedAmd := ocispecs.Platform{OS: "linux", Architecture: "amd64"}
 	require.Equal(t, []string{"cmd-main"}, args(e))

control/control_test.go

@@ -140,9 +140,9 @@ func TestParseCacheExportIgnoreError(t *testing.T) {
 		t.Run(ignoreErrStr, func(t *testing.T) {
 			ignoreErr, supported := parseCacheExportIgnoreError(ignoreErrStr)
 			t.Log("checking expectedIgnoreError")
-			require.Equal(t, ignoreErr, test.expectedIgnoreError)
+			require.Equal(t, test.expectedIgnoreError, ignoreErr)
 			t.Log("checking expectedSupported")
-			require.Equal(t, supported, test.expectedSupported)
+			require.Equal(t, test.expectedSupported, supported)
 		})
 	}
 }

executor/oci/mounts_test.go

@@ -153,7 +153,7 @@ func TestWithRemovedMounts(t *testing.T) {
 
 	oldLen := len(s.Mounts)
 	err := withRemovedMount("/run")(appcontext.Context(), nil, nil, &s)
-	assert.NoError(t, err)
+	require.NoError(t, err)
 	assert.Equal(t, oldLen-1, len(s.Mounts))
 }
 

executor/resources/io_test.go

@@ -7,6 +7,7 @@ import (
 
 	resourcestypes "github.com/moby/buildkit/executor/resources/types"
 	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 )
 
 func TestParseIOStat(t *testing.T) {
@@ -15,15 +16,15 @@ func TestParseIOStat(t *testing.T) {
 	ioStatContents := `8:0 rbytes=1024 wbytes=2048 dbytes=4096 rios=16 wios=32 dios=64
 8:1 rbytes=512 wbytes=1024 dbytes=2048 rios=8 wios=16 dios=32`
 	err := os.WriteFile(filepath.Join(testDir, "io.stat"), []byte(ioStatContents), 0644)
-	assert.NoError(t, err)
+	require.NoError(t, err)
 
 	ioPressureContents := `some avg10=1.23 avg60=4.56 avg300=7.89 total=3031
 full avg10=0.12 avg60=0.34 avg300=0.56 total=9876`
 	err = os.WriteFile(filepath.Join(testDir, "io.pressure"), []byte(ioPressureContents), 0644)
-	assert.NoError(t, err)
+	require.NoError(t, err)
 
 	ioStat, err := getCgroupIOStat(testDir)
-	assert.NoError(t, err)
+	require.NoError(t, err)
 
 	var expectedPressure = &resourcestypes.Pressure{
 		Some: &resourcestypes.PressureValues{