test(Controllers): trigger finalizers

Author: Baarsgaard

controllers/alertrulegroup_controller_test.go

@@ -162,6 +162,15 @@ var _ = Describe("AlertRulegroup Reconciler: Provoke Conditions", func() {
 			// Verify condition
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Type", test.wantCondition)))
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Reason", test.wantReason)))
+
+			// Delete resource
+			Expect(k8sClient.Delete(testCtx, resultCr)).Should(Succeed())
+
+			// Skip finalizer
+			if test.wantErr == "" {
+				_, err = r.Reconcile(testCtx, req)
+				Expect(err).ShouldNot(HaveOccurred())
+			}
 		})
 	}
 })

controllers/contactpoint_controller_test.go

@@ -124,6 +124,15 @@ var _ = Describe("ContactPoint Reconciler: Provoke Conditions", func() {
 			// Verify Condition
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Type", test.wantCondition)))
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Reason", test.wantReason)))
+
+			// Delete resource
+			Expect(k8sClient.Delete(testCtx, resultCr)).Should(Succeed())
+
+			// Skip finalizer
+			if test.wantErr == "" {
+				_, err = r.Reconcile(testCtx, req)
+				Expect(err).ShouldNot(HaveOccurred())
+			}
 		})
 	}
 })

controllers/dashboard_controller_test.go

@@ -140,6 +140,15 @@ var _ = Describe("Dashboard Reconciler: Provoke Conditions", func() {
 			// Verify Condition
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Type", test.wantCondition)))
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Reason", test.wantReason)))
+
+			// Delete resource
+			Expect(k8sClient.Delete(testCtx, resultCr)).Should(Succeed())
+
+			// Skip finalizer
+			if test.wantErr == "" {
+				_, err = r.Reconcile(testCtx, req)
+				Expect(err).ShouldNot(HaveOccurred())
+			}
 		})
 	}
 })

controllers/datasource_controller_test.go

@@ -282,6 +282,15 @@ var _ = Describe("Datasource Reconciler: Provoke Conditions", func() {
 			// Verify Condition
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Type", test.wantCondition)))
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Reason", test.wantReason)))
+
+			// Delete resource
+			Expect(k8sClient.Delete(testCtx, resultCr)).Should(Succeed())
+
+			// Skip finalizer
+			if test.wantErr == "" {
+				_, err = r.Reconcile(testCtx, req)
+				Expect(err).ShouldNot(HaveOccurred())
+			}
 		})
 	}
 })

controllers/folder_controller_test.go

@@ -100,6 +100,15 @@ var _ = Describe("Folder Reconciler: Provoke Conditions", func() {
 			// Verify Condition
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Type", test.wantCondition)))
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Reason", test.wantReason)))
+
+			// Delete resource
+			Expect(k8sClient.Delete(testCtx, resultCr)).Should(Succeed())
+
+			// Skip finalizer
+			if test.wantErr == "" {
+				_, err = r.Reconcile(testCtx, req)
+				Expect(err).ShouldNot(HaveOccurred())
+			}
 		})
 	}
 })

controllers/grafana_controller_test.go

@@ -88,6 +88,12 @@ var _ = Describe("Grafana Reconciler: Provoke Conditions", func() {
 			// Verify condition
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Type", test.wantCondition)))
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Reason", test.wantReason)))
+
+			// Delete resource
+			Expect(k8sClient.Delete(testCtx, resultCr)).Should(Succeed())
+
+			_, err = r.Reconcile(testCtx, req)
+			Expect(err).ShouldNot(HaveOccurred())
 		})
 	}
 })

controllers/librarypanel_controller_test.go

@@ -98,6 +98,15 @@ var _ = Describe("LibraryPanel Reconciler: Provoke Conditions", func() {
 			// Verify Condition
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Type", test.wantCondition)))
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Reason", test.wantReason)))
+
+			// Delete resource
+			Expect(k8sClient.Delete(testCtx, resultCr)).Should(Succeed())
+
+			// Skip finalizer
+			if test.wantErr == "" {
+				_, err = r.Reconcile(testCtx, req)
+				Expect(err).ShouldNot(HaveOccurred())
+			}
 		})
 	}
 })

controllers/mutetiming_controller_test.go

@@ -105,6 +105,15 @@ var _ = Describe("MuteTiming Reconciler: Provoke Conditions", func() {
 			// Verify Condition
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Type", test.wantCondition)))
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Reason", test.wantReason)))
+
+			// Delete resource
+			Expect(k8sClient.Delete(testCtx, resultCr)).Should(Succeed())
+
+			// Skip finalizer
+			if test.wantErr == "" {
+				_, err = r.Reconcile(testCtx, req)
+				Expect(err).ShouldNot(HaveOccurred())
+			}
 		})
 	}
 })

controllers/notificationpolicy_controller_test.go

@@ -488,6 +488,15 @@ var _ = Describe("NotificationPolicy Reconciler: Provoke Conditions", func() {
 			// Verify Condition
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Type", test.wantCondition)))
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Reason", test.wantReason)))
+
+			// Delete resource
+			Expect(k8sClient.Delete(testCtx, resultCr)).Should(Succeed())
+
+			// Skip finalizer
+			if test.wantErr == "" {
+				_, err = r.Reconcile(testCtx, req)
+				Expect(err).ShouldNot(HaveOccurred())
+			}
 		})
 	}
 })

controllers/notificationtemplate_controller_test.go

@@ -91,6 +91,15 @@ var _ = Describe("NotificationTemplate Reconciler: Provoke Conditions", func() {
 			// Verify Condition
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Type", test.wantCondition)))
 			Expect(resultCr.Status.Conditions).Should(ContainElement(HaveField("Reason", test.wantReason)))
+
+			// Delete resource
+			Expect(k8sClient.Delete(testCtx, resultCr)).Should(Succeed())
+
+			// Skip finalizer
+			if test.wantErr == "" {
+				_, err = r.Reconcile(testCtx, req)
+				Expect(err).ShouldNot(HaveOccurred())
+			}
 		})
 	}
 })