fix: disable dupl linter on some e2e tests

Linter complains that the expected event lists are too similar,
which is deliberate and should not be abstracted away. Explicit
test expectations in the test makes it easier to debug failures.

test/e2e/crd_test.go

@@ -19,6 +19,7 @@ import (
 	"sigs.k8s.io/controller-runtime/pkg/client"
 )
 
+//nolint:dupl // expEvents similar to mutation tests
 func crdTest(_ client.Client, invConfig InventoryConfig, inventoryName, namespaceName string) {
 	By("apply a set of resources that includes both a crd and a cr")
 	applier := invConfig.ApplierFactoryFunc()
@@ -38,7 +39,6 @@ func crdTest(_ client.Client, invConfig InventoryConfig, inventoryName, namespac
 		EmitStatusEvents: false,
 	}))
 
-	//nolint:dupl
 	expEvents := []testutil.ExpEvent{
 		{
 			// InitTask

test/e2e/mutation_test.go

@@ -31,6 +31,7 @@ import (
 // we test a toy example with a pod-a depending on pod-b, injecting the ip and
 // port from pod-b into an environment variable of pod-a.
 
+//nolint:dupl // expEvents similar to CRD tests
 func mutationTest(c client.Client, invConfig InventoryConfig, inventoryName, namespaceName string) {
 	By("apply resources in order with substitutions based on apply-time-mutation annotation")
 	applier := invConfig.ApplierFactoryFunc()
@@ -51,7 +52,6 @@ func mutationTest(c client.Client, invConfig InventoryConfig, inventoryName, nam
 		EmitStatusEvents: false,
 	}))
 
-	//nolint:dupl
 	expEvents := []testutil.ExpEvent{
 		{
 			// InitTask