fix ginkgolinter findins

nunnatsa · nunnatsa · commit 53946b6c78f2 · 2024-03-14T08:17:04.000+02:00
diff --git a/internal/framework/controller/register_test.go b/internal/framework/controller/register_test.go
@@ -150,7 +150,7 @@ func TestRegister(t *testing.T) {
 			} else {
 				err := register()
 				if test.expectedErr == nil {
-					g.Expect(err).To(BeNil())
+					g.Expect(err).ToNot(HaveOccurred())
 				} else {
 					g.Expect(err).To(MatchError(test.expectedErr))
 				}
diff --git a/internal/framework/events/events_test.go b/internal/framework/events/events_test.go
@@ -28,8 +28,8 @@ func TestEventLoop_SwapBatches(t *testing.T) {
 
 	eventLoop.swapBatches()
 
-	g.Expect(len(eventLoop.currentBatch)).To(Equal(len(nextBatch)))
+	g.Expect(eventLoop.currentBatch).To(HaveLen(len(nextBatch)))
 	g.Expect(eventLoop.currentBatch).To(Equal(nextBatch))
-	g.Expect(len(eventLoop.nextBatch)).To(Equal(0))
+	g.Expect(eventLoop.nextBatch).To(BeEmpty())
 	g.Expect(cap(eventLoop.nextBatch)).To(Equal(3))
 }
diff --git a/internal/framework/events/first_eventbatch_preparer_test.go b/internal/framework/events/first_eventbatch_preparer_test.go
@@ -58,7 +58,7 @@ var _ = Describe("FirstEventBatchPreparer", func() {
 			batch, err := preparer.Prepare(context.Background())
 
 			Expect(batch).Should(BeEmpty())
-			Expect(err).Should(BeNil())
+			Expect(err).ShouldNot(HaveOccurred())
 		})
 
 		It("should prepare one event for each resource type", func() {
@@ -97,7 +97,7 @@ var _ = Describe("FirstEventBatchPreparer", func() {
 			batch, err := preparer.Prepare(context.Background())
 
 			Expect(batch).Should(Equal(expectedBatch))
-			Expect(err).Should(BeNil())
+			Expect(err).ShouldNot(HaveOccurred())
 		})
 	})
 
diff --git a/internal/framework/events/loop_test.go b/internal/framework/events/loop_test.go
@@ -59,7 +59,7 @@ var _ = Describe("EventLoop", func() {
 
 			var err error
 			Eventually(errorCh).Should(Receive(&err))
-			Expect(err).To(BeNil())
+			Expect(err).ToNot(HaveOccurred())
 		})
 
 		// Because BeforeEach() creates the first batch and waits for it to be handled, in the tests below
@@ -138,7 +138,7 @@ var _ = Describe("EventLoop", func() {
 			cancel()
 			err := eventLoop.Start(ctx)
 
-			Expect(err).Should(BeNil())
+			Expect(err).ShouldNot(HaveOccurred())
 		})
 	})
 })
diff --git a/internal/framework/runnables/runnables_test.go b/internal/framework/runnables/runnables_test.go
@@ -32,7 +32,7 @@ func TestEnableAfterBecameLeader(t *testing.T) {
 	g.Expect(enabled).To(BeFalse())
 
 	err := enableAfterBecameLeader.Start(context.Background())
-	g.Expect(err).To(BeNil())
+	g.Expect(err).ToNot(HaveOccurred())
 
 	g.Expect(enabled).To(BeTrue())
 }
diff --git a/internal/mode/provisioner/handler_test.go b/internal/mode/provisioner/handler_test.go
@@ -342,7 +342,7 @@ var _ = Describe("handler", func() {
 				err := k8sclient.List(context.Background(), deps)
 
 				Expect(err).ToNot(HaveOccurred())
-				Expect(deps.Items).To(HaveLen(0))
+				Expect(deps.Items).To(BeEmpty())
 			})
 		})
 
