From 48f8e398fba6657f03e0dab746a9297b3fdc1896 Mon Sep 17 00:00:00 2001 From: Patrick Ohly Date: Thu, 2 Apr 2020 15:31:16 +0200 Subject: [PATCH] mock tests: remove redundant wrapping of error The "error waiting for expected CSI calls" is redundant because it's immediately followed by checking that error with: framework.ExpectNoError(err, "while waiting for all CSI calls") --- test/e2e/storage/csi_mock_volume.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/e2e/storage/csi_mock_volume.go b/test/e2e/storage/csi_mock_volume.go index 22f29beed37..271261792a3 100644 --- a/test/e2e/storage/csi_mock_volume.go +++ b/test/e2e/storage/csi_mock_volume.go @@ -726,7 +726,7 @@ var _ = utils.SIGDescribe("CSI mock volume", func() { err = wait.Poll(time.Second, csiUnstageWaitTimeout, func() (done bool, err error) { _, index, err := compareCSICalls(trackedCalls, test.expectedCalls, m.cs, f.Namespace.Name, driverPodName, driverContainerName) if err != nil { - return true, fmt.Errorf("error waiting for expected CSI calls: %s", err) + return true, err } if index == 0 { // No CSI call received yet