Merge pull request #823 from pafmaf/fix/exchange-webhook-validator-interface

minor cleanup: `Spec.AutoDelete` VS `Spec.Durable`

Author: Zerpet

api/v1beta1/exchange_webhook.go

@@ -83,7 +83,7 @@ func (e *Exchange) ValidateUpdate(_ context.Context, oldObj, newObj runtime.Obje
 if newExchange.Spec.Durable != oldExchange.Spec.Durable {
 allErrs = append(allErrs, field.Invalid(
 field.NewPath("spec", "durable"),
-newExchange.Spec.AutoDelete,
+newExchange.Spec.Durable,
 "durable cannot be updated",
 ))
 }

api/v1beta1/queue_webhook.go

@@ -89,7 +89,7 @@ func (q *Queue) ValidateUpdate(_ context.Context, oldObj, newObj runtime.Object)
 if newQueue.Spec.Durable != oldQueue.Spec.Durable {
 allErrs = append(allErrs, field.Invalid(
 field.NewPath("spec", "durable"),
-newQueue.Spec.AutoDelete,
+newQueue.Spec.Durable,
 "durable cannot be updated",
 ))
 }

api/v1beta1/queue_webhook_test.go

@@ -51,7 +51,7 @@ var _ = Describe("queue webhook", func() {
 
 It("does not allow non-durable quorum queues", func() {
 notAllowedQ := queue.DeepCopy()
-notAllowedQ.Spec.AutoDelete = false
+notAllowedQ.Spec.Durable = false
 _, err := notAllowedQ.ValidateCreate(rootCtx, notAllowedQ)
 Expect(err).To(MatchError(ContainSubstring("Quorum queues must have durable set to true")))
 })