[chore]: enable len rule from testifylint

mmorel-35 · mmorel-35 · commit 67afab1fb156 · 2024-09-02T06:06:39.000Z
Signed-off-by: Matthieu MOREL &lt;matthieu.morel35@gmail.com&gt;
diff --git a/.golangci.yml b/.golangci.yml
@@ -132,7 +132,6 @@ linters-settings:
       - float-compare
       - formatter
       - go-require
-      - len
       - negative-positive
       - nil-compare
       - require-error
diff --git a/Makefile b/Makefile
@@ -56,6 +56,14 @@ gotest-with-cover:
 	@$(MAKE) for-all-target TARGET="test-with-cover"
 	$(GOCMD) tool covdata textfmt -i=./coverage/unit -o ./coverage.txt
 
+.PHONY: gotestifylint
+gotestifylint:
+	@$(MAKE) for-all-target TARGET="testifylint"
+
+.PHONY: gotestifylint-fix
+gotestifylint-fix:
+	$(MAKE) for-all-target TARGET="testifylint-fix"
+
 .PHONY: goporto
 goporto: $(PORTO)
 	$(PORTO) -w --include-internal --skip-dirs "^cmd/mdatagen/third_party$$" ./
diff --git a/Makefile.Common b/Makefile.Common
@@ -36,6 +36,9 @@ MULTIMOD     := $(TOOLS_BIN_DIR)/multimod
 PORTO        := $(TOOLS_BIN_DIR)/porto
 SEMCONVGEN   := $(TOOLS_BIN_DIR)/semconvgen
 SEMCONVKIT   := $(TOOLS_BIN_DIR)/semconvkit
+TESTIFYLINT  := $(TOOLS_BIN_DIR)/testifylint
+
+TESTIFYLINT_OPT?= --enable-all --disable=compares,empty,error-is-as,error-nil,expected-actual,float-compare,formatter,go-require,negative-positive,nil-compare,require-error
 
 .PHONY: install-tools
 install-tools: $(TOOLS_BIN_NAMES)
@@ -88,3 +91,12 @@ impi: $(IMPI)
 .PHONY: moddownload
 moddownload:
 	$(GOCMD) mod download
+
+.PHONY: testifylint
+testifylint: $(TESTIFYLINT)
+	@echo "running $(TESTIFYLINT)"
+	@$(TESTIFYLINT) $(TESTIFYLINT_OPT) ./...
+
+.PHONY: testifylint-fix
+testifylint-fix:
+	@$(MAKE) testifylint TESTIFYLINT_OPT="${TESTIFYLINT_OPT} --fix"
diff --git a/connector/forwardconnector/forward_test.go b/connector/forwardconnector/forward_test.go
@@ -57,7 +57,7 @@ func TestForward(t *testing.T) {
 	assert.NoError(t, metricsToMetrics.Shutdown(ctx))
 	assert.NoError(t, logsToLogs.Shutdown(ctx))
 
-	assert.Equal(t, 1, len(tracesSink.AllTraces()))
-	assert.Equal(t, 2, len(metricsSink.AllMetrics()))
-	assert.Equal(t, 3, len(logsSink.AllLogs()))
+	assert.Len(t, tracesSink.AllTraces(), 1)
+	assert.Len(t, metricsSink.AllMetrics(), 2)
+	assert.Len(t, logsSink.AllLogs(), 3)
 }
diff --git a/consumer/consumertest/sink_test.go b/consumer/consumertest/sink_test.go
@@ -28,7 +28,7 @@ func TestTracesSink(t *testing.T) {
 	assert.Equal(t, want, sink.AllTraces())
 	assert.Equal(t, len(want), sink.SpanCount())
 	sink.Reset()
-	assert.Equal(t, 0, len(sink.AllTraces()))
+	assert.Len(t, sink.AllTraces(), 0)
 	assert.Equal(t, 0, sink.SpanCount())
 }
 
@@ -43,7 +43,7 @@ func TestMetricsSink(t *testing.T) {
 	assert.Equal(t, want, sink.AllMetrics())
 	assert.Equal(t, 2*len(want), sink.DataPointCount())
 	sink.Reset()
-	assert.Equal(t, 0, len(sink.AllMetrics()))
+	assert.Len(t, sink.AllMetrics(), 0)
 	assert.Equal(t, 0, sink.DataPointCount())
 }
 