@@ -370,7 +370,7 @@ var _ = Describe("handler", func() {
 				err := k8sclient.List(context.Background(), deps)
 
 				Expect(err).ToNot(HaveOccurred())
-				Expect(deps.Items).To(HaveLen(0))
+				Expect(deps.Items).To(BeEmpty())
 			})
 		})
 
diff --git a/internal/mode/static/handler_test.go b/internal/mode/static/handler_test.go
@@ -229,7 +229,7 @@ var _ = Describe("eventHandler", func() {
 				"Failed to update control plane configuration: logging.level: Unsupported value: " +
 					"\"invalid\": supported values: \"info\", \"debug\", \"error\"")
 			Expect(statuses).To(Equal(expStatuses(cond)))
-			Expect(len(fakeEventRecorder.Events)).To(Equal(1))
+			Expect(fakeEventRecorder.Events).To(HaveLen(1))
 			event := <-fakeEventRecorder.Events
 			Expect(event).To(Equal(
 				"Warning UpdateFailed Failed to update control plane configuration: logging.level: Unsupported value: " +
@@ -252,7 +252,7 @@ var _ = Describe("eventHandler", func() {
 
 			Expect(handler.GetLatestConfiguration()).To(BeNil())
 
-			Expect(len(fakeEventRecorder.Events)).To(Equal(1))
+			Expect(fakeEventRecorder.Events).To(HaveLen(1))
 			event := <-fakeEventRecorder.Events
 			Expect(event).To(Equal("Warning ResourceDeleted NginxGateway configuration was deleted; using defaults"))
 			Expect(zapLogLevelSetter.Enabled(zap.InfoLevel)).To(BeTrue())
diff --git a/internal/mode/static/sort/sort_test.go b/internal/mode/static/sort/sort_test.go
@@ -85,7 +85,7 @@ func TestLessObjectMeta(t *testing.T) {
 			invertedResult := LessObjectMeta(test.meta2, test.meta1)
 
 			g.Expect(result).To(Equal(test.expected))
-			g.Expect(invertedResult).To(Equal(false))
+			g.Expect(invertedResult).To(BeFalse())
 		})
 	}
 }
diff --git a/internal/mode/static/state/graph/backend_tls_policy_test.go b/internal/mode/static/state/graph/backend_tls_policy_test.go
@@ -405,7 +405,7 @@ func TestValidateBackendTLSPolicy(t *testing.T) {
 			if !test.isValid && !test.ignored {
 				g.Expect(conds).To(HaveLen(1))
 			} else {
-				g.Expect(conds).To(HaveLen(0))
+				g.Expect(conds).To(BeEmpty())
 			}
 		})
 	}
diff --git a/internal/mode/static/state/graph/httproute_test.go b/internal/mode/static/state/graph/httproute_test.go
@@ -250,7 +250,7 @@ func TestBuildSectionNameRefs(t *testing.T) {
 			if test.expectedError != nil {
 				g.Expect(err).To(Equal(test.expectedError))
 			} else {
-				g.Expect(err).To(BeNil())
+				g.Expect(err).ToNot(HaveOccurred())
 			}
 		})
 	}
diff --git a/internal/mode/static/state/graph/secret_test.go b/internal/mode/static/state/graph/secret_test.go
@@ -191,7 +191,7 @@ func TestSecretResolver(t *testing.T) {
 	for _, test := range tests {
 		err := resolver.resolve(test.nsname)
 		if test.expectedErrMsg == "" {
-			g.Expect(err).To(BeNil(), fmt.Sprintf("case %q", test.name))
+			g.Expect(err).ToNot(HaveOccurred(), fmt.Sprintf("case %q", test.name))
 		} else {
 			g.Expect(err).To(MatchError(test.expectedErrMsg), fmt.Sprintf("case %q", test.name))
 		}
diff --git a/internal/mode/static/telemetry/collector_test.go b/internal/mode/static/telemetry/collector_test.go
@@ -347,7 +347,7 @@ var _ = Describe("Collector", Ordered, func() {
 
 				data, err := dataCollector.Collect(ctx)
 
-				Expect(err).To(BeNil())
+				Expect(err).ToNot(HaveOccurred())
 				Expect(expData).To(Equal(data))
 			})
 		})
