Extend bundle diff test for modified ignored keys

e2e/single-cluster/bundle_diffs_test.go

@@ -25,6 +25,7 @@ var _ = Describe("BundleDiffs", func() {
 		targetNamespace string
 
 		k        kubectl.Command
+		kw       kubectl.Command
 		interval = 2 * time.Second
 		duration = 30 * time.Second
 	)
@@ -54,6 +55,8 @@ var _ = Describe("BundleDiffs", func() {
 		var err error
 		targetNamespace, err = utils.NewNamespaceName()
 		Expect(err).ToNot(HaveOccurred())
+		kw = k.Namespace(targetNamespace)
+
 		name = "bundle-diff-" + targetNamespace
 	})
 
@@ -105,32 +108,57 @@ var _ = Describe("BundleDiffs", func() {
 					if err != nil || status == nil {
 						return false
 					}
+					GinkgoWriter.Printf("bundledeployment status: %v", status)
 					return status.Ready && status.NonModified
 				}).Should(BeTrue())
 
 				Eventually(func() string {
-					out, _ := k.Namespace(targetNamespace).Get("services")
+					out, _ := kw.Get("services")
 
 					return out
 				}).Should(ContainSubstring("app-service"))
 			})
 		})
 
-		Context("modifying a ignored resource", func() {
+		Context("adding new values", func() {
 			JustBeforeEach(func() {
 				By("modifying the workload resources", func() {
-					kw := k.Namespace(targetNamespace)
 					out, err := kw.Patch(
 						"service", "app-service",
 						"--type=json",
 						"-p", `[{"op": "add", "path": "/spec/ports/0", "value": {"name":"test","port":1023,"protocol":"TCP"}}]`,
 					)
 					Expect(err).ToNot(HaveOccurred(), out)
 
+					// adds a value
 					out, err = kw.Patch(
 						"configmap", "app-config",
 						"--type=merge",
-						"-p", `{"data":{"value":"by-test-code"}}`,
+						"-p", `{"data":{"newvalue":"by-test-code"}}`,
+					)
+					Expect(err).ToNot(HaveOccurred(), out)
+				})
+			})
+
+			It("ignores changes", func() {
+				Consistently(func() bool {
+					status, err := bundleDeploymentStatus(name)
+					if err != nil || status == nil {
+						return false
+					}
+
+					return status.NonModified
+				}, duration, interval).Should(BeTrue(), "ignores modification")
+			})
+		})
+
+		Context("modifying existing values, that are ignored", func() {
+			JustBeforeEach(func() {
+				By("modifying the workload resources", func() {
+					out, err := kw.Patch(
+						"configmap", "app-config",
+						"--type=merge",
+						"-p", `{"data":{"test":"by-test-code"}}`,
 					)
 					Expect(err).ToNot(HaveOccurred(), out)
 				})
@@ -148,10 +176,9 @@ var _ = Describe("BundleDiffs", func() {
 			})
 		})
 
-		Context("modifying a monitored resource", func() {
+		Context("modifying existing values, that are not ignored", func() {
 			JustBeforeEach(func() {
 				By("modifying a monitored value", func() {
-					kw := k.Namespace(targetNamespace)
 					out, err := kw.Patch(
 						"service", "app-service",
 						"--type=json",