Skip to content

Commit

Permalink
Merge pull request #258 from timoreimann/gate-execution-of-NodeUnpubl…
Browse files Browse the repository at this point in the history
…ishVolume-missing-volume-test

Remove missing volume test case for NodeUnpublishVolume
  • Loading branch information
k8s-ci-robot authored May 25, 2020
2 parents 661a3cb + d3bc8bc commit e89bc15
Showing 1 changed file with 0 additions and 15 deletions.
15 changes: 0 additions & 15 deletions pkg/sanity/node.go
Original file line number Diff line number Diff line change
Expand Up @@ -419,21 +419,6 @@ 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() {

_, err := c.NodeUnpublishVolume(
context.Background(),
&csi.NodeUnpublishVolumeRequest{
VolumeId: sc.Config.IDGen.GenerateUniqueValidVolumeID(),
TargetPath: sc.StagingPath,
})
Expect(err).To(HaveOccurred(), "failed to unpublish volume from node")

serverError, ok := status.FromError(err)
Expect(ok).To(BeTrue(), "error from NodeUnpublishVolume is not a gRPC error")
Expect(serverError.Code()).To(Equal(codes.NotFound), "unexpected error code")
})
})

Describe("NodeStageVolume", func() {
Expand Down

0 comments on commit e89bc15

Please sign in to comment.