@@ -414,7 +414,7 @@ var _ = Describe("Collector", Ordered, func() {
 
 					data, err := dataCollector.Collect(ctx)
 
-					Expect(err).To(BeNil())
+					Expect(err).ToNot(HaveOccurred())
 					Expect(expData).To(Equal(data))
 				})
 			})
@@ -430,7 +430,7 @@ var _ = Describe("Collector", Ordered, func() {
 
 				data, err := dataCollector.Collect(ctx)
 
-				Expect(err).To(BeNil())
+				Expect(err).ToNot(HaveOccurred())
 				Expect(expData).To(Equal(data))
 			})
 
@@ -539,7 +539,7 @@ var _ = Describe("Collector", Ordered, func() {
 
 				data, err := dataCollector.Collect(ctx)
 
-				Expect(err).To(BeNil())
+				Expect(err).ToNot(HaveOccurred())
 				Expect(expData).To(Equal(data))
 			})
 
@@ -558,7 +558,7 @@ var _ = Describe("Collector", Ordered, func() {
 
 				data, err := dataCollector.Collect(ctx)
 
-				Expect(err).To(BeNil())
+				Expect(err).ToNot(HaveOccurred())
 				Expect(expData).To(Equal(data))
 			})
 
@@ -576,7 +576,7 @@ var _ = Describe("Collector", Ordered, func() {
 
 				data, err := dataCollector.Collect(ctx)
 
-				Expect(err).To(BeNil())
+				Expect(err).ToNot(HaveOccurred())
 				Expect(expData).To(Equal(data))
 			})
 
diff --git a/internal/mode/static/telemetry/exporter_test.go b/internal/mode/static/telemetry/exporter_test.go
@@ -18,7 +18,7 @@ func TestLoggingExporter(t *testing.T) {
 
 	err := exporter.Export(context.Background(), &Data{})
 
-	g.Expect(err).To(BeNil())
+	g.Expect(err).ToNot(HaveOccurred())
 	g.Expect(buffer.String()).To(ContainSubstring(`"level":"info"`))
 	g.Expect(buffer.String()).To(ContainSubstring(`"msg":"Exporting telemetry"`))
 }
