Add webhook rule checker to validator

Per Goncalves da Silva · Per Goncalves da Silva · commit d42452a17295 · 2025-09-19T09:48:52.000+02:00
Signed-off-by: Per Goncalves da Silva &lt;pegoncal@redhat.com&gt;
diff --git a/internal/operator-controller/rukpak/render/registryv1/registryv1.go b/internal/operator-controller/rukpak/render/registryv1/registryv1.go
@@ -27,6 +27,7 @@ var BundleValidator = render.BundleValidator{
 	validators.CheckWebhookNameIsDNS1123SubDomain,
 	validators.CheckConversionWebhookCRDReferenceUniqueness,
 	validators.CheckConversionWebhooksReferenceOwnedCRDs,
+	validators.CheckWebhookRules,
 }
 
 // ResourceGenerators a slice of ResourceGenerators required to generate plain resource manifests for
diff --git a/internal/operator-controller/rukpak/render/registryv1/registryv1_test.go b/internal/operator-controller/rukpak/render/registryv1/registryv1_test.go
@@ -31,6 +31,7 @@ func Test_BundleValidatorHasAllValidationFns(t *testing.T) {
 		validators.CheckWebhookNameIsDNS1123SubDomain,
 		validators.CheckConversionWebhookCRDReferenceUniqueness,
 		validators.CheckConversionWebhooksReferenceOwnedCRDs,
+		validators.CheckWebhookRules,
 	}
 	actualValidationFns := registryv1.BundleValidator
 

Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,7 @@ var BundleValidator = render.BundleValidator{`
`27`	`27`	`validators.CheckWebhookNameIsDNS1123SubDomain,`
`28`	`28`	`validators.CheckConversionWebhookCRDReferenceUniqueness,`
`29`	`29`	`validators.CheckConversionWebhooksReferenceOwnedCRDs,`
	`30`	`+ validators.CheckWebhookRules,`
`30`	`31`	`}`
`31`	`32`
`32`	`33`	`// ResourceGenerators a slice of ResourceGenerators required to generate plain resource manifests for`
Original file line number	Diff line number	Diff line change
`@@ -31,6 +31,7 @@ func Test_BundleValidatorHasAllValidationFns(t *testing.T) {`
`31`	`31`	`validators.CheckWebhookNameIsDNS1123SubDomain,`
`32`	`32`	`validators.CheckConversionWebhookCRDReferenceUniqueness,`
`33`	`33`	`validators.CheckConversionWebhooksReferenceOwnedCRDs,`
	`34`	`+ validators.CheckWebhookRules,`
`34`	`35`	`}`
`35`	`36`	`actualValidationFns := registryv1.BundleValidator`
`36`	`37`