Merge pull request #583 from happybhati/release-1105

loader/loader.go

@@ -160,6 +160,11 @@ func (l *loader) GetMatchingReleasePlanAdmission(ctx context.Context, cli client
 // are in the namespace specified by the ReleasePlanAdmission's origin. If the List operation fails, an
 // error will be returned.
 func (l *loader) GetMatchingReleasePlans(ctx context.Context, cli client.Client, releasePlanAdmission *v1alpha1.ReleasePlanAdmission) (*v1alpha1.ReleasePlanList, error) {
+
+	if releasePlanAdmission.Spec.Origin == "" {
+		return nil, fmt.Errorf("releasePlanAdmission has no origin, so no ReleasePlans can be found")
+	}
+
 	releasePlans := &v1alpha1.ReleasePlanList{}
 	err := cli.List(ctx, releasePlans,
 		client.InNamespace(releasePlanAdmission.Spec.Origin),

loader/loader_test.go

@@ -250,6 +250,16 @@ var _ = Describe("Release Adapter", Ordered, func() {
 				return returnedObject != &v1alpha1.ReleasePlanList{} && err == nil && contains == false
 			})
 		})
+
+		It("fails to return release plans if origin is empty", func() {
+			modifiedReleasePlanAdmission := releasePlanAdmission.DeepCopy()
+			modifiedReleasePlanAdmission.Spec.Origin = ""
+
+			returnedObject, err := loader.GetMatchingReleasePlans(ctx, k8sClient, modifiedReleasePlanAdmission)
+			Expect(err).To(HaveOccurred())
+			Expect(err.Error()).To(ContainSubstring("releasePlanAdmission has no origin, so no ReleasePlans can be found"))
+			Expect(returnedObject).To(BeNil())
+		})
 	})
 
 	When("calling GetPreviousRelease", func() {