diff --git a/tests/suite/longevity_test.go b/tests/suite/longevity_test.go
@@ -76,7 +76,7 @@ var _ = Describe("Longevity", Label("longevity-setup", "longevity-teardown"), fu
 		homeDir, err := os.UserHomeDir()
 		Expect(err).ToNot(HaveOccurred())
 
-		Expect(framework.WriteContent(resultsFile, "\n## Traffic\n"))
+		Expect(framework.WriteContent(resultsFile, "\n## Traffic\n")).To(Succeed())
 		Expect(writeTrafficResults(resultsFile, homeDir, "coffee.txt", "HTTP")).To(Succeed())
 		Expect(writeTrafficResults(resultsFile, homeDir, "tea.txt", "HTTPS")).To(Succeed())
 
diff --git a/tests/suite/system_suite_test.go b/tests/suite/system_suite_test.go
@@ -170,7 +170,7 @@ func setup(cfg setupConfig, extraInstallArgs ...string) {
 		timeoutConfig.CreateTimeout,
 	)
 	Expect(err).ToNot(HaveOccurred())
-	Expect(podNames).ToNot(HaveLen(0))
+	Expect(podNames).ToNot(BeEmpty())
 
 	if *serviceType != "LoadBalancer" {
 		portFwdPort, err = framework.PortForward(k8sConfig, installCfg.Namespace, podNames[0], portForwardStopCh)
diff --git a/tests/suite/upgrade_test.go b/tests/suite/upgrade_test.go
@@ -185,7 +185,7 @@ var _ = Describe("Upgrade testing", Label("nfr", "upgrade"), func() {
 
 		podNames, err := framework.GetReadyNGFPodNames(k8sClient, ngfNamespace, releaseName, timeoutConfig.GetTimeout)
 		Expect(err).ToNot(HaveOccurred())
-		Expect(podNames).ToNot(HaveLen(0))
+		Expect(podNames).ToNot(BeEmpty())
 
 		// ensure that the leader election lease has been updated to the new pods
 		leaseCtx, leaseCancel := context.WithTimeout(context.Background(), 1*time.Minute)

Original file line number	Diff line number	Diff line change
`@@ -150,7 +150,7 @@ func TestRegister(t *testing.T) {`
`150`	`150`	`} else {`
`151`	`151`	`err := register()`
`152`	`152`	`if test.expectedErr == nil {`
`153`		`- g.Expect(err).To(BeNil())`
	`153`	`+ g.Expect(err).ToNot(HaveOccurred())`
`154`	`154`	`} else {`
`155`	`155`	`g.Expect(err).To(MatchError(test.expectedErr))`
`156`	`156`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,8 +28,8 @@ func TestEventLoop_SwapBatches(t *testing.T) {`
`28`	`28`
`29`	`29`	`eventLoop.swapBatches()`
`30`	`30`
`31`		`- g.Expect(len(eventLoop.currentBatch)).To(Equal(len(nextBatch)))`
	`31`	`+ g.Expect(eventLoop.currentBatch).To(HaveLen(len(nextBatch)))`
`32`	`32`	`g.Expect(eventLoop.currentBatch).To(Equal(nextBatch))`
`33`		`- g.Expect(len(eventLoop.nextBatch)).To(Equal(0))`
	`33`	`+ g.Expect(eventLoop.nextBatch).To(BeEmpty())`
`34`	`34`	`g.Expect(cap(eventLoop.nextBatch)).To(Equal(3))`
`35`	`35`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ func TestEnableAfterBecameLeader(t *testing.T) {`
`32`	`32`	`g.Expect(enabled).To(BeFalse())`
`33`	`33`
`34`	`34`	`err := enableAfterBecameLeader.Start(context.Background())`
`35`		`- g.Expect(err).To(BeNil())`
	`35`	`+ g.Expect(err).ToNot(HaveOccurred())`
`36`	`36`
`37`	`37`	`g.Expect(enabled).To(BeTrue())`
`38`	`38`	`}`
Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ func TestLessObjectMeta(t *testing.T) {`
`85`	`85`	`invertedResult := LessObjectMeta(test.meta2, test.meta1)`
`86`	`86`
`87`	`87`	`g.Expect(result).To(Equal(test.expected))`
`88`		`- g.Expect(invertedResult).To(Equal(false))`
	`88`	`+ g.Expect(invertedResult).To(BeFalse())`
`89`	`89`	`})`
`90`	`90`	`}`
`91`	`91`	`}`
Original file line number	Diff line number	Diff line change
`@@ -405,7 +405,7 @@ func TestValidateBackendTLSPolicy(t *testing.T) {`
`405`	`405`	`if !test.isValid && !test.ignored {`
`406`	`406`	`g.Expect(conds).To(HaveLen(1))`
`407`	`407`	`} else {`
`408`		`- g.Expect(conds).To(HaveLen(0))`
	`408`	`+ g.Expect(conds).To(BeEmpty())`
`409`	`409`	`}`
`410`	`410`	`})`
`411`	`411`	`}`
Original file line number	Diff line number	Diff line change
`@@ -250,7 +250,7 @@ func TestBuildSectionNameRefs(t *testing.T) {`
`250`	`250`	`if test.expectedError != nil {`
`251`	`251`	`g.Expect(err).To(Equal(test.expectedError))`
`252`	`252`	`} else {`
`253`		`- g.Expect(err).To(BeNil())`
	`253`	`+ g.Expect(err).ToNot(HaveOccurred())`
`254`	`254`	`}`
`255`	`255`	`})`
`256`	`256`	`}`