@@ -58,7 +58,7 @@ func TestLogsSink(t *testing.T) {
 	assert.Equal(t, want, sink.AllLogs())
 	assert.Equal(t, len(want), sink.LogRecordCount())
 	sink.Reset()
-	assert.Equal(t, 0, len(sink.AllLogs()))
+	assert.Len(t, sink.AllLogs(), 0)
 	assert.Equal(t, 0, sink.LogRecordCount())
 }
 
@@ -72,5 +72,5 @@ func TestProfilesSink(t *testing.T) {
 	}
 	assert.Equal(t, want, sink.AllProfiles())
 	sink.Reset()
-	assert.Equal(t, 0, len(sink.AllProfiles()))
+	assert.Len(t, sink.AllProfiles(), 0)
 }
diff --git a/exporter/exporterhelper/logs_test.go b/exporter/exporterhelper/logs_test.go
@@ -418,7 +418,7 @@ func checkWrapSpanForLogsExporter(t *testing.T, sr *tracetest.SpanRecorder, trac
 
 	// Inspection time!
 	gotSpanData := sr.Ended()
-	require.Equal(t, numRequests+1, len(gotSpanData))
+	require.Len(t, gotSpanData, numRequests+1)
 
 	parentSpan := gotSpanData[numRequests]
 	require.Equalf(t, fakeLogsParentSpanName, parentSpan.Name(), "SpanData %v", parentSpan)
diff --git a/exporter/exporterhelper/metrics_test.go b/exporter/exporterhelper/metrics_test.go
@@ -424,7 +424,7 @@ func checkWrapSpanForMetricsExporter(t *testing.T, sr *tracetest.SpanRecorder, t
 
 	// Inspection time!
 	gotSpanData := sr.Ended()
-	require.Equal(t, numRequests+1, len(gotSpanData))
+	require.Len(t, gotSpanData, numRequests+1)
 
 	parentSpan := gotSpanData[numRequests]
 	require.Equalf(t, fakeMetricsParentSpanName, parentSpan.Name(), "SpanData %v", parentSpan)
diff --git a/exporter/exporterhelper/traces_test.go b/exporter/exporterhelper/traces_test.go
@@ -426,7 +426,7 @@ func checkWrapSpanForTracesExporter(t *testing.T, sr *tracetest.SpanRecorder, tr
 
 	// Inspection time!
 	gotSpanData := sr.Ended()
-	require.Equal(t, numRequests+1, len(gotSpanData))
+	require.Len(t, gotSpanData, numRequests+1)
 
 	parentSpan := gotSpanData[numRequests]
 	require.Equalf(t, fakeTraceParentSpanName, parentSpan.Name(), "SpanData %v", parentSpan)
diff --git a/exporter/otlpexporter/otlp_test.go b/exporter/otlpexporter/otlp_test.go
@@ -300,7 +300,7 @@ func TestSendTraces(t *testing.T) {
 
 	md := rcv.getMetadata()
 	require.EqualValues(t, md.Get("header"), expectedHeader)
-	require.Equal(t, len(md.Get("User-Agent")), 1)
+	require.Len(t, md.Get("User-Agent"), 1)
 	require.Contains(t, md.Get("User-Agent")[0], "Collector/1.2.3test")
 
 	// Return partial success
@@ -472,7 +472,7 @@ func TestSendMetrics(t *testing.T) {
 
 	mdata := rcv.getMetadata()
 	require.EqualValues(t, mdata.Get("header"), expectedHeader)
-	require.Equal(t, len(mdata.Get("User-Agent")), 1)
+	require.Len(t, mdata.Get("User-Agent"), 1)
 	require.Contains(t, mdata.Get("User-Agent")[0], "Collector/1.2.3test")
 
 	st := status.New(codes.InvalidArgument, "Invalid argument")
@@ -763,7 +763,7 @@ func TestSendLogData(t *testing.T) {
 	assert.EqualValues(t, ld, rcv.getLastRequest())
 
 	md := rcv.getMetadata()
-	require.Equal(t, len(md.Get("User-Agent")), 1)
+	require.Len(t, md.Get("User-Agent"), 1)
 	require.Contains(t, md.Get("User-Agent")[0], "Collector/1.2.3test")
 
 	st := status.New(codes.InvalidArgument, "Invalid argument")
diff --git a/extension/ballastextension/memory_ballast_test.go b/extension/ballastextension/memory_ballast_test.go
@@ -62,7 +62,7 @@ func TestMemoryBallast(t *testing.T) {
 			assert.Nil(t, mbExt.ballast)
 
 			assert.NoError(t, mbExt.Start(context.Background(), componenttest.NewNopHost()))
-			assert.Equal(t, tt.expect, len(mbExt.ballast))
+			assert.Len(t, mbExt.ballast, tt.expect)
 
 			assert.NoError(t, mbExt.Shutdown(context.Background()))
 			assert.Nil(t, mbExt.ballast)
diff --git a/internal/e2e/status_test.go b/internal/e2e/status_test.go
@@ -118,7 +118,7 @@ func Test_ComponentStatusReporting_SharedInstance(t *testing.T) {
 	err = s.Shutdown(context.Background())
 	require.NoError(t, err)
 
-	require.Equal(t, 2, len(eventsReceived))
+	require.Len(t, eventsReceived, 2)
 
 	for instanceID, events := range eventsReceived {
 		pipelineIDs := ""
diff --git a/internal/sharedcomponent/sharedcomponent_test.go b/internal/sharedcomponent/sharedcomponent_test.go
@@ -200,7 +200,7 @@ func TestReportStatusOnStartShutdown(t *testing.T) {
 				require.Equal(t, tc.shutdownErr, err)
 			}
 
-			require.Equal(t, tc.expectedNumReporterInstances, len(reportedStatuses))
+			require.Len(t, reportedStatuses, tc.expectedNumReporterInstances)
 
 			for _, actualStatuses := range reportedStatuses {
 				require.Equal(t, tc.expectedStatuses, actualStatuses)
diff --git a/internal/testutil/testutil.go b/internal/testutil/testutil.go
@@ -149,14 +149,14 @@ func createExclusionsList(exclusionsText string, t testing.TB) []portpair {
 	var exclusions []portpair
 
 	parts := strings.Split(exclusionsText, "--------")
-	require.Equal(t, len(parts), 3)
+	require.Len(t, parts, 3)
 	portsText := strings.Split(parts[2], "*")
 	require.Greater(t, len(portsText), 1) // original text may have a suffix like " - Administered port exclusions."
 	lines := strings.Split(portsText[0], "\n")
 	for _, line := range lines {
 		if strings.TrimSpace(line) != "" {
 			entries := strings.Fields(strings.TrimSpace(line))
-			require.Equal(t, len(entries), 2)
+			require.Len(t, entries, 2)
 			pair := portpair{entries[0], entries[1]}
 			exclusions = append(exclusions, pair)
 		}
diff --git a/internal/testutil/testutil_test.go b/internal/testutil/testutil_test.go
@@ -68,8 +68,8 @@ Start Port    End Port
 * - Administered port exclusions.
 `
 	exclusions := createExclusionsList(exclusionsText, t)
-	require.Equal(t, len(exclusions), 2)
+	require.Len(t, exclusions, 2)
 
 	emptyExclusions := createExclusionsList(emptyExclusionsText, t)
-	require.Equal(t, len(emptyExclusions), 0)
+	require.Len(t, emptyExclusions, 0)
 }
diff --git a/internal/tools/go.mod b/internal/tools/go.mod
@@ -5,6 +5,7 @@ go 1.22.1
 toolchain go1.22.6
 
 require (
+	github.com/Antonboom/testifylint v1.4.3
 	github.com/a8m/envsubst v1.4.2
 	github.com/client9/misspell v0.3.4
 	github.com/golangci/golangci-lint v1.60.1
@@ -29,7 +30,6 @@ require (
 	github.com/Abirdcfly/dupword v0.0.14 // indirect
 	github.com/Antonboom/errname v0.1.13 // indirect
 	github.com/Antonboom/nilnil v0.1.9 // indirect
-	github.com/Antonboom/testifylint v1.4.3 // indirect
 	github.com/BurntSushi/toml v1.4.1-0.20240526193622-a339e1f7089c // indirect
 	github.com/Crocmagnon/fatcontext v0.4.0 // indirect
 	github.com/Djarvur/go-err113 v0.0.0-20210108212216-aea10b59be24 // indirect
diff --git a/internal/tools/tools.go b/internal/tools/tools.go
@@ -11,6 +11,7 @@ package tools // import "go.opentelemetry.io/collector/internal/tools"
 // This ensures that all systems use the same version of tools in addition to regular dependencies.
 
 import (
+	_ "github.com/Antonboom/testifylint"
 	_ "github.com/a8m/envsubst/cmd/envsubst"
 	_ "github.com/client9/misspell/cmd/misspell"
 	_ "github.com/golangci/golangci-lint/cmd/golangci-lint"
diff --git a/otelcol/buffered_core_test.go b/otelcol/buffered_core_test.go
@@ -72,7 +72,7 @@ func Test_bufferedCore_Write(t *testing.T) {
 		e,
 		fields,
 	}
-	require.Equal(t, 1, len(bc.logs))
+	require.Len(t, bc.logs, 1)
 	require.Equal(t, expected, bc.logs[0])
 }
 
diff --git a/otelcol/collector_core_test.go b/otelcol/collector_core_test.go
@@ -72,7 +72,7 @@ func Test_collectorCore_Write(t *testing.T) {
 		e,
 		fields,
 	}
-	require.Equal(t, 1, len(cc.core.(*bufferedCore).logs))
+	require.Len(t, cc.core.(*bufferedCore).logs, 1)
 	require.Equal(t, expected, cc.core.(*bufferedCore).logs[0])
 }
 
diff --git a/otelcol/otelcoltest/nop_factories_test.go b/otelcol/otelcoltest/nop_factories_test.go
@@ -17,27 +17,27 @@ func TestNopFactories(t *testing.T) {
 	nopFactories, err := NopFactories()
 	require.NoError(t, err)
 
-	require.Equal(t, 1, len(nopFactories.Receivers))
+	require.Len(t, nopFactories.Receivers, 1)
 	nopReceiverFactory, ok := nopFactories.Receivers[nopType]
 	require.True(t, ok)
 	require.Equal(t, nopType, nopReceiverFactory.Type())
 
-	require.Equal(t, 1, len(nopFactories.Processors))
+	require.Len(t, nopFactories.Processors, 1)
 	nopProcessorFactory, ok := nopFactories.Processors[nopType]
 	require.True(t, ok)
 	require.Equal(t, nopType, nopProcessorFactory.Type())
 
-	require.Equal(t, 1, len(nopFactories.Exporters))
+	require.Len(t, nopFactories.Exporters, 1)
 	nopExporterFactory, ok := nopFactories.Exporters[nopType]
 	require.True(t, ok)
 	require.Equal(t, nopType, nopExporterFactory.Type())
 
-	require.Equal(t, 1, len(nopFactories.Extensions))
+	require.Len(t, nopFactories.Extensions, 1)
 	nopExtensionFactory, ok := nopFactories.Extensions[nopType]
 	require.True(t, ok)
 	require.Equal(t, nopType, nopExtensionFactory.Type())
 
-	require.Equal(t, 1, len(nopFactories.Connectors))
+	require.Len(t, nopFactories.Connectors, 1)
 	nopConnectorFactory, ok := nopFactories.Connectors[nopType]
 	require.True(t, ok)
 	require.Equal(t, nopType, nopConnectorFactory.Type())
diff --git a/pdata/pcommon/map_test.go b/pdata/pcommon/map_test.go
@@ -320,7 +320,7 @@ func TestMap_Range(t *testing.T) {
 		delete(rawMap, k)
 		return true
 	})
-	assert.EqualValues(t, 0, len(rawMap))
+	assert.Len(t, rawMap, 0)
 }
 
 func TestMap_FromRaw(t *testing.T) {
diff --git a/pdata/pcommon/slice_test.go b/pdata/pcommon/slice_test.go
@@ -77,7 +77,7 @@ func TestSlice_EnsureCapacity(t *testing.T) {
 	for i := 0; i < es.Len(); i++ {
 		expectedEs[es.At(i).getOrig()] = true
 	}
-	assert.Equal(t, es.Len(), len(expectedEs))
+	assert.Len(t, expectedEs, es.Len())
 	es.EnsureCapacity(ensureSmallLen)
 	assert.Less(t, ensureSmallLen, es.Len())
 	foundEs := make(map[*otlpcommon.AnyValue]bool, es.Len())
@@ -89,7 +89,7 @@ func TestSlice_EnsureCapacity(t *testing.T) {
 	// Test ensure larger capacity
 	const ensureLargeLen = 9
 	oldLen := es.Len()
-	assert.Equal(t, oldLen, len(expectedEs))
+	assert.Len(t, expectedEs, oldLen)
 	es.EnsureCapacity(ensureLargeLen)
 	assert.Equal(t, ensureLargeLen, cap(*es.getOrig()))
 }
diff --git a/processor/batchprocessor/batch_processor_test.go b/processor/batchprocessor/batch_processor_test.go
diff --git a/receiver/otlpreceiver/otlp_test.go b/receiver/otlpreceiver/otlp_test.go
diff --git a/receiver/receiverhelper/obsreport_test.go b/receiver/receiverhelper/obsreport_test.go
diff --git a/service/internal/graph/graph_test.go b/service/internal/graph/graph_test.go
diff --git a/service/internal/proctelemetry/process_telemetry_linux_test.go b/service/internal/proctelemetry/process_telemetry_linux_test.go
diff --git a/service/internal/proctelemetry/process_telemetry_test.go b/service/internal/proctelemetry/process_telemetry_test.go
diff --git a/service/internal/testcomponents/example_exporter_test.go b/service/internal/testcomponents/example_exporter_test.go

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ func TestTracesSink(t *testing.T) {`
`28`	`28`	`assert.Equal(t, want, sink.AllTraces())`
`29`	`29`	`assert.Equal(t, len(want), sink.SpanCount())`
`30`	`30`	`sink.Reset()`
`31`		`- assert.Equal(t, 0, len(sink.AllTraces()))`
	`31`	`+ assert.Len(t, sink.AllTraces(), 0)`
`32`	`32`	`assert.Equal(t, 0, sink.SpanCount())`
`33`	`33`	`}`
`34`	`34`
`@@ -43,7 +43,7 @@ func TestMetricsSink(t *testing.T) {`
`43`	`43`	`assert.Equal(t, want, sink.AllMetrics())`
`44`	`44`	`assert.Equal(t, 2*len(want), sink.DataPointCount())`
`45`	`45`	`sink.Reset()`
`46`		`- assert.Equal(t, 0, len(sink.AllMetrics()))`
	`46`	`+ assert.Len(t, sink.AllMetrics(), 0)`
`47`	`47`	`assert.Equal(t, 0, sink.DataPointCount())`
`48`	`48`	`}`
`49`	`49`
`@@ -58,7 +58,7 @@ func TestLogsSink(t *testing.T) {`
`58`	`58`	`assert.Equal(t, want, sink.AllLogs())`
`59`	`59`	`assert.Equal(t, len(want), sink.LogRecordCount())`
`60`	`60`	`sink.Reset()`
`61`		`- assert.Equal(t, 0, len(sink.AllLogs()))`
	`61`	`+ assert.Len(t, sink.AllLogs(), 0)`
`62`	`62`	`assert.Equal(t, 0, sink.LogRecordCount())`
`63`	`63`	`}`
`64`	`64`
`@@ -72,5 +72,5 @@ func TestProfilesSink(t *testing.T) {`
`72`	`72`	`}`
`73`	`73`	`assert.Equal(t, want, sink.AllProfiles())`
`74`	`74`	`sink.Reset()`
`75`		`- assert.Equal(t, 0, len(sink.AllProfiles()))`
	`75`	`+ assert.Len(t, sink.AllProfiles(), 0)`
`76`	`76`	`}`