check StringData

Author: Cory Schwartz

controllers/ipfscluster_controller_test.go

@@ -81,25 +81,36 @@ var _ = Describe("IPFS Reconciler", func() {
 		It("creates a new peer ids", func() {
 			fn, _ := ipfsReconciler.SecretConfig(ctx, ipfs, secretConfig, clusterSec, bootstrapKey)
 			Expect(fn()).To(BeNil())
-			expectedKeys := alwaysKeys + (replicas * 2)
-			Expect(len(secretConfig.Data)).To(Equal(expectedKeys))
+			expectedKeys := replicas * 2
+			// in the real world, StringData will be copied to Data as part of k8s.
+			// However, we are checking it before it has the opportunity.
+			Expect(len(secretConfig.Data)).To(Equal(alwaysKeys))
+			Expect(len(secretConfig.StringData)).To(Equal(expectedKeys))
 
 			// save this so we can check for changes later.
 			dataCopy := make(map[string][]byte)
+			stringCopy := make(map[string]string)
 			for k, v := range secretConfig.Data {
 				dataCopy[k] = v
 			}
+			for k, v := range secretConfig.StringData {
+				stringCopy[k] = v
+			}
 
 			// increase the replica count. Expect to see new keys generated.
 			ipfs.Spec.Replicas++
 			fn, _ = ipfsReconciler.SecretConfig(ctx, ipfs, secretConfig, clusterSec, bootstrapKey)
 			Expect(fn()).To(BeNil())
-			Expect(len(secretConfig.Data)).To(Equal(expectedKeys + 2))
+			Expect(len(secretConfig.Data)).To(Equal(alwaysKeys))
+			Expect(len(secretConfig.StringData)).To(Equal(expectedKeys + 2))
 
 			// expect the old keys to still be the same
 			for k := range dataCopy {
 				Expect(dataCopy[k]).To(Equal(secretConfig.Data[k]))
 			}
+			for k := range stringCopy {
+				Expect(stringCopy[k]).To(Equal(secretConfig.StringData[k]))
+			}
 		})
 	})
 })