test(Controllers): trigger finalizers

Author: Baarsgaard

controllers/alertrulegroup_controller_test.go

@@ -163,6 +163,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

@@ -125,6 +125,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

@@ -141,6 +141,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

@@ -283,6 +283,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

@@ -101,6 +101,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

@@ -99,6 +99,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

@@ -106,6 +106,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

@@ -489,6 +489,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

@@ -92,6 +92,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())
+			}
 		})
 	}
 })