Merge pull request #50406 from alrs/fix-vsphere-volume-dropped-errors

Automatic merge from submit-queue Fix dropped errors in vsphere_volume **What this PR does / why we need it**: Error variables are being assigned in the vsphere_volume tests, but their values are being ignored. **Release note**: ```release-note NONE ```

Merge pull request #50406 from alrs/fix-vsphere-volume-dropped-errors
acf7b571 · Kubernetes Submit Queue · GitHub · 6b0d4754 · f56bf94a · acf7b571
Commit acf7b571 authored Aug 14, 2017 by Kubernetes Submit Queue Committed by GitHub Aug 14, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

vsphere_volume_test.go pkg/volume/vsphere_volume/vsphere_volume_test.go +6 -0

No files found.
--- a/pkg/volume/vsphere_volume/vsphere_volume_test.go
+++ b/pkg/volume/vsphere_volume/vsphere_volume_test.go
@@ -153,6 +153,9 @@ func TestPlugin(t *testing.T) {
 		PersistentVolumeReclaimPolicy: v1.PersistentVolumeReclaimDelete,
 	}
 	provisioner, err := plug.(*vsphereVolumePlugin).newProvisionerInternal(options, &fakePDManager{})
+	if err != nil {
+		t.Errorf("newProvisionerInternal() failed: %v", err)
+	}
 	persistentSpec, err := provisioner.Provision()
 	if err != nil {
 		t.Errorf("Provision() failed: %v", err)
@@ -177,6 +180,9 @@ func TestPlugin(t *testing.T) {
 		PersistentVolume: persistentSpec,
 	}
 	deleter, err := plug.(*vsphereVolumePlugin).newDeleterInternal(volSpec, &fakePDManager{})
+	if err != nil {
+		t.Errorf("newDeleterInternal() failed: %v", err)
+	}
 	err = deleter.Delete()
 	if err != nil {
 		t.Errorf("Deleter() failed: %v", err)