test(STONEINTG-883): add unit test coverage

Signed-off-by: Dheeraj<[email protected]>

internal/controller/snapshot/snapshot_adapter_test.go

@@ -1925,7 +1925,6 @@ var _ = Describe("Snapshot Adapter", Ordered, func() {
 			result, err := adapter.EnsureOverrideSnapshotValid()
 			Expect(result.CancelRequest).To(BeFalse())
 			Expect(result.RequeueRequest).To(BeFalse())
-			Expect(controllerutil.HasControllerReference(hasInvalidOverrideSnapshot)).To(BeTrue())
 			Expect(buf.String()).Should(ContainSubstring("Snapshot has been marked as invalid"))
 			Expect(err).ToNot(HaveOccurred())
 

internal/controller/snapshot/snapshot_controller_test.go

@@ -178,6 +178,30 @@ var _ = Describe("SnapshotController", func() {
 		Expect(err).To(BeNil())
 	})
 
+	It("can add the Application as a Controller OwnerReference on Snapshot", func() {
+		Expect(hasSnapshot.ObjectMeta.OwnerReferences).To(BeNil())
+
+		req := ctrl.Request{
+			NamespacedName: types.NamespacedName{
+				Name:      hasSnapshot.Name,
+				Namespace: "default",
+			},
+		}
+		result, err := snapshotReconciler.Reconcile(ctx, req)
+		Expect(reflect.TypeOf(result)).To(Equal(reflect.TypeOf(reconcile.Result{})))
+		Expect(err).ToNot(HaveOccurred())
+
+		Eventually(func() bool {
+			err := k8sClient.Get(ctx, types.NamespacedName{
+				Namespace: hasSnapshot.Namespace,
+				Name:      hasSnapshot.Name,
+			}, hasSnapshot)
+			return err == nil
+		}).Should(BeTrue())
+		Expect(hasSnapshot.ObjectMeta.OwnerReferences).ToNot(BeNil())
+		Expect(hasSnapshot.ObjectMeta.GetOwnerReferences()[0].Name).To(Equal(hasApp.Name))
+	})
+
 	It("Does not return an error if the component cannot be found", func() {
 		err := k8sClient.Delete(ctx, hasComp)
 		Eventually(func() bool {