From 71fb3ce9209c397a1025927dd1eef061234a3702 Mon Sep 17 00:00:00 2001 From: Timo Reimann Date: Thu, 16 Apr 2020 21:45:01 +0200 Subject: [PATCH] Add descriptions to To() expectations in NodeUnpublishVolume Without descriptions, To() expectations are unnecessarily difficult to understand. --- pkg/sanity/node.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/sanity/node.go b/pkg/sanity/node.go index cd759101..64345c18 100644 --- a/pkg/sanity/node.go +++ b/pkg/sanity/node.go @@ -428,11 +428,11 @@ var _ = DescribeSanity("Node Service", func(sc *TestContext) { VolumeId: sc.Config.IDGen.GenerateUniqueValidVolumeID(), TargetPath: sc.StagingPath, }) - Expect(err).To(HaveOccurred()) + Expect(err).To(HaveOccurred(), "failed to unpublish volume from node") serverError, ok := status.FromError(err) - Expect(ok).To(BeTrue()) - Expect(serverError.Code()).To(Equal(codes.NotFound)) + Expect(ok).To(BeTrue(), "error from NodeUnpublishVolume is not a gRPC error") + Expect(serverError.Code()).To(Equal(codes.NotFound), "unexpected error code") }) })