Added test for updated annotation; added timeout for teardown

Author: seans3

pkg/client/client_test.go

@@ -78,7 +78,7 @@ var _ = Describe("Client", func() {
 		scheme.AddKnownTypeWithName(ngvk, node)
 
 		close(done)
-	})
+	}, serverSideTimeoutSeconds)
 
 	var delOptions *metav1.DeleteOptions
 	AfterEach(func(done Done) {
@@ -100,7 +100,7 @@ var _ = Describe("Client", func() {
 			Expect(err).NotTo(HaveOccurred())
 		}
 		close(done)
-	})
+	}, serverSideTimeoutSeconds)
 
 	Describe("New", func() {
 		It("should return a new Client", func(done Done) {
@@ -280,10 +280,11 @@ var _ = Describe("Client", func() {
 			err = cl.Update(context.TODO(), dep)
 			Expect(err).NotTo(HaveOccurred())
 
-			By("validating updated Deployment exists")
+			By("validating updated Deployment has new annotation")
 			actual, err := clientset.AppsV1().Deployments(ns).Get(dep.Name, metav1.GetOptions{})
 			Expect(err).NotTo(HaveOccurred())
 			Expect(actual).NotTo(BeNil())
+			Expect(actual.Annotations["foo"]).To(Equal("bar"))
 
 			close(done)
 		})
@@ -306,10 +307,11 @@ var _ = Describe("Client", func() {
 			err = cl.Update(context.TODO(), u)
 			Expect(err).NotTo(HaveOccurred())
 
-			By("fetching newly created unstructured Deployment")
+			By("fetching newly created unstructured Deployment has new annotation")
 			actual, err := clientset.AppsV1().Deployments(ns).Get(dep.Name, metav1.GetOptions{})
 			Expect(err).NotTo(HaveOccurred())
 			Expect(actual).NotTo(BeNil())
+			Expect(actual.Annotations["foo"]).To(Equal("bar"))
 
 			close(done)
 		})
@@ -327,10 +329,11 @@ var _ = Describe("Client", func() {
 			err = cl.Update(context.TODO(), node)
 			Expect(err).NotTo(HaveOccurred())
 
-			By("writing the result back to the go struct")
+			By("validate updated Node had new annotation")
 			actual, err := clientset.CoreV1().Nodes().Get(node.Name, metav1.GetOptions{})
 			Expect(err).NotTo(HaveOccurred())
 			Expect(actual).NotTo(BeNil())
+			Expect(actual.Annotations["foo"]).To(Equal("bar"))
 
 			close(done)
 		})