sanity: test NOT_FOUND on NodePublishVolume

According to the CSI spec, a plugin must return a `NOT_FOUND` error when
`NodePublishVolume` is called with a `volume_id` of a volume that does
not (or no longer?) exist.

The `sanity` package has a test asserting this `NOT_FOUND` is returned
on `NodeUnpublishVolume`, but not for `NodePublishVolume`. This patch
adds this for completeness.

See: https://github.com/container-storage-interface/spec/blob/87a27a2496565857ed564284eaa762587feb9453/spec.md#nodepublishvolume-errors

Author: NicolasT

pkg/sanity/node.go

@@ -391,6 +391,28 @@ var _ = DescribeSanity("Node Service", func(sc *TestContext) {
 			Expect(ok).To(BeTrue())
 			Expect(serverError.Code()).To(Equal(codes.InvalidArgument))
 		})
+
+		It("should fail when the volume is missing", func() {
+			stagingPath := ""
+			if nodeStageSupported {
+				stagingPath = sc.StagingPath
+			}
+
+			_, err := c.NodePublishVolume(
+				context.Background(),
+				&csi.NodePublishVolumeRequest{
+					VolumeId:          sc.Config.IDGen.GenerateUniqueValidVolumeID(),
+					VolumeCapability:  TestVolumeCapabilityWithAccessType(sc, csi.VolumeCapability_AccessMode_SINGLE_NODE_WRITER),
+					StagingTargetPath: stagingPath,
+					TargetPath:        sc.TargetPath + "/target",
+					Secrets:           sc.Secrets.NodePublishVolumeSecret,
+				})
+			Expect(err).To(HaveOccurred())
+
+			serverError, ok := status.FromError(err)
+			Expect(ok).To(BeTrue())
+			Expect(serverError.Code()).To(Equal(codes.NotFound))
+		})
 	})
 
 	Describe("NodeUnpublishVolume", func() {